Skip to content

travis: Compile a more compatible libc.a for musl #40019

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 2 commits into from
Feb 25, 2017

Conversation

alexcrichton
Copy link
Member

The mitigations for #34978 involve passing -Wa,-mrelax-relocations=no to all C
code we compile, and we just forgot to pass it when compiling musl itself.

Closes #39979

The mitigations for rust-lang#34978 involve passing `-Wa,-mrelax-relocations=no` to all C
code we compile, and we just forgot to pass it when compiling musl itself.

Closes rust-lang#39979
This doesn't belong in rustbuild itself, and now that we have only rustbuild we
can move this out of the build system.
@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@brson
Copy link
Contributor

brson commented Feb 21, 2017

@bors r+

@bors
Copy link
Collaborator

bors commented Feb 21, 2017

📌 Commit 9a08f40 has been approved by brson

@nikomatsakis
Copy link
Contributor

r? @brson

@rust-highfive rust-highfive assigned brson and unassigned nikomatsakis Feb 22, 2017
@BurntSushi
Copy link
Member

@alexcrichton Did this make it into the 2017-02-23 nightly? I think I'm still seeing failures: https://travis-ci.org/BurntSushi/ripgrep/jobs/204969471 --- Should I open a new bug?

@alexcrichton
Copy link
Member Author

@BurntSushi this PR hasn't landed yet?

eddyb added a commit to eddyb/rust that referenced this pull request Feb 25, 2017
travis: Compile a more compatible libc.a for musl

The mitigations for rust-lang#34978 involve passing `-Wa,-mrelax-relocations=no` to all C
code we compile, and we just forgot to pass it when compiling musl itself.

Closes rust-lang#39979
eddyb added a commit to eddyb/rust that referenced this pull request Feb 25, 2017
travis: Compile a more compatible libc.a for musl

The mitigations for rust-lang#34978 involve passing `-Wa,-mrelax-relocations=no` to all C
code we compile, and we just forgot to pass it when compiling musl itself.

Closes rust-lang#39979
bors added a commit that referenced this pull request Feb 25, 2017
@bors bors merged commit 9a08f40 into rust-lang:master Feb 25, 2017
@alexcrichton alexcrichton deleted the fix-musl branch March 8, 2017 16:17
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants