From: Phillip Berndt <phillip.berndt@googlemail.com>
Date: Wed, 23 Nov 2016 20:25:09 +0000 (+0100)
Subject: batch mode: Skip all uid < 1000 programs instead of only uid = 0
X-Git-Tag: 1.0~3
X-Git-Url: https://git.donarmstrong.com/?p=deb_pkgs%2Fautorandr.git;a=commitdiff_plain;h=53d29f99275aebf14240ea95f2d7022b305738d5

batch mode: Skip all uid < 1000 programs instead of only uid = 0

This fixes bug #63, but might have unexpected side effects. From a
comment in the commit:

> The following line assumes that user accounts start at 1000 and that
> no one works using the root or another system account. This is rather
> restrictive, but de facto default.  Alternatives would be to use the
> UID_MIN from /etc/login.defs or FIRST_UID from /etc/adduser.conf; but
> effectively, both values aren't binding in any way.  If this breaks
> your use case, please file a bug on Github.
---

diff --git a/autorandr.py b/autorandr.py
index 84fb3b3..af1c8c3 100755
--- a/autorandr.py
+++ b/autorandr.py
@@ -798,7 +798,14 @@ def dispatch_call_to_sessions(argv):
         if not os.path.isfile(environ_file):
             continue
         uid = os.stat(environ_file).st_uid
-        if uid == 0:
+
+        # The following line assumes that user accounts start at 1000 and that
+        # no one works using the root or another system account. This is rather
+        # restrictive, but de facto default. Alternatives would be to use the
+        # UID_MIN from /etc/login.defs or FIRST_UID from /etc/adduser.conf;
+        # but effectively, both values aren't binding in any way.
+        # If this breaks your use case, please file a bug on Github.
+        if uid < 1000:
             continue
 
         process_environ = {}