Exclude module dependencies in layered builds doesn't seem to work about requirejs-rails HOT 7 CLOSED

Hey Julian,

I think you should exclude: ["jquery"] because common isn't a dependency of your page1.js.

Though, I have the following solution that doesn't seem to work:

My app.js: define["module1", "module2"], { // ... })

My module1.js: define["ckeditor/ckeditor", { //... })

My requirejs.yml:

paths:
  ckeditor: "ui/ckeditor"
modules:
  - name: 'app'
    exclude: ['ckeditor/ckeditor']

Though, ckeditor is still included. I tried to exclude ckeditor with ckeditor, ui/ckeditor/ckeditor and ckeditor/ckeditor. None seems to work.

Any clue?

from requirejs-rails.

I created a fork up until John merges or fixes it:

#31

If you want to test it, replace the requirejs-rails line in your gemfile with:
gem 'requirejs-rails', :git => 'https://github.com/karellm/requirejs-rails'

from requirejs-rails.

Thanks for you bugfix @karellm.

Take a look at the example in the docs: http://requirejs.org/docs/faq-optimization.html#priority

If I exclude common then the optimized page1 module shouldn't include common's modules and their nested dependencies.

from requirejs-rails.

Indeed it makes a lot of sense as it excludes all dependencies of the excluded module. Thanks for the info.

from requirejs-rails.

I've also experienced this problem. The exclude directive doesn't seem to work for either direct dependencies or their nested dependencies. My workaround has been to force exclusion at the module level by declaring dependencies using variables, as per the example on http://requirejs.org/docs/optimization.html .

from requirejs-rails.

That is a nice workaround but my fork fixes the bug in the gem directly so your code can remain clean of any hack.

from requirejs-rails.

Hi all. I'm going get a fix for this into the next release, which I expect to kick out over the weekend. @karelim, thanks for pull request #31. I'll review that and pull it assuming that I have no feedback/changes.

from requirejs-rails.