Merge pull request #389 from benmoss/fix-verify-checksums

Don't dirty the working directory with release checksums
carvel-dev · May 11, 2022 · 2fcf08d · 2fcf08d
2 parents 714912c + d575388
commit 2fcf08d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -87,9 +87,9 @@ jobs:
           set -e -x
           VERSION=`echo ${{ github.ref }}  | grep -Eo '[0-9].*'`
 
-          ./hack/build-binaries.sh "$VERSION" > ./go-checksums
-          cat ./go-checksums
-          diff ./go-checksums <(cat <<EOF
+          ./hack/build-binaries.sh "$VERSION" > /tmp/go-checksums
+          cat /tmp/go-checksums
+          diff /tmp/go-checksums <(cat <<EOF
           ${{steps.get-checksums-from-draft-release.outputs.result}}
           EOF
           )
diff --git a/hack/build.sh b/hack/build.sh
@@ -21,7 +21,7 @@ git diff --exit-code vendor/github.com/vdemeester || {
 }
 
 # export GOOS=linux GOARCH=amd64
-go build -ldflags="$LDFLAGS" -trimpath -o "imgpkg${IMGPKG_BINARY_EXT-}" ./cmd/imgpkg/...
+go build -trimpath -o "imgpkg${IMGPKG_BINARY_EXT-}" ./cmd/imgpkg/...
 ./imgpkg version
 
 # compile tests, but do not run them: https://github.com/golang/go/issues/15513#issuecomment-839126426