Skip to content

Commit 5c7501b

Browse files
authored
Merge pull request #52 from hoesler/master
fix: make dist should put diff binaries in bin folder
2 parents 0cafd7c + e229181 commit 5c7501b

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

Makefile

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,14 +31,14 @@ endif
3131
dist: export COPYFILE_DISABLE=1 #teach OSX tar to not put ._* files in tar archive
3232
dist:
3333
rm -rf build/diff/* release/*
34-
mkdir -p build/diff release/
34+
mkdir -p build/diff/bin release/
3535
cp README.md LICENSE plugin.yaml build/diff
36-
GOOS=linux GOARCH=amd64 go build -o build/diff/diff -ldflags="$(LDFLAGS)"
36+
GOOS=linux GOARCH=amd64 go build -o build/diff/bin/diff -ldflags="$(LDFLAGS)"
3737
tar -C build/ -zcvf $(CURDIR)/release/helm-diff-linux.tgz diff/
38-
GOOS=darwin GOARCH=amd64 go build -o build/diff/diff -ldflags="$(LDFLAGS)"
38+
GOOS=darwin GOARCH=amd64 go build -o build/diff/bin/diff -ldflags="$(LDFLAGS)"
3939
tar -C build/ -zcvf $(CURDIR)/release/helm-diff-macos.tgz diff/
40-
rm build/diff/diff
41-
GOOS=windows GOARCH=amd64 go build -o build/diff/diff.exe -ldflags="$(LDFLAGS)"
40+
rm build/diff/bin/diff
41+
GOOS=windows GOARCH=amd64 go build -o build/diff/bin/diff.exe -ldflags="$(LDFLAGS)"
4242
tar -C build/ -zcvf $(CURDIR)/release/helm-diff-windows.tgz diff/
4343

4444
.PHONY: release

install-binary.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ installFile() {
9797
HELM_TMP="/tmp/$PROJECT_NAME"
9898
mkdir -p "$HELM_TMP"
9999
tar xf "$PLUGIN_TMP_FILE" -C "$HELM_TMP"
100-
HELM_TMP_BIN="$HELM_TMP/diff/diff"
100+
HELM_TMP_BIN="$HELM_TMP/diff/bin/diff"
101101
echo "Preparing to install into ${HELM_PLUGIN_PATH}"
102102
mkdir -p "$HELM_PLUGIN_PATH/bin"
103103
cp "$HELM_TMP_BIN" "$HELM_PLUGIN_PATH/bin"

0 commit comments

Comments
 (0)