On 2019-12-03 06:03, Tom Lane wrote:
Peter Eisentraut <peter.eisentr...@2ndquadrant.com> writes:
Currently, configure puts the configure args into the makefiles and
then have the makefiles pass them to the build of pg_config. That looks
like an unnecessary redirection, and indeed that method was
put in place when pg_config was a shell script. We can simplify that
by having configure put the value into pg_config.h directly. This
also makes the standard build system match how the MSVC build system
already does it.
I dunno, is this really an improvement? It makes the handling of
VAL_CONFIGURE different from every other one of the values passed
into pg_config, and I don't see any countervailing addition of
some other regularity.
The other values come from the makefiles, so we have to do it that way.
The configure args come from configure, so why make them go through the
makefile? (PG_VERSION also comes in that way. ;-) )
There is also the weird difference with how the MSVC build system
handles it. It appends VAL_CONFIGURE to pg_config.h instead of passing
it on the command line.
I'm also a bit suspicious of the ad-hoc escaping step ...
Hmm, the current way doesn't handle embedded quotes at all, so perhaps
this wouldn't be necessary. But it would add some robustness.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
