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
bazel-buildtools: 0.22.0 -> unstable-2019-06-21 #64537
bazel-buildtools: 0.22.0 -> unstable-2019-06-21 #64537
Conversation
@Profpatsch @uri-canva Can you have a look please? |
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 good to me but I'm not sure about taking head over the latest tagged version. The last tag was only a month ago and the previous one was one month before that so it's unlikely we'll have to wait very long until the next tag.
@@ -1,16 +1,16 @@ | |||
{ stdenv, buildGoPackage, fetchFromGitHub }: | |||
{ stdenv, buildGoPackage, fetchgit, fetchhg, fetchbzr, fetchsvn }: |
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.
fetchhg, fetchbzr, fetchsvn
don't seem to be used, were they added by go2nix
?
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.
Yes, they were added by go2nix.
@uri-canva True, but there is no reason to wait either. There are a few useful features in the latest commits and this will not break any other package. |
@uri-canva Looks like you might get your wish after all: bazelbuild/buildtools@0ce0811#commitcomment-34252722 |
Closing in favor of #65277: bumping to an newer upstream version release. |
Motivation for this change
Continuation from #63856
Update to HEAD of master. There are a few nice features there, so it might make sense to go directly to HEAD.
I used go2nix to produce the files, even though I patched
default.nix
to include the meta info.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)