Antoine Pitrou added the comment:
Thank you Martin. I will take a look soon at the patch.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21722
___
Roundup Robot added the comment:
New changeset d86214c98a9c by Antoine Pitrou in branch '3.4':
Issue #21722: The distutils upload command now exits with a non-zero return
code when uploading fails.
http://hg.python.org/cpython/rev/d86214c98a9c
New changeset 979aaa08c067 by Antoine Pitrou in
Antoine Pitrou added the comment:
I've now committed the fix. Thank you for your contribution!
--
resolution: - fixed
stage: - resolved
status: open - closed
type: - behavior
versions: +Python 3.4
___
Python tracker rep...@bugs.python.org
Roundup Robot added the comment:
New changeset cf70f030a744 by Antoine Pitrou in branch '2.7':
Issue #21722: The distutils upload command now exits with a non-zero return
code when uploading fails.
http://hg.python.org/cpython/rev/cf70f030a744
--
Martin Dengler added the comment:
I've got the contrib form bit now.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21722
___
___
Martin Dengler added the comment:
The attached file is a patch for backporting this fix to 2.7.7.
--
Added file:
http://bugs.python.org/file35573/cpython2-patch-Lib-distutils-command-upload.py.patch
___
Python tracker rep...@bugs.python.org
New submission from Martin Dengler:
This patch teaches distutils/command/upload.py to return a non-zero exit code
when uploading fails. Currently a zero error code is returned (specifically,
no Exception is raised by upload.run(...)) regardless of the HTTP response from
the server or any
Changes by Ned Deily n...@acm.org:
--
components: +Distutils
nosy: +dstufft, eric.araujo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21722
___
Antoine Pitrou added the comment:
Thanks you for the patch. It may be good to add a unit test for this.
Also, for development, it is probably better to use a Mercurial clone instead
of trying to generate diffs by hand. See the devguide for more information:
https://docs.python.org/devguide/
Martin Dengler added the comment:
Here is the patch against hg tip.
--
Added file:
http://bugs.python.org/file35581/cpython-patch-Lib-distutils-command-upload.py.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21722
Changes by Martin Dengler mar...@martindengler.com:
Removed file:
http://bugs.python.org/file35573/cpython2-patch-Lib-distutils-command-upload.py.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21722
Martin Dengler added the comment:
Here is the patch against hg 2.7.
--
Added file:
http://bugs.python.org/file35582/cpython2-patch-Lib-distutils-command-upload.py.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21722
Changes by Martin Dengler mar...@martindengler.com:
Removed file:
http://bugs.python.org/file35572/cpython-patch-Lib-distutils-command-upload.py.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21722
Martin Dengler added the comment:
I will submit patches with tests as soon as the before after tests finish.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21722
___
Martin Dengler added the comment:
Here is the patch against tip, including a new test case.
--
Added file:
http://bugs.python.org/file35583/cpython-patch-Lib-distutils-command-upload.py.patch
___
Python tracker rep...@bugs.python.org
Martin Dengler added the comment:
Here is the patch against 2.7, including a new test case.
--
Added file:
http://bugs.python.org/file35584/cpython2-patch-Lib-distutils-command-upload.py.patch
___
Python tracker rep...@bugs.python.org
Martin Dengler added the comment:
If you'd like me to change anything about the test case please let me know.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21722
___
17 matches
Mail list logo