Re: [patch] Build pgoutput with MSVC

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: MauMau <maumau307(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [patch] Build pgoutput with MSVC
Date: 2017-05-05 09:58:21
Message-ID: CAB7nPqT3Rf+zhECxoTDXySNDa6ayqg-cZ=CogvaySo4rnxjKkw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, May 5, 2017 at 6:10 PM, MauMau <maumau307(at)gmail(dot)com> wrote:
> The pgoutput is not built with MSVC. The attached patch fixes this.
> I confirmed that a few INSERTs were replicated correctly.
>
> Should I add this matter in the PostgreSQL 10 Open Items page?

Yes, with Peter as committer and Petr as owner.

+ my $pgoutput = $solution->AddProject(
+ 'pgoutput', 'dll', '',
+ 'src/backend/replication/pgoutput');
+ $pgoutput->AddReference($postgres);
Yup, that's correct.

You have forgotten to update clean.bat, which should clean up pgoutput.dll.
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Magnus Hagander 2017-05-05 10:10:06 Re: [patch] Build pgoutput with MSVC
Previous Message MauMau 2017-05-05 09:10:12 [patch] Build pgoutput with MSVC