-p relative not working correctly

[bdkjones]

Gave the new -p relative functionality a go, but things still don't look right. First, here's the file structure for the project:

I first compile "arrays.coffee" into "arrays-out.js" and have CoffeeScript generate a corresponding source map that gets saved to "arrays-out.map". (CoffeeScript is retarded, so I manually write that source map's file paths --- in other words, I know the paths in the source map I get from CoffeeScript are correct because I set them myself.)

Next, I tell Uglify to minify the "arrays-out.js" file and write it back to the exact same spot. I also tell Uglify to create a source map and write that map file to the exact same spot (arrays-out.map). I pass arrays-out.map as the --in-source-map parameter.

I pass -p relative to the Uglify command and I get the following source map file (I've deleted the mappings data for brevity):

There are a few problems:

1. The "file" path is not relative at all. When the relative flag is passed, this path should be relative to the map file's location. In this example, the path should be: "./arrays-out.js" or just "arrays-out.js" because the map file is right next to the output file.

2. The files in the "sources" array are indeed relative to the location of the map file, but Uglify has also included the optional sourceRoot parameter and set sourceRoot equal to: "./../" This is incorrect because the files in the sources array already have that prefix appended to them. Right now, when the browser loads this source map, it's going to take the sourceRoot value and prefix that to every item in the sources array, so it will look for sources at this path: "./.././../arrays.coffee". Obviously, this will break.

The solution is EITHER:

A) the sourceRoot parameter (which is optional according to the source map spec) needs to be omitted if the file paths in the sources array are ALREADY relative to the map file.

OR

B) if a sourceRoot path is included, the file paths in the sources array must be set relative to the SOURCE ROOT folder, not the *.map file itself.

Hopefully that all makes sense!