Re: pgsql: Move pg_dump memory routines into pg_dumpmem.c/h and restore com

On 11/26/2011 10:36 PM, Bruce Momjian wrote:
> Move pg_dump memory routines into pg_dumpmem.c/h and restore common.c
> with its original functions.  The previous function migration would
> cause too many difficulties in back-patching.


MSVC is still broken with this change, but now I think we've exposed a 
long-standing error in the MSVC build system.

Mkvcbuild.pm has:

    my $pgdumpall = AddSimpleFrontend('pg_dump', 1);
    $pgdumpall->{name} = 'pg_dumpall';
    $pgdumpall->AddIncludeDir('src\backend');
    $pgdumpall->AddFile('src\bin\pg_dump\pg_dumpall.c');
    $pgdumpall->AddFile('src\bin\pg_dump\keywords.c');
    $pgdumpall->AddFile('src\backend\parser\kwlookup.c')


AddSimpleFrontend() calls AddDir() which harvests the contents of 
$(OBJS) from the Makefile for the target. But pg_dumpall doesn't want 
$(OBJS). We've been benignly but mistakenly building it with them for a 
quite a few years, but now we can't do that any more, given Bruce's 
changes. It looks like the fix is to call AddProject() for pg_dumpall 
instead of AddSimpleFrontend() and then do a little more work ourselves 
to select exactly what we need.


I don't have time to do that and test it immediately, as I'll be away 
most of the day, so if anyone else can please go for it.

cheers

andrew