On 28/06/2010 10:29, Bert Huijben wrote:
>
>
>> -----Original Message-----
>> From: Julian Foad [mailto:julian.foad_at_wandisco.com]
>> Sent: maandag 28 juni 2010 10:48
>> To: Paul Smedley
>> Cc: dev_at_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
Received on 2010-06-28 11:51:20 CEST