diff options
| author | Craig Jennings <c@cjennings.net> | 2026-04-22 11:45:19 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-04-22 11:45:19 -0500 |
| commit | 7b235ee0e9298a81d8acc839b7b6e8abe9aca170 (patch) | |
| tree | 467fd148b41c95598fc83b4f744e4f8576101a5e /modules/coverage-core.el | |
| parent | 1fa817ab7390c694766d94215d4f63dd9dc31b95 (diff) | |
| download | dotemacs-7b235ee0e9298a81d8acc839b7b6e8abe9aca170.tar.gz dotemacs-7b235ee0e9298a81d8acc839b7b6e8abe9aca170.zip | |
fix(host-environment): detect battery correctly on Linux desktops
`env-laptop-p` treated any `battery-format "%B"` value that wasn't literally "N/A" as "has a battery." On a Linux desktop using `battery-upower`, the result is "unknown". The AC adapter and USB-C power entries exist in /sys but there's no BAT*. That made desktops look like laptops. The per-machine font height switch in `font-config.el` broke as a result.
The fix uses /sys/class/power_supply/BAT* as the canonical Linux signal. That's what the kernel exposes, and what upower itself reads. Other platforms keep the `battery-format` path, but the fallback now checks for a live battery status char ("!", "+", "-") instead of only excluding "N/A".
Two pure helpers (`env--battery-status-char-indicates-battery-p`, `env--power-supply-has-battery-p`) keep the logic testable. The new test file covers Normal, Boundary, and Error cases for each helper.
Diffstat (limited to 'modules/coverage-core.el')
0 files changed, 0 insertions, 0 deletions