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

Re: [PATCH] Make configure report the components detected

From: Kamesh Jayachandran <kamesh_at_collab.net>
Date: 2006-11-16 13:15:23 CET

What happened to this patch?

With regards
Kamesh Jayachandran
Kamesh Jayachandran wrote:
> Erik Huelsmann wrote:
>> On 10/9/06, Kamesh Jayachandran <kamesh@collab.net> wrote:
>>> I like this change.
>>> One small suggestion, Can you attach the patch instead of inline text?
>>>
>>> Erik Huelsmann wrote:
>>> > Our current configure script warns at the end if BDB hasn't been
>>> > found. Given that it's now just one of the components that can be
>>> used
>>> > Index: configure.in
>>> > ===================================================================
>>> > --- configure.in (revision 21833)
>>> > +++ configure.in (working copy)
>>> >
>>> > -You don't seem to have Berkeley DB version $db_version or newer
>>> > -installed and linked to APR-UTIL. We have created Makefiles which
>>> > -will build without the Berkeley DB back-end; your repositories will
>>> > -use FSFS as the default back-end. You can find the latest
>>> version of
>>> > -Berkeley DB here:
>>> > - http://www.sleepycat.com/download/index.shtml
>>> > -])
>>> > -;;
>>> > + AC_MSG_NOTICE([ BerkeleyDB no])
>>> > + ;;
>>> > + "yes" )
>>> > + AC_MSG_NOTICE([ BerkeleyDB $db_version])
>>> > + ;;
>>> > esac
>>> > -
>>> > +case "!$APXS!" in
>>>
>>> > + "!no!"|"!!" )
>>> Why we have cruft of "!" here?
>>
>> Because I didn't know how to test for the empty string, but I think
>> that could just be "", right?
>>
>> bye,
>>
>> Erik.
>>
> case "$APXS" in
> "no"|"")
> echo "no"
> ;;
> * )
> echo "default"
> ;;
> esac
>
> The above snippet works fine in my bash. If testing the empty string
> is really a problem, should we not do the same for others too like
> "$SWIG"?
>
> With regards
> Kamesh Jayachandran
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Nov 16 13:15:54 2006

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.