-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Backport LLVM 12.0.1-4 bump to 1.7 #42535
Conversation
@vchuravy Can you rebase this PR (and force-push) and see if that triggers Buildkite? |
Thanks! Okay, it looks like Buildkite is now running on this PR. |
Note: the Buildkite failures in the |
FreeBSD tells me that my backport was not sufficient:
but this might also be RPATH? @vtjnash |
Might I ask that we hold off with merging this backport till we are certain it will not introduce new build breakage for consumers of RC2? :) |
d01b0f7
to
b1f569a
Compare
@Sacha0 can you test whether this branch also breaks your NixOs setup? |
Sure thing, I'll pull this patch onto our 1.7 testing branch and report back. Thanks again! :) |
To confirm, this branch breaks our 1.7 test builds with the same failure mode that the bump on master broke our master test builds:
|
@staticfloat any ideas how we can fix the RPATH issue? @Sacha0 currently we have no CI against NixOS so this not a supported build-configuration. Could you look into adding one? |
Regrets, I know little about NixOS and dearly want to avoid learning any more 😄. |
b1f569a
to
0a3718e
Compare
@Sacha0 can you test this against your infrastructure? |
Firing off a test of this branch and master with Rob's patch in now, will report back :). |
Preliminarily it looks like both our latest-tester and 1.7-tester succeed with the addition of Rob's patch 🎉 |
Includes: