Skip to content

bpo-32295: User friendly message when invoking bdist_wheel sans wheel package#4824

Closed
ewdurbin wants to merge 1 commit into
python:masterfrom
ewdurbin:distutils-clippy-bdist_wheel
Closed

bpo-32295: User friendly message when invoking bdist_wheel sans wheel package#4824
ewdurbin wants to merge 1 commit into
python:masterfrom
ewdurbin:distutils-clippy-bdist_wheel

Conversation

@ewdurbin

@ewdurbin ewdurbin commented Dec 12, 2017

Copy link
Copy Markdown
Member

Wheels are a well-known part of the Python packaging ecosystem at this point!

Many tutorials, gists, and other tools suggest to build wheels using setuptools and distutils, but may not remind users to install the wheel package.

Given that this does not interfere with experienced users implementing their own bdist_wheel command, it should be a welcome addition to help guide novice packagers in the right direction.

Result for a Python environment without the wheel package installed:

$ python3 setup.py bdist_wheel
usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
   or: setup.py --help [cmd1 cmd2 ...]
   or: setup.py --help-commands
   or: setup.py cmd --help

error: invalid command 'bdist_wheel'
note: The wheel package provides this command. See https://packaging.python.org/tutorials/distributing-packages/#wheels for information on packaging wheels

https://bugs.python.org/issue32295

Wheels are a well-known part of the Python packaging ecosystem at this point!

Many tutorials, gists, and other tools suggest to build wheels using setuptools and distutils, but may not remind users to install the wheel package.

Given that this does not interfere with experienced users implementing their own bdist_wheel command, it should be a welcome addition to help guide novice packagers in the right direction.
@the-knights-who-say-ni

Copy link
Copy Markdown

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept your contribution by verifying you have signed the PSF contributor agreement (CLA).

Unfortunately our records indicate you have not signed the CLA. For legal reasons we need you to sign this before we can look at your contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

Thanks again to your contribution and we look forward to looking at it!

@merwok merwok changed the title User friendly message when invoking bdist_wheel sans wheel package bpo-32295: User friendly message when invoking bdist_wheel sans wheel package Dec 12, 2017
@Mariatta

Copy link
Copy Markdown
Member

@merwok Any update? I saw in bpo that you'd like to close this PR.

@merwok

merwok commented Jun 15, 2018

Copy link
Copy Markdown
Member

Closing in the absence of counter-arguments on bpo.

@merwok merwok closed this Jun 15, 2018
@ewdurbin ewdurbin deleted the distutils-clippy-bdist_wheel branch June 19, 2018 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants