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

Re: [PATCH] Add LZ4 version info to svn --version -v

From: Johan Corveleyn <jcorvel_at_gmail.com>
Date: Sun, 13 Aug 2017 14:22:20 +0200

On Sun, Aug 13, 2017 at 6:53 AM, Ivan Zhakov <ivan_at_visualsvn.com> wrote:
> On 11 August 2017 at 23:28, Ivan Zhakov <ivan_at_visualsvn.com> wrote:
>> Log message:
>> [[[
>> Add LZ4 library verison info to svn --version -v.
>>
>> * subversion/libsvn_subr/sysinfo.c
>> (): Include lz4/lz4internal.h or lz4.h depending on SVN_INTERNAL_LZ4.
>> (svn_sysinfo__linked_libs): Add LZ4 library version info.
>> ]]]
>>
> V2 version of the patch:
> - display 'static' for statically linked LZ4
> - do not use LZ4_VERSION_STRING and LZ4_versionString() API, because
> they are not available in older version of LZ4 library.
>
>
>
> --
> Ivan Zhakov

Hi Ivan!

Looks good to me. Go ahead and commit I'd say :-).

Thanks,

-- 
Johan
Received on 2017-08-13 14:22:45 CEST

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.