-
Notifications
You must be signed in to change notification settings - Fork 198
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
Enable wayland
feature for bevy
in dev
#556
Enable wayland
feature for bevy
in dev
#556
Conversation
Uhh... Need to add the deps to CI... https://github.com/bevyengine/bevy/blob/release-0.14.1/docs/linux_dependencies.md#ubuntu |
e4e8c03
to
cd48d32
Compare
cd48d32
to
c9a2dbe
Compare
Oops... Missed adding the step for the |
bevyengine/bevy#10792 (comment) states that
I largely don't have an opinion on doing this for dev. It looks like the current state of things is we either potentially break one group of users or another group of users (until bevy merges the fix upstream), but wayland is the future so I'm a soft +1. Would like to get thoughts from @rparrett though. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm fine with doing this before Bevy if it helps out the people who want to work on bevy_ecs_tilemap
now.
Actually no, it's already broken. If you're on Wayland, and have no Vulkan driver, nothing would help at this time. Every possible combination is broken. XWayland + OpenGL does not work: bevyengine/bevy#13753 (comment) and bevyengine/bevy#8788 (comment) Wayland + OpenGL does not work: bevyengine/bevy#13923 |
merging then, we can re-evaluate if bevy changes upstream. thanks @teohhanhui ! |
Rationale:
BadDisplay
after winit 0.30 on Wayland OpenGL bevyengine/bevy#13923)