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 | 24a9d1aa4793bac1653beb844dc967072f0dfbe8 (patch) | |
| tree | dfa2345fb1e1107555efaa721d51479d900c20f8 /modules/custom-ordering.el | |
| parent | 70a12559e223e1dd00924178099b4acd5dacbe75 (diff) | |
| download | dotemacs-24a9d1aa4793bac1653beb844dc967072f0dfbe8.tar.gz dotemacs-24a9d1aa4793bac1653beb844dc967072f0dfbe8.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/custom-ordering.el')
0 files changed, 0 insertions, 0 deletions