Skip to content

waybar: fix command not found when reloading#2865

Merged
berbiche merged 1 commit intonix-community:masterfrom
danth:fix-waybar-reload
Apr 16, 2022
Merged

waybar: fix command not found when reloading#2865
berbiche merged 1 commit intonix-community:masterfrom
danth:fix-waybar-reload

Conversation

@danth
Copy link
Copy Markdown
Member

@danth danth commented Apr 5, 2022

Description

Currently, trying to run systemctl --user reload waybar.service fails with:

Apr 05 19:31:39 StarBook-MkV systemd[2313]: Reloading Highly customizable Wayland bar for Sway and Wlroots based compositors....
Apr 05 19:31:39 StarBook-MkV systemd[46289]: waybar.service: Failed to locate executable kill: No such file or directory
Apr 05 19:31:39 StarBook-MkV systemd[46289]: waybar.service: Failed at step EXEC spawning kill: No such file or directory
Apr 05 19:31:39 StarBook-MkV systemd[2313]: waybar.service: Control process exited, code=exited, status=203/EXEC
Apr 05 19:31:39 StarBook-MkV systemd[2313]: Reload failed for Highly customizable Wayland bar for Sway and Wlroots based compositors..

Fixed this by inserting the full path to the kill command from the busybox package.

Checklist

  • Change is backwards compatible.

  • Code formatted with ./format.

  • Code tested through nix-shell --pure tests -A run.all.

  • Test cases updated/added. See example.

  • Commit messages are formatted like

    {component}: {description}
    
    {long description}
    

    See CONTRIBUTING for more information and recent commit messages for examples.

  • (Extra:) Code tested on a live system.

@danth danth requested a review from berbiche as a code owner April 5, 2022 18:45
Copy link
Copy Markdown
Member

@berbiche berbiche left a comment

Choose a reason for hiding this comment

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

Thanks!

Comment thread modules/programs/waybar.nix Outdated
@danth danth requested a review from berbiche April 6, 2022 09:55
@berbiche
Copy link
Copy Markdown
Member

berbiche commented Apr 7, 2022

Once #2867 is merged, you will have to rebase your changes to fix the failing tests.
I am unable to merge a PR with failing tests.

@danth
Copy link
Copy Markdown
Member Author

danth commented Apr 8, 2022

@berbiche Sorry for the delay, I've rebased the changes now 😄

@sumnerevans
Copy link
Copy Markdown
Contributor

Looks like the test needs to be fixed. It also needs test.stubs.coreutils = { }

@berbiche
Copy link
Copy Markdown
Member

berbiche commented Apr 8, 2022

Looks like the test needs to be fixed. It also needs test.stubs.coreutils = { }

👍 though coreutils is very likely already part of the test closure.

@danth
Copy link
Copy Markdown
Member Author

danth commented Apr 10, 2022

@sumnerevans test.stubs.coreutils causes an infinite recursion. What would you suggest I do?

@berbiche
Copy link
Copy Markdown
Member

@sumnerevans test.stubs.coreutils causes an infinite recursion. What would you suggest I do?

In that case, there's no need for a stub 👍

@danth
Copy link
Copy Markdown
Member Author

danth commented Apr 11, 2022

Used normalizeStorePaths instead of a stub; the test should be working now.

@danth
Copy link
Copy Markdown
Member Author

danth commented Apr 11, 2022

Fixed formatting.

@berbiche berbiche merged commit a640ddd into nix-community:master Apr 16, 2022
@danth danth deleted the fix-waybar-reload branch April 17, 2022 10:19
@teto teto mentioned this pull request Aug 22, 2022
7 tasks
teto pushed a commit to teto/home-manager that referenced this pull request Aug 22, 2022
spacekookie pushed a commit to spacekookie/home-manager that referenced this pull request Feb 10, 2023
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