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

Fix Hyprland socketpath changed to XDG_RUNTIME_DIR #3183

Merged
merged 1 commit into from
Apr 29, 2024

Conversation

zjeffer
Copy link
Contributor

@zjeffer zjeffer commented Apr 29, 2024

Since hyprwm/Hyprland@a5a6480, the hyprland IPC socket can now be found under $XDG_RUNTIME_DIR/hypr/<signature>/.socket.sock and /.socket2.sock.

This PR adds a fix for this, while keeping backwards compatibility (=fallback to /tmp/hypr) for people not yet using this Hyprland commit.

@zjeffer
Copy link
Contributor Author

zjeffer commented Apr 29, 2024

Oops, I just realized #3180 already exists. But that PR doesn't take into account the fallback to /tmp, and duplicates code...

@Alexays can you merge one of these? The Hyprland change breaks the hyprland/workspaces module entirely.

@Alexays Alexays merged commit f41458e into Alexays:master Apr 29, 2024
9 checks passed
@Alexays
Copy link
Owner

Alexays commented Apr 29, 2024

Thanks!

@leon-erd
Copy link

leon-erd commented May 2, 2024

I think it would be really nice to maybe make another minor release that includes this PR. Without it waybar is broken on hyprland.

@zjeffer
Copy link
Contributor Author

zjeffer commented May 4, 2024

@Alexays A new Hyprland release (0.40.0) that contains the breaking change has just been released. Can you make a new Waybar release?

drawbu added a commit to drawbu/dotfiles that referenced this pull request May 6, 2024
- switch from `v0.38.1` -> `v0.40.0`
- add pkgs.hyprpkgs to have packages follow hyprland
- remove hyprland-plugins as not used
- fix hyprland socket in waybar (Alexays/Waybar#3183)
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.

3 participants