-
-
Notifications
You must be signed in to change notification settings - Fork 732
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
Waybar doesn't start and can't quit it, need to force kill #2598
Comments
Thanks for bisecting already, luckily the commit isn't too complex. :) I have no immediate idea, but my first guess would be that there might be an overflow because a too small type is used somewhere. |
Reverted the PR until we fix the issue :) |
Found the issue, was a little bit tricky because the cause wasn't in the user code but in the standard library.
I will open a PR later this evening. Minimal config to reproduce the config: {
"modules-center": [
"tray",
"custom/wlogout"
],
"custom/wlogout": {
"format": "",
"interval": "once",
"on-click": "true",
"tooltip": false
}
} For whatever reason, the code hangs in an infinite-loop in |
Related PR from bisect: #2586
Specifically 89e85db introduces the borked waybar.
Can no longer run waybar on master. Doing a git bisect shows that this commit breaks waybar for me. If I delete that commit my waybar launches and operates, as normal.
Setup: Using NixOS with the nixpkgs-wayland overlay to build latest from master. Tested locally and bisected using the in repo flake devenv with nix-build.
Waybar config
@taminob any thoughts on why this would break waybar for me? If I simply revert that commit, it works fine.
The text was updated successfully, but these errors were encountered: