Skip to content

Commit

Permalink
Release script binary upload bug, tarball bug
Browse files Browse the repository at this point in the history
Removes the '$CLI' portion of uploading the binaries / tarballs. Had a
naming error when uploading.

Updates it so we cd to `bin` so we can create the tarballs without the
`/bin` folder being located within it.
  • Loading branch information
cdrage committed Apr 28, 2017
1 parent 0f92e3f commit ef627ee
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions script/release.sh
Original file line number Diff line number Diff line change
Expand Up @@ -114,10 +114,13 @@ build_binaries() {
}

create_tarballs() {
for f in $CLI/bin/*
# cd into the bin directory so we don't have '/bin' inside the tarball
cd bin
for f in *
do
tar cvzf $f.tar.gz $f
done
cd ..
}

git_commit() {
Expand Down Expand Up @@ -172,10 +175,10 @@ push() {
fi

# Upload all the binaries and tarballs generated in bin/
for f in $CLI/bin/*
for f in bin/*
do
echo "Uploading file $f"
NAME=`echo $f | sed "s,$CLI/bin/,,g"`
NAME=`echo $f | sed "s,bin/,,g"`
github-release upload \
--user $UPSTREAM_REPO \
--repo $CLI \
Expand All @@ -198,7 +201,7 @@ push() {
}

clean() {
rm -r $CLI changes.txt
rm changes.txt
}

main() {
Expand Down

0 comments on commit ef627ee

Please sign in to comment.