changeset:   99440:c65e135df1dc
branch:      3.5
parent:      99437:ae27ad306dbf
parent:      99439:b63fd82a8528
user:        R David Murray <rdmurray@bitdance.com>
date:        Fri Dec 04 23:04:37 2015 -0500
files:       Lib/compileall.py Lib/test/test_compileall.py Misc/ACKS Misc/NEWS
description:
Merge: #24903: Remove misleading error message to fix regression.


diff -r ae27ad306dbf -r c65e135df1dc Lib/compileall.py
--- a/Lib/compileall.py	Sat Dec 05 02:03:42 2015 +0000
+++ b/Lib/compileall.py	Fri Dec 04 23:04:37 2015 -0500
@@ -238,9 +238,6 @@
     args = parser.parse_args()
     compile_dests = args.compile_dest
 
-    if (args.ddir and (len(compile_dests) != 1
-            or not os.path.isdir(compile_dests[0]))):
-        parser.exit('-d destdir requires exactly one directory argument')
     if args.rx:
         import re
         args.rx = re.compile(args.rx)
diff -r ae27ad306dbf -r c65e135df1dc Lib/test/test_compileall.py
--- a/Lib/test/test_compileall.py	Sat Dec 05 02:03:42 2015 +0000
+++ b/Lib/test/test_compileall.py	Fri Dec 04 23:04:37 2015 -0500
@@ -398,14 +398,6 @@
         self.assertCompiled(init2fn)
         self.assertCompiled(bar2fn)
 
-    def test_d_takes_exactly_one_dir(self):
-        rc, out, err = self.assertRunNotOK('-d', 'foo')
-        self.assertEqual(out, b'')
-        self.assertRegex(err, b'-d')
-        rc, out, err = self.assertRunNotOK('-d', 'foo', 'bar')
-        self.assertEqual(out, b'')
-        self.assertRegex(err, b'-d')
-
     def test_d_compile_error(self):
         script_helper.make_script(self.pkgdir, 'crunchyfrog', 'bad(syntax')
         rc, out, err = self.assertRunNotOK('-q', '-d', 'dinsdale', self.pkgdir)
diff -r ae27ad306dbf -r c65e135df1dc Misc/ACKS
--- a/Misc/ACKS	Sat Dec 05 02:03:42 2015 +0000
+++ b/Misc/ACKS	Fri Dec 04 23:04:37 2015 -0500
@@ -477,6 +477,7 @@
 Nitin Ganatra
 Fred Gansevles
 Lars Marius Garshol
+Jake Garver
 Dan Gass
 Andrew Gaul
 Matthieu Gautier
diff -r ae27ad306dbf -r c65e135df1dc Misc/NEWS
--- a/Misc/NEWS	Sat Dec 05 02:03:42 2015 +0000
+++ b/Misc/NEWS	Fri Dec 04 23:04:37 2015 -0500
@@ -22,6 +22,10 @@
 Library
 -------
 
+- Issue #24903: Fix regression in number of arguments compileall accepts when
+  '-d' is specified.  The check on the number of arguments has been dropped
+  completely as it never worked correctly anyway.
+
 - Issue #25764: In the subprocess module, preserve any exception caused by
   fork() failure when preexec_fn is used.