Normalize directory path to forward slash in sourcemap source property #110
Labels
Comments
|
Yes. I having the same issue. its windows only. when I track it down. I got a temporary fix. node_modules\grunt-contrib-concat\tasks\lib\sourcemap.js may be line number 96 : SourceMapConcatHelper.prototype.addlines = function(src, filename) {
var relativeFilename = path.relative(path.dirname(this.dest), filename);TO SourceMapConcatHelper.prototype.addlines = function(src, filename) {
var relativeFilename = path.relative(path.dirname(this.dest), filename).replace(/\\/g,'/');Good Luck. |
riaann
added a commit
to riaann/grunt-contrib-concat
that referenced
this issue
Feb 17, 2015
On windows, node's path api returns backslashes for path separators. Because the source-map library expects forward slashes in all cases, their relative path logic (specifically their "normalize" function) gives incorrect results when passing in backslash. The mozilla/source-map library won't change this, because they are actually expecting URLs as input - see mozilla/source-map#91 (comment). This fix is similar to the one made for grunt-contrib-uglify: gruntjs/grunt-contrib-uglify#175. This should fix issue gruntjs#110 and gruntjs#95.
riaann
added a commit
to riaann/grunt-contrib-concat
that referenced
this issue
Feb 18, 2015
On windows, node's path api returns backslashes for path separators. Because the source-map library expects forward slashes in all cases, their relative path logic (specifically their "normalize" function) gives incorrect results when passing in backslash. The mozilla/source-map library won't change this, because they are actually expecting URLs as input - see mozilla/source-map#91 (comment). This fix is similar to the one made for grunt-contrib-uglify: gruntjs/grunt-contrib-uglify#175. Standardized line endings for test fixtures by enforcing LF end of files for fixtures directory (through repository .gitattributes). This change was needed because the expected output of source maps are based on input files with LF, rather than CRLF. Before this change the tests were breaking due to git autocrlf on Windows. The difference between the source maps generated from input files using CRLF and the source maps generated from input files using only LF can't be normalized in the test, due to the nature of the changes: the offsets in the source map are changed, as well as the embedded source. These kinds of changes aren't as simple to normalize as just line feeds in output files - and such normalization would be too invasive, to the point of making the test less effective. This should fix issue gruntjs#110 and gruntjs#95 and all unit tests should now pass.
|
Duplicate of #95. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Under windows,
sourcemap.jsproduces:sources:[ "..\\main.js"].In this case, devtools cannot map it to the correct file. It could be the
path.relativealways returns backslash in windows.The text was updated successfully, but these errors were encountered: