jpbetz / auger

Directly access data objects stored in etcd by kubernetes.
Apache License 2.0
392 stars 62 forks source link

Issues at "make release" #37

Open Ninsbean opened 2 years ago

Ninsbean commented 2 years ago

Tried to install auger on my linux machine. Did the steps in the instructions until "make release".

$ make release Building release in temp directory /tmp/tmp.el6JOnIEwF docker run \ -v /tmp/tmp.el6JOnIEwF/auger:/go/src/github.com/jpbetz/auger \ -w /go/src/github.com/jpbetz/auger \ golang:1.16.5 \ /bin/bash -c "make -f /go/src/github.com/jpbetz/auger/Makefile release-docker-build GOARCH=amd64 GOOS=linux" Emulate Docker CLI using podman. Create /etc/containers/nodocker to quiet msg. export GOPATH=/go GOOS=linux GOARCH=amd64 GO111MODULE=on go build go: github.com/google/safetext@v0.0.0-20220914124124-e18e3fe012bf requires github.com/niemeyer/pretty@v0.0.0-20200227124842-a10e7caefd8e: missing go.sum entry; to add it: go mod download github.com/niemeyer/pretty make: [/go/src/github.com/jpbetz/auger/Makefile:53: release-docker-build] Error 1 make: [Makefile:41: release] Error 2

I checked if pretty is in the go.sum file and it is. Also checked if the module is listed at the go mods.

$ go list -m all | grep pretty github.com/kr/pretty v0.2.1 github.com/niemeyer/pretty

What can be the problem here?

Thanks for your help :-)

hrrrsn commented 1 year ago

I know absolutely nothing about Go so there's probably a better way of achieving this, but I resolved this by editing this line in the Makefile:

/bin/bash -c "make -f /go/src/$(PKG)/Makefile release-docker-build GOARCH=$(GOARCH) GOOS=$(GOOS)"

to:

/bin/bash -c "go mod download github.com/niemeyer/pretty; make -f /go/src/$(PKG)/Makefile release-docker-build GOARCH=$(GOARCH) GOOS=$(GOOS)"