fix(build): use consistent dashed naming for release assets
All checks were successful
CI / Test (push) Successful in 25s
CI / Lint (push) Successful in 19s
CI / Build (push) Successful in 13s
Release / Create Release (push) Successful in 1m57s

- Move `-trimpath` flag before `-ldflags` in go build command for correct ordering.
- Update asset filename in install script from underscores to dashes for consistency.
This commit is contained in:
Greg Gauthier 2026-03-01 23:33:51 +00:00
parent b8482b6fb0
commit 9278c2c718
2 changed files with 2 additions and 2 deletions

View File

@ -33,7 +33,7 @@ jobs:
IFS='/' read -r OS ARCH <<< "$plat"
BIN="grokkit-${OS}-${ARCH}"
if [ "$OS" = "windows" ]; then BIN="${BIN}.exe"; fi
GOOS="$OS" GOARCH="$ARCH" go build -ldflags "-s -w -X 'gmgauthier.com/grokkit/internal/version.Version=${VERSION}' -X 'gmgauthier.com/grokkit/internal/version.Commit=${COMMIT}' -X 'gmgauthier.com/grokkit/internal/version.BuildDate=${DATE}' -trimpath" -o "build/${BIN}" .
GOOS="$OS" GOARCH="$ARCH" go build -trimpath -ldflags "-s -w -X 'gmgauthier.com/grokkit/internal/version.Version=${VERSION}' -X 'gmgauthier.com/grokkit/internal/version.Commit=${COMMIT}' -X 'gmgauthier.com/grokkit/internal/version.BuildDate=${DATE}'" -o "build/${BIN}" .
done
- name: Prepare assets
run: |

View File

@ -18,7 +18,7 @@ case &quot;$ARCH&quot; in
arm64|aarch64) ARCH=arm64 ;;
esac
ASSET=&quot;grokkit_${OS}_${ARCH}_${VERSION}.tar.gz&quot;
ASSET=&quot;grokkit-${OS}-${ARCH}-${VERSION}.tar.gz&quot;
echo &quot;Installing grokkit ${VERSION} for ${OS}/${ARCH}...&quot;