From: Dave Chinner Date: Tue, 7 Apr 2020 18:29:39 +0000 (-0400) Subject: xfsprogs: LDFLAGS comes from configure, not environment X-Git-Tag: v5.6.0-rc1~11 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=2582ae0d770560960f87d8e921be543fc2caa40d;p=thirdparty%2Fxfsprogs-dev.git xfsprogs: LDFLAGS comes from configure, not environment When doing: $ LDFLAGS=foo make bad things happen because we don't initialise LDFLAGS to an empty string in include/builddefs.in and hence make takes wahtever is in the environment and runs with it. This causes problems with linker options specified correctly through configure. We don't support overriding build flags (like CFLAGS) though the make environment, so it was an oversight 13 years ago to allow LDFLAGS to be overridden when adding support to custom LDFLAGS being passed from the the configure script. This ensures we only ever use linker flags from configure, not the make environment. Signed-off-by: Dave Chinner Reviewed-by: Christoph Hellwig Signed-off-by: Eric Sandeen --- diff --git a/include/builddefs.in b/include/builddefs.in index 891bf93d7..6ed9d2951 100644 --- a/include/builddefs.in +++ b/include/builddefs.in @@ -16,6 +16,10 @@ LTLDFLAGS = @LDFLAGS@ CFLAGS = @CFLAGS@ -D_FILE_OFFSET_BITS=64 -Wno-address-of-packed-member BUILD_CFLAGS = @BUILD_CFLAGS@ -D_FILE_OFFSET_BITS=64 +# make sure we don't pick up whacky LDFLAGS from the make environment and +# only use what we calculate from the configured options above. +LDFLAGS = + LIBRT = @librt@ LIBUUID = @libuuid@ LIBPTHREAD = @libpthread@