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
fly: init at 5.3.0 #64714
fly: init at 5.3.0 #64714
Conversation
postInstall = '' | ||
install -D -m 444 ${ | ||
writeText "fly.bash" (builtins.readFile ./fly.bash) | ||
} $out/share/bash-completion/completions/fly |
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.
} $out/share/bash-completion/completions/fly | |
install -D -m 444 fly.bash $out/share/bash-completion/completions/fly |
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.
@kalbasit Because fly.bash
isn't present in the project's source, I had to implement this slightly differently: 9e465d2#diff-bd215dfd96e2fb6f91323ab49fa3563fR28
Add `fly` CLI for working with Concourse CI, and include Bash completions.
@GrahamcOfBorg build fly |
The writeText argument is unused. It was originally used to copy a bash completion script into the derivation, but a different approach was chosen, and I simply forgot to remove the writeText argument. NixOS@78013b4 NixOS#64714 (comment)
Add
fly
CLI for working with Concourse.Motivation for this change
There was no existing derivation for fly CLI.
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)