From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | Christoph Berg <christoph(dot)berg(at)credativ(dot)de>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCH] Pass COPT and PROFILE to CXXFLAGS as well |
Date: | 2018-11-13 22:27:17 |
Message-ID: | 22230.1542148037@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Andres Freund <andres(at)anarazel(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2018-11-13 22:33:01 | Re: add the source of param misconfigurations to error messages |
Previous Message | Jordan Deitch | 2018-11-13 22:02:48 | add the source of param misconfigurations to error messages |