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

Re: Subversion1.10.0-rc1 up for testing/signing

From: Branko Čibej <brane_at_apache.org>
Date: Sat, 24 Feb 2018 21:57:28 +0100

On 24.02.2018 21:55, Julian Foad wrote:
> Branko Čibej wrote:
>>>    - Warning from JavaHL build:
>>>
>>>      subversion/bindings/javahl/native/SVNRepos.cpp:371:15: warning:
>>> 'svn_repos_load_fs5'
>>>            is deprecated [-Wdeprecated-declarations]
>>>        SVN_JNI_ERR(svn_repos_load_fs5(repos,
>>> dataIn.getStream(requestPool),
>>
>> I'm assuming we can fix this during the soak period, even if it means
>> adding another overload to the JavaHL Repos API, yes? Fixing bindings
>> shouldn't extend the soak (or only by a week)?
>>
>> In other words, it doesn't make sense to block RC1 over this detail.
>
> We should not require this to be fixed for 1.10.0.
>
> We have a policy of not calling deprecated APIs from within our own
> code, but that is basically for our own convenience -- it does not
> matter to the users of our code. We have a requirement that deprecated
> APIs are safe and correct to use. Therefore it is fine to release code
> like this.
>
> It can be fixed at leisure, any time.

The fix is already done and is approved for backport to 1.10.x (so it
gets merged tonight). It adds another API overload to JavaHL, so it
really needs to be in .0.

-- Brane
Received on 2018-02-24 21:57:34 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.