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
mtail: 3.0.0-rc4 -> 3.0.0-rc34 #79356
Conversation
Another tiny thing is that since I only set |
It seems my problem with |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review-may-2019/3032/115 |
done, thanks @doronbehar |
I done all the changes and updated to rc35. thanks |
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.
Great, tested basic --help
and no comments left :).
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Motivation for this change
3.0.0-rc4
is over 2 years old. no release yet...Things done
This update has a tiny lie.I saw from the manual thatgo2nix
is normally used forgoDeps
but on thego2nix
repo it says to usevgo2nix
when there ago.mod
file present.The thing is thatvgo2nix
fails with3.0.0-rc34
but not with master, so I generated the file with master. master only has tiny changes. It's mostly a deps update. (check google/mtail@v3.0.0-rc34...0150dad)If it's not OK I could another way. I also wanted to switch the package to buildGoModule but I got some problem: #79355Also, is gotools needed?
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)/nix/store/z0dk0mhfsni2lpaqpw2pnjgs0y9vzsyl-mtail-3.0.0-rc4-bin 40.4M
/nix/store/1m6f7yfly52fpa4rwwsvwqkrxmfd7h8a-mtail-3.0.0-rc35 45.5M