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

Update dependencies #2878

Merged
merged 1 commit into from Jul 11, 2018
Merged

Update dependencies #2878

merged 1 commit into from Jul 11, 2018

Conversation

@Eijebong
Copy link
Member

Eijebong commented Jul 10, 2018

Including glutin, winit, core-* stuff.


This change is Reviewable

@@ -259,7 +260,7 @@ fn make_window(
let window_builder = winit::WindowBuilder::new()
.with_title("WRench")
.with_multitouch()
.with_dimensions(size.width, size.height);
.with_dimensions(glutin::dpi::LogicalSize::new(size.width as f64, size.height as f64));

This comment has been minimized.

@Eijebong

Eijebong Jul 10, 2018

Author Member

Woups

@@ -171,10 +172,10 @@ impl Window {
}

let framebuffer_size = {
let (width, height) = self.window.get_inner_size().unwrap();
DeviceUintSize::new(width, height)
let glutin::dpi::LogicalSize {width, height} = self.window.get_inner_size().unwrap();

This comment has been minimized.

@Eijebong

Eijebong Jul 10, 2018

Author Member

woups²

@Eijebong Eijebong force-pushed the Eijebong:bump-stuff branch from a5f599e to d1ef283 Jul 10, 2018
@Eijebong
Copy link
Member Author

Eijebong commented Jul 10, 2018

Failure is legit on both. I'd need help updating the rust version on taskcluster though

@staktrace
Copy link
Contributor

staktrace commented Jul 10, 2018

Right now all the taskcluster failures are just linting problems.

But I can help with updating rust versions - what version of rust do you want to be running on?

@Eijebong
Copy link
Member Author

Eijebong commented Jul 10, 2018

Mhh maybe it's not necessary then, 1.25 seems to be working but if we can get to the same version as firefox it'd be nice :)

@staktrace
Copy link
Contributor

staktrace commented Jul 10, 2018

The linux runs in the taskcluster CI always update rust to the latest stable version (so should be 1.27 now). The mac runs do not, but I manually updated that to 1.27 as well sometime last week. So everything in taskcluster CI should be using 1.27.

@Eijebong
Copy link
Member Author

Eijebong commented Jul 10, 2018

Oh ok, nice !

@Eijebong Eijebong force-pushed the Eijebong:bump-stuff branch 8 times, most recently from 3437087 to 781dd42 Jul 10, 2018
Including glutin, winit, core-* stuff.
@Eijebong Eijebong force-pushed the Eijebong:bump-stuff branch from 781dd42 to c5edc4d Jul 10, 2018
@nical
Copy link
Collaborator

nical commented Jul 11, 2018

@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented Jul 11, 2018

📌 Commit c5edc4d has been approved by nical

@bors-servo
Copy link
Contributor

bors-servo commented Jul 11, 2018

Testing commit c5edc4d with merge a30fd22...

bors-servo added a commit that referenced this pull request Jul 11, 2018
Update dependencies

Including glutin, winit, core-* stuff.

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

bors-servo commented Jul 11, 2018

☀️ Test successful - status-appveyor, status-taskcluster
Approved by: nical
Pushing a30fd22 to master...

@bors-servo bors-servo merged commit c5edc4d into servo:master Jul 11, 2018
3 checks passed
3 checks passed
Taskcluster (pull_request) TaskGroup: success
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
bors-servo added a commit that referenced this pull request Jul 13, 2018
…monSapin

use WS_EX_NOREDIRECTIONBITMAP direct-composition app

`with_no_redirection_bitmap` API added to master branch of `winit=0.16.0`.
It is useful for direct-composition samples.

resubmit #2838 PR,
Thanks! #2878, #2877.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/2886)
<!-- Reviewable:end -->
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

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