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
osm-cli: init at 0.1.0 #94748
osm-cli: init at 0.1.0 #94748
Conversation
@danieldk Excellent suggestions! I have made those updates. Thank you for your speedy and thorough review 💯 |
One more question, I saw that the derivation is added to |
@danieldk I arranged it that way because I think there may ultimately be multiple subpackages under this project, such as the controller. |
I see. Wouldn't it make more sense to have a single package If not, I'd put this derivation in But it seems most logical to have just a single package with both, unless there are very good reasons against that. |
let | ||
pname = "osm-cli"; | ||
version = "0.1.0"; | ||
sha256 = "0hc63xvww4p4bqsq2ihz5in5ia9x7ibfa7ixd6fkxvdb1rbwxrx6"; | ||
vendorSha256 = "03qy4gl9lii4amsfg2y4ib5xwg74gri601pkfa3c7jwc034xf5kf"; | ||
in buildGoModule { | ||
inherit pname version vendorSha256; | ||
|
||
src = fetchFromGitHub { | ||
inherit sha256; | ||
|
||
owner = "openservicemesh"; | ||
repo = "osm"; | ||
rev = "v${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.
let | |
pname = "osm-cli"; | |
version = "0.1.0"; | |
sha256 = "0hc63xvww4p4bqsq2ihz5in5ia9x7ibfa7ixd6fkxvdb1rbwxrx6"; | |
vendorSha256 = "03qy4gl9lii4amsfg2y4ib5xwg74gri601pkfa3c7jwc034xf5kf"; | |
in buildGoModule { | |
inherit pname version vendorSha256; | |
src = fetchFromGitHub { | |
inherit sha256; | |
owner = "openservicemesh"; | |
repo = "osm"; | |
rev = "v${version}"; | |
}; | |
buildGoModule rec { | |
pname = "osm-cli"; | |
version = "0.1.0"; | |
src = fetchFromGitHub { | |
owner = "openservicemesh"; | |
repo = "osm"; | |
rev = "v${version}"; | |
sha256 = "0hc63xvww4p4bqsq2ihz5in5ia9x7ibfa7ixd6fkxvdb1rbwxrx6"; | |
}; | |
vendorSha256 = "03qy4gl9lii4amsfg2y4ib5xwg74gri601pkfa3c7jwc034xf5kf"; |
Result of 1 package built:
|
I marked this as stale due to inactivity. → More info |
Motivation for this change
Added a package for the Open Service Mesh CLI tool.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)