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.

Cheers,

Daniel

Reply via email to