golang: Fix conditionals not stripped
authorJeffery To <jeffery.to@gmail.com>
Fri, 15 Apr 2022 20:55:04 +0000 (04:55 +0800)
committerRosen Penev <rosenp@gmail.com>
Fri, 15 Apr 2022 22:18:12 +0000 (15:18 -0700)
Signed-off-by: Jeffery To <jeffery.to@gmail.com>
lang/golang/golang-host-build.mk
lang/golang/golang-package.mk

index ee4f1ea311b44691d883fe2f42663ba27a8c88ce..4b9e29873f0fc38a7fa01be9176eb09064c1111f 100644 (file)
@@ -107,10 +107,10 @@ GO_HOST_CUSTOM_LDFLAGS= \
 GO_HOST_INSTALL_ARGS= \
        -v \
        -ldflags "all=$(GO_HOST_DEFAULT_LDFLAGS)" \
-       $(if $(filter $(GO_HOST_ENABLE_PIE),1),-buildmode pie) \
-       $(if $(GO_HOST_GCFLAGS),-gcflags "$(GO_HOST_GCFLAGS)") \
-       $(if $(GO_HOST_CUSTOM_LDFLAGS),-ldflags "$(GO_HOST_CUSTOM_LDFLAGS) $(GO_HOST_DEFAULT_LDFLAGS)") \
-       $(if $(GO_HOST_TAGS),-tags "$(GO_HOST_TAGS)")
+       $(if $(GO_HOST_ENABLE_PIE),-buildmode pie) \
+       $(if $(strip $(GO_HOST_GCFLAGS)),-gcflags "$(GO_HOST_GCFLAGS)") \
+       $(if $(strip $(GO_HOST_CUSTOM_LDFLAGS)),-ldflags "$(GO_HOST_CUSTOM_LDFLAGS) $(GO_HOST_DEFAULT_LDFLAGS)") \
+       $(if $(strip $(GO_HOST_TAGS)),-tags "$(GO_HOST_TAGS)")
 
 define GoHost/Host/Configure
        $(GO_GENERAL_BUILD_CONFIG_VARS) \
index fb13e55d778182fc76112c63f03e701087ce1411..20a993726dce8a18bfedbf01180b60f0639c267d 100644 (file)
@@ -241,15 +241,15 @@ GO_PKG_INSTALL_ARGS= \
        -buildvcs=false \
        -trimpath \
        -ldflags "all=$(GO_PKG_DEFAULT_LDFLAGS)" \
-       $(if $(GO_PKG_DEFAULT_GCFLAGS),-gcflags "all=$(GO_PKG_DEFAULT_GCFLAGS)") \
-       $(if $(GO_PKG_DEFAULT_ASMFLAGS),-asmflags "all=$(GO_PKG_DEFAULT_ASMFLAGS)") \
-       $(if $(filter $(GO_PKG_ENABLE_PIE),1),-buildmode pie) \
+       $(if $(strip $(GO_PKG_DEFAULT_GCFLAGS)),-gcflags "all=$(GO_PKG_DEFAULT_GCFLAGS)") \
+       $(if $(strip $(GO_PKG_DEFAULT_ASMFLAGS)),-asmflags "all=$(GO_PKG_DEFAULT_ASMFLAGS)") \
+       $(if $(GO_PKG_ENABLE_PIE),-buildmode pie) \
        $(if $(filter $(GO_ARCH),arm),-installsuffix "v$(GO_ARM)") \
        $(if $(filter $(GO_ARCH),mips mipsle),-installsuffix "$(GO_MIPS)") \
        $(if $(filter $(GO_ARCH),mips64 mips64le),-installsuffix "$(GO_MIPS64)") \
-       $(if $(GO_PKG_GCFLAGS),-gcflags "$(GO_PKG_GCFLAGS) $(GO_PKG_DEFAULT_GCFLAGS)") \
-       $(if $(GO_PKG_CUSTOM_LDFLAGS),-ldflags "$(GO_PKG_CUSTOM_LDFLAGS) $(GO_PKG_DEFAULT_LDFLAGS)") \
-       $(if $(GO_PKG_TAGS),-tags "$(GO_PKG_TAGS)")
+       $(if $(strip $(GO_PKG_GCFLAGS)),-gcflags "$(GO_PKG_GCFLAGS) $(GO_PKG_DEFAULT_GCFLAGS)") \
+       $(if $(strip $(GO_PKG_CUSTOM_LDFLAGS)),-ldflags "$(GO_PKG_CUSTOM_LDFLAGS) $(GO_PKG_DEFAULT_LDFLAGS)") \
+       $(if $(strip $(GO_PKG_TAGS)),-tags "$(GO_PKG_TAGS)")
 
 define GoPackage/Build/Configure
        $(GO_GENERAL_BUILD_CONFIG_VARS) \