Incorrect path resolving on Windows with multiple disks

[itoys]

Hello!
When executable code and sources located on separate disk(i.e C:/ and D:/) method
require('source-map-resolve").resolveSync works incorrect.
It strips off disk letter(urix package) and after when it trying read from disk, it catch error.
For example:
server running on disk C:, sources located on disk D:

let filePath = 'D:\\projects\\test\\index.js';
sourceMapResolve.resolveSync(someCode, filePath, fs.readFileSync);

I have error:
ENOENT: no such file or directory, open 'C:\projects\test\build\index.js.map'
Because map file located on disk D:

[lydell]

Could you show input, actual output and expected output?

[lydell]

Idea: Maybe source-map-resolve shouldn’t try to be smart with URL-to-path conversion at all, and leave it completely up to the read parameter to do any necessary conversions. Then it’ll be up to the users of source-map-resolve to use whatever is needed for their use case.

[jens-duttke]

I have same issue, is there a chance to get that fixed?

Maybe source-map-resolve shouldn’t try to be smart with URL-to-path conversion at all, and leave it completely up to the read parameter to do any necessary conversions

For my usecase, that would be the perfect solution.

[lydell]

@jens-duttke I really dislike this package, so don’t expect anything. Try to use a different dependency if you can. I might accept a PR, but last time I said that I ended up merging it to another branch then left it hanging 😢 #20

[jens-duttke]

@jens-duttke I really dislike this package, so don’t expect anything. Try to use a different dependency if you can. I might accept a PR, but last time I said that I ended up merging it to another branch then left it hanging 😢 #20

Do you have any recommendations? I tried different packages, but this one works best and is easy to use.

The workaround I've implemented in my code now, is:

const result = sourceMapResolve.resolveSync(code, codePath, (sourceMapFilePath) => {
	if ((/^[a-z]:/ui).test(codePath)) {
		sourceMapFilePath= `${codePath.substr(0, 2)}${sourceMapFilePath}`;
	}

	return compiler.outputFileSystem.readFileSync(sourceMapFilePath, 'utf8');
});

Anyhow, it would be good to get rid of this hack.