Skip to content

Commit c4397da

Browse files
committedOct 10, 2021
Final fix for building release GitHub asset
Signed-off-by: Jordan Borean <[email protected]>
1 parent 9889db2 commit c4397da

File tree

2 files changed

+7
-16
lines changed

2 files changed

+7
-16
lines changed
 

‎.github/workflows/release.yml

-3
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,6 @@ jobs:
88
runs-on: ubuntu-latest
99

1010
steps:
11-
- name: Setup Python
12-
uses: actions/setup-python@v2
13-
1411
- name: Check out code
1512
uses: actions/checkout@v2
1613

‎ci/before-deploy.sh

+7-13
Original file line numberDiff line numberDiff line change
@@ -12,26 +12,18 @@ yum -y install tar git
1212
# build the docs
1313
deploy::build-docs
1414

15-
# NB(directxman12): this is a *terrible* hack, but basically,
16-
# `twine` gets called like this:
17-
# - python setup.py $PYPI_DISTRIBUTIONS
18-
# - twine upload -r pypi dist/*
19-
# - [some other stuff]
20-
#
21-
# so if we set $PYPI_DISTRIBUTIONS to something harmless, like `check`,
22-
# and then build the dist ourselves (and save it from the cleanup),
23-
# dpl will upload that
24-
25-
# build the sdist
15+
# build the sdist and save the dirs before the clean
2616
python setup.py sdist
2717
mv dist dist_saved
18+
mv .venv .venv_saved
2819

2920
# for the tarball upload
3021
# clean up
31-
git clean -Xdf
22+
git clean -Xdf --exclude .venv
3223

33-
# restore the saved "dist" directory
24+
# restore the saved "dist"/".venv" directory
3425
mv dist_saved dist
26+
mv .venv_saved .venv
3527

3628
# make the dir
3729
rm -rf ./tag_build || true
@@ -53,6 +45,7 @@ tar -cvf ./tag_build/${PKG_NAME_VER}.tar \
5345
--exclude='tag_build' \
5446
--exclude='.git' \
5547
--exclude='travis_docs_build' \
48+
--exclude='.venv' \
5649
--exclude='README.rst' \
5750
--transform="s,^\.,${PKG_NAME_VER}," .
5851

@@ -67,3 +60,4 @@ EOF
6760
gzip ./tag_build/${PKG_NAME_VER}.tar
6861

6962
sha512sum --binary ./tag_build/${PKG_NAME_VER}.tar.gz > ./tag_build/${PKG_NAME_VER}.sha512sum
63+

0 commit comments

Comments
 (0)
Please sign in to comment.