Re: svn commit: r1099094 - /subversion/trunk/subversion/libsvn_ra_serf/commit.c

On 8/28/13 7:35 AM, Ivan Zhakov wrote:
> Bert refers to r712674 [1] -- 4 years old change in APR, so it should
> not be issue now.
>
> @Bert: Do you remember why temporary file could be not removed on file close?
>
> [1] http://svn.apache.org/viewvc?view=revision&revision=r712674

Unless we've raised our requirement of APR to not allow versions that
have this issue we should leave any workarounds we have for it in. We
end up with a lot of users running on old Linux distros that use
whatever APR is provided by the distros. Heck our own CentOS buildbot
is one of these cases, it's using APR/APR-Util 1.2.7 which was released
April 14th, 2006.

If you want to avoid the workarounds for newer APR versions that you
detect at runtime I'd support that.

I'd support us raising our APR version requirement in Subversion 1.9.x
to APR 1.3.x, not sure if that helps in this case.
Received on 2013-08-28 19:04:00 CEST