pip
pip copied to clipboard
pypa
Stop explicitly listing wheel as a build dependency
setuptools' get_requires_for_build_wheel() hook already injects this dependency when building wheels is requested.
See also 64d89385ce4b5ae2d05d176e7746bff0baaabafd.
cc @webknjaz
This should have no impact on users. I deem it a trivial change.
If a news entry is required anyway, what category should I use?
Yeah, it won't have any impact. I wonder how I missed it earlier..
Not sure if a change note is needed — I proposed having more granular categories some time ago but that discussion is stuck.. I'll let the maintainers decide.
None of the categories fit, so let's put it down as a feature since this is worth calling out in the changelog IMO.
For the record, I don't think this is worth calling out in the changelog. It only impacts users who build sdists of pip.
Nevertheless, the changelog fragment has been added.
FWIW, the audience is downstreams, which is separate from the end-users. Hence, my proposal @ https://github.com/pypa/pip/issues/12555.
I'll leave it to @pradyunsg as RM to decide if this is OK to go in 24.1 when we're already at beta 2.
For such a trivial change, the changelog discussion kinda makes me regret proposing it.
To whoever is merging this: feel free to adjust the changlog fragment however you desire.
For such a trivial change, the changelog discussion konda makes me regret proposing it.
To whoever is merging this: feel free to adjust the changlog fragment however you desire.
I don't think you should be angry or regret things. I think communicating a change is as important (or even more important) than the change itself. This is actually a very good discussion here, and various perspective have been presented, so @hroncok - if I were you I would be rather happy to see it and learn how open source maintainers think.
I think it's worth considering different maintainers perspectives on it, because they want to make sure that all kinds of users (pip end users but also - as mentioned earlier - downstream maintainers) - are aware of the change - and it's perfectly OK that different people have different opinions, so I would rather wait for @pradyunsg as RM to make final decision on it :).
I don't wish to spark any conflict here, but I do want to remind everyone that we're all volunteers. I can understand where @hroncok is coming from. I've been frustrated myself when one of my PRs is stuck in review hell, taking way longer than it should. I get that you're simply trying to offer another perspective @potiuk (which we can all agree is useful!) but saying someone shouldn't feel their anger right off the bat doesn't seem super nice (even though the second half of your message is cordial!).
Anyway, @hroncok apologies for the considerable back and forth on this PR. We've historically been burnt by various seemingly minor changes causing issues for downstream repackagers, so we are wary of making changes like this one. We appreciate the PR :)
