-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
New installs of Gatsby fail on builds with source-map error #15249
Comments
Perhaps related to this webpack-contrib/babel-minify-webpack-plugin#68 |
I can confirm this as well. I just saw this issue about 10 minutes ago and came here to check if anyone else was too. I just upgraded to 2.11.1 before that build. Perhaps there is something related there? |
actually i just reverted back to 2.8.5 and am still having the same error, so probably not that. |
Started happening today in vue-cli also it looks like: vuejs/vue-cli#4210 I imagine its a dependency issue somewhere down the tree. Might be worth cross referencing pack-lock.json files of gatsby-cli and vue-cli, match commonalities in packages and versions, then find most recently updated on the list and triage from there? |
Seems a bug in terser 4.0.1 |
I thought I was losing my mind. |
Happened to me while presenting 😂 |
Here's a workaround, if you use
|
@szimek Didn't change anything for me |
@KyleAMathews @szimek seems it this has been resolved? See this sandbox link, build runs with no errors. |
@joeynimu This sandbox uses terser 4.0.0. You can check it using @iRoachie I tried it with a new project and it fixed this issue for me. It's possible that you have other library that has You can try this to force terser 4.0.0 for all libraries (if I understand correctly how resolutions work in yarn):
|
Hey @joeynimu that codesandbox is using @szimek I have It's not really clear on where this should go either:
|
@spences10 As @sodatea mentioned before, this issue is caused by The When a new Gatsby project is generated, it uses And once again - one can check which version(s) of |
@szimek thanks, I was just looking at the terser error when I saw you referenced this issue there 😁 Ok, so I added the additional yarn gatsby clean
rm -rf node_modules/
yarn
yarn build And it's built ok this time, I'm going to push it up to netlify and see if it still doesn't break. Thanks |
I see. I did a Note The sandbox is the default |
Temporary fix published in |
Somehow I can only get v2.10.0 working with the resolution pinning terser to v4.0.0. And v2.13.1 works neither with the resolution nor without. So I will have to wait for a working version. |
I also had this issue. At first I just did a webpack workaround in gatsby-node.js, but when I added the gatsby-plugin-netlify-cms, my build broke. And I am using npm, and had no intention of switching to yarn just because of the source maps issue. So I ended up doing the npm version of resolutions. Note: I actually tried the latest version of terser (at least last night), so am using 4.1.2. And it worked on my build. At least locally. We'll see about what happens when I push to Netlify. |
original.line and original.column are not numbers -- you probably meant to omit the original mapping entirely and only map the generated position. If so, pass null for the original mapping instead of an object with empty or null values.
It seems to just have started happening ~8-12 hours ago.
Neither webpack, webpack-sources, nor source-map have had new releases.
The text was updated successfully, but these errors were encountered: