Skip to content

Commit 6f6d11e

Browse files
authored
Merge pull request tinkerbell#31 from mmlb/fix-yamllint-path
2 parents a21adc1 + 944e9c5 commit 6f6d11e

File tree

2 files changed

+14
-4
lines changed

2 files changed

+14
-4
lines changed

Makefile

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ GOLINT_CONFIG = $(LINT_ROOT)/.golangci.yml
2222
YAMLLINT_ROOT = out/linters/yamllint-$(YAMLLINT_VERSION)
2323

2424
.PHONY: lint
25-
lint: out/linters/shellcheck-$(SHELLCHECK_VERSION)-$(LINT_ARCH)/shellcheck out/linters/hadolint-$(HADOLINT_VERSION)-$(LINT_ARCH) out/linters/golangci-lint-$(GOLINT_VERSION)-$(LINT_ARCH) $(YAMLLINT_ROOT)/bin/yamllint
25+
lint: out/linters/shellcheck-$(SHELLCHECK_VERSION)-$(LINT_ARCH)/shellcheck out/linters/hadolint-$(HADOLINT_VERSION)-$(LINT_ARCH) out/linters/golangci-lint-$(GOLINT_VERSION)-$(LINT_ARCH) $(YAMLLINT_ROOT)/dist/bin/yamllint
2626
out/linters/golangci-lint-$(GOLINT_VERSION)-$(LINT_ARCH) run
2727
out/linters/hadolint-$(HADOLINT_VERSION)-$(LINT_ARCH) $(shell find . -name "*Dockerfile")
2828
out/linters/shellcheck-$(SHELLCHECK_VERSION)-$(LINT_ARCH)/shellcheck $(shell find . -name "*.sh")
@@ -35,20 +35,25 @@ fix: out/linters/shellcheck-$(SHELLCHECK_VERSION)-$(LINT_ARCH)/shellcheck out/li
3535

3636
out/linters/shellcheck-$(SHELLCHECK_VERSION)-$(LINT_ARCH)/shellcheck:
3737
mkdir -p out/linters
38+
rm -rf out/linters/shellcheck-*
3839
curl -sSfL https://github.com/koalaman/shellcheck/releases/download/$(SHELLCHECK_VERSION)/shellcheck-$(SHELLCHECK_VERSION).$(LINT_LOWER_OS).$(LINT_ARCH).tar.xz | tar -C out/linters -xJf -
3940
mv out/linters/shellcheck-$(SHELLCHECK_VERSION) out/linters/shellcheck-$(SHELLCHECK_VERSION)-$(LINT_ARCH)
4041

4142
out/linters/hadolint-$(HADOLINT_VERSION)-$(LINT_ARCH):
4243
mkdir -p out/linters
44+
rm -rf out/linters/hadolint-*
4345
curl -sfL https://github.com/hadolint/hadolint/releases/download/v2.6.1/hadolint-$(LINT_OS)-$(LINT_ARCH) > out/linters/hadolint-$(HADOLINT_VERSION)-$(LINT_ARCH)
4446
chmod u+x out/linters/hadolint-$(HADOLINT_VERSION)-$(LINT_ARCH)
4547

4648
out/linters/golangci-lint-$(GOLINT_VERSION)-$(LINT_ARCH):
4749
mkdir -p out/linters
50+
rm -rf out/linters/golangci-lint-*
4851
curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b out/linters $(GOLINT_VERSION)
4952
mv out/linters/golangci-lint out/linters/golangci-lint-$(GOLINT_VERSION)-$(LINT_ARCH)
5053

51-
$(YAMLLINT_ROOT)/bin/yamllint:
54+
$(YAMLLINT_ROOT)/dist/bin/yamllint:
55+
mkdir -p out/linters
56+
rm -rf out/linters/yamllint-*
5257
curl -sSfL https://github.com/adrienverge/yamllint/archive/refs/tags/v$(YAMLLINT_VERSION).tar.gz | tar -C out/linters -zxf -
5358
cd $(YAMLLINT_ROOT) && pip3 install . -t dist
5459
# END: lint-install .

