Skip to content
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

bdist_wheel failing with setuptools with bad wheel name #265

Closed
OmSaran opened this issue Oct 4, 2018 · 6 comments
Closed

bdist_wheel failing with setuptools with bad wheel name #265

OmSaran opened this issue Oct 4, 2018 · 6 comments
Labels

Comments

@OmSaran
Copy link

OmSaran commented Oct 4, 2018

python setup.py bdist_wheel uses bdist_wheel.py from wheel package. Since version 0.32.0 it no longer works with version names that does not match https://github.com/pypa/wheel/blob/master/wheel/wheelfile.py#L16:L19

Version names not matching the above regex did not fail with older versions of wheel.
Also PEP 427 does not enforce using PEP 440 standard version names.

@benoit-pierre
Copy link
Member

benoit-pierre commented Oct 4, 2018

What's the name of your wheel?

@benoit-pierre
Copy link
Member

FYI, this is the regex used by pip: https://github.com/pypa/pip/blob/master/src/pip/_internal/wheel.py#L572

Way more lax indeed...

@agronholm
Copy link
Contributor

Need more information. Can you point me to your project or at least give the name of the wheel name?

@OmSaran
Copy link
Author

OmSaran commented Oct 5, 2018

@agronholm
The version name is branch_commit
This is the error seen with wheel version > 0.32.0

wheel.cli.WheelError: Bad wheel filename 'TestProject-branch_commit-py2-none-any.whl'

The same works fine with wheel version <= 0.31.1

@benoit-pierre
The regex used in pip was relaxed via pypa/pip#4384
(refer pypa/pip#4169)

@agronholm
Copy link
Contributor

@OmSaran Can you verify that the fix works for you?

@OmSaran
Copy link
Author

OmSaran commented Oct 8, 2018

@agronholm It works, thanks for the fix!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants