Options handling if multiple input/output paths are used.

[mkempe]

Hey there,

if you use multiple input/output paths with ember-cli-less, multiple instances of LESSCompiler (broccoli-less-single) are created:

var trees = Object.keys(paths).map(function(file) {
  var input  = path.join(inputPath, file + '.' + ext);
  var output = paths[file];

  return new LESSCompiler([tree], input, output, options);
});

The problem: The options are shared / overridden by the last instance, respectively all instances have the same options. This results in the problem, that multiple Source Maps with the same name are created.

For example:

var app = new EmberApp(defaults, {
  outputPaths: {
    app: {
      html: 'index.html',
      css: {
        'foo': '/assets/foo.css',
        'bar': '/assets/bar.css'
      },
      js: '/assets/app.js'
    },

    // ...
  },

  // ...
};

For /assets/bar.css a Source Map with the name /assets/foo.css.map is created instead of /assets/bar.css.map.

I fixed the problem, but i do not really understand, why this happens , and if this is the right solution.

[knownasilya]

Looks great, and works as expected.

[jasonmit]

Ok I will review both ember cli less and your changes tonight to get a better understanding. Thanks!

[mkempe]

Hey Jason,

thanks for looking into.

PS: I merged the latest changes in.

[jasonmit]

Okay, I have a better understanding of the change. I'm all for not mutating objects that are passed in. Thanks for the PR, I will version accordingly in case people relied on this bug.

[jasonmit]

Thanks again 👍

[jasonmit]

Versioned as 0.5.0

[mkempe]

Nice! Thanks again Jason for looking over and the new release.