Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix source map paths in Uglify configuration #1499

Merged
merged 1 commit into from
Feb 17, 2023

Conversation

joshkel
Copy link
Contributor

@joshkel joshkel commented Feb 16, 2023

Source maps were written with "0" as the source filename. This could result in errors similar to the following in Webpack (depending on the Webpack configuration):

WARNING in ../../node_modules/jsoneditor/dist/jsoneditor.min.js
Module Warning (from ../../node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from '/Users/josh/app/node_modules/jsoneditor/dist/0' file: Error: ENOENT: no such file or directory, open '/Users/josh/app/node_modules/jsoneditor/dist/0'

Uglify configuration based on https://github.com/mishoo/UglifyJS#api-reference.

Source maps were written with `"0"` as the source filename.  This could
result in errors similar to the following in Webpack (depending on the
Webpack configuration):

```
WARNING in ../../node_modules/jsoneditor/dist/jsoneditor.min.js
Module Warning (from ../../node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from '/Users/josh/app/node_modules/jsoneditor/dist/0' file: Error: ENOENT: no such file or directory, open '/Users/josh/app/node_modules/jsoneditor/dist/0'
```
@josdejong
Copy link
Owner

josdejong commented Feb 17, 2023

Thanks! So this will fix #1208, right?

@josdejong josdejong merged commit 56b7494 into josdejong:develop Feb 17, 2023
@joshkel
Copy link
Contributor Author

joshkel commented Feb 17, 2023

@josdejong, yes, this should fix #1208.

@joshkel joshkel deleted the source-map-fix branch February 17, 2023 14:17
@josdejong
Copy link
Owner

Awesome 😎

@josdejong
Copy link
Owner

Fix published now in v9.10.0, thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants