process.cwd() + package.json should be dynamic about webpack-asset-relocator-loader HOT 5 CLOSED

Is this because the contents of the cwd of the build are not the same as the contents of the cwd of the runtime? If so, perhaps set the cwd option to point to the correct folder? The package.json should be emitted strictly speaking as that's what the asset emitter does.

We could possibly have a cwd: false option to disable cwd-based emissions.

from webpack-asset-relocator-loader.

Thanks you for the information about cwd: false. That is exactly what i want.

IMHO, process.cwd() should be not treated as static in most cases, while static is a feature. i.e. cwd: false as default.

from webpack-asset-relocator-loader.

I'm open to switching the default.

from webpack-asset-relocator-loader.

IMHO, process.cwd() should be not treated as static in most cases

I think I agree. I am using ncc in the context of CI/CD (Github Actions), and I need to be able to compile scripts locally and run them on various remote machines. But if I include a dependency that uses process.cwd, then the scripts I compile locally won't work on the remote runners, because they'll include filepaths on my local machine.

from webpack-asset-relocator-loader.

@roryabraham the safe route here is always to set the cwd option. A PR to improve the default would be welcome too.

from webpack-asset-relocator-loader.