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-ecr-credential-helper: init at 0.1.0 #44050
Conversation
|
||
goPackagePath = "github.com/awslabs/amazon-ecr-credential-helper"; | ||
|
||
src = fetchurl { |
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.
Please use fetchFromGitHub
instead of fetchurl
because it relies on the hash of the contents of the repository instead of the hash of a tarball, and it would allow us to fix all GitHub src fetchers, if GitHub changes something about how it works.
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, thank you for pointing that out! I was always thinking about the time it takes to fetch the src.
@GrahamcOfBorg build amazon-ecr-credential-helper |
Success on x86_64-linux (full log) Attempted: amazon-ecr-credential-helper Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: amazon-ecr-credential-helper Partial log (click to expand)
|
4a5d0ab
to
1b25e6a
Compare
@ryantm PTAL |
@GrahamcOfBorg build amazon-ecr-credential-helper |
Success on aarch64-linux (full log) Attempted: amazon-ecr-credential-helper Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: amazon-ecr-credential-helper Partial log (click to expand)
|
@GrahamcOfBorg eval |
Success on aarch64-linux (full log) Attempted: amazon-ecr-credential-helper Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: amazon-ecr-credential-helper Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: amazon-ecr-credential-helper Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: amazon-ecr-credential-helper Partial log (click to expand)
|
Motivation for this change
The Amazon ECR Docker Credential Helper is a credential helper for the Docker daemon that makes it easier to use Amazon Elastic Container Registry.
https://github.com/awslabs/amazon-ecr-credential-helper
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)