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
gdm: init at 1.4 #21870
gdm: init at 1.4 #21870
Conversation
Does somebody else also gets a "Bad commit or revision" error, when building this package like travis does? |
Nitpick: I wonder if gdm might be too ambiguous name; in particular, around Linux packaging this name tends to be mostly associated with gnome display manager – and we use the name as well:
|
src = fetchgit { | ||
url = "https://github.com/sparrc/gdm"; | ||
sha256 = "0kpqmbg144qcvd8k88j9yx9lrld85ray2viw161xajafk16plvld"; | ||
rev = 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.
Have you tried "refs/tags/${version}"
, also is there a reason not to use fetchFromGitHub
here?
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 the reason for not using fetchFromGitHub
could be how go2nix
generated it :-/ I need to implement it there to make maintainers life easier.
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.
oh, usually I replace fetchgit manually. However in this case I forgot it. @kamilchm if you take a look at how nix-prefetch-zip
is implemented, it should be also possible to generate fetchFromGitHub
for dependencies (the hash is calculated using the content of the directory, which should be the same as fresh checkout of the local repo). This should you also give a nice speed-up because fetchgit is kind of slow for both downloading and disk i/o.
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 was trying to do it not only for the main source, but for all goDeps
#19311
I need to get back to it in near future.
909329b
to
ffe2736
Compare
@vcunat I also thought about this one, but did not find a better naming. Are people aware that they have to use modules to install gdm or would it lead to confusion, when they try to add |
How about using |
Motivation for this change
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
./result/bin/
)