Re: [HACKERS] [patch] Build pgoutput with MSVC - Mailing list pgsql-hackers

From MauMau
Subject Re: [HACKERS] [patch] Build pgoutput with MSVC
Date
Msg-id 1BD7FF5AD24442739AE96FF59D028AE5@tunaPC
Whole thread Raw
In response to Re: [HACKERS] [patch] Build pgoutput with MSVC  (Magnus Hagander <magnus@hagander.net>)
Responses Re: [HACKERS] [patch] Build pgoutput with MSVC  (Michael Paquier <michael.paquier@gmail.com>)
List pgsql-hackers
From: Magnus Hagander
If that's all that's required, I'll just go ahead and commit it right
away, including the clean.bat.

I think the problem with clean.bat isn't cleaning up pgoutput.dll -- 
that one goes in a different directory. But it does need to clean up
the win32ver.rc file that gets dropped there automaticaly.

The attached patch itself seems broken (it has some sort of byte order
marker at the beginning, but removing that still breaks with "patch
unexpectedly ends in middle of line patch: **** Only garbage was found
in the patch input.". But I can just copy/paste it manually :)


Thanks, fixed clean.bat, too.  My original patch was in UTF-16
unexpectedly.  With Git Shell in GitHub Desktop on Windows, "git diff
> filename" seems to produce output in UTF-16.  I guess that's due to
PowerShell.  I think there's nothing else to do, so please commit
this.  Also, I added an item in the Open Items page.

Regards
MauMau

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Attachment

pgsql-hackers by date:

Previous
From: Peter Geoghegan
Date:
Subject: Re: [HACKERS] modeling parallel contention (was: Parallel Append implementation)
Next
From: Bruce Momjian
Date:
Subject: Re: [HACKERS] PG 10 release notes