search

sproutcore / sproutcore

JavaScript Application Framework - JS library only
sproutcore.com
Other
2.15k stars 291 forks source link

sc_require TOO strict, rejects/ignores camel case file names #13

Closed geapi closed 14 years ago

geapi commented 14 years ago

after switching from 1.0.46 to 1.4.1 I see this warning:

WARN ~ Could not find entry 'system/loader' required in sproutcore/bootstrap:source/setup_body_class_names.js

and then none of my sc_requires are found, the files are loaded anyways but not in the right order which breaks the app.

turned out to be the sc_require parser not liking camel case anymore (used to in 1.0)

wagenet commented 14 years ago

A few questions: 1) Are you running from gem? 2) Did you clear your tmp directory? 3) Did you override the SproutCore framework in your project's framework/spoutcore directory? If so, what version?

geapi commented 14 years ago

1) from gem? yes, sproutcore (1.4.1) 2) tmp clean? YES 3) framework overwritten? yes (latest from master at sproutcore/sproutcore) and no (removed it to see if that was the culprit but the warning and subsequent failing of the app persists)

devinus commented 14 years ago

This can be fixed if the commented-out sc_require('system/loader') is taken out of setup_body_class_names.js.

geapi commented 14 years ago

if I do that I get: ReferenceError: Can't find variable: sc_require which confirms that something with the sc_require is off, I'm not sure where to dig deeper though

geapi commented 14 years ago

updated title based on these comments and IRC chat with rxcfc

wagenet commented 14 years ago

Fixed in: http://github.com/sproutcore/abbot/commit/1e116951bd55c2a3235580cb717c5cf1b9f6cf6e

wagenet commented 14 years ago

Also fixed system/loader warning: http://github.com/sproutcore/sproutcore/commit/fc62b5c03f75de52b52ccf928f97bfc1defa720d