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
telegraf: patch go.mod, remove runVend #95485
Conversation
runVend = true; | ||
vendorSha256 = "0c2sayg49b2rq3fnrbf741b6zy8byhwxlnxkhf5160gzqn6jy2rw"; | ||
patches = [ | ||
./gomod.patch |
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.
Could you use fetchpatch here?
https://github.com/influxdata/telegraf/commit/a0d0c80c373dd1896d449e9f3534a157d78fc866.patch
This is the PR:
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.
vendor/github.com/shirou/gopsutil/host/host_bsd.go:25:13: undefined: unix.SysctlTimeval
Looks like it works with 2.20.6 and breaks with a different issue on 2.20.7.
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.
Ok. I will have a look.
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.
Bumped to 2.20.6: Let's see if that makes CI happy: https://github.com/influxdata/telegraf/commit/63e1f41d8ff246d191d008ff7f69d69cc34b4fae.patch
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.
Builds with patch, generates the same vendorSha256
.
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.
Looks this patch breaks FreeBSD now.... https://app.circleci.com/pipelines/github/influxdata/telegraf/1327/workflows/6b7efed4-74fb-45bf-a3c7-2978f9bdf5bb/jobs/62496
However I think this should be good enough for nixpkgs.
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)