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

Fix Rustbuild linking on Illumos #43104

Merged
merged 1 commit into from
Jul 8, 2017

Conversation

nbaksalyar
Copy link
Contributor

Illumos (an OpenSolaris fork) expects to get several extra library references for some system functions used by Rust standard library. This commit adds required linker options to rustbuild, which is currently doesn't work on Illumos-based operating systems.

Illumos (an OpenSolaris fork) expects to get several
extra library references for some system functions used
by Rust standard library. This commit adds required linker
options to rustbuild, which is currently doesn't work on
Illumos-based operating systems.
@rust-highfive
Copy link
Collaborator

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@sanxiyn
Copy link
Member

sanxiyn commented Jul 7, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Jul 7, 2017

📌 Commit e084bb2 has been approved by sanxiyn

@shepmaster shepmaster added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jul 7, 2017
@bors
Copy link
Contributor

bors commented Jul 8, 2017

⌛ Testing commit e084bb2 with merge 35e8ec787dcefd9656195890a84fedd80d4f9759...

@bors
Copy link
Contributor

bors commented Jul 8, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: sanxiyn
Pushing 35e8ec787dcefd9656195890a84fedd80d4f9759 to master...

@bors
Copy link
Contributor

bors commented Jul 8, 2017

👀 Test was successful, but fast-forwarding failed: 422 Update is not a fast forward

@Mark-Simulacrum
Copy link
Member

Hm... I haven't seen that before, but I guess:

@bors retry

@bors
Copy link
Contributor

bors commented Jul 8, 2017

⌛ Testing commit e084bb2 with merge ac1b675...

bors added a commit that referenced this pull request Jul 8, 2017
Fix Rustbuild linking on Illumos

Illumos (an OpenSolaris fork) expects to get several extra library references for some system functions used by Rust standard library. This commit adds required linker options to rustbuild, which is currently doesn't work on Illumos-based operating systems.
@bors
Copy link
Contributor

bors commented Jul 8, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: sanxiyn
Pushing ac1b675 to master...

@bors bors merged commit e084bb2 into rust-lang:master Jul 8, 2017
@nbaksalyar nbaksalyar deleted the rustbuild-illumos-fix branch July 8, 2017 21:06
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants