-
Notifications
You must be signed in to change notification settings - Fork 47
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[Travis] Work around Windows not officially supporting python
- Loading branch information
1 parent
826c02d
commit c5d2be5
Showing
3 changed files
with
45 additions
and
18 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
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
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,26 @@ | ||
#!/bin/bash -e | ||
|
||
# Temporary hack while issue DPL issue persists | ||
# Manually upload wheels via twine for windows | ||
# https://github.com/travis-ci/dpl/issues/1009 | ||
|
||
success="yes" | ||
|
||
# Sigh, go find paths again | ||
PYPATH="/c/Python${PYENV:0:1}${PYENV:2:1}" | ||
export PATH="$PYPATH:$PYPATH/Scripts:/c/Program Files/MIT/Kerberos/bin:$PATH" | ||
|
||
echo 'Running: python -m pip install twine ...' | ||
python -m pip install twine | ||
|
||
echo 'Running: set +x; python -m twine upload...' | ||
# Please note this cannot be set -x or passwords will leak! | ||
set +x | ||
|
||
python -m twine upload -u $TWINE_USER -p $TWINE_PASSWORD dist/gssapi* > out.log 2>&1 || true | ||
|
||
# and restore... | ||
set -x | ||
egrep -i 'fail|error' out.log && cat out.log && exit 1 | ||
|
||
exit 0 |
c5d2be5
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.
Looks great. I like the solution of the repo env variables paired with temporarily disabling command echoing. Hopefully the deployment bugs will be ironed out soon so this can be simplified.