Skip to content

Commit

Permalink
Use gnu99 instead of c99, as strdup is not part of C99
Browse files Browse the repository at this point in the history
Illumina#10

Not sure if this is necessary, but it didn't break anything.
  • Loading branch information
lgruen authored and edmundmiller committed Dec 7, 2023
1 parent e6dea0a commit 022704e
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion config.mk
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,9 @@ CPPFLAGS += -DLOCAL_BUILD
CPPFLAGS += -D'DRAGEN_OS_VERSION="$(DRAGEN_OS_VERSION)"'
CPPFLAGS += -DVERSION_STRING="$(VERSION_STRING)"
CXXFLAGS+=$(CXXWARNINGS) $(CXXSTD)
CFLAGS+=$(CWARNINGS)
# Use gnu99 instead of c99, as strdup is not part of C99:
# https://stackoverflow.com/questions/26284110/strdup-confused-about-warnings-implicit-declaration-makes-pointer-with
CFLAGS+=$(CWARNINGS) -std=gnu99
LDFLAGS?=

ifneq (,$(BOOST_INCLUDEDIR))
Expand Down

0 comments on commit 022704e

Please sign in to comment.