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

Show all shared libraries found in purelib #143

Merged
merged 4 commits into from
Jun 5, 2019

Conversation

di
Copy link
Member

@di di commented Mar 21, 2019

Instead of just outputting the first shared library found in purelib, include a list of all offending files.

@di di requested a review from ehashman March 21, 2019 18:18
@di di force-pushed the list-all-shared-libraries-in-purelib branch from 2fd9f26 to 0e94f7a Compare March 21, 2019 20:07
@auvipy
Copy link
Contributor

auvipy commented Apr 12, 2019

Dustin, in case is it possible to add a test for the proposed changes? The code looks nice though

@di di force-pushed the list-all-shared-libraries-in-purelib branch from 0e94f7a to 7f64f91 Compare June 4, 2019 23:46
@di di requested review from auvipy and removed request for ehashman June 4, 2019 23:46
@di
Copy link
Member Author

di commented Jun 4, 2019

@auvipy I've added a test and resolved the merge conflicts.

auditwheel/wheel_abi.py Outdated Show resolved Hide resolved
@lkollar lkollar merged commit 7e2928e into pypa:master Jun 5, 2019
@di di deleted the list-all-shared-libraries-in-purelib branch June 5, 2019 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants