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

0.10.2 Regression: Windows installer fails due to malformatted tar #1097

Closed
nickbabcock opened this issue Dec 19, 2021 · 14 comments · Fixed by #1144
Closed

0.10.2 Regression: Windows installer fails due to malformatted tar #1097

nickbabcock opened this issue Dec 19, 2021 · 14 comments · Fixed by #1144

Comments

@nickbabcock
Copy link

nickbabcock commented Dec 19, 2021

🐛 Bug description

curl -L "https://github.com/rustwasm/wasm-pack/releases/download/v0.10.2/wasm-pack-v0.10.2-x86_64-pc-windows-msvc.tar.gz" | \
  tar -xz

The above outputs:

tar: Skipping to next header
tar: A lone zero block at 12975
tar: Exiting with failure status due to previous errors

🤔 Expected Behavior

The tar should be able to be extracted like v0.10.1

curl -L "https://github.com/rustwasm/wasm-pack/releases/download/v0.10.1/wasm-pack-v0.10.1-x86_64-pc-windows-msvc.tar.gz" | \
  tar -xz

Workaround

Current workaround to downgrade to 0.10.1:

-curl https://rustwasm.github.io/wasm-pack/installer/init.sh -sSf | sh 
+npm install -g wasm-pack@0.10.1
@tmcguire
Copy link

This is causing our GitHub Actions CI to fail on Windows with the following:

/c/Rust/.cargo/bin/wasm-pack: cannot execute binary file: Exec format error

Just putting it here in case someone searches for that error message.

Downgrading wasm-pack to 0.10.1 fixed our CI problem.

@nickbabcock
Copy link
Author

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.

@magcius
Copy link

magcius commented Feb 23, 2022

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?

@Keavon
Copy link

Keavon commented Apr 3, 2022

@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.

@RReverser
Copy link
Member

@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?

RReverser added a commit to GoogleChromeLabs/wasm-bindgen-rayon that referenced this issue May 7, 2022
0.10.0 was broken on Windows, and 0.10.2 is broken on Windows... rustwasm/wasm-pack#1097
@drager
Copy link
Member

drager commented May 11, 2022

@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?

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!

@RReverser
Copy link
Member

@drager Totally get it. Thanks!

@RReverser
Copy link
Member

@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?

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 I'm guessing it didn't work out 😅

@drager
Copy link
Member

drager commented May 18, 2022

@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?

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 I'm guessing it didn't work out 😅

No, sorry 😞 Will take a look soon!

@ConnorJamesLow
Copy link

Hi, can confirm this is still an issue :)

@nasso
Copy link
Contributor

nasso commented Jun 9, 2022

I just opened a pull request with a very simple fix: #1144

@drager
Copy link
Member

drager commented Jun 13, 2022

Released in 0.10.3.

@Keavon
Copy link

Keavon commented Jun 13, 2022

Thanks @drager! And just checking, these failing CI tests are not a problem?

@drager
Copy link
Member

drager commented Jun 13, 2022

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.

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 a pull request may close this issue.

8 participants