> -----Original Message-----
> From: Eric Zeitler [mailto:eric.zeitler_at_gmail.com]
> Sent: vrijdag 23 juli 2010 1:48
> To: Bert Huijben
> Cc: dev_at_subversion.apache.org
> Subject: Re: [Patch] - Fix compilation of direct_uri-test.c on OS/2
>
> On 7/22/2010 4:02 PM, Bert Huijben wrote:
> >
> >> -----Original Message-----
> >> From: Eric Zeitler [mailto:eric.zeitler_at_gmail.com]
> >> Sent: donderdag 22 juli 2010 23:50
> >> To: dev_at_subversion.apache.org
> >> Subject: Re: [Patch] - Fix compilation of direct_uri-test.c on OS/2
> >>
> >> Matthew Bentham <mjb67 <at> artvps.com> writes:
> >>
> >>> 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
> >>>
> >>>
> >> I have created a patch to fix this very issue for Cygwin.
> > This patch disables all C:/ like path support in Subversion if you
> compile
> > it for Cygwin. I would guess that is not what you intended?
> >
> > We probably need a specific patch to target that function instead of
> just
> > switching all path handling to unix-like.
> >
> > Bert
> >
> Cygwin, by default, gives a big scary warning if you use DOS paths so I
> figure anyone who really needs it can use cypath. I don't think anyone
> was depending on this as it didn't even compile before.
This specific define you are patching was added on trunk a few weeks ago,
until then the specific testcase was only enabled for Microsoft compilers.
I think it is hard to tell the real number of users of a specific
distribution from the number of users compiling current trunk themselves.
(Especially with all the warnings around the current working copy state)
Bert
Received on 2010-07-23 02:42:33 CEST