-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Add gitcommit to runc builds #636
Conversation
@marcosnils i honestly have no clue what is going on in that thread. I think it's good to have to git version compiled into the runc binary so that you know the exact commit that it was built from. Its also very simple to do. And if people are building without git then they can build without our makefile as well |
if git errors out, use "local" instead? |
i guess we can just remove that part if there is no git, that maybe the best outcome for everyone? |
Signed-off-by: Michael Crosby <crosbymichael@gmail.com>
10b4c86
to
beb3e5e
Compare
Ok, i changed it to exclude the commit information if it cannot be retrieved, i.e. you don't have git or the git checkout. |
@crosbymichael LGTM! |
@marcosnils thanks for the review |
LGTM |
1 similar comment
LGTM |
[Config Solaris] Add anchor tags for headings
Signed-off-by: Michael Crosby crosbymichael@gmail.com