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

UWP build broken with nightly update #24766

Closed
Manishearth opened this issue Nov 18, 2019 · 7 comments
Closed

UWP build broken with nightly update #24766

Manishearth opened this issue Nov 18, 2019 · 7 comments

Comments

@Manishearth
Copy link
Member

@Manishearth Manishearth commented Nov 18, 2019

warning: Patch `rustc-std-workspace-std v1.99.0 (C:\Users\manis\.rustup\toolchains\nightly-2019-11-16-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\tools\rustc-std-workspace-std)` was not used in the crate graph.

...

   Compiling rustc-std-workspace-alloc v1.99.0 (C:\Users\manis\.rustup\toolchains\nightly-2019-11-16-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\tools\rustc-std-workspace-alloc)
error[E0635]: unknown feature `compiler_builtins_lib`
   --> C:\Users\manis\.rustup\toolchains\nightly-2019-11-16-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\libstd\lib.rs:247:12
    |
247 | #![feature(compiler_builtins_lib)]
    |            ^^^^^^^^^^^^^^^^^^^^^

error: aborting due to previous error
C:\Users\manis\dev\servo>python mach build --uwp --win-arm64 -r
info: component 'rust-src' is up to date
    Updating crates.io index
warning: Patch `rustc-std-workspace-std v1.99.0 (C:\Users\manis\.rustup\toolchains\nightly-2019-11-16-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\tools\rustc-std-workspace-std)` was not used in the crate graph.
Check that the patched package version and available features are compatible
with the dependency requirements. If the patch has a different version from
what is locked in the Cargo.lock file, run `cargo update` to use the new
version. This may also occur with an optional dependency that is not enabled.
   Compiling compiler_builtins v0.1.21
   Compiling core v0.0.0 (C:\Users\manis\.rustup\toolchains\nightly-2019-11-16-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\libcore)
   Compiling cc v1.0.47
   Compiling libc v0.2.65
^Cmach interrupted by signal or user action. Stopping.                  ] 0/28: compiler_builtins(build.rs), core,...

C:\Users\manis\dev\servo>rustup self update
info: checking for self-updates
info: downloading self-update
info: rustup updated successfully to 1.20.2

C:\Users\manis\dev\servo>python mach build --uwp --win-arm64 -r
info: component 'rust-src' is up to date
    Updating crates.io index
warning: Patch `rustc-std-workspace-std v1.99.0 (C:\Users\manis\.rustup\toolchains\nightly-2019-11-16-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\tools\rustc-std-workspace-std)` was not used in the crate graph.
Check that the patched package version and available features are compatible
with the dependency requirements. If the patch has a different version from
what is locked in the Cargo.lock file, run `cargo update` to use the new
version. This may also occur with an optional dependency that is not enabled.
   Compiling compiler_builtins v0.1.21
   Compiling core v0.0.0 (C:\Users\manis\.rustup\toolchains\nightly-2019-11-16-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\libcore)
   Compiling cc v1.0.47
   Compiling libc v0.2.65
   Compiling autocfg v0.1.7
   Compiling std v0.0.0 (C:\Users\manis\.rustup\toolchains\nightly-2019-11-16-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\libstd)
   Compiling hashbrown v0.6.3
   Compiling backtrace-sys v0.1.32
   Compiling unwind v0.0.0 (C:\Users\manis\.rustup\toolchains\nightly-2019-11-16-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\libunwind)
   Compiling rustc-std-workspace-core v1.99.0 (C:\Users\manis\.rustup\toolchains\nightly-2019-11-16-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\tools\rustc-std-workspace-core)
   Compiling cfg-if v0.1.10
   Compiling alloc v0.0.0 (C:\Users\manis\.rustup\toolchains\nightly-2019-11-16-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\liballoc)
   Compiling rustc-demangle v0.1.16
   Compiling panic_abort v0.0.0 (C:\Users\manis\.rustup\toolchains\nightly-2019-11-16-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\libpanic_abort)
   Compiling backtrace v0.3.40
   Compiling rustc-std-workspace-alloc v1.99.0 (C:\Users\manis\.rustup\toolchains\nightly-2019-11-16-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\tools\rustc-std-workspace-alloc)
error[E0635]: unknown feature `compiler_builtins_lib`
   --> C:\Users\manis\.rustup\toolchains\nightly-2019-11-16-x86_64-pc-windows-msvc\lib\rustlib\src\rust\src\libstd\lib.rs:247:12
    |
247 | #![feature(compiler_builtins_lib)]
    |            ^^^^^^^^^^^^^^^^^^^^^

error: aborting due to previous error

For more information about this error, try `rustc --explain E0635`.
error: could not compile `std`.

To learn more, run the command again with --verbose.
error: `"cargo" "build" "--release" "--manifest-path" "C:\\Users\\manis\\AppData\\Local\\Temp\\xargo.knWhSgxFtJnz\\Cargo.toml" "--target" "aarch64-uwp-windows-msvc" "-p" "std"` failed with exit code: Some(101)
note: run with `RUST_BACKTRACE=1` for a backtrace
Build FAILED in 0:00:29

C:\Users\manis\dev\servo>

Seems like #24749 and #24750 updated us to a rustc which doesn't build on UWP, and Servo's UWP build builds rustc locally.

I'm unclear why this didn't fail CI, though.

cc @saschanaz @SimonSapin @jdm

@Manishearth
Copy link
Member Author

@Manishearth Manishearth commented Nov 18, 2019

Hmm, I'm still getting this error when checking out b9cdf9e , which is pre-rustups

@Manishearth
Copy link
Member Author

@Manishearth Manishearth commented Nov 18, 2019

This occurs with a fresh target directory and cleared rustup toolchain directory

@Manishearth
Copy link
Member Author

@Manishearth Manishearth commented Nov 18, 2019

Broken as far back as 18dae9e

Unsure what's going on here, that feature definitely exists.

@Manishearth
Copy link
Member Author

@Manishearth Manishearth commented Nov 18, 2019

Deleted entire servo clone, still getting this error. The compiler being used is the latest pinned nightly (checked via task manager), it's definitely able to find that feature.

@Manishearth
Copy link
Member Author

@Manishearth Manishearth commented Nov 18, 2019

This isn't a servo issue, I can reproduce this with xargo build --target aarch64-uwp-windows-msvc on a new crate with Servo's Xargo.toml

@Manishearth
Copy link
Member Author

@Manishearth Manishearth commented Nov 18, 2019

I can also reproduce this with the same test crate and an older nightly. Unsure what's going on, this obviously used to work.

bors-servo added a commit that referenced this issue Nov 18, 2019
Update xargo

Fixes #24766 (I already built and uploaded a package, and have a local build running using it)

I don't know why this fixes that issue, or why that issue turned up in the first place. But it seems to work!

r? @jdm
@Manishearth
Copy link
Member Author

@Manishearth Manishearth commented Nov 18, 2019

Xargo update seemed to fix it, though I don't have time to investigate why, since I was only trying to start an overnight build for tomorrow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

1 participant
You can’t perform that action at this time.