Re: [PATCH] Pass COPT and PROFILE to CXXFLAGS as well
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andres Freund <andres@anarazel.de>
Cc: Christoph Berg <christoph.berg@credativ.de>,
PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2018-11-13T22:27:17Z
Lists: pgsql-hackers
Andres Freund <andres@anarazel.de> writes: > On 2018-11-13 11:40:05 +0100, Christoph Berg wrote: >> While working on making extension modules built reproducibly, I >> noticed that extra flags passed via COPT (notably -ffile-prefix-map) >> do not get added to CXXFLAGS. > PROFILE I can see, but COPT I'm less sure. The name suggests it's about > C not C++. How about adding CXXOPT? > Secondary question: Why aren't you using CFLAGS/CXXFLAGS for this? COPT (and PROFILE) are handy for injecting additional flags manually; they're even documented for that (cf installation.sgml, around line 1550). I agree that CXXOPT would be a better idea than COPT for this. Not sure about PROFILE. But we could inject the latter into both flag sets and then document that if that's not what you want, use COPT. regards, tom lane
Commits
-
Add PG_CFLAGS, PG_CXXFLAGS, and PG_LDFLAGS variables to PGXS
- 42b204db298c 9.4.21 landed
- 12ff406f35b6 9.5.16 landed
- 5e63df82708b 9.6.12 landed
- da14c9b19048 10.7 landed
- 946430da6a0b 11.2 landed
- ac3a9afdbefd 12.0 landed