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
amazon-ecs-cli: init at 1.12.1 #52605
Conversation
cd8de7a
to
2fadbe6
Compare
sha256 = "100iv4cchnxl1s02higga5v3hvawi4c7sqva97x34qigr4r7fxwm"; | ||
}; | ||
|
||
unpackCmd = "mkdir bin; cp $src ./bin/ecs-cli"; |
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.
Seems to me that unpackCmd
and installPhase
can be replaced with
install -D $src $out/bin/ecs-cli
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 not found a way to disable the unpack phase, could you point me in the right direction?
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.
A simple way to disable specific phases is to set them to ":", as in
{
unpackPhase = ":";
}
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.
like this?
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.
LGTM. Using install
would shorten it further, as noted above. Even simpler would be
runCommand "foo" { src = fetchurl { /* ... */ }; } ''install -D $src $out/bin/foo''
There may even be a suitable canned builder that captures this pattern entirely. That said I think what you have is acceptable as is ...
2fadbe6
to
7ad77f8
Compare
7ad77f8
to
8cb8b2b
Compare
Motivation for this change
SInce we needed this for the deployment process I figured I push it upstream
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)