Skip to content

Commit 5cd7907

Browse files
committed
test: Disable gometalinter as it fails to install
To be replaced in the near future with golangci-lint, which fully replaces gometalinter in terms of functionality, plus adds some more. # gopkg.in/alecthomas/gometalinter.v1 Error: ../../../../pkg/mod/gopkg.in/alecthomas/[email protected]/main.go:129:69: cannot use loadConfig (type func(*kingpin.Application, *kingpin.ParseElement, *kingpin.ParseContext) error) as type kingpin.Action in argument to kingpin.Flag("config", "Load JSON configuration from file.").Action Error: ../../../../pkg/mod/gopkg.in/alecthomas/[email protected]/main.go:130:104: cannot use disableAction (type func(*kingpin.Application, *kingpin.ParseElement, *kingpin.ParseContext) error) as type kingpin.Action in argument to kingpin.Flag("disable", "Disable previously enabled linters.").PlaceHolder("LINTER").Short('D').Action Error: ../../../../pkg/mod/gopkg.in/alecthomas/[email protected]/main.go:131:103: cannot use enableAction (type func(*kingpin.Application, *kingpin.ParseElement, *kingpin.ParseContext) error) as type kingpin.Action in argument to kingpin.Flag("enable", "Enable previously disabled linters.").PlaceHolder("LINTER").Short('E').Action Error: ../../../../pkg/mod/gopkg.in/alecthomas/[email protected]/main.go:135:60: cannot use disableAllAction (type func(*kingpin.Application, *kingpin.ParseElement, *kingpin.ParseContext) error) as type kingpin.Action in argument to kingpin.Flag("disable-all", "Disable all linters.").Action Error: ../../../../pkg/mod/gopkg.in/alecthomas/[email protected]/main.go:136:58: cannot use enableAllAction (type func(*kingpin.Application, *kingpin.ParseElement, *kingpin.ParseContext) error) as type kingpin.Action in argument to kingpin.Flag("enable-all", "Enable all linters.").Action Signed-off-by: Joe Groocock <[email protected]>
1 parent 7f8fb34 commit 5cd7907

File tree

2 files changed

+1
-6
lines changed

2 files changed

+1
-6
lines changed

docker/Dockerfile.build

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,5 @@ ENV PKG_CONFIG_PATH=/opt/rh/rh-ruby22/root/usr/lib64/pkgconfig${PKG_CONFIG_PATH:
88

99
RUN yum -y install epel-release wget unzip git make which centos-release-scl gcc && sed -i "s/enabled=0/enabled=1/" /etc/yum.repos.d/epel-testing.repo && yum -y install rh-ruby22 && wget -O /opt/go1.16.8.linux-amd64.tar.gz https://storage.googleapis.com/golang/go1.16.8.linux-amd64.tar.gz && tar -C /usr/local -xzf /opt/go1.16.8.linux-amd64.tar.gz
1010
RUN mkdir -p $GOPATH/src/github.com/purpleidea && cd $GOPATH/src/github.com/purpleidea && git clone --recursive https://github.com/purpleidea/mgmt
11-
RUN go get -u gopkg.in/alecthomas/gometalinter.v1 && cd $GOPATH/src/github.com/purpleidea/mgmt && make deps && make build
11+
RUN cd $GOPATH/src/github.com/purpleidea/mgmt && make deps && make build
1212
CMD ["/bin/bash"]

misc/make-deps.sh

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -134,11 +134,6 @@ go get golang.org/x/lint/golint # for `golint`-ing
134134
go get golang.org/x/tools/cmd/goimports # for fmt
135135
go get github.com/kevinburke/go-bindata/go-bindata # for compiling in non golang files
136136
go get github.com/dvyukov/go-fuzz/go-fuzz # for fuzzing the mcl lang bits
137-
if in_ci; then
138-
go get -u gopkg.in/alecthomas/gometalinter.v1 && \
139-
mv "$(dirname $(command -v gometalinter.v1))/gometalinter.v1" "$(dirname $(command -v gometalinter.v1))/gometalinter" && \
140-
gometalinter --install # bonus
141-
fi
142137
fold_end "Install golang tools"
143138

144139
fold_start "Install miscellaneous tools"

0 commit comments

Comments
 (0)