Issue 7677: upload: improve display for error messages from gpg (original) (raw)

Created on 2010-01-11 15:22 by illume, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
patchfix Parimala.Rao,2014-03-19 17:29 Improved Error message review
Messages (7)
msg97589 - (view) Author: Rene Dudfield (illume) Date: 2010-01-11 15:22
hi, When using setup.py upload --sign without --identity, gpg can give weird messages. Rather than this error message: """running upload gpg --detach-sign --local-user gn -a dist/pywebsite-0.1.18pre.tar.gz gpg: skipped "gn": secret key not available gpg: signing failed: secret key not available error: command 'gpg' failed with exit status 2 make: *** [upload] Error 1""" It might be nicer to also mention using --identity=, as well as a link to the documentation for --sign (url and also the help from "--help upload"). cheers,
msg101630 - (view) Author: Zubin Mithra (zubin71) Date: 2010-03-24 12:25
are you suggesting that a warning stating that '--identity has`nt been used' has to be generated?
msg159660 - (view) Author: Éric Araujo (eric.araujo) * (Python committer) Date: 2012-04-29 23:47
We had a look at this bug during a sprint last week. First, it is reasonable to use --sign without --identity, as gpg will fall back to a default identity (see the description of --local-user and --default-key in the gpg man page). Second, it looks like you run distutils commands from a script or makefile and you have a bug there (the “skipped "gn"” message looks suspicious, maybe -sign was used instead of --sign and interpreted as -s -i gn). Finally, even though we thing this is not a bug, we agreed that it would be nicer to print a message to make clear that distutils2 is not broken but shows the error message from gpg on purpose; this may be done during the next sprint.
msg213460 - (view) Author: Éric Araujo (eric.araujo) * (Python committer) Date: 2014-03-13 20:32
Improving error messages is an acceptable change for stable branches IMO.
msg213943 - (view) Author: Éric Araujo (eric.araujo) * (Python committer) Date: 2014-03-18 04:52
Notes for contributors: * Ignore what I said about distutils2, it’s not an active project anymore. This ticket is about distutils in the standard library. * To test changes, you can run setup.py commands without really uploading stuff to PyPI, there is a special sandbox repository for that: https://wiki.python.org/moin/TestPyPI
msg214114 - (view) Author: Parimala Rao (Parimala.Rao) * Date: 2014-03-19 17:29
This error is seen when there are no secret keys/default keys present for the person running "upload" command. Hence using --sign does not fall back to default identity.
msg386409 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2021-02-03 18:29
Distutils is now deprecated (see PEP 632) and all tagged issues are being closed. From now until removal, only release blocking issues will be considered for distutils. If this issue does not relate to distutils, please remove the component and reopen it. If you believe it still requires a fix, most likely the issue should be re-reported at https://github.com/pypa/setuptools
History
Date User Action Args
2022-04-11 14:56:56 admin set github: 51926
2021-02-03 18:29:05 steve.dower set status: open -> closednosy: + steve.dowermessages: + resolution: out of datestage: needs patch -> resolved
2014-03-19 17:29:19 Parimala.Rao set files: + patchfixnosy: + Parimala.Raomessages: +
2014-03-18 04:52:35 eric.araujo set messages: +
2014-03-13 20:32:36 eric.araujo set messages: + components: + Distutils, - Distutils2versions: + Python 2.7, Python 3.4, - 3rd party
2012-04-29 23:47:52 eric.araujo set title: improve error message for setup.py upload --sign without --identity -> upload: improve display for error messages from gpg
2012-04-29 23:47:05 eric.araujo set versions: + Python 3.3, - Python 3.1, Python 2.7, Python 3.2nosy: + alexismessages: + assignee: tarek -> eric.araujocomponents: - Distutils
2010-11-18 01:56:56 eric.araujo set keywords: + easynosy:illume, tarek, eric.araujo, zubin71title: distutils, better error message for setup.py upload -sign without identity. -> improve error message for setup.py upload --sign without --identitycomponents: + Distutils2versions: + 3rd party, - Python 2.6
2010-04-09 00:04:34 eric.araujo set nosy: + eric.araujo
2010-03-24 12:25:23 zubin71 set nosy: + zubin71messages: +
2010-01-11 15:31:31 brian.curtin set priority: normalstage: needs patchversions: + Python 2.6, Python 3.1, Python 2.7, Python 3.2
2010-01-11 15:22:33 illume create