New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
restic: 0.9.2 -> 0.9.3 #48627
restic: 0.9.2 -> 0.9.3 #48627
Conversation
Success on aarch64-linux (full log) Attempted: restic Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: restic Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: restic Partial log (click to expand)
|
buildPhase = '' | ||
cd go/src/${goPackagePath} | ||
go run build.go | ||
GO111MODULE=on GOCACHE=`mktemp -d` go run -mod=vendor build.go |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually we already set GOCACHE = "off"
, but this does not work?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, I'm getting either build flag -mod=vendor only valid when using modules
or go: cannot use modules with build cache disabled
if I I also remove -mod=vendor
. It also doesn't work if I remove GO111MODULE=on
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if we could detect go modules automatically.
FYI: restic-0.9.4 was just released: |
restic recently switched to using Go 1.11 modules: restic/restic#1920
I don't know any Go so tried to guess how to do that the right way in Nixpkgs. Please have a look.
Again: I have no idea whether I'm using
GO111MODULE
andGOCACHE
the right way.All I can say is that restic seems to run fine according to my short test.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after) (new: 17178568, old: 16659688)cc @adisbladis @Mic92