Skip to content

fix linking error on i586 #45991

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Nov 17, 2017
Merged

fix linking error on i586 #45991

merged 1 commit into from
Nov 17, 2017

Conversation

gnzlbg
Copy link
Contributor

@gnzlbg gnzlbg commented Nov 14, 2017

Try to fix this linking error on i586 in cross:

https://travis-ci.org/japaric/cross/builds/302095949#L8670

The problem is that std is built in Ubuntu 16.04 and cross uses a linker from 12.04.

Currently this fix solves the problem for i686-musl making it "supercompatible", this PR applies the fix to i586 as well.

The cross PR is here: cross-rs/cross#157

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Nov 14, 2017

r? @alexcrichton

@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 14, 2017
@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented Nov 14, 2017

📌 Commit 5146663 has been approved by alexcrichton

@kennytm kennytm 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-review Status: Awaiting review from the assignee but also interested parties. labels Nov 14, 2017
@bors
Copy link
Collaborator

bors commented Nov 17, 2017

⌛ Testing commit 5146663 with merge 3bcb00d...

bors added a commit that referenced this pull request Nov 17, 2017
fix linking error on i586

Try to fix this linking error on i586 in cross:

https://travis-ci.org/japaric/cross/builds/302095949#L8670

The problem is that `std` is built in Ubuntu 16.04 and `cross` uses a linker from 12.04.

Currently this fix solves the problem for `i686-musl`  making it "supercompatible", this PR applies the fix to `i586` as well.

The cross PR is here: cross-rs/cross#157
@bors
Copy link
Collaborator

bors commented Nov 17, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 3bcb00d to master...

@bors bors merged commit 5146663 into rust-lang:master Nov 17, 2017
# for free to join this conversation on GitHub. Already have an account? # 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.

4 participants