Éric Araujo added the comment:
distutils2/packaging development has stopped.
--
resolution: - out of date
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14183
Francisco Martín Brugué franci...@email.de added the comment:
I've updated the patch. Let me know if something has to be changed.
--
Added file: http://bugs.python.org/file24727/issue14183_fbb9847b8f43.patch
___
Python tracker rep...@bugs.python.org
Éric Araujo mer...@netwok.org added the comment:
Thanks for the patch.
-# Test that the isntalled raises an exception if the project does not
+# Test that the installed raises an exception if the project does not
It took me many seconds to find the change :)
+def
Francisco Martín Brugué franci...@email.de added the comment:
Hi Éric,
- # Test that the isntalled raises an exception if the project does not
+ # Test that the installed raises an exception if the project does not
It took me many seconds to find the change :)
The editor told me :)
+
Nadeem Vawda nadeem.va...@gmail.com added the comment:
-install.install_dists = lambda x, y=None: None
+install.install_dists = lambda x, y = None: None
PEP 8: Never put spaces in a function (or lambda) signature.
I cannot find that exactly in pep8.
From the section
Francisco Martín Brugué franci...@email.de added the comment:
Nadeem:
- Don't use spaces around the '=' sign when used to indicate a
keyword argument or a default parameter value.
ok, code formating is not working as expected (at least for lambdas...)
Éric :
If you grep packaging tests