chore: refactor Makefile (#354)

* refactor: extract build logic from build.sh to Makefile

* chore: add test rule to Makefile

* apply review suggestion

Co-authored-by: Abiola Ibrahim <[email protected]>

* fix: use defined `OUTPUT_DIR` variable

* chore: remove -race flag from test as it needs CGO_ENABLED=1

* chore: generate sha in binaries directory

* chore: propagate Go build environment variables

Co-authored-by: Abiola Ibrahim <[email protected]>

Author: tricktron

Makefile

@@ -8,27 +8,53 @@ GOARCH_aarch64 = arm64
 GOARCH_arm64 = arm64
 GOARCH ?= $(shell echo "$(GOARCH_$(ARCH))")
 
+VERSION := $(shell git describe --tags --always)
+REVISION := $(shell git rev-parse HEAD)
+PACKAGE := github.com/abiosoft/colima/config
+VERSION_VARIABLES := -X $(PACKAGE).appVersion=$(VERSION) -X $(PACKAGE).revision=$(REVISION)
+
+OUTPUT_DIR := _output/binaries
+OUTPUT_BIN := colima-$(OS)-$(ARCH)
+INSTALL_DIR := /usr/local/bin
+BIN_NAME := colima
+
+LDFLAGS := $(VERSION_VARIABLES)
+
+.PHONY: all
 all: build 
 
+.PHONY: clean
 clean:
 	rm -rf _output _build
 
+.PHONY: gopath
 gopath:
 	go get -v ./cmd/colima
 
+.PHONY: fmt
 fmt:
 	go fmt ./...
 	goimports -w .
 
+.PHONY: build
 build:
-	GOOS=$(GOOS) GOARCH=$(GOARCH) OS=$(OS) ARCH=$(ARCH) sh scripts/build.sh
+	GOOS=$(GOOS) GOARCH=$(GOARCH) go build -ldflags="$(LDFLAGS)" -o $(OUTPUT_DIR)/$(OUTPUT_BIN) ./cmd/colima
+	cd $(OUTPUT_DIR) && openssl sha256 -r -out $(OUTPUT_BIN).sha256sum $(OUTPUT_BIN)
+
+.PHONY: test
+test:
+	go test -v -ldflags="$(LD_FLAGS)" ./cmd/...
 
+.PHONY: vmnet
 vmnet:
 	sh scripts/build_vmnet.sh
 
+.PHONY: install
 install:
-	cp _output/binaries/colima-$(OS)-$(ARCH) /usr/local/bin/colima
-	chmod +x /usr/local/bin/colima
+	mkdir -p $(INSTALL_DIR)
+	rm -f $(INSTALL_DIR)/$(BIN_NAME)
+	cp $(OUTPUT_DIR)/colima-$(OS)-$(ARCH) $(INSTALL_DIR)/$(BIN_NAME)
+	chmod +x $(INSTALL_DIR)/$(BIN_NAME)
 
 .PHONY: lint
 lint: ## Assumes that golangci-lint is installed and in the path.  To install: https://golangci-lint.run/usage/install/