-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Migrate cdylib-dylib-linkage
run-make
test to rmake
#128245
Conversation
This comment has been minimized.
This comment has been minimized.
I'll take a look at this (eventually™), I'm looking at some other PRs that also need some holy water. |
uh... does the sysroot path have a newline?
|
d0ff2cd
to
648ce51
Compare
The run-make-support library was changed cc @jieyouxu This PR modifies cc @jieyouxu |
@bors try |
Migrate `cdylib-dylib-linkage` `run-make` test to rmake Part of rust-lang#121876 and the associated [Google Summer of Code project](https://blog.rust-lang.org/2024/05/01/gsoc-2024-selected-projects.html). ~~Those sysroot tests are always fun. I'm getting local errors that don't make a lot of sense about my own sysroot not existing, so I am trying this in CI to see what happens.~~ ~~EDIT: I am getting the same error here. The strange thing is, when I try to navigate to `/checkout/obj/build/x86_64-unknown-linux-gnu/stage2/lib/rustlib/x86_64-unknown-linux-gnu/lib` on my personal computer, the directory does exist, but the error message is that the directory does not.~~ EDIT 2: The sysroot path just needed to be trimmed! Please try: try-job: x86_64-msvc
☀️ Try build successful - checks-actions |
@bors try |
Migrate `cdylib-dylib-linkage` `run-make` test to rmake Part of rust-lang#121876 and the associated [Google Summer of Code project](https://blog.rust-lang.org/2024/05/01/gsoc-2024-selected-projects.html). ~~Those sysroot tests are always fun. I'm getting local errors that don't make a lot of sense about my own sysroot not existing, so I am trying this in CI to see what happens.~~ ~~EDIT: I am getting the same error here. The strange thing is, when I try to navigate to `/checkout/obj/build/x86_64-unknown-linux-gnu/stage2/lib/rustlib/x86_64-unknown-linux-gnu/lib` on my personal computer, the directory does exist, but the error message is that the directory does not.~~ EDIT 2: The sysroot path just needed to be trimmed! Please try: // try-job: x86_64-msvc // passed previously try-job: x86_64-mingw try-job: x86_64-gnu-llvm-18 try-job: i686-msvc try-job: aarch64-apple
This comment has been minimized.
This comment has been minimized.
💔 Test failed - checks-actions |
648ce51
to
3c6d739
Compare
The dynamic library std filter was incorrect - it was only looking for files that are "not an rlib, with std in the name somewhere", which could include rmeta files. This new implementation explicitly looks for dylibs with std in their filename. Please try again. @rustbot review |
@bors try |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, just a few suggestions
@rustbot author |
3c6d739
to
8f599f3
Compare
@rustbot review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, a few more nits then r=me.
@bors delegate+ |
✌️ @Oneirical, you can now approve this pull request! If @jieyouxu told you to " |
@bors rollup=iffy |
@rustbot author |
☔ The latest upstream changes (presumably #128356) made this pull request unmergeable. Please resolve the merge conflicts. |
8f599f3
to
92f5216
Compare
92f5216
to
20332da
Compare
☀️ Test successful - checks-actions |
Finished benchmarking commit (60d1465): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results (primary 2.8%)This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesThis benchmark run did not return any relevant results for this metric. Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 760.454s -> 760.99s (0.07%) |
Part of #121876 and the associated Google Summer of Code project.
Those sysroot tests are always fun. I'm getting local errors that don't make a lot of sense about my own sysroot not existing, so I am trying this in CI to see what happens.EDIT: I am getting the same error here. The strange thing is, when I try to navigate to/checkout/obj/build/x86_64-unknown-linux-gnu/stage2/lib/rustlib/x86_64-unknown-linux-gnu/lib
on my personal computer, the directory does exist, but the error message is that the directory does not.EDIT 2: The sysroot path just needed to be trimmed!
Please try:
// try-job: x86_64-msvc // passed previously
try-job: x86_64-mingw
try-job: x86_64-gnu-llvm-18
try-job: i686-msvc
try-job: aarch64-apple