-
Notifications
You must be signed in to change notification settings - Fork 13k
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
fixes #27124 for bitrig #27137
fixes #27124 for bitrig #27137
Conversation
r? @huonw (rust_highfive has picked a reviewer for you, use r? to override) |
You should be actually able to delete This may also just indicate a bug in LLVM's implementation? Unsure :( |
@alexcrichton the goal of Bitrig is to not use any GNU software so using the default |
Oh "gnu" in this case is just the format of the archive, not the actual tooling used to produce it. |
@alexcrichton what's wrong with using the built-in ar? |
It generally works, but there are a number of wins for not depending on it:
|
@alexcrichton alright, I tested "gnu" and it works just fine, let's go with that. rebased and fixed. |
@bors: rollup |
…excrichton fixes the linking error on bitrig.
fixes the linking error on bitrig.