Skip to content

Commit

Permalink
fix: In Makefile, download zot as ZOT rather than regclient
Browse files Browse the repository at this point in the history
This was a copy/paste error added by my last change to Makefile.
Instead of downloading zot to TOOLS_D/bin/zot, we were downloading
regclient.  Big surprise, regclient binary doesn't work like zot
binary.

Fix the download url, and also update the ZOT_VERSION to include
the 'v' for consistency with REGCLIENT_VERSION and
GOLANGCI_LINT_VERSION.

Signed-off-by: Scott Moser <[email protected]>
  • Loading branch information
smoser committed Sep 27, 2023
1 parent 24fbadd commit 54226bf
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ REGCLIENT := $(TOOLS_D)/bin/regctl
REGCLIENT_VERSION := v0.5.1
# OCI registry
ZOT := $(TOOLS_D)/bin/zot
ZOT_VERSION := 2.0.0-rc6
ZOT_VERSION := v2.0.0-rc6

GOLANGCI_LINT_VERSION = v1.54.2
GOLANGCI_LINT = $(TOOLS_D)/golangci-lint/$(GOLANGCI_LINT_VERSION)/golangci-lint
Expand Down Expand Up @@ -94,7 +94,7 @@ $(REGCLIENT):
$(call dlbin,$@,https://github.com/regclient/regclient/releases/download/$(REGCLIENT_VERSION)/regctl-linux-amd64)

$(ZOT):
$(call dlbin,$@,https://github.com/regclient/regclient/releases/download/$(REGCLIENT_VERSION)/regctl-linux-amd64)
$(call dlbin,$@,https://github.com/project-zot/zot/releases/download/$(ZOT_VERSION)/zot-linux-amd64-minimal)

TEST?=$(patsubst test/%.bats,%,$(wildcard test/*.bats))
PRIVILEGE_LEVEL?=
Expand Down

0 comments on commit 54226bf

Please sign in to comment.