From 5f84abc333fffb378d7a538ac2efda684d05881e Mon Sep 17 00:00:00 2001
From: vigress8 <150687949+vigress8@users.noreply.github.com>
Date: Wed, 11 Sep 2024 22:02:54 +0400
Subject: [PATCH] feat(Makefile): GO variable, several improvements

* add GO variable to specify the path to the `go` binary
* make `BINARY_NAME` a target, so `make install` only rebuilds if the binary is not present
* assign variables using `:=` (simple expansion) to avoid potential gotchas caused by recursive expansion
---
 Makefile | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/Makefile b/Makefile
index c5a732d9..084ebac8 100644
--- a/Makefile
+++ b/Makefile
@@ -1,11 +1,15 @@
-PREFIX=/usr/
-DESTDIR=/
-BINARY_NAME=apx
+PREFIX := /usr
+DESTDIR := /
+BINARY_NAME := apx
 
-all: build
+GO := go
 
-build:
-	CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -a -tags netgo -ldflags '-w -extldflags "-static"' -o ${BINARY_NAME}
+all: clean build
+
+build: ${BINARY_NAME}
+
+${BINARY_NAME}:
+	CGO_ENABLED=0 GOOS=linux GOARCH=amd64 ${GO} build -a -tags netgo -ldflags '-w -extldflags "-static"' -o $@
 
 install: build
 	install -Dm755 ${BINARY_NAME} ${DESTDIR}${PREFIX}/bin/${BINARY_NAME}
@@ -22,7 +26,7 @@ install-manpages:
 	chmod 644 ${DESTDIR}${PREFIX}/share/man/man1/apx*
 
 uninstall: uninstall-manpages
-	rm ${DESTDIR}${PREFIX}/bin/apx
+	rm ${DESTDIR}${PREFIX}/bin/${BINARY_NAME}
 	rm -rf ${DESTDIR}/etc/apx
 	rm -rf ${DESTDIR}${PREFIX}/share/apx
 
@@ -31,4 +35,4 @@ uninstall-manpages:
 
 clean:
 	rm -f ${BINARY_NAME}
-	go clean
+	${GO} clean