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

Re: svn commit: r1829344 - in /subversion/trunk/build/ac-macros: lz4.m4 utf8proc.m4

From: Branko ─îibej <brane_at_apache.org>
Date: Tue, 17 Apr 2018 15:07:35 +0200

On 17.04.2018 13:34, Johan Corveleyn wrote:
> On Tue, Apr 17, 2018 at 1:15 PM, Philip Martin <philip_at_codematters.co.uk> wrote:
>> Johan Corveleyn <jcorvel_at_gmail.com> writes:
>>
>>> Maybe we could add an extra hint to the error message along the lines
>>> of "Maybe you want --with-lz4=internal"? To help self-serviceness and
>>> avoid mails to the users list ...
>> I don't think that is a good recommendation. Internal libraries should
>> only be used if the OS does not provide the library.
> OK.
>
> I just thought a common scenario of users running into this would be:
> - Try to build Subversion.
> - Fail because lz4 library is missing.
> - "Hmm, haven't seen this before, let's try if we can build without
> it, let's try --without-lz4".
> - Error ... Ah, it's required, damn. Now what? Do I have to go looking
> for yet another dependency of SVN?
> - Lose time, and perhaps give up.
>
> But if it's not recommended, then okay.
>
> Or would the following be useful and acceptable?
> "Subversion requires LZ4. It is recommended you use the most
> up-to-date version of the LZ4 library provided by your OS. If your OS
> does not provide one, you can use --with-lz4=internal to use the
> library included in the SVN release package."
>
> A bit long perhaps...
>
> It's just a thought ... not terribly important.

build/ac-macros/lz4.m4 suggests --with-lz4=internal in the version check
error message.

-- Brane
Received on 2018-04-17 15:07:39 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.