[3.5] bpo-30584: Fix test_os fails on non-English Windows (GH-1980) (#2000)

denis-osipov · vstinner · commit a0b9088881b8 · 2017-06-08T14:16:48.000+02:00
* Fix bpo-30584 * Adding a comment mentionning the bpo and explaining what is the identifier * Add Denis Osipov to Misc/ACKS (cherry picked from commit 897bba7)
diff --git a/Lib/test/test_os.py b/Lib/test/test_os.py
@@ -448,7 +448,9 @@ def test_access_denied(self):
         # force CreateFile to fail with ERROR_ACCESS_DENIED.
         DETACHED_PROCESS = 8
         subprocess.check_call(
-            ['icacls.exe', fname, '/deny', 'Users:(S)'],
+            # bpo-30584: Use security identifier *S-1-5-32-545 instead
+            # of localized "Users" to not depend on the locale.
+            ['icacls.exe', fname, '/deny', '*S-1-5-32-545:(S)'],
             creationflags=DETACHED_PROCESS
         )
         result = os.stat(fname)
diff --git a/Misc/ACKS b/Misc/ACKS
@@ -1101,6 +1101,7 @@ William Orr
 Michele Orrù
 Tomáš Orsava
 Oleg Oshmyan
+Denis Osipov
 Denis S. Otkidach
 Peter Otten
 Michael Otteneder
