Opened 10 years ago
Closed 10 years ago
#12663 closed defect (fixed)
Rhino Build includes Browser specific modules
Reported by: | zet4080 | Owned by: | Rawld Gill |
---|---|---|---|
Priority: | high | Milestone: | 1.7 |
Component: | BuildSystem | Version: | 1.6.0 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
Download src release of dojo 1.6
Go to util/buildscripts
build profile=rhino action=release
The build includes browser specific modules like _base/html.js and some more.
Change History (5)
comment:1 Changed 10 years ago by
comment:2 Changed 10 years ago by
Milestone: | tbd → 1.7 |
---|---|
Status: | new → assigned |
comment:3 Changed 10 years ago by
Status: | assigned → new |
---|
comment:4 Changed 10 years ago by
Status: | new → assigned |
---|
comment:5 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
I tracked the error down to util/buildscripts/jslib/fileUtil.js, Method transformAsyncModule.
At the very beginning of this method are some variable definitions, and there is the line
dojo = { isBrowser:true },
included.
If I set this value to false the build is done as expected (means: like in Version 1.5.0).
It is not the correct place to do this, because if I run the base.profile.js now, the Browser parts are missing, so I guess it is not the right place to set the isBrowser property at all, but I can't figure out what the right place is ...