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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
dobi: init at 0.8 #20124
dobi: init at 0.8 #20124
Conversation
@vdemeester, thanks for your PR! By analyzing the history of the files in this pull request, we identified @zimbatm, @zraexy and @edolstra to be potential reviewers. |
You could try to pass |
ooh I missed that one 馃槄 Thanks @kamilchm, I'll do that \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.
This package generates a $out/bin/script
binary. What's that used for? It's a very generic name and seems to fail to load some config to generate documentation?
If it's not useful for using dobi, please simply remove it for instance in the postInstall
hook.
@fpletz updated by adding |
Signed-off-by: Vincent Demeester <vincent@sbr.pm>
|
||
buildGoPackage rec { | ||
name = "dobi-${version}"; | ||
version = "v0.8"; |
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.
Version strings must begin with a digit. In this case, the package will end up with the name "dobi-v0.8" and no version (see builtins.parseDrvName
).
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.
ohh 馃槄
Motivation for this change
dobi is a build automation tool for Docker applications that I tend to use on several projects.
I have a question though : Is there a way to specify
-ldflags
withbuildGoPackage
? (didn't find it but I might have miss something).Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
Tested compilation of all pkgs that depend on this change usingnix-shell -p nox --run "nox-review wip"
./result/bin/
)馃惛
Signed-off-by: Vincent Demeester vincent@sbr.pm