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

Add wlr layer shell check #1752

Merged
merged 1 commit into from
Feb 26, 2024
Merged

Add wlr layer shell check #1752

merged 1 commit into from
Feb 26, 2024

Conversation

Caellian
Copy link
Collaborator

Closes: #1751

When a global is registered with wl_registry_add_listener it receives events for all supported extensions in bulk. wl_display_roundtrip call blocks until the initial events are exchanged so it's safe to check whether wl_globals.layer_shell is initialized right after that.

@github-actions github-actions bot added the sources PR modifies project sources label Feb 25, 2024
Copy link

netlify bot commented Feb 25, 2024

Deploy Preview for conkyweb canceled.

Name Link
🔨 Latest commit 1da6d30
🔍 Latest deploy log https://app.netlify.com/sites/conkyweb/deploys/65dbce3d5815050009288593

Print a nicer error message if the compositor doesn't support
wlr-layer-shell-unstable-v1 spec.

Signed-off-by: Tin Švagelj <[email protected]>
@Caellian
Copy link
Collaborator Author

Caellian commented Feb 25, 2024

This needs to be tested. I don't have a WM without layer shell installed so I just made a quick PR as it was quicker than describing the change in the issue.

Copy link
Owner

@brndnmtthws brndnmtthws left a comment

Choose a reason for hiding this comment

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

I tested this, seems to work.

@brndnmtthws brndnmtthws merged commit 97b6233 into brndnmtthws:main Feb 26, 2024
51 of 55 checks passed
@brndnmtthws brndnmtthws added the bug Bug report or bug fix PR label Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug report or bug fix PR sources PR modifies project sources
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: segfault when zwlr_layer_shell_v1 not available on Wayland
2 participants