Skip to content

Commit

Permalink
Download golangci-lint via wget
Browse files Browse the repository at this point in the history
Kubernetes now requires golang 1.13.4. When using golang 1.13.4 to build critest, we run into golangci-lint#659. Fix this by installing the golangci-lint binary for 1.17.1 with curl (similar to how CRI-O currently does), instead of using go mods.

Signed-off-by: Peter Hunt <[email protected]>
  • Loading branch information
haircommander committed Nov 7, 2019
1 parent bc50b06 commit 0092115
Show file tree
Hide file tree
Showing 1,016 changed files with 14,879 additions and 189,787 deletions.
7 changes: 6 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,12 @@ lint: $(GOLANGCI_LINT)
install.tools: $(GOLANGCI_LINT) $(GINKGO)

$(GOLANGCI_LINT):
$(call go-build,./vendor/github.com/golangci/golangci-lint/cmd/golangci-lint)
export \
VERSION=v1.17.1 \
URL=https://raw.githubusercontent.com/golangci/golangci-lint \
BINDIR=${BUILD_BIN_PATH} && \
curl -sfL $$URL/$$VERSION/install.sh | sh -s $$VERSION


$(GINKGO):
$(call go-build,./vendor/github.com/onsi/ginkgo/ginkgo)
Expand Down
7 changes: 3 additions & 4 deletions go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -9,18 +9,17 @@ require (
github.com/ghodss/yaml v1.0.0
github.com/golang/glog v0.0.0-20160126235308-23def4e6c14b
github.com/golang/protobuf v1.3.2
github.com/golangci/golangci-lint v1.17.1
github.com/onsi/ginkgo v1.10.1
github.com/onsi/gomega v1.7.0
github.com/opencontainers/go-digest v1.0.0-rc1
github.com/opencontainers/selinux v1.2.2
github.com/pborman/uuid v1.2.0
github.com/sirupsen/logrus v1.4.2
github.com/urfave/cli v1.22.1
golang.org/x/net v0.0.0-20190812203447-cdfb69ac37fc
golang.org/x/sys v0.0.0-20190730183949-1393eb018365
golang.org/x/net v0.0.0-20190923162816-aa69164e4478
golang.org/x/sys v0.0.0-20190922100055-0a153f010e69
google.golang.org/grpc v1.23.0
gopkg.in/yaml.v2 v2.2.2
gopkg.in/yaml.v2 v2.2.4
k8s.io/api v0.0.0
k8s.io/apimachinery v0.0.0
k8s.io/client-go v11.0.0+incompatible
Expand Down
149 changes: 7 additions & 142 deletions go.sum

Large diffs are not rendered by default.

1 change: 0 additions & 1 deletion tools/tools.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,5 @@ limitations under the License.
package tools

import (
_ "github.com/golangci/golangci-lint/cmd/golangci-lint"
_ "github.com/onsi/ginkgo/ginkgo"
)
5 changes: 0 additions & 5 deletions vendor/github.com/BurntSushi/toml/.gitignore

This file was deleted.

15 changes: 0 additions & 15 deletions vendor/github.com/BurntSushi/toml/.travis.yml

This file was deleted.

3 changes: 0 additions & 3 deletions vendor/github.com/BurntSushi/toml/COMPATIBLE

This file was deleted.

21 changes: 0 additions & 21 deletions vendor/github.com/BurntSushi/toml/COPYING

This file was deleted.

19 changes: 0 additions & 19 deletions vendor/github.com/BurntSushi/toml/Makefile

This file was deleted.

218 changes: 0 additions & 218 deletions vendor/github.com/BurntSushi/toml/README.md

This file was deleted.

Loading

0 comments on commit 0092115

Please sign in to comment.