Skip to content

Commit 591e87f

Browse files
committedSep 17, 2024
chore: support tools target for arm64
Signed-off-by: Pranshu Srivastava <rexagod@gmail.com>
1 parent a52ed9b commit 591e87f

File tree

2 files changed

+14
-16
lines changed

2 files changed

+14
-16
lines changed
 

‎.github/tools

+2-2
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
golangci-lint v1.55.2
44
controller-gen v0.14.0
55
kustomize v4.5.2
6-
oc v4.8.11
6+
oc v4.16.10
77
operator-sdk v1.34.1
88
opm v1.39.0
99
promq v0.0.1
@@ -13,4 +13,4 @@ jsonnetfmt v0.20.0
1313
jsonnet-lint v0.20.0
1414
jb v0.5.1
1515
gojsontoyaml v0.1.0
16-
shellcheck 0.10.0
16+
shellcheck v0.10.0

‎Makefile.tools

+12-14
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ PROMQ_VERSION = v0.0.1
2929

3030
# NOTE: oc is NOT downloadable using the OC_VERSION in its URL, so this has to be manually updated
3131
OC = $(TOOLS_DIR)/oc
32-
OC_VERSION = v4.8.11
32+
OC_VERSION = 4.16.10
3333

3434
CRDOC = $(TOOLS_DIR)/crdoc
3535
CRDOC_VERSION = v0.5.2
@@ -125,12 +125,11 @@ $(OC) oc: $(TOOLS_DIR)
125125
set -ex ;\
126126
[[ -f $(OC) ]] && exit 0 ;\
127127
OS=$(shell go env GOOS) && ARCH=$(shell go env GOARCH) && \
128-
curl -sSLo $(OC) https://mirror.openshift.com/pub/openshift-v4/$${ARCH}/clients/oc/latest/$${OS}/oc.tar.gz ;\
129-
tar -xf $(TOOLS_DIR)/oc -C $(TOOLS_DIR) ;\
130-
rm -f $(TOOLS_DIR)/README.md ;\
131-
$(OC) version ;\
132-
version=$(OC_VERSION) ;\
133-
$(OC) version | grep -q $${version##v} ;\
128+
if [[ $${OS} == "darwin" ]]; then OS="mac"; fi ;\
129+
curl -sSLo $(OC).tar.gz https://mirror.openshift.com/pub/openshift-v4/$${ARCH}/clients/ocp/stable-4.16/openshift-client-$${OS}-$(OC_VERSION).tar.gz ;\
130+
tar -xf $(OC).tar.gz -C $(TOOLS_DIR) ;\
131+
rm -f $(OC).tar.gz ;\
132+
rm -f $(TOOLS_DIR)/-.md ;\
134133
}
135134

136135
.PHONY: crdoc
@@ -189,11 +188,10 @@ $(SHELLCHECK) shellcheck: $(TOOLS_DIR)
189188
[[ -f $(SHELLCHECK) ]] && exit 0 ;\
190189
cd $$(mktemp -d) ;\
191190
OS=$(shell go env GOOS) && ARCH=$(shell uname -m) && \
192-
curl -sSLo shellcheck-stable.tar.xz https://github.com/koalaman/shellcheck/releases/download/stable/shellcheck-stable.$${OS}.$${ARCH}.tar.xz ;\
193-
tar -xJf shellcheck-stable.tar.xz ;\
194-
cp shellcheck-stable/shellcheck $(SHELLCHECK) ;\
195-
version=$(SHELLCHECK_VERSION) ;\
196-
$(SHELLCHECK) -V | grep -q $${version##v} ;\
191+
if [[ "$$OS" == "darwin" && "$$ARCH" == "arm64" ]]; then shellcheck_arch="aarch64"; fi ;\
192+
curl -sSLo shellcheck.tar.xz https://github.com/koalaman/shellcheck/releases/download/v$(SHELLCHECK_VERSION)/shellcheck-v$(SHELLCHECK_VERSION).$${OS}.$${shellcheck_arch}.tar.xz ;\
193+
tar -xJf shellcheck.tar.xz ;\
194+
cp shellcheck-v$(SHELLCHECK_VERSION)/shellcheck $(SHELLCHECK) ;\
197195
}
198196

199197
.PHONY: jsonnet-tools
@@ -219,7 +217,7 @@ tools: $(CONTROLLER_GEN) \
219217
echo $$(basename $(GOLANGCI_LINT)) $(GOLANGCI_LINT_VERSION) >> $$tools_file ;\
220218
echo $$(basename $(CONTROLLER_GEN)) $(CONTROLLER_GEN_VERSION) >> $$tools_file ;\
221219
echo $$(basename $(KUSTOMIZE)) $(KUSTOMIZE_VERSION) >> $$tools_file ;\
222-
echo $$(basename $(OC)) $(OC_VERSION) >> $$tools_file ;\
220+
echo $$(basename $(OC)) v$(OC_VERSION) >> $$tools_file ;\
223221
echo $$(basename $(OPERATOR_SDK)) $(OPERATOR_SDK_VERSION) >> $$tools_file ;\
224222
echo $$(basename $(OPM)) $(OPM_VERSION) >> $$tools_file ;\
225223
echo $$(basename $(PROMQ)) $(PROMQ_VERSION) >> $$tools_file ;\
@@ -229,7 +227,7 @@ tools: $(CONTROLLER_GEN) \
229227
echo $$(basename $(JSONNET_LINT)) $(JSONNET_VERSION) >> $$tools_file ;\
230228
echo $$(basename $(JB)) $(JB_VERSION) >> $$tools_file ;\
231229
echo $$(basename $(GOJSONTOYAML)) $(GOJSONTOYAML_VERSION) >> $$tools_file ;\
232-
echo $$(basename $(SHELLCHECK)) $(SHELLCHECK_VERSION) >> $$tools_file ;\
230+
echo $$(basename $(SHELLCHECK)) v$(SHELLCHECK_VERSION) >> $$tools_file ;\
233231
}
234232

235233
.PHONY: clean-tools

0 commit comments

Comments
 (0)