pg.ddx.io pgsql-committers@postgresql.org mailing list archivehelp / color / mirror / Atom feed
pgsql: Use CXXFLAGS instead of CFLAGS for linking C++ code 5+ messages / 2 participants [nested] [flat]
* pgsql: Use CXXFLAGS instead of CFLAGS for linking C++ code @ 2024-08-04 09:24 Peter Eisentraut <peter@eisentraut.org> 0 siblings, 0 replies; 5+ messages in thread From: Peter Eisentraut @ 2024-08-04 09:24 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org Use CXXFLAGS instead of CFLAGS for linking C++ code Otherwise, this would break if using C and C++ compilers from different families and they understand different options. It already used the right flags for compiling, this is only for linking. Also, the meson setup already did this correctly. Reported-by: Tom Lane <tgl@sss.pgh.pa.us> Discussion: https://www.postgresql.org/message-id/228700.1722717983@sss.pgh.pa.us Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/365b5a345b2680615527b23ee6befa09a2f784f2 Modified Files -------------- src/backend/jit/llvm/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ^ permalink raw reply [nested|flat] 5+ messages in thread
* pgsql: Use CXXFLAGS instead of CFLAGS for linking C++ code @ 2026-02-26 17:07 Tom Lane <tgl@sss.pgh.pa.us> 0 siblings, 0 replies; 5+ messages in thread From: Tom Lane @ 2026-02-26 17:07 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org Use CXXFLAGS instead of CFLAGS for linking C++ code Otherwise, this would break if using C and C++ compilers from different families and they understand different options. It already used the right flags for compiling, this is only for linking. Also, the meson setup already did this correctly. Back-patch of v18 commit 365b5a345 into older supported branches. At the time we were only aware of trouble in v18, but as shown by buildfarm member siren, older branches can hit the problem too. Reported-by: Tom Lane <tgl@sss.pgh.pa.us> Author: Peter Eisentraut <peter@eisentraut.org> Discussion: https://www.postgresql.org/message-id/228700.1722717983@sss.pgh.pa.us Discussion: https://postgr.es/m/3109540.1771698685@sss.pgh.pa.us Backpatch-through: 14-17 Branch ------ REL_17_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/b6b7e96365eb7b5f79d4e655afcfb4f9cb12ab88 Modified Files -------------- src/backend/jit/llvm/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ^ permalink raw reply [nested|flat] 5+ messages in thread
* pgsql: Use CXXFLAGS instead of CFLAGS for linking C++ code @ 2026-02-26 17:07 Tom Lane <tgl@sss.pgh.pa.us> 0 siblings, 0 replies; 5+ messages in thread From: Tom Lane @ 2026-02-26 17:07 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org Use CXXFLAGS instead of CFLAGS for linking C++ code Otherwise, this would break if using C and C++ compilers from different families and they understand different options. It already used the right flags for compiling, this is only for linking. Also, the meson setup already did this correctly. Back-patch of v18 commit 365b5a345 into older supported branches. At the time we were only aware of trouble in v18, but as shown by buildfarm member siren, older branches can hit the problem too. Reported-by: Tom Lane <tgl@sss.pgh.pa.us> Author: Peter Eisentraut <peter@eisentraut.org> Discussion: https://www.postgresql.org/message-id/228700.1722717983@sss.pgh.pa.us Discussion: https://postgr.es/m/3109540.1771698685@sss.pgh.pa.us Backpatch-through: 14-17 Branch ------ REL_16_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/b9f7ceabd820369280d4ca6bfaab43cf8345f1bd Modified Files -------------- src/backend/jit/llvm/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ^ permalink raw reply [nested|flat] 5+ messages in thread
* pgsql: Use CXXFLAGS instead of CFLAGS for linking C++ code @ 2026-02-26 17:07 Tom Lane <tgl@sss.pgh.pa.us> 0 siblings, 0 replies; 5+ messages in thread From: Tom Lane @ 2026-02-26 17:07 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org Use CXXFLAGS instead of CFLAGS for linking C++ code Otherwise, this would break if using C and C++ compilers from different families and they understand different options. It already used the right flags for compiling, this is only for linking. Also, the meson setup already did this correctly. Back-patch of v18 commit 365b5a345 into older supported branches. At the time we were only aware of trouble in v18, but as shown by buildfarm member siren, older branches can hit the problem too. Reported-by: Tom Lane <tgl@sss.pgh.pa.us> Author: Peter Eisentraut <peter@eisentraut.org> Discussion: https://www.postgresql.org/message-id/228700.1722717983@sss.pgh.pa.us Discussion: https://postgr.es/m/3109540.1771698685@sss.pgh.pa.us Backpatch-through: 14-17 Branch ------ REL_15_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/41d75b9a4a1c7f84bafd331428c005e5743bac3d Modified Files -------------- src/backend/jit/llvm/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ^ permalink raw reply [nested|flat] 5+ messages in thread
* pgsql: Use CXXFLAGS instead of CFLAGS for linking C++ code @ 2026-02-26 17:07 Tom Lane <tgl@sss.pgh.pa.us> 0 siblings, 0 replies; 5+ messages in thread From: Tom Lane @ 2026-02-26 17:07 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org Use CXXFLAGS instead of CFLAGS for linking C++ code Otherwise, this would break if using C and C++ compilers from different families and they understand different options. It already used the right flags for compiling, this is only for linking. Also, the meson setup already did this correctly. Back-patch of v18 commit 365b5a345 into older supported branches. At the time we were only aware of trouble in v18, but as shown by buildfarm member siren, older branches can hit the problem too. Reported-by: Tom Lane <tgl@sss.pgh.pa.us> Author: Peter Eisentraut <peter@eisentraut.org> Discussion: https://www.postgresql.org/message-id/228700.1722717983@sss.pgh.pa.us Discussion: https://postgr.es/m/3109540.1771698685@sss.pgh.pa.us Backpatch-through: 14-17 Branch ------ REL_14_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/231570d33581e34cdfbacfa0f89a3d6fff790ef3 Modified Files -------------- src/backend/jit/llvm/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ^ permalink raw reply [nested|flat] 5+ messages in thread
end of thread, other threads:[~2026-02-26 17:07 UTC | newest] Thread overview: 5+ messages (download: mbox mbox.gz follow: Atom feed) -- links below jump to the message on this page -- 2024-08-04 09:24 pgsql: Use CXXFLAGS instead of CFLAGS for linking C++ code Peter Eisentraut <peter@eisentraut.org> 2026-02-26 17:07 pgsql: Use CXXFLAGS instead of CFLAGS for linking C++ code Tom Lane <tgl@sss.pgh.pa.us> 2026-02-26 17:07 pgsql: Use CXXFLAGS instead of CFLAGS for linking C++ code Tom Lane <tgl@sss.pgh.pa.us> 2026-02-26 17:07 pgsql: Use CXXFLAGS instead of CFLAGS for linking C++ code Tom Lane <tgl@sss.pgh.pa.us> 2026-02-26 17:07 pgsql: Use CXXFLAGS instead of CFLAGS for linking C++ code Tom Lane <tgl@sss.pgh.pa.us>
This inbox is served by DDX for PostgreSQL; see mirroring instructions for how to clone and mirror all data and code used for this inbox