> -----Original Message----- > From: Sergei Antonov [mailto:sap...@gmail.com] > Sent: donderdag 14 maart 2013 14:13 > To: Bert Huijben > Subject: Re: Windows build still requires apr-util? > > On 14 March 2013 13:44, Bert Huijben <b...@qqmail.nl> wrote: > > > > > >> -----Original Message----- > >> From: Sergei Antonov [mailto:sap...@gmail.com] > >> Sent: donderdag 14 maart 2013 13:31 > >> To: Bert Huijben > >> Subject: Re: Windows build still requires apr-util? > >> > >> On 13 March 2013 19:15, Bert Huijben <b...@qqmail.nl> wrote: > >> > > >> > > >> >> -----Original Message----- > >> >> From: Sergei Antonov [mailto:sap...@gmail.com] > >> >> Sent: woensdag 13 maart 2013 18:47 > >> >> To: dev@subversion.apache.org > >> >> Subject: Re: Windows build still requires apr-util? > >> >> > >> >> Gentle reminder: VS 2010 doesn't compile the latest revision. > >> >> Please, add "#include <time.h>" into > >> >> subversion\libsvn_subr\win32_crashrpt.c and commit. > >> > > >> > I think that is somehow dependant on other settings as it just compiles > for > >> me with 2010 (and 2012 and 2008). > >> > >> Do not compile this file at all? > >> > >> > Can you explain what makes your environment different > >> > >> You do not have SVN_USE_WIN32_CRASHHANDLER? > >> You use different revision? (mine is 1456399) > >> > >> Open VS 2010 .sln, navigate to 'win32_crashrpt.c' in project > >> 'libsvn_subr', choose Build->Compile. What do you see? I see this: > > > > I use the crash handler. (Standard settings) and I usually build Subversion > trunk multiple times per day with different Visual Studio versions. The file > hasn't changed in this respect for years. > > > > time.h is included by apr.h for me. > > My apr.h is attached. Can you compare it with yours? > > > I don't see why we would have to duplicate the include here in my > environment. > > Is time.h from apr.h guaranteed?
Yes, in apr 0.9.X and 1.x it is always included on Windows, see apr.hw from the apr project. Their compatibility rules are the same as ours. We don't declare apr 2.X support in Subversion, so I would guess that is untested. But I would guess it is still included there. (And unless it was an actual decision to remove it, I would call it a bug if it is no longer included) But APR is supported on another mailinglist, than this one. Bert