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

freebsd add subset of libprocstat #2313

Merged
merged 1 commit into from Aug 19, 2021
Merged

freebsd add subset of libprocstat #2313

merged 1 commit into from Aug 19, 2021

Conversation

devnexen
Copy link
Contributor

@devnexen devnexen commented Aug 2, 2021

No description provided.

@rust-highfive
Copy link

r? @JohnTitor

(rust-highfive has picked a reviewer for you, use r? to override)

@JohnTitor
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Aug 3, 2021

📌 Commit 735c993 has been approved by JohnTitor

bors added a commit that referenced this pull request Aug 3, 2021
freebsd add subset of libprocstat
@bors
Copy link
Contributor

bors commented Aug 3, 2021

⌛ Testing commit 735c993 with merge 427dc35...

@bors
Copy link
Contributor

bors commented Aug 3, 2021

💔 Test failed - checks-actions

@bors
Copy link
Contributor

bors commented Aug 16, 2021

☔ The latest upstream changes (presumably #2337) made this pull request unmergeable. Please resolve the merge conflicts.

@JohnTitor
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Aug 19, 2021

📌 Commit be8fc00 has been approved by JohnTitor

@bors
Copy link
Contributor

bors commented Aug 19, 2021

⌛ Testing commit be8fc00 with merge 9247b83...

bors added a commit that referenced this pull request Aug 19, 2021
freebsd add subset of libprocstat
@bors
Copy link
Contributor

bors commented Aug 19, 2021

💔 Test failed - checks-actions

@JohnTitor
Copy link
Member

Seems you messed up a branch by a rebase.

@JohnTitor
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Aug 19, 2021

📌 Commit 250965a has been approved by JohnTitor

@bors
Copy link
Contributor

bors commented Aug 19, 2021

⌛ Testing commit 250965a with merge b82289f...

@bors
Copy link
Contributor

bors commented Aug 19, 2021

☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing b82289f to master...

@bors bors merged commit b82289f into rust-lang:master Aug 19, 2021
@m-ou-se
Copy link
Member

m-ou-se commented Sep 3, 2021

This change broke the freebsd build in the rust-lang/rust CI. See rust-lang/rust#88478 (comment)

/usr/local/x86_64-unknown-freebsd11/bin/ld: cannot find -lprocstat

@lnicola
Copy link
Member

lnicola commented Sep 3, 2021

Maybe libprocstat needs to be installed on the system?

@devnexen
Copy link
Contributor Author

devnexen commented Sep 3, 2021

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

@lnicola
Copy link
Member

lnicola commented Sep 3, 2021

Well, I ran ./src/ci/docker/run.sh dist-x86_64-freebsd on my branch (oh well, I just wanted a shell, not a dist build) and got the same error:

  = note: /usr/local/x86_64-unknown-freebsd11/bin/ld: cannot find -lprocstat
          clang: error: linker command failed with exit code 1 (use -v to see invocation)

So this is a Docker container, not a real FreeBSD install, right? Then it could be missing some libraries.

@devnexen
Copy link
Contributor Author

devnexen commented Sep 3, 2021

It seems ; you did all right in your side.

@m-ou-se
Copy link
Member

m-ou-se commented Sep 3, 2021

cc @rust-lang/infra ^

@pietroalbini
Copy link
Member

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 :(

@devnexen
Copy link
Contributor Author

devnexen commented Sep 5, 2021

@pietroalbini
Copy link
Member

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

@devnexen
Copy link
Contributor Author

devnexen commented Sep 5, 2021

ah right in that case it makes sense.

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

Successfully merging this pull request may close these issues.

None yet

7 participants