Makefile.tmpl

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ endif
2222
{{ if .YAML }}YAMLLINT_ROOT = out/linters/yamllint-$(YAMLLINT_VERSION){{ end }}
2323

2424
.PHONY: lint
25-
lint: {{ if .Shell }}out/linters/shellcheck-$(SHELLCHECK_VERSION)-$(LINT_ARCH)/shellcheck {{ end }}{{ if .Dockerfile }}out/linters/hadolint-$(HADOLINT_VERSION)-$(LINT_ARCH) {{ end }}{{ if .Go}}out/linters/golangci-lint-$(GOLINT_VERSION)-$(LINT_ARCH) {{ end }}{{ if .YAML}}$(YAMLLINT_ROOT)/bin/yamllint{{ end }}
25+
lint: {{ if .Shell }}out/linters/shellcheck-$(SHELLCHECK_VERSION)-$(LINT_ARCH)/shellcheck {{ end }}{{ if .Dockerfile }}out/linters/hadolint-$(HADOLINT_VERSION)-$(LINT_ARCH) {{ end }}{{ if .Go}}out/linters/golangci-lint-$(GOLINT_VERSION)-$(LINT_ARCH) {{ end }}{{ if .YAML}}$(YAMLLINT_ROOT)/dist/bin/yamllint{{ end }}
2626
{{- range .LintCommands }}
2727
{{ .}}{{ end}}
2828

@@ -34,27 +34,32 @@ fix: {{ if .Shell }}out/linters/shellcheck-$(SHELLCHECK_VERSION)-$(LINT_ARCH)/sh
3434
{{ if .Shell -}}
3535
out/linters/shellcheck-$(SHELLCHECK_VERSION)-$(LINT_ARCH)/shellcheck:
3636
mkdir -p out/linters
37+
rm -rf out/linters/shellcheck-*
3738
curl -sSfL https://github.com/koalaman/shellcheck/releases/download/$(SHELLCHECK_VERSION)/shellcheck-$(SHELLCHECK_VERSION).$(LINT_LOWER_OS).$(LINT_ARCH).tar.xz | tar -C out/linters -xJf -
3839
mv out/linters/shellcheck-$(SHELLCHECK_VERSION) out/linters/shellcheck-$(SHELLCHECK_VERSION)-$(LINT_ARCH)
3940

4041
{{ end -}}
4142
{{ if .Dockerfile -}}
4243
out/linters/hadolint-$(HADOLINT_VERSION)-$(LINT_ARCH):
4344
mkdir -p out/linters
45+
rm -rf out/linters/hadolint-*
4446
curl -sfL https://github.com/hadolint/hadolint/releases/download/v2.6.1/hadolint-$(LINT_OS)-$(LINT_ARCH) > out/linters/hadolint-$(HADOLINT_VERSION)-$(LINT_ARCH)
4547
chmod u+x out/linters/hadolint-$(HADOLINT_VERSION)-$(LINT_ARCH)
4648

4749
{{ end -}}
4850
{{ if .Go -}}
4951
out/linters/golangci-lint-$(GOLINT_VERSION)-$(LINT_ARCH):
5052
mkdir -p out/linters
53+
rm -rf out/linters/golangci-lint-*
5154
curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b out/linters $(GOLINT_VERSION)
5255
mv out/linters/golangci-lint out/linters/golangci-lint-$(GOLINT_VERSION)-$(LINT_ARCH)
5356

5457
{{ end -}}
5558

5659
{{ if .YAML -}}
57-
$(YAMLLINT_ROOT)/bin/yamllint:
60+
$(YAMLLINT_ROOT)/dist/bin/yamllint:
61+
mkdir -p out/linters
62+
rm -rf out/linters/yamllint-*
5863
curl -sSfL https://github.com/adrienverge/yamllint/archive/refs/tags/v$(YAMLLINT_VERSION).tar.gz | tar -C out/linters -zxf -
5964
cd $(YAMLLINT_ROOT) && pip3 install . -t dist
6065
{{ end -}}

0 commit comments

Comments
 (0)