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

Port to wlroots 0.18.x #2452

Merged
merged 24 commits into from
Dec 18, 2024
Merged

Port to wlroots 0.18.x #2452

merged 24 commits into from
Dec 18, 2024

Conversation

soreau
Copy link
Member

@soreau soreau commented Aug 26, 2024

Gotchas:

wl_drm was dropped, so clients must update. A notable one is libva, which stopped working with wlroots 0.18 until upgrading libva.

wlr_input_inhibit was dropped and as of now, there is no replacement.

Possibilities:

linux-drm-syncobj-v1

@mark-herbert42
Copy link

Compiled 0.9 with this PR - works ok so far. Seems like the new wlroots is better in terms of battery life at least for firefox-youtube.

@soreau
Copy link
Member Author

soreau commented Aug 27, 2024

Compiled 0.9 with this PR - works ok so far. Seems like the new wlroots is better in terms of battery life at least for firefox-youtube.

Thanks for testing!

@soreau soreau force-pushed the track-wlroots branch 28 times, most recently from 0e5e05e to c9c1798 Compare September 1, 2024 10:35
soreau added 19 commits October 4, 2024 23:27
This leads to a crash on wlroots 0.18, as wlroots handles this already.
xdg wayland surfaces were not committing because compositors are expected
to send a configure when wlr_xdg_surface.initial_commit is set.
There is nothing needed to do here when using wlr_output_state.
Without edge, latest only has 1.22 until possibly Nov '24, when alpine image
3.21 is released.
If a subsurface is created and then becomes smaller without repositioning,
the old larger size was never damaged, leaving artifacts on the output.
This fixes the build.
bob-beck pushed a commit to openbsd/ports that referenced this pull request Nov 7, 2024
update to the latest git commit from the 'port to wlroots 0.18.x'
upstream branch, cf WayfireWM/wayfire#2452
@soreau
Copy link
Member Author

soreau commented Dec 16, 2024

Since wlroots 0.18.2 has been released, I propose that we update to it before merging this.

Copy link
Member

@ammen99 ammen99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, also @soreau ran the tests and there are no regressions, so we can finally merge this :) Thanks for the patience and for your work @soreau 🎉

@soreau soreau merged commit 0f9d50d into master Dec 18, 2024
8 checks passed
@ammen99 ammen99 deleted the track-wlroots branch December 18, 2024 15:50
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