Remove required metadata warnings for sdist

[voidspace]

BPO	7726
Nosy	@tarekziade, @merwok, @voidspace

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/tarekziade'
closed_at = <Date 2020-10-22.21:27:35.096>
created_at = <Date 2010-01-17.17:13:15.635>
labels = ['type-feature', 'library']
title = 'Remove required metadata warnings for sdist'
updated_at = <Date 2020-10-22.21:27:35.095>
user = 'https://github.com/voidspace'

bugs.python.org fields:

activity = <Date 2020-10-22.21:27:35.095>
actor = 'eric.araujo'
assignee = 'tarek'
closed = True
closed_date = <Date 2020-10-22.21:27:35.096>
closer = 'eric.araujo'
components = ['Distutils']
creation = <Date 2010-01-17.17:13:15.635>
creator = 'michael.foord'
dependencies = []
files = []
hgrepos = []
issue_num = 7726
keywords = []
message_count = 4.0
messages = ['97957', '119369', '228454', '379356']
nosy_count = 4.0
nosy_names = ['tarek', 'eric.araujo', 'michael.foord', 'francismb']
pr_nums = []
priority = 'normal'
resolution = 'wont fix'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue7726'
versions = ['Python 2.7', 'Python 3.2']

[voidspace]

When building releases with sdist (and other distribution building options) missing metadata should not cause warnings. For public projects this is useful (i.e. projects uploaded to PyPI), but not for internally built distributions. (If a distribution is not public then why should it have a URL for example.)

[merwok]

I don’t think warnings should be removed, at least not by default. They’re warnings, not errors, which is IMO a nice compromise between accepting anything and requiring that rules be followed. We could add an option like --no-warnings in distutils2, but I’m worried people could just disable it globally instead of fixing the actual problems. I’m inclined to reject this report. Tarek, opinion?

Since the warnings go to stderr, you can suppress them with shell redirection, although I wouldn’t advise that since you could miss unrelated warnings or errors. Does that cover your need?

*musing* Maybe we could revamp the required fields. Name and version are required by various distutils[2] and pkgutil APIs, so not setting them will screw up queries or uninstallation for example. We could make distutils2 fail when they’re not present. But then, why not fail if a distribution to be installed has the same name+version that an already present distribution? What I’m saying is that it’s better to do no checks than incomplete checks, and this is the realm of a distribution manager, not the underlying library, but then again, distutils2 provides a simple distribution manager too, so I’m not sure this should be in or out of it, and at which level (lib/command/manager).

(Removing Terry from nosy at his request in private email.)

[francismb]

AFAICS, this is an enhancement issue for the currently in maintenance branches 2.7 and 3.2.

Does this still applies to 3.5? (otherwise this issue should/could be closed).

[merwok]

Distutils won’t be changed at this point in time, this request should go to setuptools.