Run the FreeBSD tests on GitHub Actions instead of Cirrus - #1683
Merged
Conversation
This comment has been minimized.
This comment has been minimized.
bjorn3
requested changes
Aug 4, 2026
Contributor
|
Raised a fix to hopefully avoid needing the TLS workaround. rust-lang/rust#160522. Though I do want to note in my local VM testing I didn't encounter this error in FreeBSD 15, while I did in 13 and 14. |
neilpang
force-pushed
the
ci/freebsd-vmactions
branch
from
August 5, 2026 02:08
bef180b to
392bf49
Compare
Contributor
Author
|
All three done.
On the timeout: this run took 15m25s, and the two before it 13m32s and @clubby789 I left LD_STATIC_TLS_EXTRA alone since the Readme documents it |
bjorn3
reviewed
Aug 5, 2026
Cirrus CI shut down on 2026-06-01, which removed the FreeBSD coverage. Run the same commands in a FreeBSD VM booted under QEMU/KVM on a regular ubuntu-latest runner, keeping both workarounds the Cirrus task carried. Also removes .cirrus.yml, which no longer runs anywhere.
neilpang
force-pushed
the
ci/freebsd-vmactions
branch
from
August 5, 2026 10:19
392bf49 to
eb75f0c
Compare
Contributor
Author
|
Removed. rustc.yml and cranelift-release-branch.yml do not set it either, |
Member
|
Thanks! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #1640
Cirrus CI shut down on 2026-06-01, which removed the FreeBSD coverage. This
runs the same commands in a FreeBSD VM booted under QEMU/KVM on a regular
ubuntu-latestrunner.Both workarounds the Cirrus task carried are kept verbatim:
LD_STATIC_TLS_EXTRA=4096for Dependency of rustc_fluent_macros getting compiled with -Ztls-model=initial-exec rust#134863--skip-test test.rust-random/randfor rust-random/rand#1355Verification
https://github.com/neilpang/rustc_codegen_cranelift/actions/runs/30730256715
cache-after-prepareskips the rustup install on subsequent runs.