-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
need more specific diagnoses/error messages when uploading/registering #107
Comments
Author-Email accepts many forms, but only one value: https://www.python.org/dev/peps/pep-0345/#author-email-optional (I agree on your other points about developer experience) |
@MikeHart85 You are correct, PyPI was being overly restrictive with the |
@di Cool, thank you! |
@MikeHart85 Just to double-check -- now that this is addressed in Warehouse, shall we talk about the other related points you brought up? On whether We've removed |
@brainwane Adding publication capabilities to pip is far from being an agreed proposal, since it's like saying "I should be able to use my eReader to publish new books as well downloading and reading existing ones". Publishing & maintaining new libraries is a very different activity from installing and using existing ones, and has a correspondingly steeper learning curve. |
Thanks, @ncoghlan - I was going off an older comment by Donald that I took as a decision that had already been made. Other folks who want to take a look at that conversation: it's ongoing in #60. @MikeHart85 Since we've addressed some of the concerns you brought up and are addressing others elsewhere, I'm taking the liberty of changing the title and prepending a problem restatement on this issue. Thanks. |
After further conversation below, this issue is now focused on:
This isn't a very useful error message... unless it's really just a random disconnect, which it isn't here.
Potentially related to:
pypi/warehouse#2533
pypi/legacy#675
pypa/twine#263
I was getting much the same behaviour. Turned out to be caused by a tiny detail in
setup.py
in my case.Using
twine register
on any package ortwine upload dist/*
would return499 Client Error: Client Disconnected
every time after a bit of delay. In the case oftwine upload
, it even appeared to upload but then failed after reaching 100%.Then I tried uploading specifically the
.tar.gz
instead ofdist/*
(which defaulted to attempting.whl
first):$ twine upload dist/project.tar.gz
That's different!(?!)
Our
author_email
field has always followed a"Some Name <[email protected]>, Other Name <...>, ..."
format to list multiple authors. This has always worked fine in the past. Because of that last error, I tried changing it to simply:author_email="[email protected]"
. Suddenly, everything works again:Thoughts on this:
author_email
should not be a failure condition. It's just a string with some meta information.twine register
exist, especially now that it's not necessary / doesn't work?twine
really need to exist? Why not simplypip upload
? Or an upload button on the website?I don't think this is the exact issue @s-m-e is having, since the
author_email
in his project is just a single email. But it could be some other detail in his setup.py isn't formatted right, or maybe it's something else entirely. It's hard to tell how related the issues are... because this isn't a very useful error message.The text was updated successfully, but these errors were encountered: