Re: pgsql/src (Makefile.global.in) - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql/src (Makefile.global.in)
Date
Msg-id 23635.974698392@sss.pgh.pa.us
Whole thread Raw
In response to pgsql/src (Makefile.global.in)  (vadim@postgresql.org)
Responses Re: pgsql/src (Makefile.global.in)  (Peter Eisentraut <peter_e@gmx.net>)
List pgsql-committers
vadim@postgresql.org writes:

*** 145,150 ****
--- 145,152 ----
  ifeq ($(GCC), yes)
    CFLAGS += -Wall -Wmissing-prototypes -Wmissing-declarations
  endif
+
+ CFLAGS += -DXLOG

  CXX = @CXX@
  GXX = @GXX@

Ick.  Couldn't we do this by #define XLOG in config.h.in, instead?
CFLAGS are horribly vulnerable to people overriding them with
make options.

I do recall that when I last tried to build with WAL, my attempt
to add #define XLOG to config.h failed --- but that indicates poor
coding practices elsewhere.  It *should* work to add or remove a
feature symbol in config.h.

            regards, tom lane

pgsql-committers by date:

Previous
From: vadim@postgresql.org
Date:
Subject: pgsql/src (Makefile.global.in)
Next
From: vadim@postgresql.org
Date:
Subject: pgsql/src/include/catalog (catversion.h)