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

subprojects: remove wlroots.wrap #4572

Merged
merged 1 commit into from
Jan 31, 2024
Merged

subprojects: remove wlroots.wrap #4572

merged 1 commit into from
Jan 31, 2024

Conversation

fufexan
Copy link
Member

@fufexan fufexan commented Jan 31, 2024

Describe your PR, what does it fix/add?

From @jbeich's testing, the wrap file doesn't work as it should, and having it
is not consistent with the other subprojects. See
hyprwm/hyprland-wiki#200 (comment) for
more info.

Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)

The wiki command meson subprojects update --reset won't work anymore (if it
even worked in the first place). It will be removed in the PR linked above.

Is it ready for merging, or does it need work?

Might be ready. @jbeich thoughts?

@jbeich
Copy link
Contributor

jbeich commented Jan 31, 2024

Looks fine. With d179501 there's no need for soversion hack while plugin API uses wlroots headers but doesn't need the library. I'm not sure if any plugins end up using wlroots symbols (only tested hy3) but even then Hyprland binary should provide wlroots symbols (on dlopen) due to static linking.

@fufexan
Copy link
Member Author

fufexan commented Jan 31, 2024

Thanks for the input, merging then.

@fufexan fufexan merged commit e6f7724 into hyprwm:main Jan 31, 2024
11 checks passed
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.

2 participants