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 the build step #5664

Merged
merged 5 commits into from
Jan 31, 2023
Merged

Conversation

elliottt
Copy link
Member

@elliottt elliottt commented Jan 31, 2023

Change the permissions on libwasmtime.a before copying it, to avoid errors stemming from new behavior in rustc-1.67. Mitigates the effects of rust-lang/rust#107495. See #5665.

  • successful build of the Build workflow
  • remove the change that enabled the Build workflow for PRs

@elliottt elliottt changed the title trevor/fix build Debug the build step Jan 31, 2023
@elliottt elliottt marked this pull request as ready for review January 31, 2023 00:33
@elliottt elliottt requested a review from alexcrichton January 31, 2023 00:33
ci/build-tarballs.sh Show resolved Hide resolved
@elliottt elliottt enabled auto-merge (squash) January 31, 2023 02:26
@elliottt elliottt changed the title Debug the build step Fix the build step Jan 31, 2023
@elliottt elliottt merged commit cc768f2 into bytecodealliance:main Jan 31, 2023
alexcrichton added a commit to alexcrichton/wasmtime that referenced this pull request Jan 31, 2023
This fixes the build issue identified in bytecodealliance#5664 at the toolchain level
rather than working around it in our own build. The next step in fixing
this will be to remove the nightly override in the future when the
toolchain becomes stable.
alexcrichton added a commit that referenced this pull request Jan 31, 2023
This fixes the build issue identified in #5664 at the toolchain level
rather than working around it in our own build. The next step in fixing
this will be to remove the nightly override in the future when the
toolchain becomes stable.
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