Skip to content

Commit a21adc1

Browse files
authored
Merge pull request tinkerbell#29 from tstromberg/py3
Use pip3 to install yamllint instead of setuptools
2 parents 279db92 + 57fe15b commit a21adc1

File tree

3 files changed

+4
-6
lines changed

3 files changed

+4
-6
lines changed

Makefile

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ lint: out/linters/shellcheck-$(SHELLCHECK_VERSION)-$(LINT_ARCH)/shellcheck out/l
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")
29-
PYTHONPATH=$(YAMLLINT_ROOT)/lib $(YAMLLINT_ROOT)/bin/yamllint .
29+
PYTHONPATH=$(YAMLLINT_ROOT)/dist $(YAMLLINT_ROOT)/dist/bin/yamllint .
3030

3131
.PHONY: fix
3232
fix: out/linters/shellcheck-$(SHELLCHECK_VERSION)-$(LINT_ARCH)/shellcheck out/linters/golangci-lint-$(GOLINT_VERSION)-$(LINT_ARCH)
@@ -49,7 +49,6 @@ out/linters/golangci-lint-$(GOLINT_VERSION)-$(LINT_ARCH):
4949
mv out/linters/golangci-lint out/linters/golangci-lint-$(GOLINT_VERSION)-$(LINT_ARCH)
5050

5151
$(YAMLLINT_ROOT)/bin/yamllint:
52-
mkdir -p $(YAMLLINT_ROOT)/lib
5352
curl -sSfL https://github.com/adrienverge/yamllint/archive/refs/tags/v$(YAMLLINT_VERSION).tar.gz | tar -C out/linters -zxf -
54-
cd $(YAMLLINT_ROOT) && PYTHONPATH=lib python setup.py -q install --prefix . --install-lib lib
53+
cd $(YAMLLINT_ROOT) && pip3 install . -t dist
5554
# END: lint-install .

Makefile.tmpl

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,8 @@ out/linters/golangci-lint-$(GOLINT_VERSION)-$(LINT_ARCH):
5555

5656
{{ if .YAML -}}
5757
$(YAMLLINT_ROOT)/bin/yamllint:
58-
mkdir -p $(YAMLLINT_ROOT)/lib
5958
curl -sSfL https://github.com/adrienverge/yamllint/archive/refs/tags/v$(YAMLLINT_VERSION).tar.gz | tar -C out/linters -zxf -
60-
cd $(YAMLLINT_ROOT) && PYTHONPATH=lib python setup.py -q install --prefix . --install-lib lib
59+
cd $(YAMLLINT_ROOT) && pip3 install . -t dist
6160
{{ end -}}
6261

6362
# END: lint-install {{.Args}}

lint-install.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,7 @@ func yamlLintCmd(_ string, level string) string {
279279
if level == "warn" {
280280
suffix = " || true"
281281
}
282-
return fmt.Sprintf(`PYTHONPATH=$(YAMLLINT_ROOT)/lib $(YAMLLINT_ROOT)/bin/yamllint .%s`, suffix)
282+
return fmt.Sprintf(`PYTHONPATH=$(YAMLLINT_ROOT)/dist $(YAMLLINT_ROOT)/dist/bin/yamllint .%s`, suffix)
283283
}
284284

285285
// main creates peanut butter & jelly sandwiches with utmost precision.

0 commit comments

Comments
 (0)