sourceMappingUrl is wrong

[frankrrr]

Same problem here as #58, even when devtool = source-map.

/*# sourceMappingURL=../../../sass.map */

Just tried to run the demo test/sourceMap too, which isn't working either:

/*# sourceMappingURL=test/output/sass.map */

The file sass.map isn't generated. I'm using node-sass 3.0.0-beta.5 and sass-loader 1.0.1 and also tried other versions. Any ideas?

[jhnns]

Is the URL wrong or is the map file just not generated?

[frankrrr]

I think both, it should generate for example: styles.sourcemap.css.map (in stead of sass.map?)

[jhnns]

You're right. It looks like this has changed in webpack somehow... need to investigate.

[jeloi]

+1

[jjeising]

With the current webpack 1.8.9 and extract-text-plugin the following will be outputted in the .css file:

…
/*# sourceMappingURL=js/sass.map */
/*# sourceMappingURL=app.css.map*/

The later app.css.map exists and contains the .scss content, while the first isn't generated (as before). Config is

        loader: ExtractTextPlugin.extract('css?sourceMap!sass?sourceMap&sourceMapContents')

When deleting the first line manually everything looks fine in the browser.

[jhnns]

Thx for investigating this.... so maybe we just need to remove all the sourceMappingURL references added by libsass?

[jjeising]

That could be a solution, though I haven't tested other configurations. Removing this line does not work, as the map from libsass is needed. Maybe there should be an option for libsass to generate a source map without adding a reference?

[jhnns]

Maybe there should be an option for libsass to generate a source map without adding a reference

Well, that option doesn't make sense from libsass' point of view ^^

[paddycarver]

Forgive me if I'm an idiot, but isn't there already an option for that?

[jjeising]

Adding

  if (opt.sourceMap) {
    …
    opt.sourceMap = this.options.output.path + '/sass.map';
+   opt.omitSourceMapUrl = true;
  }

works fine for me.

[jhnns]

I'll try that out, thanks!

[jjeising]

Will this be added to the official distribution at some point?

[jhnns]

Yep. See #123

[jjeising]

Thanks!