-
Notifications
You must be signed in to change notification settings - Fork 113
make: Add build option STATIC=1 to statically link #523
Conversation
/test |
This PR doesn't change the default behavior of kata-agent. jenkins-ci-Power8-ubuntu-16-04-initrd failed:
jenkins-ci-fedora failed:
|
Makefile
Outdated
@@ -79,7 +79,11 @@ else | |||
SECCOMP=no | |||
endif | |||
# go build common flags | |||
BUILDFLAGS := -buildmode=pie | |||
ifndef STATIC |
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.
maybe you should change this condition, I think the following would be easier to understand
ifdef STATIC
LDFLAGS := -extldflags '-static'
else
BUILDFLAGS := -buildmode=pie
endif
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.
maybe you should change this condition, I think the following would be easier to understand
ifdef STATIC LDFLAGS := -extldflags '-static' else BUILDFLAGS := -buildmode=pie endif
Updated.
/test |
/retest |
Add build option STATIC=1 to statically link kata-agent. Fixes: kata-containers#522 Signed-off-by: Hui Zhu <[email protected]>
/test |
jenkins-ci-Power8-ubuntu-16-04-initrd fail:
This PR doesn't change the default behavior of kata-agent. |
@nitkon ^^^ can you have a check of the power8 CI slave status? |
Add build option STATIC=1 to statically link kata-agent.
Fixes: #522
Signed-off-by: Hui Zhu [email protected]