Skip to content
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

Try to fix the nightlies #34519

Merged
merged 2 commits into from
Jun 28, 2016
Merged

Try to fix the nightlies #34519

merged 2 commits into from
Jun 28, 2016

Conversation

alexcrichton
Copy link
Member

They look to be failing right after the CMake PR landed. I've diagnosed and confirmed the first issue fixed, the second is a bit of a shot in the dark to see if it fixes things.

Looks like compiler-rt has been tweaked slightly during the last update,
propagate these changes to rustbuild as well.
* Implement the clean-llvm target for those cases where makefiles are being used
* Have all cross-compiled LLVMs depend on the **host** LLVM as they'll require
  the llvm-tablegen executable from there
@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

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

@alexcrichton
Copy link
Member Author

r? @brson

@rust-highfive rust-highfive assigned brson and unassigned nikomatsakis Jun 28, 2016
@brson
Copy link
Contributor

brson commented Jun 28, 2016

@bors r+ p=1

@bors
Copy link
Contributor

bors commented Jun 28, 2016

📌 Commit 3fd411e has been approved by brson

@bors
Copy link
Contributor

bors commented Jun 28, 2016

⌛ Testing commit 3fd411e with merge 7d751c4...

@bors
Copy link
Contributor

bors commented Jun 28, 2016

💥 Test timed out

@alexcrichton
Copy link
Member Author

@bors: retry

  • Android bot apparently disappeared...

@bors
Copy link
Contributor

bors commented Jun 28, 2016

⌛ Testing commit 3fd411e with merge 366de83...

bors added a commit that referenced this pull request Jun 28, 2016
Try to fix the nightlies

They look to be failing right after the CMake PR landed. I've diagnosed and confirmed the first issue fixed, the second is a bit of a shot in the dark to see if it fixes things.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants