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
gitAndTools.gitstatus: init at 20190506 #61116
Conversation
@@ -127,6 +127,8 @@ let | |||
|
|||
gitflow = callPackage ./gitflow { }; | |||
|
|||
gitflow = callPackage ./gitstatus { }; |
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.
gitstatus I think :p
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.
derp :)
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.
Thats what I get for not cherry-picking from my other branch :|
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.
fixed
{callPackage, stdenv, fetchFromGitHub, ...}: | ||
|
||
stdenv.mkDerivation rec { | ||
name = "gitstatus-${version}"; |
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.
use pname = "gistatus"
instead, mkDerivation will generate the same name
automatically.
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.
oic!
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.
fixed
pkgs/applications/version-management/git-and-tools/gitstatus/default.nix
Outdated
Show resolved
Hide resolved
pkgs/applications/version-management/git-and-tools/gitstatus/default.nix
Show resolved
Hide resolved
@GrahamcOfBorg gitAndTools.gitstatus |
@nlewo pinging, requested changes are in |
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.
Oups! Sorry ;)
Still minor review, otherwise LGTM.
|
||
stdenv.mkDerivation rec { | ||
pname = "gitstatus"; | ||
version = "20190506"; |
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.
According to the manual:
version = "20190506"; | |
version = "unstable-2019-05-06"; |
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.
@nlewo done
@GrahamcOfBorg build gitAndTools.gitstatus |
This doesn't build on darwin. Could you please either fix the build for darwin or restrict the supported platforms (see |
@nlewo the build log indicates an issue with clang not being able to build a binary... which indicates ci env issues? I'd rather not just blacklist darwin, but don't really know how to proceed. |
Ok. I don't think we can do more on this darwin error. So, let's merge it as it is currently. |
:toot: |
Motivation for this change
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)