-
-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
dateutils: init at 0.4.1 #22512
dateutils: init at 0.4.1 #22512
Conversation
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.
Thanks for the PR. Some minor changes are needed.
{ stdenv, fetchurl }: | ||
|
||
stdenv.mkDerivation rec { | ||
version = "0.4.1"; |
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 fix indentation in this file
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.
I hope I've done that correctly, 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.
two spaces are needed. Here it's correct.
version = "0.4.1"; | ||
name = "dateutils-${version}"; | ||
|
||
src =fetchurl { |
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.
I think you're supposed to use fetchFromBitbucket
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 this the preference? It can't find the make file correctly when I use the fetchFromBitbucket
.
homepage = http://www.fresse.org/dateutils/; | ||
license = licenses.bsd3; | ||
platforms = platforms.linux; | ||
maintainers = [ maintainers.paperdigits ]; |
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.
did you add yourself to lib/maintainers.nix
?
}; | ||
|
||
meta = with stdenv.lib; { | ||
description = "Virtual lighttable and darkroom for photographers"; |
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 also fix the description, and any other metadata that may have been copypasted
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.
I've done that, thanks.
8e66607
to
720efc1
Compare
lib/maintainers.nix
Outdated
@@ -358,6 +358,7 @@ | |||
paholg = "Paho Lurie-Gregg <paho@paholg.com>"; | |||
pakhfn = "Fedor Pakhomov <pakhfn@gmail.com>"; | |||
palo = "Ingolf Wanger <palipalo9@googlemail.com>"; | |||
paperdigits = "Mica Semrick <mica@silentumbrella.com>"; |
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.
Should be spaces, not tab
name = "dateutils-${version}"; | ||
|
||
src =fetchurl { | ||
url = "https://bitbucket.org/hroptatyr/dateutils/downloads/${name}.tar.xz"; |
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.
but here its not
720efc1
to
70a7fe9
Compare
Thanks for your patients, I've attempted to correct the cited problems. |
Motivation for this change
Package a nice CLI app for computing dates. Particularly useful for finding relative dates.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)