diff options
| author | Bruce Weirdan <weirdan@gmail.com> | 2015-12-24 04:11:22 +0200 |
|---|---|---|
| committer | Marc Cornellà <marc.cornella@live.com> | 2020-02-12 15:29:57 +0100 |
| commit | b8b87629157b30fc00d0b4af62dd0cf422896346 (patch) | |
| tree | 68ca2f23661fb8ce36469398a0fa5ccba3c1948e /plugins/battery | |
| parent | 5f6f7b6e8de8e6b64f07deb6b185535a8cdbe03e (diff) | |
| download | zsh-b8b87629157b30fc00d0b4af62dd0cf422896346.tar.gz zsh-b8b87629157b30fc00d0b4af62dd0cf422896346.tar.bz2 zsh-b8b87629157b30fc00d0b4af62dd0cf422896346.zip | |
Report only active battery (#4726)
On a system with multiple batteries (like thinkpads) report percentage
and time remaining only for the active battery (the one being
discharged).
Ideally we should report all batteries, but acpi only shows time remaining
for the active battery. Also callers of these functions expect a single
return value. This is still better than reporting 596% remaining (like it
did on my laptop).
For the reference, the output of acpi command with multiple batteries looks
like this:
Battery 0: Unknown, 5%
Battery 1: Discharging, 86%, 03:14:04 remaining
Diffstat (limited to 'plugins/battery')
| -rw-r--r-- | plugins/battery/battery.plugin.zsh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/plugins/battery/battery.plugin.zsh b/plugins/battery/battery.plugin.zsh index 7c5bf6f82..d1adcd0b6 100644 --- a/plugins/battery/battery.plugin.zsh +++ b/plugins/battery/battery.plugin.zsh @@ -117,7 +117,7 @@ elif [[ "$OSTYPE" = linux* ]]; then function battery_pct() { if (( $+commands[acpi] )); then - acpi 2>/dev/null | cut -f2 -d ',' | tr -cd '[:digit:]' + acpi 2>/dev/null | command grep -E '^Battery.*(Disc|C)harging' | cut -f2 -d ',' | tr -cd '[:digit:]' fi } |