Skip to content

Conversation

james7132
Copy link
Member

Objective

Follow-up to #19232. X11 support is on the downturn, with other players like GTK announcing that they're considering deprecating X11 support in the next major release and Fedora 43 later this year is going to be Wayland only.

Solution

Remove x11 as a default feature from the top level bevy crate.

Testing

Local build run of 3d_example. This doesn't seem to impact compile times significantly. Release builds shave off about 1MB.

@james7132 james7132 added A-Windowing Platform-agnostic interface layer to run your app in O-Linux Specific to the Linux desktop operating system labels Sep 21, 2025
@alice-i-cecile alice-i-cecile added this to the 0.18 milestone Sep 21, 2025
@alice-i-cecile alice-i-cecile added M-Needs-Migration-Guide A breaking change to Bevy's public API that needs to be noted in a migration guide S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged labels Sep 21, 2025
Copy link
Contributor

It looks like your PR is a breaking change, but you didn't provide a migration guide.

Please review the instructions for writing migration guides, then expand or revise the content in the migration guides directory to reflect your changes.

@hukasu
Copy link
Contributor

hukasu commented Sep 22, 2025

update this comment

# Display server protocol support (X11 is enabled by default)
wayland = ["bevy_winit/wayland"]
x11 = ["bevy_winit/x11"]

@james7132 james7132 added S-Needs-Review Needs reviewer attention (from anyone!) to move forward and removed S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged labels Sep 22, 2025
@hukasu hukasu added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Sep 22, 2025
@alice-i-cecile alice-i-cecile added the X-Contentious There are nontrivial implications that should be thought through label Sep 22, 2025
@james7132 james7132 requested a review from cart September 22, 2025 23:34
@cart
Copy link
Member

cart commented Sep 25, 2025

This feels a bit premature. If we do this now we're close to being a "first mover". Id prefer to not trade platform support for leanness in this particular case / at this phase of the x11 deprecation. Linux is already a compatibility nightmare. I see no reason to cut out a percentage of users by default.

@alice-i-cecile alice-i-cecile added X-Controversial There is active debate or serious implications around merging this PR S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged and removed S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it X-Contentious There are nontrivial implications that should be thought through labels Sep 25, 2025
@james7132
Copy link
Member Author

Looking deeper into it, it seems like they're just dropping the X session, and not the headers. Right now it seems like it's only KDE and GNOME that have made the jump and Cosmic that was built natively in Wayland. MATE and LXQt both have some support for Wayland at this point, but haven't dropped X11 support yet. Cinnamon and XFCE has it in experimental.

We'll have to come back to this at a later point in time.

@james7132 james7132 removed this from the 0.18 milestone Sep 29, 2025
@james7132 james7132 added S-Blocked This cannot move forward until something else changes and removed S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged labels Sep 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Windowing Platform-agnostic interface layer to run your app in M-Needs-Migration-Guide A breaking change to Bevy's public API that needs to be noted in a migration guide O-Linux Specific to the Linux desktop operating system S-Blocked This cannot move forward until something else changes X-Controversial There is active debate or serious implications around merging this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants