On Sun, Oct 23, 2016 at 5:48 PM, Tom Lane <[email protected]> wrote:

> Magnus Hagander <[email protected]> writes:
> > The remaining windows buildfarm build issue looks like this:
> >   src/bin/pg_basebackup/receivelog.c(135): error C2039: '_commit' : is
> > not a member of 'WalWriteMethod'
> > [C:\buildfarm\buildenv\HEAD\pgsql.build\pg_basebackup.vcxproj]
> > AFAICT this comes from using walmethod->fsync(), and we have a #define
> > changing fsync() to _commit (port/win32.h, line 70).
>
> Yeah.
>
> > The easiest might just be to rename walmethod->fsync() to
> > walmethod->do_fsync(), so we don't have to mess with any other code.
>
> How about calling the method just "sync()"?  Seems a bit less
> artificial.
>

Sure, that should work too. Seems we're not defining that one to something
else, so let's use that.

I'll go fix.


-- 
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/

Reply via email to