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

Unbreak angle #23768

Merged
merged 2 commits into from Jul 13, 2019
Merged

Unbreak angle #23768

merged 2 commits into from Jul 13, 2019

Conversation

@jdm
Copy link
Member

jdm commented Jul 12, 2019

This updates our ANGLE integration to a working version that allows nightly builds to run with --angle once more.


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes fix #23656
  • There are tests for these changes

This change is Reviewable

@highfive
Copy link

highfive commented Jul 12, 2019

Heads up! This PR modifies the following files:

@jdm
Copy link
Member Author

jdm commented Jul 12, 2019

@bors-servo try=windows

@bors-servo
Copy link
Contributor

bors-servo commented Jul 12, 2019

Trying commit 13cfa10 with merge 33e3fbb...

bors-servo added a commit that referenced this pull request Jul 12, 2019
Unbreak angle

This updates our ANGLE integration to a working version that allows nightly builds to run with `--angle` once more.

---
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #23656
- [x] There are tests for these changes

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

jdm commented Jul 12, 2019

The new automated test to verify that ANGLE works passed: https://treeherder.allizom.org/#/jobs?repo=servo-try&revision=33e3fbb97cdba9c8149fe962983d076277d3e17c

Oh frabjous day! Calloo callay!

This PR is ready for review.

@jdm
Copy link
Member Author

jdm commented Jul 12, 2019

@bors-servo r=asajeffrey

@bors-servo
Copy link
Contributor

bors-servo commented Jul 12, 2019

📌 Commit 13cfa10 has been approved by asajeffrey

@bors-servo
Copy link
Contributor

bors-servo commented Jul 13, 2019

Testing commit 13cfa10 with merge 7006f8f...

bors-servo added a commit that referenced this pull request Jul 13, 2019
Unbreak angle

This updates our ANGLE integration to a working version that allows nightly builds to run with `--angle` once more.

---
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #23656
- [x] There are tests for these changes

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

bors-servo commented Jul 13, 2019

💔 Test failed - linux-rel-css

@jdm jdm force-pushed the jdm:unbreak-angle branch from 13cfa10 to 02797a3 Jul 13, 2019
@jdm
Copy link
Member Author

jdm commented Jul 13, 2019

@bors-servo r=asajeffrey

@bors-servo
Copy link
Contributor

bors-servo commented Jul 13, 2019

📌 Commit 02797a3 has been approved by asajeffrey

@bors-servo
Copy link
Contributor

bors-servo commented Jul 13, 2019

Testing commit 02797a3 with merge 2b84348...

bors-servo added a commit that referenced this pull request Jul 13, 2019
Unbreak angle

This updates our ANGLE integration to a working version that allows nightly builds to run with `--angle` once more.

---
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #23656
- [x] There are tests for these changes

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

bors-servo commented Jul 13, 2019

☀️ Test successful - linux-rel-css, linux-rel-wpt, status-taskcluster
Approved by: asajeffrey
Pushing 2b84348 to master...

@bors-servo bors-servo merged commit 02797a3 into servo:master Jul 13, 2019
3 checks passed
3 checks passed
Taskcluster (pull_request) TaskGroup: success
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
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.

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