-
Notifications
You must be signed in to change notification settings - Fork 315
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
Default binlink to all bins in package #2007
Conversation
Fixes #1921 This change makes the BINARY argument to the binlink command optional. If omitted, all binaries in the package will be symlinked. ``` ../../target/debug/hab pkg binlink -d /usr/local/bin core/net-tools » Symlinking dnsdomainname from core/net-tools into /usr/local/bin ★ Binary dnsdomainname from core/net-tools/1.60/20161214060044 symlinked to /usr/local/bin/dnsdomainname » Symlinking ypdomainname from core/net-tools into /usr/local/bin ★ Binary ypdomainname from core/net-tools/1.60/20161214060044 symlinked to /usr/local/bin/ypdomainname » Symlinking domainname from core/net-tools into /usr/local/bin ★ Binary domainname from core/net-tools/1.60/20161214060044 symlinked to /usr/local/bin/domainname » Symlinking hostname from core/net-tools into /usr/local/bin ★ Binary hostname from core/net-tools/1.60/20161214060044 symlinked to /usr/local/bin/hostname » Symlinking netstat from core/net-tools into /usr/local/bin ★ Binary netstat from core/net-tools/1.60/20161214060044 symlinked to /usr/local/bin/netstat ``` Signed-off-by: Nolan Davidson <ndavidson@chef.io>
This is great! |
Note that the travis failure in not pertinent to this pr. |
@thesentinels approve |
🤘 I am testing your branch against master before merging it. We do this to ensure that the master branch is never failing tests. |
Travis CI has started testing this PR. |
1 similar comment
Travis CI has started testing this PR. |
💖 Travis CI reports this PR passed. It always makes me feel nice when humans approve of one anothers work. I'm merging this PR now. I just want you and the contributor to answer me one question: |
Fixes #1921
This change makes the BINARY argument to the binlink command optional.
If omitted, all binaries in the package will be symlinked.
Signed-off-by: Nolan Davidson ndavidson@chef.io