-
Notifications
You must be signed in to change notification settings - Fork 409
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
0.10.2 Regression: Windows installer fails due to malformatted tar #1097
Comments
This is causing our GitHub Actions CI to fail on Windows with the following:
Just putting it here in case someone searches for that error message. Downgrading wasm-pack to 0.10.1 fixed our CI problem. |
Current workaround to downgrade to 0.10.1 (OP has been updated to include this): -curl https://rustwasm.github.io/wasm-pack/installer/init.sh -sSf | sh
+npm install -g wasm-pack@0.10.1 As for the root cause, maybe there is a bug in 7zip? I see it is used to create the tarball, and verified reported size matches download size. 7zip can extract the generated tarball, but it prints a warning, so if 0.10.2 is necessary, one can manually download the windows tarball and extract with 7zip while ignoring warnings. |
because wasm-pack 0.10.2 is broken rustwasm/wasm-pack#1097
Would it be possible to just push out a new release to see if the bug happens again? How can we help solve this going forward? |
@drager just checking if there's a possible timeline to this getting resolved. We're having to pin to the previous version right now, but that old one is broken for Apple Silicon users. |
@drager Ping. This is a pretty significant issue that's blocking usage of latest wasm-pack on Windows, and was open for 5 months. Any chance of making another release? |
0.10.0 was broken on Windows, and 0.10.2 is broken on Windows... rustwasm/wasm-pack#1097
Sorry for my late response and acting here. Got limited time recently... I will take a look this weekend and try to create a new release! |
@drager Totally get it. Thanks! |
@drager I'm guessing it didn't work out 😅 |
No, sorry 😞 Will take a look soon! |
Hi, can confirm this is still an issue :) |
I just opened a pull request with a very simple fix: #1144 |
Released in 0.10.3. |
Thanks @drager! And just checking, these failing CI tests are not a problem? |
No, I just merged some new PRs after the release was made that caused some tests to fail. I.e only in latest master. |
🐛 Bug description
The above outputs:
🤔 Expected Behavior
The tar should be able to be extracted like v0.10.1
Workaround
Current workaround to downgrade to 0.10.1:
The text was updated successfully, but these errors were encountered: