diff options
| author | Joel Kitching <kitching@google.com> | 2019-12-05 15:02:32 +0800 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-01-03 21:13:26 +0000 |
| commit | 59c9d59bfd5a2a1ed2c1814fbdcb0a0c82fb447a (patch) | |
| tree | 5aee76c774f7e881a4b7d67d695cf752eb33e797 | |
| parent | 1bdf613b1d364b97fed62da60c52062732b5c8d2 (diff) | |
| download | vboot-59c9d59bfd5a2a1ed2c1814fbdcb0a0c82fb447a.tar.gz | |
vboot: remove static linking support from Makefile
Previously, futility and crossystem needed to be statically
linked. (futility had both futility and futility_s.)
Since there's no longer any need to statically link any binaries
built by vboot_reference, remove this support from the Makefile.
BUG=b:124141368, chromium:765499
TEST=make clean && make runtests
BRANCH=none
Change-Id: I8c2b5c5b5aa86bbecb0c264f688dfdac4b19ca7e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/1954976
Tested-by: Joel Kitching <kitching@chromium.org>
Commit-Queue: Joel Kitching <kitching@chromium.org>
Reviewed-by: Julius Werner <jwerner@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/1986997
Reviewed-by: Shelley Chen <shchen@chromium.org>
Commit-Queue: Shelley Chen <shchen@chromium.org>
Tested-by: Shelley Chen <shchen@chromium.org>
| -rw-r--r-- | Makefile | 15 |
1 files changed, 3 insertions, 12 deletions
@@ -574,9 +574,6 @@ UTIL_SCRIPTS += \ utility/vbutil_what_keys endif -# These utilities should also provide static linked version (*_s). -UTIL_NAMES_STATIC = - UTIL_NAMES = \ utility/crossystem \ utility/dumpRSAPublicKey \ @@ -593,8 +590,7 @@ LZMA_LIBS = $(shell ${PKG_CONFIG} --libs liblzma) YAML_LIBS = $(shell ${PKG_CONFIG} --libs yaml-0.1) endif -UTIL_BINS_STATIC := $(addsuffix _s,$(addprefix ${BUILD}/,${UTIL_NAMES_STATIC})) -UTIL_BINS = $(addprefix ${BUILD}/,${UTIL_NAMES}) ${UTIL_BINS_STATIC} +UTIL_BINS = $(addprefix ${BUILD}/,${UTIL_NAMES}) ALL_OBJS += $(addsuffix .o,${UTIL_BINS}) @@ -1005,12 +1001,8 @@ cgpt_wrapper_install: cgpt_install ${CGPT_WRAPPER} # These have their own headers too. ${BUILD}/utility/%: INCLUDES += -Iutility/include -${UTIL_BINS} ${UTIL_BINS_STATIC}: ${UTILLIB} -${UTIL_BINS} ${UTIL_BINS_STATIC}: LIBS = ${UTILLIB} - -# Utilities for auto-update toolkits must be statically linked. -${UTIL_BINS_STATIC}: LDFLAGS += -static - +${UTIL_BINS}: ${UTILLIB} +${UTIL_BINS}: LIBS = ${UTILLIB} .PHONY: utils utils: ${UTIL_BINS} ${UTIL_SCRIPTS} @@ -1146,7 +1138,6 @@ ${UTIL_DEFAULTS}: # Some utilities need external crypto functions CRYPTO_LIBS := $(shell ${PKG_CONFIG} --libs libcrypto) -CRYPTO_STATIC_LIBS := $(shell ${PKG_CONFIG} --libs libcrypto --static) ${BUILD}/utility/dumpRSAPublicKey: LDLIBS += ${CRYPTO_LIBS} ${BUILD}/utility/pad_digest_utility: LDLIBS += ${CRYPTO_LIBS} |