-
Notifications
You must be signed in to change notification settings - Fork 247
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Script for building manylinux wheels
- Loading branch information
Showing
2 changed files
with
16 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,6 +5,7 @@ | |
build/ | ||
dist/ | ||
.tox/ | ||
wheelhouse/ | ||
greenlet.egg-info/ | ||
/doc/_build | ||
__pycache__/ | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
#!/bin/bash | ||
set -e | ||
|
||
if [ -d /greenlet -a -d /opt/python ]; then | ||
# Running inside docker | ||
cd /greenlet | ||
for version in cp27-cp27mu cp34-cp34m cp35-cp35m; do | ||
/opt/python/${version}/bin/pip wheel greenlet | ||
auditwheel repair greenlet*${version}*.whl | ||
rm greenlet*${version}*whl | ||
done | ||
exit 0 | ||
fi | ||
|
||
docker run --rm -ti -v "$(pwd):/greenlet" quay.io/pypa/manylinux1_x86_64 /greenlet/make-manylinux |
1a45be2
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.
Our TravisCI build picked up new wheels, and because we're using hashed requirements, it broke our TravisCI build. This isn't a problem, we can update our hashes, but I want to confirm: Did you upload these new wheels to PyPI over the weekend?
1a45be2
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.
Yes, I built and uploaded them in response to #100.
1a45be2
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.
To be honest given #101 I think that was a mistake, but I don't want to make matters even worse by deleting them now.
1a45be2
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 for the heads-up - I'll test accordingly.
1a45be2
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.
After some more thinking I pulled manylinux1 wheels for greenlet 0.4.9, since uploading them in the first place was a big mistake. I will bring them back in greenlet 0.4.10.