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

Install cmake 3.2 on ubuntu builders. #747

Merged
merged 1 commit into from Nov 9, 2017
Merged

Install cmake 3.2 on ubuntu builders. #747

merged 1 commit into from Nov 9, 2017

Conversation

@jdm
Copy link
Member

jdm commented Nov 8, 2017

This is needed for servo/servo#19152.


This change is Reviewable

@metajack
Copy link
Contributor

metajack commented Nov 8, 2017

@bors-servo r+

Man, we really need to upgrade the builders to a recent Ubuntu.

@bors-servo
Copy link
Contributor

bors-servo commented Nov 8, 2017

📌 Commit d77248b has been approved by metajack

@highfive highfive assigned metajack and unassigned aneeshusa Nov 8, 2017
@bors-servo
Copy link
Contributor

bors-servo commented Nov 8, 2017

Testing commit d77248b with merge c3fb6a2...

bors-servo added a commit that referenced this pull request Nov 8, 2017
Install cmake 3.2 on ubuntu builders.

This is needed for servo/servo#19152.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/747)
<!-- Reviewable:end -->
@jdm jdm force-pushed the jdm:cmakeup branch from d77248b to d55eb38 Nov 8, 2017
@jdm jdm force-pushed the jdm:cmakeup branch 2 times, most recently from 5b8a677 to 18f2a4d Nov 8, 2017
@nox
Copy link
Member

nox commented Nov 9, 2017

Seems like this needs a rebase.

@jdm
Copy link
Member Author

jdm commented Nov 9, 2017

No, that's just the new github UI showing that it could be rebased, but there are no actual merge conflicts yet.

@jdm jdm force-pushed the jdm:cmakeup branch from 18f2a4d to d4e2afe Nov 9, 2017
@jdm
Copy link
Member Author

jdm commented Nov 9, 2017

@bors-servo: r=metajack

@bors-servo
Copy link
Contributor

bors-servo commented Nov 9, 2017

📌 Commit d4e2afe has been approved by metajack

bors-servo added a commit that referenced this pull request Nov 9, 2017
Install cmake 3.2 on ubuntu builders.

This is needed for servo/servo#19152.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/747)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 9, 2017

Testing commit d4e2afe with merge 2272cbc...

@bors-servo
Copy link
Contributor

bors-servo commented Nov 9, 2017

☀️ Test successful - status-travis
Approved by: metajack
Pushing 2272cbc to master...

@bors-servo bors-servo merged commit d4e2afe into servo:master Nov 9, 2017
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@jdm jdm removed the S-needs-deploy label Nov 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
You can’t perform that action at this time.