commit: a0e75219444993d81826f7650bcccad80c968b5d Author: Dongsu Park <dongsu <AT> kinvolk <DOT> io> AuthorDate: Mon Apr 29 05:42:50 2019 +0000 Commit: Zac Medico <zmedico <AT> gentoo <DOT> org> CommitDate: Mon Apr 29 17:32:51 2019 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=a0e75219
app-emulation/cri-o: disable go build install flag There has been a corner case where `make bin/crio` failed like that: ``` go build runtime/cgo: open /usr/lib/go1.12/pkg/linux_amd64/runtime/cgo.a: permission denied make: *** [Makefile:99: bin/crio] Error 1 ``` The failure happens only under certain circumstances, for example in Jenkins pipelines, where the standard Go runtime path `/usr/lib/go1.12` is not writable for normal users like `jenkins`. On the other hand, the failure is not reproducible when testing locally, nor when testing with the root user who can write to the Go runtime path. What happens is, the standard Go runtime sometime has incorrect timestamps or checksums of `cgo.a`, which makes `go build -i` think that it has to install its own library to the system directory. That's obviously not what the build process should do. Since Go 1.10, it's unnecessary to run `go build` with an `-i` option, because now `go build` command itself maintains a cache of recently build packages. Build speed will not be better by adding an `-i` option. So let's remove `-i` flag from `go build`. That's actually what upstream Go maintainers recommend to do. See: https://github.com/golang/go/issues/24674 Closes: https://github.com/gentoo/gentoo/pull/11863 Package-Manager: Portage-2.3.65, Repoman-2.3.12 Signed-off-by: Dongsu Park <dongsu <AT> kinvolk.io> Signed-off-by: Zac Medico <zmedico <AT> gentoo.org> app-emulation/cri-o/cri-o-1.10.6.ebuild | 2 +- app-emulation/cri-o/cri-o-1.13.5.ebuild | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app-emulation/cri-o/cri-o-1.10.6.ebuild b/app-emulation/cri-o/cri-o-1.10.6.ebuild index dbb1db95a32..845da94a93b 100644 --- a/app-emulation/cri-o/cri-o-1.10.6.ebuild +++ b/app-emulation/cri-o/cri-o-1.10.6.ebuild @@ -43,7 +43,7 @@ src_prepare() { default sed -e '/^GIT_.*/d' \ - -e 's/$(GO) build/$(GO) build -v -work -x/' \ + -e 's/$(GO) build -i/$(GO) build -v -work -x/' \ -e 's/\${GIT_COMMIT}/'${EGIT_COMMIT}'/' \ -i Makefile || die diff --git a/app-emulation/cri-o/cri-o-1.13.5.ebuild b/app-emulation/cri-o/cri-o-1.13.5.ebuild index bf1e755e142..2478498c2a7 100644 --- a/app-emulation/cri-o/cri-o-1.13.5.ebuild +++ b/app-emulation/cri-o/cri-o-1.13.5.ebuild @@ -44,7 +44,7 @@ src_prepare() { sed -e '/^GIT_.*/d' \ -e '/ git diff --exit-code/d' \ - -e 's/$(GO) build/$(GO) build -v -work -x/' \ + -e 's/$(GO) build -i/$(GO) build -v -work -x/' \ -e 's/\${GIT_COMMIT}/'${EGIT_COMMIT}'/' \ -i Makefile || die
