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
git-appraise: init unstable at 2018-02-26 #46781
Conversation
|
||
buildGoPackage rec { | ||
name = "git-appraise-unstable-${version}"; | ||
version = "2018-02-26"; |
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.
It seems like the commit was on February 27th.
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.
@haslersn weird… go2nix did put this 🤔 but indeed GitHub shows Feb. 27th
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.
ah… because of my timezone I guess
git show 2414523
commit 2414523905939525559e4b2498c5597f86193b61 (HEAD -> master, origin/master, origin/HEAD)
Merge: 0cd354a 7c60f9d
Author: Omar Jarjur <ojarjur@google.com>
Date: Mon Feb 26 21:36:54 2018 -0500
@@ -0,0 +1,23 @@ | |||
{ 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.
Remove unused fetchers. :)
|
||
goPackagePath = "github.com/google/git-appraise"; | ||
|
||
src = fetchgit { |
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.
Is there any reason to not use fetchFromGitHub here? It saves a lot of bandwidth.
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.
Not really, it's just how go2nix
did it. I'll update 😉
|
||
meta = { | ||
description = "Distributed code review system for Git repos"; | ||
homepage = "https://github.com/google/git-appraise"; |
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.
Prefer unquoted urls when you're not doing subtitutions within them
Signed-off-by: Vincent Demeester <vincent@sbr.pm>
1c42cc2
to
69f028e
Compare
@srhb updated 👼 |
@GrahamcOfBorg build gitAndTools.git-appraise |
Success on x86_64-linux (full log) Attempted: gitAndTools.git-appraise Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: gitAndTools.git-appraise Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: gitAndTools.git-appraise Partial log (click to expand)
|
Thanks! |
Signed-off-by: Vincent Demeester vincent@sbr.pm
Motivation for this change
Package up a recent version of https://github.com/google/git-appraise… That's why it's starting from
master
(a.k.a. unstable-2018-02-26)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)