Skip to content

Commit

Permalink
Do not redefine compiler flags that are up to build env to define
Browse files Browse the repository at this point in the history
Yet another case where -D_FORTIFY_SOURCE=2 conflicts with distribution
provided one. Flags like this should be up to distribution policy to
define, not individual packages.
  • Loading branch information
marmarek committed Jun 22, 2024
1 parent c231fe5 commit 973fe96
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion not-script/Makefile
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
CC=gcc
CFLAGS := $(CFLAGS) -g3 -O2 -Wall -Wextra -Werror -fPIE -D_FORTIFY_SOURCE=2 -D_GNU_SOURCE -Wmaybe-uninitialized
CFLAGS += -g -O2 -Wall -Wextra -Werror -fPIE -D_GNU_SOURCE -Wmaybe-uninitialized
.PHONY: clean install all
all: not-script

Expand Down
2 changes: 1 addition & 1 deletion qrexec-lib/Makefile
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
CC=gcc
CFLAGS := $(CFLAGS) -I. -g3 -O2 -Wall -Wextra -Werror -pie -fPIC -Wmissing-declarations -Wmissing-prototypes
CFLAGS += -I. -g -O2 -Wall -Wextra -Werror -pie -fPIC -Wmissing-declarations -Wmissing-prototypes
SO_VER=2
LDFLAGS+=-Wl,--no-undefined,--as-needed,-Bsymbolic -L .
.PHONY: all clean install check
Expand Down

0 comments on commit 973fe96

Please sign in to comment.