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

Add optional ANGLE support to glutin port #22856

Merged
merged 3 commits into from May 10, 2019

Conversation

Projects
7 participants
@jdm
Copy link
Member

commented Feb 8, 2019

Supporting ANGLE in Servo has two important benefits:

  • we can actually run Servo instances on our Windows CI machines, which gives us more confidence that we're not breaking it
  • we can continue to use OpenGL on devices like the Hololens, rather than creating platform-specific graphical backends

This change is Reviewable

@highfive

This comment has been minimized.

Copy link

commented Feb 8, 2019

Heads up! This PR modifies the following files:

  • @wafflespeanut: python/servo/testing_commands.py
  • @paulrouget: ports/servo/glutin_app/egl.rs, ports/servo/glutin_app/window.rs, ports/servo/glutin_app/mod.rs, ports/servo/Cargo.toml
@highfive

This comment has been minimized.

Copy link

commented Feb 8, 2019

warning Warning warning

  • These commits modify unsafe code. Please review it carefully!
@jdm

This comment has been minimized.

Copy link
Member Author

commented Feb 8, 2019

@bors-servo try=windows

@bors-servo

This comment has been minimized.

Copy link
Contributor

commented Feb 8, 2019

⌛️ Trying commit 7e36817 with merge 63d72b9...

bors-servo added a commit that referenced this pull request Feb 8, 2019

Auto merge of #22856 - jdm:angle, r=<try>
WIP add angle support

<!-- 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/22856)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

commented Feb 8, 2019

💔 Test failed - status-taskcluster

@jdm

This comment has been minimized.

Copy link
Member Author

commented Feb 8, 2019

@bors-servo try=windows

@jdm jdm force-pushed the jdm:angle branch from 9d5d67f to 99b5242 Feb 8, 2019

@jdm

This comment has been minimized.

Copy link
Member Author

commented Feb 8, 2019

@bors-servo try=windows

@bors-servo

This comment has been minimized.

Copy link
Contributor

commented Feb 9, 2019

⌛️ Trying commit 99b5242 with merge 6fce713...

bors-servo added a commit that referenced this pull request Feb 9, 2019

Auto merge of #22856 - jdm:angle, r=<try>
WIP add angle support

<!-- 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/22856)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

commented Feb 9, 2019

💔 Test failed - status-taskcluster

@jdm jdm force-pushed the jdm:angle branch from 99b5242 to c36cb62 Feb 9, 2019

@highfive highfive removed the S-tests-failed label Feb 9, 2019

@jdm

This comment has been minimized.

Copy link
Member Author

commented Feb 9, 2019

@bors-servo try=windows

@bors-servo

This comment has been minimized.

Copy link
Contributor

commented Feb 9, 2019

⌛️ Trying commit c36cb62 with merge 91c588e...

bors-servo added a commit that referenced this pull request Feb 9, 2019

Auto merge of #22856 - jdm:angle, r=<try>
WIP add angle support

<!-- 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/22856)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

commented Feb 9, 2019

💔 Test failed - status-taskcluster

@jdm jdm force-pushed the jdm:angle branch from c36cb62 to 211138d Feb 9, 2019

@highfive highfive removed the S-tests-failed label Feb 9, 2019

@jdm

This comment has been minimized.

Copy link
Member Author

commented Feb 9, 2019

@bors-servo try=windows

@bors-servo

This comment has been minimized.

Copy link
Contributor

commented Feb 9, 2019

⌛️ Trying commit 211138d with merge c5ecb00...

bors-servo added a commit that referenced this pull request Feb 9, 2019

Auto merge of #22856 - jdm:angle, r=<try>
WIP add angle support

<!-- 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/22856)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

commented Feb 9, 2019

💔 Test failed - status-taskcluster

@bors-servo

This comment has been minimized.

Copy link
Contributor

commented May 9, 2019

⌛️ Testing commit a690d6f with merge 5fff2a8...

bors-servo added a commit that referenced this pull request May 9, 2019

Auto merge of #22856 - jdm:angle, r=paulrouget
Add optional ANGLE support to glutin port

Supporting ANGLE in Servo has two important benefits:
* we can actually run Servo instances on our Windows CI machines, which gives us more confidence that we're not breaking it
* we can continue to use OpenGL on devices like the Hololens, rather than creating platform-specific graphical backends

<!-- 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/22856)
<!-- Reviewable:end -->
@jdm

This comment has been minimized.

Copy link
Member Author

commented May 10, 2019

@bors-servo retry

@bors-servo

This comment has been minimized.

Copy link
Contributor

commented May 10, 2019

⌛️ Testing commit a690d6f with merge 4d8be3e...

bors-servo added a commit that referenced this pull request May 10, 2019

Auto merge of #22856 - jdm:angle, r=paulrouget
Add optional ANGLE support to glutin port

Supporting ANGLE in Servo has two important benefits:
* we can actually run Servo instances on our Windows CI machines, which gives us more confidence that we're not breaking it
* we can continue to use OpenGL on devices like the Hololens, rather than creating platform-specific graphical backends

<!-- 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/22856)
<!-- Reviewable:end -->
@jdm

This comment has been minimized.

Copy link
Member Author

commented May 10, 2019

@jdm jdm closed this May 10, 2019

UWP port automation moved this from Probably unnecessary to Done May 10, 2019

@jdm jdm reopened this May 10, 2019

UWP port automation moved this from Done to In progress May 10, 2019

@jdm

This comment has been minimized.

Copy link
Member Author

commented May 10, 2019

@bors-servo r=paulrouget

@bors-servo

This comment has been minimized.

Copy link
Contributor

commented May 10, 2019

📌 Commit a690d6f has been approved by paulrouget

@bors-servo

This comment has been minimized.

Copy link
Contributor

commented May 10, 2019

⌛️ Testing commit a690d6f with merge 7304606...

bors-servo added a commit that referenced this pull request May 10, 2019

Auto merge of #22856 - jdm:angle, r=paulrouget
Add optional ANGLE support to glutin port

Supporting ANGLE in Servo has two important benefits:
* we can actually run Servo instances on our Windows CI machines, which gives us more confidence that we're not breaking it
* we can continue to use OpenGL on devices like the Hololens, rather than creating platform-specific graphical backends

<!-- 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/22856)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

commented May 10, 2019

💔 Test failed - linux-rel-wpt

@jdm

This comment has been minimized.

Copy link
Member Author

commented May 10, 2019

@bors-servo

This comment has been minimized.

Copy link
Contributor

commented May 10, 2019

⌛️ Testing commit a690d6f with merge b1b47d8...

bors-servo added a commit that referenced this pull request May 10, 2019

Auto merge of #22856 - jdm:angle, r=paulrouget
Add optional ANGLE support to glutin port

Supporting ANGLE in Servo has two important benefits:
* we can actually run Servo instances on our Windows CI machines, which gives us more confidence that we're not breaking it
* we can continue to use OpenGL on devices like the Hololens, rather than creating platform-specific graphical backends

<!-- 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/22856)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

commented May 10, 2019

☀️ Test successful - android-mac, arm32, arm64, linux-rel-css, linux-rel-wpt, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, magicleap, status-taskcluster
Approved by: paulrouget
Pushing b1b47d8 to master...

@bors-servo bors-servo merged commit a690d6f into servo:master May 10, 2019

4 checks passed

Taskcluster (pull_request) TaskGroup: success
Details
Travis CI - Pull Request Build Passed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details

UWP port automation moved this from In progress to Done May 10, 2019

@bors-servo bors-servo referenced this pull request May 10, 2019

Merged

Glutin 0.21 #23288

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.