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 to glutin 0.8, Winit 0.6 #1251

Merged
merged 1 commit into from
May 4, 2017
Merged

Update to glutin 0.8, Winit 0.6 #1251

merged 1 commit into from
May 4, 2017

Conversation

csherratt
Copy link
Contributor

This will require gfx_window_glutin to be published.

I'm on linux and didn't get a chance to test the trianglell example.

@csherratt csherratt force-pushed the master branch 6 times, most recently from 228b499 to e7d9af3 Compare May 3, 2017 02:56
@kvark
Copy link
Member

kvark commented May 3, 2017

Looks great! One small thing - gfx_app version also needs a bump.

@kvark
Copy link
Member

kvark commented May 3, 2017

I wonder, where did travis go?..

@kvark
Copy link
Member

kvark commented May 3, 2017

@homu r+

@homu
Copy link
Contributor

homu commented May 3, 2017

📌 Commit 55d1347 has been approved by kvark

@homu
Copy link
Contributor

homu commented May 3, 2017

⌛ Testing commit 55d1347 with merge 2844afc...

homu added a commit that referenced this pull request May 3, 2017
Update to glutin 0.8, Winit 0.6

This will require gfx_window_glutin to be published.

I'm on linux and didn't get a chance to test the trianglell example.
@homu
Copy link
Contributor

homu commented May 3, 2017

💔 Test failed - status

@kvark
Copy link
Member

kvark commented May 3, 2017

@homu r+

@homu
Copy link
Contributor

homu commented May 3, 2017

📌 Commit 0705a00 has been approved by kvark

homu added a commit that referenced this pull request May 3, 2017
Update to glutin 0.8, Winit 0.6

This will require gfx_window_glutin to be published.

I'm on linux and didn't get a chance to test the trianglell example.
@homu
Copy link
Contributor

homu commented May 3, 2017

⌛ Testing commit 0705a00 with merge d5de10c...

@homu
Copy link
Contributor

homu commented May 3, 2017

💔 Test failed - status

@kvark
Copy link
Member

kvark commented May 3, 2017

Almost there! OSX just needs a fixup.

@kvark
Copy link
Member

kvark commented May 4, 2017

@homu r+

@homu
Copy link
Contributor

homu commented May 4, 2017

📌 Commit 23ae748 has been approved by kvark

homu added a commit that referenced this pull request May 4, 2017
Update to glutin 0.8, Winit 0.6

This will require gfx_window_glutin to be published.

I'm on linux and didn't get a chance to test the trianglell example.
@homu
Copy link
Contributor

homu commented May 4, 2017

⌛ Testing commit 23ae748 with merge 22b0d06...

@homu
Copy link
Contributor

homu commented May 4, 2017

☀️ Test successful - status

@homu homu merged commit 23ae748 into gfx-rs:master May 4, 2017
@kvark
Copy link
Member

kvark commented May 4, 2017

Woohoo!

@kvark kvark mentioned this pull request May 4, 2017
@ishitatsuyuki
Copy link
Contributor

ishitatsuyuki commented May 5, 2017

May I ask for it to be published?

@kvark
Copy link
Member

kvark commented May 5, 2017

@ishitatsuyuki I can publish gfx_window_glutin later tonight. Do you need any other crates among the updated ones?

@ishitatsuyuki
Copy link
Contributor

I only need the glutin one. Thanks!

@kvark
Copy link
Member

kvark commented May 5, 2017

@ishitatsuyuki ok, will do. Please use an override to a locally-checked-out crate in the meantime

@kvark
Copy link
Member

kvark commented May 6, 2017

@ishitatsuyuki version 0.16.0 is published now from rev 162221c

adamnemecek pushed a commit to adamnemecek/gfx that referenced this pull request Apr 1, 2021
1251: Don't panic on pipeline creation errors r=kvark a=kvark

**Connections**
Related to Gecko crashes.

**Description**
We shouldn't ever panic in `wgpu`. The panic can only occur if there is internal inconsistency. For any invalid workloads, submitted by the user, we should produce errors instead.

**Testing**
Not tested

Co-authored-by: Dzmitry Malyshau <[email protected]>
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.

4 participants