-
Notifications
You must be signed in to change notification settings - Fork 1.1k
freebsd add subset of libprocstat #2313
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
Conversation
r? @JohnTitor (rust-highfive has picked a reviewer for you, use r? to override) |
@bors r+ |
📌 Commit 735c993 has been approved by |
💔 Test failed - checks-actions |
☔ The latest upstream changes (presumably #2337) made this pull request unmergeable. Please resolve the merge conflicts. |
@bors r+ |
📌 Commit be8fc00 has been approved by |
freebsd add subset of libprocstat
💔 Test failed - checks-actions |
Seems you messed up a branch by a rebase. |
@bors r+ |
📌 Commit 250965a has been approved by |
☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13 |
This change broke the freebsd build in the
|
Maybe |
normally freebsd comes with since it s a system library not a third party package one and even looking at the git repo in the 11.0 release era it was already that way https://github.com/freebsd/freebsd-src/tree/releng/11.0/lib |
Well, I ran
So this is a Docker container, not a real FreeBSD install, right? Then it could be missing some libraries. |
It seems ; you did all right in your side. |
cc @rust-lang/infra ^ |
We're cross-compiling FreeBSD from a Linux container, yes. https://github.com/rust-lang/rust/blob/master/src/ci/docker/host-x86_64/dist-x86_64-freebsd/Dockerfile would need to be updated to make sure the library is present, but I'm not that familiar with FreeBSD cross-compilation so can't give more accurate pointers :( |
would it be possible to do similarly as here instead ? https://github.com/uutils/coreutils/blob/2bd556e252fd562777cabe052a2eb811d44c3167/.github/workflows/CICD.yml#L534 |
That's not really feasible with our current CI configuration: we're using cross-compilation for most platforms (everything but Windows and macOS) to standardize our CI as much as possible, and running FreeBSD in a VM would add too much complexity. I took a quick look at the Docker image, and it seems like the relevant libraries are extracted here, maybe try to include the missing library? https://github.com/rust-lang/rust/blob/e2750baf53aaa60db95f10759f6cf9463dc5a6bd/src/ci/docker/scripts/freebsd-toolchain.sh#L52-L58 |
ah right in that case it makes sense. |
No description provided.