[Python-checkins] bpo-33687: Fix call to os.chmod() in uu.decode() (GH-7282)
Berker Peksag
webhook-mailer at python.org
Thu Jan 17 09:33:03 EST 2019
https://github.com/python/cpython/commit/a261b737617ca8d52e04bf3ead346b1b8786a212
commit: a261b737617ca8d52e04bf3ead346b1b8786a212
branch: 3.7
author: Miss Islington (bot) <31488909+miss-islington at users.noreply.github.com>
committer: Berker Peksag <berker.peksag at gmail.com>
date: 2019-01-17T17:32:59+03:00
summary:
bpo-33687: Fix call to os.chmod() in uu.decode() (GH-7282)
(cherry picked from commit 17f05bbc78dbcd1db308266c31370da9ec1b1d47)
Co-authored-by: Timo Furrer <tuxtimo at gmail.com>
files:
A Misc/NEWS.d/next/Library/2018-06-10-14-08-52.bpo-33687.1zZdnA.rst
M Lib/test/test_uu.py
M Lib/uu.py
diff --git a/Lib/test/test_uu.py b/Lib/test/test_uu.py
index 1147205a3b53..c9f05e5b760d 100644
--- a/Lib/test/test_uu.py
+++ b/Lib/test/test_uu.py
@@ -6,6 +6,8 @@
import unittest
from test import support
+import os
+import stat
import sys
import uu
import io
@@ -218,6 +220,23 @@ def test_decodetwice(self):
with open(self.tmpin, 'rb') as f:
self.assertRaises(uu.Error, uu.decode, f)
+ def test_decode_mode(self):
+ # Verify that decode() will set the given mode for the out_file
+ expected_mode = 0o444
+ with open(self.tmpin, 'wb') as f:
+ f.write(encodedtextwrapped(expected_mode, self.tmpout))
+
+ # make file writable again, so it can be removed (Windows only)
+ self.addCleanup(os.chmod, self.tmpout, expected_mode | stat.S_IWRITE)
+
+ with open(self.tmpin, 'rb') as f:
+ uu.decode(f)
+
+ self.assertEqual(
+ stat.S_IMODE(os.stat(self.tmpout).st_mode),
+ expected_mode
+ )
+
if __name__=="__main__":
unittest.main()
diff --git a/Lib/uu.py b/Lib/uu.py
index 8333e864d8f9..9b1e5e607207 100755
--- a/Lib/uu.py
+++ b/Lib/uu.py
@@ -133,10 +133,7 @@ def decode(in_file, out_file=None, mode=None, quiet=False):
out_file = sys.stdout.buffer
elif isinstance(out_file, str):
fp = open(out_file, 'wb')
- try:
- os.path.chmod(out_file, mode)
- except AttributeError:
- pass
+ os.chmod(out_file, mode)
out_file = fp
opened_files.append(out_file)
#
diff --git a/Misc/NEWS.d/next/Library/2018-06-10-14-08-52.bpo-33687.1zZdnA.rst b/Misc/NEWS.d/next/Library/2018-06-10-14-08-52.bpo-33687.1zZdnA.rst
new file mode 100644
index 000000000000..63c5bfcac474
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2018-06-10-14-08-52.bpo-33687.1zZdnA.rst
@@ -0,0 +1,2 @@
+Fix the call to ``os.chmod()`` for ``uu.decode()`` if a mode is given or
+decoded. Patch by Timo Furrer.
More information about the Python-checkins
mailing list