-
Notifications
You must be signed in to change notification settings - Fork 13.3k
rustc_llvm: Link to libkstat on Solaris/SPARC #114224
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
rustc_llvm: Link to libkstat on Solaris/SPARC #114224
Conversation
getHostCPUName calls into libkstat but as of LLVM 16.0.6 libLLVMTargetParser is not explicitly linked against libkstat causing builds to fail due to undefined symbols. See also: llvm/llvm-project#64186
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @cuviper (or someone else) soon. Please see the contribution instructions for more information. Namely, in order to ensure the minimum review times lag, PR authors and assigned reviewers should ensure that the review label (
|
Switching to waiting on author to incorporate changes. Feel free to request a review with @rustbot author |
@inferiorhumanorgans any updates on this? |
Short version: Sorry, no. If time is an issue, this would be a decent opportunity for someone else to pick it up and refine the PR. Long version: The continued enshittification of Github (mostly) and the resource intensive nature of Rust (secondarily) really did a number on my motivation to iterate on these PRs so these PRs moved to the back burner. By the time you replied my daily driver's SSD died and I'm currently in the process of getting a new machine set up. I know that there's a lot of dancing around how big the build artifacts are with rust programs, but combined with having to configure set up and configure a suite of VMs it's a big pain point and part of why they didn't get backed up. As for Github, I'll try not to belabor the point, but I've moved what I can to Sourcehut and I wish that Rust would standardize on something more open source and user friendly. Contributing to open source projects is a passion thing, I'm not getting paid to do this. So part of the attraction to rust is that the tooling and DX are that good. Github stands in stark contrast. As it stands I'm typing this on a brand new MacBook Pro with the most recent version of Firefox and watching the text input widget and everything below it bounce up and down as I type. I would've replied sooner but not even pages of pure static text on Github load anymore on anything without a very recent javascript interpreter. |
OK, I only wanted a comment added -- I'll do that myself when I get a chance. |
@bors r+ rollup |
…iaskrgr Rollup of 4 pull requests Successful merges: - rust-lang#114224 (rustc_llvm: Link to libkstat on Solaris/SPARC) - rust-lang#117695 (Reorder checks to make sure potential missing expect on Option/Result…) - rust-lang#117870 (`fn args_ref_X` to `fn args_X`) - rust-lang#117879 (tests: update check for inferred nneg on zext) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of rust-lang#114224 - inferiorhumanorgans:solaris-llvm-wrapper, r=cuviper rustc_llvm: Link to libkstat on Solaris/SPARC getHostCPUName calls into libkstat but as of LLVM 16.0.6 libLLVMTargetParser is not explicitly linked against libkstat causing builds to fail due to undefined symbols. See also: llvm/llvm-project#64186
@cuviper Ah, I saw emails for few related PRs, so that was meant as a catchall instead of spamming each issue. With some luck I'll have more time to deal with GH and get Solaris and dfbsd envs setup next year. |
getHostCPUName calls into libkstat but as of
LLVM 16.0.6 libLLVMTargetParser is not explicitly
linked against libkstat causing builds to fail
due to undefined symbols.
See also: llvm/llvm-project#64186