Skip to content

Commit

Permalink
Merge pull request #4970 from jackchenjc/issue-4916-2
Browse files Browse the repository at this point in the history
fix: Revise the flag name and add detailed comments
  • Loading branch information
cloudxxx8 authored Oct 14, 2024
2 parents 39873e1 + 858ca2e commit 66e3c9b
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@ INCLUDE_DELAYED_START_BUILD_CORE:="false"
# change the following boolean flag to include or exclude the delayed start libs for builds for support services exculsively
INCLUDE_DELAYED_START_BUILD_SUPPORT:="true"

# change the following boolean flag to enable or disable the ASLR for builds
ENABLE_ASLR:="true"
# change the following boolean flag to enable or disable the PIE for builds
# change the following boolean flag to enable or disable the Full RELRO (RELocation Read Only) for linux ELF (Executable and Linkable Format) binaries
ENABLE_FULL_RELRO:="true"
# change the following boolean flag to enable or disable PIE for linux binaries which is needed for ASLR (Address Space Layout Randomization) on Linux, the ASLR support on Windows is enabled by default
ENABLE_PIE:="true"

GO=go
Expand Down Expand Up @@ -64,7 +64,7 @@ DOCKER_TAG=$(VERSION)-dev
GOFLAGS=-ldflags "-s -w -X github.com/edgexfoundry/edgex-go.Version=$(VERSION)" -trimpath -mod=readonly
GOTESTFLAGS?=-race

ifeq ($(ENABLE_ASLR), "true")
ifeq ($(ENABLE_FULL_RELRO), "true")
GOFLAGS += -ldflags "-bindnow"
endif

Expand Down

0 comments on commit 66e3c9b

Please sign in to comment.