forked from dylanaraps/neofetch
-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
from-upstream-prMigrated from original dylanaraps/neofetch PRMigrated from original dylanaraps/neofetch PRhas-codePR with code ready to review/mergePR with code ready to review/merge
Description
Migrated from Original PR
Original PR: dylanaraps#2430
Author: @jcorbin
Branch: dev
Changes: +3/-3 lines
Original Description
Description
On my arch system running the river wm, get_wm wasn't working.
First off, I'd had no reason to install lsof as yet, having only fuser.
Next up the output of fuser was way more than the expected "just a pid": it included a bunch of permission denied entries while enumerating all non-user procs, and also some extra whitespace around the output pid, which was interacting poorly with the defensive quoting inside the sub-shell expansion of $tmp_pid.
TODO
- maybe add river to the guess branch when using neither lsof nor fuser?
This issue was automatically migrated from the archived dylanaraps/neofetch repository.
The code from the original PR can be cherry-picked or the author can submit a new PR to this fork.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
from-upstream-prMigrated from original dylanaraps/neofetch PRMigrated from original dylanaraps/neofetch PRhas-codePR with code ready to review/mergePR with code ready to review/merge