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
Init shiori #69255
Init shiori #69255
Conversation
@@ -0,0 +1,22 @@ | |||
{ stdenv, buildGoModule, fetchFromGitHub }: | |||
|
|||
buildGoModule rec { |
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.
See #69229. I think buildGoPackage
is preferred, though I also know little about go.
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 have tried with buildGoPackage
, but it failed after compilation and tests succeeded :-/
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 have never used either so cannot really help. I just remembered seeing the pr I linked earlier
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 lots of other prs are using buildGoModule, so I wouldn't worry about it unless someone else suggests changing 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.
Well, the Nixpkgs documentation does say that:
The function
buildGoPackage
builds legacy Go programs, not supporting Go modules.
This gives the impression that buildGoModule
is preferred ^^
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.
sure does
3c1257e
to
367cd2c
Compare
Thanks for all the feedback, @alexarice ! |
@GrahamcOfBorg build shiori |
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.
Builds and test passes with nix-review
Thanks @minijackson and @alexarice |
Motivation for this change
Shiori, the simple bookmarks manager. I don't know much about the Go programming lanuage, and much less how to package it. Please tell me if there could be improvements.
This PR comes with a module and a test for the module
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)