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

Warn when pip is not building a wheel due to lack of wheel package #7768

Merged
merged 5 commits into from Feb 23, 2020
Merged

Warn when pip is not building a wheel due to lack of wheel package #7768

merged 5 commits into from Feb 23, 2020

Conversation

anudit
Copy link
Contributor

@anudit anudit commented Feb 22, 2020

Add a logger.info to log a message about why pip is not building a wheel. In reference to #7709

@anudit
Copy link
Contributor Author

anudit commented Feb 22, 2020

@anudit
Copy link
Contributor Author

anudit commented Feb 23, 2020

All checks passing now.

@anudit anudit requested review from sbidoul and chrahunt and removed request for sbidoul Feb 23, 2020
@uranusjr
Copy link
Member

uranusjr commented Feb 23, 2020

A couple of thoughts. Both are minor and I can live with it being merged without change though:

  • This could use a news entry, since users can see the addition.
  • I would say something like Could not build wheels for XXX since package wheel is not installed. The word “legacy” is’t particularly relevent to the user, and the word “wheel” appearing twice in the sentence might confuse the user. Also “skipping” sounds like we’re not installing the package at all (but in fact we are, only with a different method).

@anudit anudit requested a review from uranusjr Feb 23, 2020
Copy link
Member

@pradyunsg pradyunsg left a comment

Minor nit-pick wrt the NEWS fragment. LGTM otherwise!

news/7768.feature Outdated Show resolved Hide resolved
@xavfernandez xavfernandez merged commit e6ed38d into pypa:master Feb 23, 2020
36 checks passed
@xavfernandez
Copy link
Member

xavfernandez commented Feb 23, 2020

Thanks @anuditnagar 👍

@lock lock bot added the auto-locked Outdated issues that have been locked by automation label Mar 27, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Mar 27, 2020
@pradyunsg pradyunsg changed the title Add Logger Alert Warn when pip is not building a wheel for installation Apr 21, 2020
@pradyunsg pradyunsg changed the title Warn when pip is not building a wheel for installation Warn when pip is not building a wheel due to lack of wheel package Apr 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto-locked Outdated issues that have been locked by automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants