Skip to content

Commit c338b2c

Browse files
authored
Merge pull request #3692 from xrstf/uget
use a script to download premade binaries instead of compiling them
2 parents a738325 + 55f22c0 commit c338b2c

7 files changed

Lines changed: 683 additions & 100 deletions

File tree

Makefile

Lines changed: 35 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,10 @@ ifeq ($(CI),true)
2323
$(shell git config --global --add safe.directory '*')
2424
endif
2525

26-
GO_INSTALL = ./hack/go-install.sh
27-
28-
TOOLS_DIR=hack/tools
26+
TOOLS_DIR = hack/tools
27+
export UGET_DIRECTORY = $(TOOLS_DIR)
28+
export UGET_CHECKSUMS = hack/tools.checksums
29+
export UGET_VERSIONED_BINARIES = true
2930
ROOT_DIR=$(abspath .)
3031
TOOLS_GOBIN_DIR := $(abspath $(TOOLS_DIR))
3132
GOBIN_DIR=$(abspath ./bin)
@@ -50,20 +51,20 @@ YAML_PATCH_BIN := yaml-patch
5051
YAML_PATCH := $(TOOLS_DIR)/$(YAML_PATCH_BIN)-$(YAML_PATCH_VER)
5152
export YAML_PATCH # so hack scripts can use it
5253

53-
GOLANGCI_LINT_VER := v2.1.6
54+
GOLANGCI_LINT_VER := 2.1.6
5455
GOLANGCI_LINT_BIN := golangci-lint
5556
GOLANGCI_LINT := $(TOOLS_GOBIN_DIR)/$(GOLANGCI_LINT_BIN)-$(GOLANGCI_LINT_VER)
5657
GOLANGCI_LINT_FLAGS ?=
5758

58-
HTTEST_VER := v0.3.2
59+
HTTEST_VER := 0.3.4
5960
HTTEST_BIN := httest
60-
HTTEST := $(TOOLS_GOBIN_DIR)/$(HTTEST_BIN)-$(HTTEST_VER)
61+
export HTTEST := $(TOOLS_GOBIN_DIR)/$(HTTEST_BIN)-$(HTTEST_VER)
6162

62-
GOTESTSUM_VER := v1.12.3
63+
GOTESTSUM_VER := 1.12.3
6364
GOTESTSUM_BIN := gotestsum
6465
GOTESTSUM := $(abspath $(TOOLS_DIR))/$(GOTESTSUM_BIN)-$(GOTESTSUM_VER)
6566

66-
LOGCHECK_VER := v0.9.0
67+
LOGCHECK_VER := d35c84c015fe03a1421e5f2ce1e3c0c3bc38d077
6768
LOGCHECK_BIN := logcheck
6869
LOGCHECK := $(TOOLS_GOBIN_DIR)/$(LOGCHECK_BIN)-$(LOGCHECK_VER)
6970
export LOGCHECK # so hack scripts can use it
@@ -129,14 +130,29 @@ install: require-jq require-go require-git verify-go-versions ## Install the pro
129130
.PHONY: install
130131

131132
$(GOLANGCI_LINT):
132-
GOBIN=$(TOOLS_GOBIN_DIR) $(GO_INSTALL) github.com/golangci/golangci-lint/v2/cmd/golangci-lint $(GOLANGCI_LINT_BIN) $(GOLANGCI_LINT_VER)
133+
@hack/uget.sh \
134+
https://github.com/golangci/golangci-lint/releases/download/v{VERSION}/golangci-lint-{VERSION}-{GOOS}-{GOARCH}.tar.gz \
135+
${GOLANGCI_LINT_BIN} \
136+
${GOLANGCI_LINT_VER}
137+
138+
# help staging modules to install the same linter version
139+
golangci-lint-version:
140+
@echo $(GOLANGCI_LINT_VER)
141+
.PHONY: golangci-lint-version
133142

134143
$(HTTEST):
135-
GOBIN=$(TOOLS_GOBIN_DIR) $(GO_INSTALL) go.xrstf.de/httest $(HTTEST_BIN) $(HTTEST_VER)
144+
@hack/uget.sh \
145+
https://codeberg.org/xrstf/httest/releases/download/v{VERSION}/httest_{VERSION}_{GOOS}_{GOARCH}.tar.gz \
146+
${HTTEST_BIN} \
147+
${HTTEST_VER}
136148

137149
$(LOGCHECK):
138-
GOBIN=$(TOOLS_GOBIN_DIR) $(GO_INSTALL) sigs.k8s.io/logtools/logcheck $(LOGCHECK_BIN) $(LOGCHECK_VER)
150+
@GO_MODULE=true hack/uget.sh \
151+
sigs.k8s.io/logtools/logcheck \
152+
${LOGCHECK_BIN} \
153+
$(LOGCHECK_VER)
139154

155+
.PHONY: $(KCP_APIGEN_GEN)
140156
$(KCP_APIGEN_GEN):
141157
pushd . && cd staging/src/github.com/kcp-dev/sdk && GOBIN=$(TOOLS_GOBIN_DIR) go install ./cmd/apigen && popd
142158

@@ -197,13 +213,17 @@ tools: $(GOLANGCI_LINT) $(HTTEST) $(CONTROLLER_GEN) $(KCP_APIGEN_GEN) $(YAML_PAT
197213
.PHONY: tools
198214

199215
$(CONTROLLER_GEN):
200-
GOBIN=$(TOOLS_GOBIN_DIR) $(GO_INSTALL) sigs.k8s.io/controller-tools/cmd/controller-gen $(CONTROLLER_GEN_BIN) $(CONTROLLER_GEN_VER)
216+
@UNCOMPRESSED=true hack/uget.sh https://github.com/kubernetes-sigs/controller-tools/releases/download/{VERSION}/controller-gen-{GOOS}-{GOARCH} ${CONTROLLER_GEN_BIN} $(CONTROLLER_GEN_VER) controller-gen*
201217

202218
$(YAML_PATCH):
203-
GOBIN=$(TOOLS_GOBIN_DIR) $(GO_INSTALL) github.com/pivotal-cf/yaml-patch/cmd/yaml-patch $(YAML_PATCH_BIN) $(YAML_PATCH_VER)
219+
@GO_MODULE=true hack/uget.sh github.com/pivotal-cf/yaml-patch/cmd/yaml-patch $(YAML_PATCH_BIN) $(YAML_PATCH_VER)
204220

205221
$(GOTESTSUM):
206-
GOBIN=$(TOOLS_GOBIN_DIR) $(GO_INSTALL) gotest.tools/gotestsum $(GOTESTSUM_BIN) $(GOTESTSUM_VER)
222+
@hack/uget.sh \
223+
https://github.com/gotestyourself/gotestsum/releases/download/v{VERSION}/gotestsum_{VERSION}_{GOOS}_{GOARCH}.tar.gz \
224+
${GOTESTSUM_BIN} \
225+
${GOTESTSUM_VER} \
226+
${GOTESTSUM_BIN}
207227

208228
crds: $(CONTROLLER_GEN) $(YAML_PATCH) ## Generate crds
209229
./hack/update-codegen-crds.sh
@@ -267,7 +287,7 @@ BOILERPLATE_KUBERNETES_FILES := $(abspath ./hack/boilerplate/boilerplate_kuberne
267287

268288
.PHONY: verify-boilerplate
269289
verify-boilerplate: ## Verify boilerplate
270-
hack/verify_boilerplate.py --boilerplate-dir=hack/boilerplate --skip docs/venv --skip pkg/network/dialer --skip-files-list $(BOILERPLATE_MODIFIED_FILES) --skip-files-list $(BOILERPLATE_KUBERNETES_FILES)
290+
hack/verify_boilerplate.py --boilerplate-dir=hack/boilerplate --skip hack/uget.sh --skip docs/venv --skip pkg/network/dialer --skip-files-list $(BOILERPLATE_MODIFIED_FILES) --skip-files-list $(BOILERPLATE_KUBERNETES_FILES)
271291
hack/verify_boilerplate.py --boilerplate-dir=hack/boilerplate/boilerplate_modified --filenames-list $(BOILERPLATE_MODIFIED_FILES)
272292
hack/verify_boilerplate.py --boilerplate-dir=hack/boilerplate/boilerplate_kubernetes --filenames-list $(BOILERPLATE_KUBERNETES_FILES)
273293

hack/go-install.sh

Lines changed: 0 additions & 62 deletions
This file was deleted.

hack/tools.checksums

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
controller-gen|GOARCH=amd64;GOOS=linux|21e5f3239666fc0c5e2d23c2a3a83fd655af40a969ede7a118b86832c35a829f
2+
controller-gen|GOARCH=arm64;GOOS=darwin|2ca28be7185d9279ed82e3355529b0543938f392cb812add3f25a62196ed7441
3+
controller-gen|GOARCH=arm64;GOOS=linux|a1a1f758435d05933c4b2f8c292f8ab2448e81a02c45f14dbd81c10e87ec4b20
4+
golangci-lint|GOARCH=amd64;GOOS=linux|7009324a8aad93c1f84dce1c3cf61181bdd6b68e5f1b8b5d6971662258255050
5+
golangci-lint|GOARCH=arm64;GOOS=darwin|96dc1dbbb686fc7982b9ea4b800f3d234a2d9fb78b0344f287e8d28ee11bc72b
6+
golangci-lint|GOARCH=arm64;GOOS=linux|c51ff5b21be688b043baea44de7dd855cf07b855c14f0de405bfaf922b1d7634
7+
gotestsum|GOARCH=amd64;GOOS=linux|2e505a9368568aa7422e0a90ef77acc8807c0d3272ab81c7a69e3e8688d1cf65
8+
gotestsum|GOARCH=arm64;GOOS=darwin|020be8d14358c7ac4155e296436057cf4b1f1232f8f8f3d71f22a0e7a5504340
9+
gotestsum|GOARCH=arm64;GOOS=linux|2f8517768c2831750cb372e379404a059dbd20f2b1f79bcc235c4cab4540cb10
10+
httest|GOARCH=amd64;GOOS=linux|d524eea4fa0bcb9376466a6da7892d88556ed6c03d70f282946e5ca8474b4232
11+
httest|GOARCH=arm64;GOOS=darwin|12ac43f851459bc1e82302ff2d2bde987970c2c0a47933f79ddd3c277bf4a93d
12+
httest|GOARCH=arm64;GOOS=linux|6f317e0e9857a7582bbf881d30c8c64ead353312d3141b31cf1261de002d91ea
13+
yaml-patch|GOARCH=amd64;GOOS=linux|6eae938136a18a5730700676f89ce7a0d4a89d332219fad180202b329501a88b
14+
yaml-patch|GOARCH=arm64;GOOS=darwin|ce2f9448472b0e69e44c28c7e6879caae46664aa61e251fe7f181b78158ff9b0
15+
yaml-patch|GOARCH=arm64;GOOS=linux|f379170ba41d9d02446cbdf42fd48b47892a23f9333d9385ad8306449fc22d8b

0 commit comments

Comments
 (0)