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

Handle standalone pip creation from pip wheel #9960

Merged
merged 1 commit into from
May 21, 2021
Merged

Conversation

abn
Copy link
Contributor

@abn abn commented May 6, 2021

This change ensures that when pip is executed from a wheel/zip,
standalone pip creation for build environment reuses the source.

Resolves: #9953

@uranusjr
Copy link
Member

uranusjr commented May 7, 2021

Actually I think maybe it’s best if we just remove the filename check altogether. If it’s gonna fail, it’s gonna fail either way, so the check doesn’t really mean much, now that I look at it. Just the if not source.is_dir() part along should be enough.

@abn
Copy link
Contributor Author

abn commented May 7, 2021

Made the suggested change.

@abn
Copy link
Contributor Author

abn commented May 7, 2021

I am guessing someone needs to approve the GitHub CI runs?

@abn abn force-pushed the issue/9953 branch 2 times, most recently from b5f1060 to c5501f0 Compare May 8, 2021 22:12
@abn abn requested a review from uranusjr May 8, 2021 22:12
@abn
Copy link
Contributor Author

abn commented May 8, 2021

@uranusjr applied your suggestions as-is.

This change ensures that when pip is executed from a wheel/zip,
standalone pip creation for build environment reuses the source.

Resolves: pypa#9953
Co-authored-by: Tzu-ping Chung <[email protected]>
@uranusjr uranusjr added this to the 21.1.2 milestone May 9, 2021
@abn
Copy link
Contributor Author

abn commented May 9, 2021

Looks like GitHub actions might need another "Approve and run".

@sbidoul sbidoul merged commit 1f68338 into pypa:main May 21, 2021
@abn abn deleted the issue/9953 branch May 24, 2021 20:15
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sdist installation fails when pip is run from wheel
3 participants