From b10f0e5b9a9ba6d8e9a27b3a81d2fc1d65d9810d Mon Sep 17 00:00:00 2001 From: Adam Borowski Date: Wed, 28 Feb 2018 23:12:04 +0100 Subject: [PATCH] hooks/pre-auto-gc-battery: allow gc to run on non-laptops Desktops and servers tend to have no power sensor, thus on_ac_power returns 255 ("unknown"). Thus, let's take any answer other than 1 ("battery") as no contraindication to run gc. If that tool returns "unknown", there's no point in querying other sources as it already queried them, and is smarter than us (can handle multiple adapters). Reported by: Xin Li Signed-off-by: Adam Borowski Signed-off-by: Junio C Hamano (cherry picked from git.git commit 781262c5e7ad4a7813c528803117ed0d2e8c5172) Signed-off-by: Fredrik Roubert Signed-off-by: Jonathan Nieder Change-Id: I51fe2eb1eb879492a61e8e09c86ee34d049036c1 --- hooks/pre-auto-gc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hooks/pre-auto-gc b/hooks/pre-auto-gc index c4107f51..ec29be43 100755 --- a/hooks/pre-auto-gc +++ b/hooks/pre-auto-gc @@ -29,7 +29,7 @@ then exit 0 fi -if test -x /sbin/on_ac_power && /sbin/on_ac_power +if test -x /sbin/on_ac_power && (/sbin/on_ac_power;test $? -ne 1) then exit 0 elif test "$(cat /sys/class/power_supply/AC/online 2>/dev/null)" = 1