-
Notifications
You must be signed in to change notification settings - Fork 144
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
fixes for auditwheel to depend on wheel 0.34 #223
Conversation
Codecov Report
@@ Coverage Diff @@
## master #223 +/- ##
=======================================
Coverage 87.19% 87.19%
=======================================
Files 19 19
Lines 976 976
Branches 214 214
=======================================
Hits 851 851
Misses 86 86
Partials 39 39
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Czaki for the PR! This is a much needed upgrade. I just have one comment.
auditwheel/wheeltools.py
Outdated
@@ -27,6 +28,16 @@ class WheelToolsError(Exception): | |||
pass | |||
|
|||
|
|||
def open_for_csv(name, mode): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Given that auditwheel only runs on Python 3 and there is only one place where this function is called, we could safely replace that call with open(name, mode, newline='', encoding='utf-8')
. That avoids the unnecessary version check and makes the code more readable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I introduce this change. Could you restart failed steep. It fail on timeout error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Restarted the ppc64 build.
@lkollar When you think it will be next release? |
wheel in version 0.34 introduce fix for pypy tags pypa/wheel#328