waybar: show the bar on this machine's primary output
The deployed config pinned "output": "eDP-1", the laptop's internal display. On the PC waybar started fine but never mapped a layer surface, so there was no bar and SIGUSR1 had nothing to toggle. Listed both profiles' primaries instead -- waybar skips names that are not connected, so the one file works on the laptop and the PC. Two bugs in the same config that this uncovered: - bars/main.jsonc included modules/temperature.cpu.jsonc, but the file is temperature-cpu.jsonc, so the temperature#cpu module the bar referenced was silently absent. - get_wg_status.sh ran `sudo wg show` with no tty, logging "a terminal is required to read the password" on every poll. Now `sudo -n`, which fails instantly and falls into the existing "disconnected" branch, and starts working as-is if a NOPASSWD rule is ever added. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01StGBghPW9Dm7z7uj2VNWHr
This commit is contained in:
@@ -44,7 +44,9 @@ function format {
|
||||
}
|
||||
|
||||
# Read wg show output and format
|
||||
wg_show_output=$(sudo wg show)
|
||||
# -n so a missing NOPASSWD sudoers rule fails instantly instead of blocking
|
||||
# the module on a password prompt it has no terminal to ask on.
|
||||
wg_show_output=$(sudo -n wg show 2>/dev/null)
|
||||
|
||||
if [[ -z $wg_show_output ]]; then
|
||||
echo "{ \"text\": \"---\", \"tooltip\": \"disconnected\", \"class\": \"\", \"percentage\": 0 }"
|
||||
|
||||
Reference in New Issue
Block a user