Build Fail

[youngho]

Hello,

I got following build error

[INFO] E:\project\dojo\util\buildscripts>java -Xms256m -Xmx512m -classpath ../shrinksafe/js.jar;../shrinksafe/shrinksafe.jar org.mozilla.javascript.tools.shell.Main build.js action=release profile=standard version=1.5-SNAPSHOT loader=xdomain xdDojoPath=​http://mydomain/js/dojo optimize=shrinksafe cssOptimize=comments stripConsole=all
[INFO] release: Using profile: profiles/standard.profile.js
[INFO] release: Using version number: 1.5-SNAPSHOT for the release.
[INFO] release: Deleting: ../../release/dojo
[INFO] release: Copying: ../../dojo/../dijit to: ../../release/dojo/dijit
[INFO] release: Copying: ../../dojo/../dojox to: ../../release/dojo/dojox
[INFO] release: Copying: ../../dojo to: ../../release/dojo/dojo
[INFO] release: Building dojo.js and layer files
[INFO] error loading uri: ./../../release/dojo/dojo/_base/declare.js, exception: TypeError?: Cannot call method "split" of undefined

[bill]

Assuming this was caused by the recent changes to declare.

[Eugene Lazutkin]

Hmm, declare doesn't use split at all. Is it really called by Rhino? Reassigning to James to look at.

[Eugene Lazutkin]

Heh. somehow missed my previous post --- recreating + updating it with more info.

Something is wrong. And as far as I can tell it manifests itself in the XD loader. I got a similar error with "debugAtAllCosts". The offending line is 279 in the loader:

TypeError: modulename is undefined
[Break on this error] var syms = modulename.split(".");\n

I am attaching my test file (it is a minimal file, which does nothing but loads Dojo).

[James Burke]

(In [21809]) Fixes #10980, build and xd loader choke on the dojo.require() reference, even though it is in a string. the regexps used for the build/xd loader are string-aware. Technically not dojo.declare fault, but this is the easiest fix. \!strict

[Eugene Lazutkin]

:-D