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.

Sign up for GitHub

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

Jump to bottom

build-wasm-atomics CI check failing #14662

Closed
BD103 opened this issue Aug 8, 2024 · 3 comments
Closed

build-wasm-atomics CI check failing #14662

BD103 opened this issue Aug 8, 2024 · 3 comments
Labels
A-Build-System Related to build systems or continuous integration C-Bug An unexpected or incorrect behavior D-Complex Quite challenging from either a design or technical perspective. Ask for help! D-Domain-Expert Requires deep knowledge in a given domain P-Regression Functionality that used to work but no longer does. Add a test for this! S-Blocked This cannot move forward until something else changes

Comments

@BD103
Copy link
Member

BD103 commented Aug 8, 2024

The build-wasm-atomics check has started failing CI.

  Downloading crates ...
  Downloaded cc v1.1.8
  Downloaded serde_derive v1.0.205
  Downloaded serde v1.0.205
error: "/home/runner/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/Cargo.lock" does not exist, unable to build with the standard library, try:
        rustup component add rust-src --toolchain nightly-x86_64-unknown-linux-gnu
Error: Process completed with exit code 101.

This is the run that first had this issue. I believe the cause was updating from Rust nightly 60d146580 2024-08-06 to 8b3870784 2024-08-07.

I'm going to do further investigation to find which Rust commit is responsible.
@BD103 BD103 added C-Bug An unexpected or incorrect behavior A-Build-System Related to build systems or continuous integration P-Regression Functionality that used to work but no longer does. Add a test for this! S-Needs-Investigation This issue requires detective work to figure out what's going wrong D-Complex Quite challenging from either a design or technical perspective. Ask for help! D-Domain-Expert Requires deep knowledge in a given domain labels Aug 8, 2024
@BD103 BD103 mentioned this issue Aug 8, 2024
Closed
@BD103
Copy link
Member Author

BD103 commented Aug 8, 2024

I opened up rust-lang/rust#128840 with the bisection. rust-lang/cargo#14370 should fix this, which will be merged in rust-lang/rust#128827.

Ideally, next nightly will include the fix. I'm going to wait until tomorrow and see if CI starts passing.

@BD103 BD103 added S-Blocked This cannot move forward until something else changes and removed S-Needs-Investigation This issue requires detective work to figure out what's going wrong labels Aug 8, 2024
@BD103 BD103 mentioned this issue Aug 8, 2024
Closed
@BD103
Copy link
Member Author

BD103 commented Aug 9, 2024

Looks like we'll need to wait one more day before this is released.

@mockersf
Copy link
Member

back to working now. thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Build-System Related to build systems or continuous integration C-Bug An unexpected or incorrect behavior D-Complex Quite challenging from either a design or technical perspective. Ask for help! D-Domain-Expert Requires deep knowledge in a given domain P-Regression Functionality that used to work but no longer does. Add a test for this! S-Blocked This cannot move forward until something else changes
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mockersf @BD103