On 28/06/2010 10:29, Bert Huijben wrote:


-----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


_getdcwd does not exist on the cygwin platform (which is a Unix platform), but _getdcwd and _wgetdcwd are available under mingw32 (which is a Windows platform) through direct.h.

Matthew

Reply via email to