Andrew Dunstan <[EMAIL PROTECTED]> writes:
> Unless it also lies on the echoed command line this seems an 
> unconvincing explanation. The seahorse log says:

> gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Winline 
> -Wdeclaration-after-statement -Wendif-labels -fno-strict-aliasing  
> -I../../../src/include -I./src/include/port/win32 -DEXEC_BACKEND  
> -I/c/tcl/include "-I../../../src/include/port/win32" 
> '-DPGBINDIR="/home/pgbuild/pgfarmbuild/HEAD/inst/bin"' 
> '-DLIBDIR="/home/pgbuild/pgfarmbuild/HEAD/inst/lib"' 
> '-DPGSHAREDIR="/home/pgbuild/pgfarmbuild/HEAD/inst/share/postgresql"' 
> '-DHOST_TUPLE="i686-pc-mingw32"' '-DMAKEPROG="make"' 
> '-DSHELLPROG="/bin/sh.exe"' -c -o pg_regress.o pg_regress.c

> If those -D values are not what it gets then that would be quite evil.

Indeed ... but if those *are* what it gets then how can you explain the
constructed paths?

I just committed a change to extract the paths via pg_config_paths.h.
If that doesn't fix it then I guess the next thing is to put in some
debug printout to show what values are really getting compiled in :-(

> We used to pass these values almost same way when we first did initdb in 
> C, and I don't recall any such problems. We had:

> override CPPFLAGS := -DPGBINDIR=\"$(*bindir*)\" -DPGDATADIR=\"$(*datadir*)\" 
> -DFRONTEND -I$(*libpq_srcdir*) $(*CPPFLAGS*)

That seems a bit interesting.  What are the stars for?  I don't see
anything about a syntax like that in my gmake documentation.

> There is also this warning, by the way:
> pg_regress.c:63: warning: 'shellprog' defined but not used

Good catch, fix committed.

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster

Reply via email to