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
blsd: init at unstable-2017-07-27 #32569
Conversation
Are you still using |
It's a utility you don't need often, so tbh I've mostly tested it as opposed to used. |
pkgs/tools/misc/blsd/default.nix
Outdated
|
||
buildGoPackage rec { | ||
name = "blsd-${version}"; | ||
version = "unstable-2017-07-27"; |
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.
The version begins with a digit, so unstable is not part of the version, but the suffix of the name. (And for packages which are unlikely to ever have a version number it is fine to drop it.)
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 could have been handy for fzf
as the author shows on the home page, but it lists directories not in sorted order which makes it feel unfinished and not something you'd want to use.
Nevertheless it's fine to merge. I've asked in case you've changed your mind, and because maintaining a dependent of libgit is not zero effort.
pkgs/tools/misc/blsd/default.nix
Outdated
description = "List directories in breadth-first order"; | ||
license = licenses.mit; | ||
platforms = platforms.unix; | ||
}; |
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.
Please list maintainers.
b41f2b3
to
9d95309
Compare
@orivej |
@GrahamcOfBorg eval |
Motivation for this change
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)