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

Updated: raw-window-handle & winit #838

Closed
wants to merge 1 commit into from
Closed

Updated: raw-window-handle & winit #838

wants to merge 1 commit into from

Conversation

codecnotsupported
Copy link

@codecnotsupported codecnotsupported commented Nov 29, 2023

Not my finest work.
This pull request updates the following dependencies:
raw-window-handle = "0.6"
winit = "0.29.4"

I'm aware of a pull request to update raw-window-handle to 0.6, (after I was done with this pull request >_>)
This pull request also updates the latest Winit version.

I ran and tested the code on Linux KDE Wayland, on exit it outputs "zwp_linux_dmabuf_v1@38 still attached".
This is a improvement over the dozen Wayland warnings it gives using the current git master Winit version.

Please validate for correctness.

raw-window-handle = "0.6"
winit = "0.29.4"
@MarijnS95 MarijnS95 closed this Dec 1, 2023
@codecnotsupported
Copy link
Author

No comment @MarijnS95?

@Ralith
Copy link
Collaborator

Ralith commented Dec 3, 2023

See #799.

@codecnotsupported
Copy link
Author

codecnotsupported commented Dec 3, 2023

See #799.

Thanks, I misunderstood and thought that PR only updated raw-window-handle to 0.6 whilst staying on a old Winit version.

@MarijnS95
Copy link
Collaborator

whilst staying on a old Winit version.

The winit version is only relevant for examples anyway. We need the latest version only to support raw-window-handle 0.6.

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.

3 participants