Skip to content

Run no_core rustdoc tests only on Linux - #89171

Merged
bors merged 1 commit into
rust-lang:masterfrom
rusticstuff:aarch64_macos_rustdoc_ci_fixes
Sep 24, 2021
Merged

Run no_core rustdoc tests only on Linux#89171
bors merged 1 commit into
rust-lang:masterfrom
rusticstuff:aarch64_macos_rustdoc_ci_fixes

Conversation

@hkratz

Copy link
Copy Markdown
Contributor

I don't think this is really worth investigating further so I just disabled them.

They currently fail like this:

error: linking with `cc` failed: exit status: 1
|
= note: "cc" "-Wl,-exported_symbols_list,/var/folders/pg/gjhzdz7n361f0pv665d0723w0000gn/T/rustcPUsMpx/list" "-arch" "arm64" "/Users/hans/dev/rust/build/aarch64-apple-darwin/test/rustdoc/intra-doc/prim-methods-external-core/auxiliary/my-core.my_core.9f3c60de-cgu.0.rcgu.o" "/Users/hans/dev/rust/build/aarch64-apple-darwin/test/rustdoc/intra-doc/prim-methods-external-core/auxiliary/my-core.50fu2g9urkmisdsg.rcgu.o" "-L" "/Users/hans/dev/rust/build/aarch64-apple-darwin/native/rust-test-helpers" "-L" "/Users/hans/dev/rust/build/aarch64-apple-darwin/test/rustdoc/intra-doc/prim-methods-external-core/auxiliary" "-L" "/Users/hans/dev/rust/build/aarch64-apple-darwin/stage1/lib/rustlib/aarch64-apple-darwin/lib" "-L" "/Users/hans/dev/rust/build/aarch64-apple-darwin/stage1/lib/rustlib/aarch64-apple-darwin/lib" "-o" "/Users/hans/dev/rust/build/aarch64-apple-darwin/test/rustdoc/intra-doc/prim-methods-external-core/auxiliary/libmy_core.dylib" "-Wl,-dead_strip" "-dynamiclib" "-Wl,-dylib" "-Wl,-install_name" "-Wl,@rpath/libmy_core.dylib" "-nodefaultlibs"
= note: ld: dynamic main executables must link with libSystem.dylib for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

r? @jyn514

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 22, 2021
@jyn514

Copy link
Copy Markdown
Member

I don't really feel like playing wack-a-mole with these tests. Can you just change it to only-linux or something instead?

@jyn514jyn514 added A-testsuite Area: The testsuite used to check the correctness of rustc T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 22, 2021
Windows on Macos ARM64 produce linker errors.
@hkratz
hkratzforce-pushed the aarch64_macos_rustdoc_ci_fixes branch from 3dcf5b9 to da6f01dCompareSeptember 22, 2021 10:30
@hkratz

Copy link
Copy Markdown
ContributorAuthor

I don't really feel like playing wack-a-mole with these tests. Can you just change it to only-linux or something instead?

Done.

@hkratzhkratz changed the title Disable no_core rustdoc tests on aarch64-apple-darwin due to linker errorRun no_core rustdoc tests only on LinuxSep 22, 2021
@jyn514jyn514 added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Sep 22, 2021
@jyn514

Copy link
Copy Markdown
Member

(this is waiting on me running this locally on a Linux host and making sure it actually runs rather than being ignored - if you want to speed it up and have a Linux machine you could do that yourself, otherwise I'll try to get to it by this weekend.)

@hkratz

Copy link
Copy Markdown
ContributorAuthor

(this is waiting on me running this locally on a Linux host and making sure it actually runs rather than being ignored - if you want to speed it up and have a Linux machine you could do that yourself, otherwise I'll try to get to it by this weekend.)

I can confirm that the test now runs on Linux and is ignored on Macos.

Linux:

$ ./x.py test --force-rerun src/test/rustdoc/cross-crate-primitive-doc.rs
[...]
running 1 test
.
test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 469 filtered out; finished in 0.11s

Macos:

$ ./x.py test --force-rerun src/test/rustdoc/cross-crate-primitive-doc.rs
[...]
running 1 test
i
test result: ok. 0 passed; 0 failed; 1 ignored; 0 measured; 469 filtered out; finished in 0.00s

@jyn514

Copy link
Copy Markdown
Member

@bors r+ rollup

Thanks!

@bors

bors commented Sep 23, 2021

Copy link
Copy Markdown
Collaborator

📌 Commit da6f01d has been approved by jyn514

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 23, 2021
workingjubilee added a commit to workingjubilee/rustc that referenced this pull request Sep 23, 2021
…i_fixes, r=jyn514
Run `no_core` rustdoc tests only on Linux
I don't think this is really worth investigating further so I just disabled them.
They currently fail like this:
```
error: linking with `cc` failed: exit status: 1
|
= note: "cc" "-Wl,-exported_symbols_list,/var/folders/pg/gjhzdz7n361f0pv665d0723w0000gn/T/rustcPUsMpx/list" "-arch" "arm64" "/Users/hans/dev/rust/build/aarch64-apple-darwin/test/rustdoc/intra-doc/prim-methods-external-core/auxiliary/my-core.my_core.9f3c60de-cgu.0.rcgu.o" "/Users/hans/dev/rust/build/aarch64-apple-darwin/test/rustdoc/intra-doc/prim-methods-external-core/auxiliary/my-core.50fu2g9urkmisdsg.rcgu.o" "-L" "/Users/hans/dev/rust/build/aarch64-apple-darwin/native/rust-test-helpers" "-L" "/Users/hans/dev/rust/build/aarch64-apple-darwin/test/rustdoc/intra-doc/prim-methods-external-core/auxiliary" "-L" "/Users/hans/dev/rust/build/aarch64-apple-darwin/stage1/lib/rustlib/aarch64-apple-darwin/lib" "-L" "/Users/hans/dev/rust/build/aarch64-apple-darwin/stage1/lib/rustlib/aarch64-apple-darwin/lib" "-o" "/Users/hans/dev/rust/build/aarch64-apple-darwin/test/rustdoc/intra-doc/prim-methods-external-core/auxiliary/libmy_core.dylib" "-Wl,-dead_strip" "-dynamiclib" "-Wl,-dylib" "-Wl,-install_name" "-Wl,`@rpath/libmy_core.dylib"` "-nodefaultlibs"
= note: ld: dynamic main executables must link with libSystem.dylib for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
```
r? `@jyn514`
This was referenced Sep 23, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 24, 2021
…ingjubilee
Rollup of 7 pull requests
Successful merges:
- rust-lang#88612 (Add a better error message for rust-lang#39364)
- rust-lang#89023 (Resolve issue : Somewhat confusing error with extended_key_value_attributes)
- rust-lang#89148 (Suggest `_` in turbofish if param will be inferred from fn argument)
- rust-lang#89171 (Run `no_core` rustdoc tests only on Linux)
- rust-lang#89176 (Change singular to plural)
- rust-lang#89184 (Temporarily rename int_roundings functions to avoid conflicts)
- rust-lang#89200 (Fix typo)
Failed merges:
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit 23efc96 into rust-lang:masterSep 24, 2021
@rustbotrustbot added this to the 1.57.0 milestone Sep 24, 2021
@hkratz
hkratz deleted the aarch64_macos_rustdoc_ci_fixes branch November 6, 2021 20:38
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-testsuiteArea: The testsuite used to check the correctness of rustcS-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-rustdocRelevant to the rustdoc team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@hkratz@jyn514@bors@rust-highfive@rustbot