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

rustc_llvm: Link to libkstat on Solaris/SPARC #114224

Merged
merged 2 commits into from
Nov 13, 2023

Conversation

inferiorhumanorgans
Copy link

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

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
@rustbot
Copy link
Collaborator

rustbot commented Jul 29, 2023

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 (S-waiting-on-review and S-waiting-on-author) stays updated, invoking these commands when appropriate:

  • @rustbot author: the review is finished, PR author should check the comments and take action accordingly
  • @rustbot review: the author is ready for a review, this PR will be queued again in the reviewer's queue

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jul 29, 2023
@apiraino
Copy link
Contributor

apiraino commented Oct 5, 2023

Switching to waiting on author to incorporate changes. Feel free to request a review with @rustbot ready, thanks!

@rustbot author

@apiraino apiraino added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 5, 2023
@Dylan-DPC
Copy link
Member

@inferiorhumanorgans any updates on this?

@inferiorhumanorgans
Copy link
Author

@Dylan-DPC

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.

@cuviper
Copy link
Member

cuviper commented Nov 11, 2023

OK, I only wanted a comment added -- I'll do that myself when I get a chance.

@cuviper
Copy link
Member

cuviper commented Nov 13, 2023

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Nov 13, 2023

📌 Commit 855388e has been approved by cuviper

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Nov 13, 2023
bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 13, 2023
…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
@bors bors merged commit 03d6e7a into rust-lang:master Nov 13, 2023
11 checks passed
@rustbot rustbot added this to the 1.76.0 milestone Nov 13, 2023
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Nov 13, 2023
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
@inferiorhumanorgans
Copy link
Author

@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants