Backbone shim about cajon

[jrburke]

cajon detects usage of node module APIs (like require/exports) and if detected, wraps the content in a define() function wrapper, then evals it. Since backbone tries to call node APIs if in node, cajon detects the API usage and wraps in a define() function wrapper.

This changes the scope and "this" value when Backbone executes. Since backbone does not try to use the node APIs to fetch jquery (instead tries to grab it off "this"/global, which is goofy in some respects), it fails.

What I can look at doing is to avoid doing the define() wrapping if there is a shim config set up for the file. I need to set up a backbone test to confirm the fix though. Will try to sort it out soon.

[grydstedt]

Ah, gotcha. Is the temp workaround to change the root.jQuery to a requjre('jquery') in backbone? I'll try it out

On Mar 20, 2013, at 7:08 PM, James Burke [email protected] wrote:

cajon detects usage of node module APIs (like require/exports) and if detected, wraps the content in a define() function wrapper, then evals it. Since backbone tries to call node APIs if in node, cajon detects the API usage and wraps in a define() function wrapper.

This changes the scope and "this" value when Backbone executes. Since backbone does not try to use the node APIs to fetch jquery (instead tries to grab it off "this"/global, which is goofy in some respects), it fails.

What I can look at doing is to avoid doing the define() wrapping if there is a shim config set up for the file. I need to set up a backbone test to confirm the fix though. Will try to sort it out soon.

—
Reply to this email directly or view it on GitHub.

[jrburke]

Another option is to use the amdified version of backbone from https://github.com/amdjs/backbone, but understand if you do not want to take fork like that.

[jrburke]

This is now fixed in the just-releases cajon 0.1.4. However, a change was also needed in the optimizer, to not do the cjsTranslate wrapping, tracked in requirejs/r.js#408. That fix will go out in r.js 2.1.6, which may be another week or so away. In the meantime, you can use the latest master snapshot if you want to do builds that work with this setup.

This is also a demo-shim directory in this repo that I used to confirm the changes.