[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1874057 - in /subversion/trunk/subversion: libsvn_subr/cmdline.c tests/cmdline/update_tests.py

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Sun, 16 Feb 2020 15:22:30 +0000

James McCoy wrote on Sun, 16 Feb 2020 10:06 -0500:
> However, the svn-x64-macosx-apr1.3-nothread buildbot informed me that
> apr_pescape_shell() isn't available in that version. It wasn't added
> until 1.5.0.
> Now we need to either:
> a) Raise the minimum APR version

apr 1.5.0 was tagged on 2013-11-13, so this should be fine, for 1.14 at least.

> b) Conditionally use the new code when built against an acceptable APR
> version and
> i) maintain the status quo for anyone using latest svn with an 8 year
> old version of APR or

That would make svn's behaviour depend on the build-time version of
libapr. I don't think that's a good idea.

> ii) add a minimal version of APR's escaping code that can be used
> when built again a pre-1.5.0 APR.

That would work too, and may be the preferred solution for 1.10 if we intend
to backport the fix there.


Received on 2020-02-16 16:22:45 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.