-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Upgrade the Travis CI macOS Xcode version to 8.3 #44338
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
Conversation
a355c83
to
508f9ca
Compare
@bors try (PR queue is too crowded. Sorry for abuse.) |
[WIP] [DO NOT MERGE] Change the Travis CI macOS Xcode version to 9.0 Just to experiment if it can make #44221 appear less often.
@bors try- |
@bors force clean retry Wat, shouldn't |
Next, |
LLVM needing 50 minutes to rebuild might be an issue though. I'm not sure if the dedicated VM for |
Summary of the experiment:
|
📌 Commit a6cd374 has been approved by |
@bors p=8 (might reduce spurious failure) Note: If any of the two |
⌛ Testing commit a6cd374 with merge 85b48bd37ed5833b559748aeb6ca1e290e293447... |
💔 Test failed - status-travis |
It is the
@bors retry — with the assumption that |
Upgrade the Travis CI macOS Xcode version to 8.3 Just to experiment if it can make #44221 appear less often.
💔 Test failed - status-travis |
Time (in minutes) spent on `dist x86_64-apple-darwin`
Currently reducing priority back to normal to check the behavior of other PRs first. If two other PRs can pass normally than we don't need PR. @bors retry p=0 |
@bors p=8 |
⌛ Testing commit a6cd374 with merge db86d16b833aba9cca7b63bcf63a174bf1411fb9... |
💔 Test failed - status-travis |
So that shows the Xcode version is likely irrelevant to the actual problem. Since Travis is already investigating the cause, hopefully it will be fixed by upstream. @bors p=0 rollup |
The incident should be resolved. Since Xcode 9 is expected to be released in a few weeks, as a regular PR it is better to just upgrade to Xcode 9-stable directly after Travis supported it. Closing. |
No description provided.