> -----Original Message-----
> From: Julian Foad [mailto:julian.f...@wandisco.com]
> Sent: maandag 28 juni 2010 10:48
> To: Paul Smedley
> Cc: dev@subversion.apache.org
> Subject: Re: [Patch] - Fix compilation of direct_uri-test.c on OS/2
> 
> On Sun, 2010-06-27, Paul Smedley wrote:
> > [[[
> > Fix compilation of direct_uri-test.c on OS/2
> >
> > * subversion/tests/libsvn_subr/dirent_uri-test.c : add define of getdcwd
> > as _getdcwd for __OS2__ as well as _MSC_VER
> > ]]]
> 
> Hi Paul.  To see if there were other similar places, I searched for
> _MSC_VER and found two more.  Can you confirm whether the following is a
> good patch?

_MSC_VER is a check for the Microsoft compiler (and it's compiler specific 
include files; we check for Windows in a different way), __OS2__ checks for a 
platform. 

Maybe we should just try to move this to platform specific checks. 

The only problem is that I have no way to confirm if this function is available 
under the less common cygwin/mingw environments. Borland C++ is most likely 
compatible with the Microsoft compilers.

        Bert

Reply via email to