Re: Changeset 2050b44407bf and qdbm
- To: Mutt-Devel <mutt-dev@xxxxxxxx>
- Subject: Re: Changeset 2050b44407bf and qdbm
- From: Brendan Cully <brendan@xxxxxxxxxx>
- Date: Wed, 1 Oct 2008 09:06:40 -0700
- Dkim-signature: v=1; a=rsa-sha1; c=relaxed; d=kublai.com; h=date:to: subject:message-id:references:mime-version:content-type: content-transfer-encoding:in-reply-to:from; q=dns/txt; s=dk; bh= B2X8ddL/umq2C+vZ2fKaNHTxCPw=; b=W1eWJePZqsbMg/Ow0vshP/0Q9/QWiINp 8lFNfx738fwKIngY3Y7Awssg1VhOZ47EpePbZEkOMQoc3TuHDaRXBmbks8oCX7Xk B7zXok/XHsWCwAbB0hmQ7alur+bK66uDnvpMLlze45+mzaO8ldTTyUccLUb1FL21 Tv514MN1QZU=
- In-reply-to: <20080929224750.GJ41326@xxxxxxxxxxxxx>
- List-post: <mailto:mutt-dev@mutt.org>
- List-unsubscribe: send mail to majordomo@mutt.org, body only "unsubscribe mutt-dev"
- Mail-followup-to: mutt-dev@xxxxxxxx
- References: <20080929224750.GJ41326@xxxxxxxxxxxxx>
- Sender: owner-mutt-dev@xxxxxxxx
- User-agent: Mutt/1.5.18 (2008-09-01)
On Monday, 29 September 2008 at 17:47, Kyle Wheeler wrote:
> After changeset 2050b44407bf, I’m unable to compile with qdbm. I see that
> configure.in is testing for $use_qdbm, but never sets it. Equivalent logic
> is used in the tokyocabinet stuff, but I don’t see how that could possibly
> work *either*, since $use_tokyocabinet is never set.
Ah. It looks like this bug is tripped only if you explicitly set
--use-qdbm or --use-tokyocabinet. If you leave out the options and let
mutt select the first one it finds, everything's fine. I'll push a
modified version of your patch.
> A small patch is attached that corrects that and lets it build properly.
>
> ~Kyle
> diff -r 9e569296190b configure.ac
> +++ b/configure.ac Mon Sep 29 17:47:29 2008 -0500
> @@ -851,7 +851,7 @@
> db_found=yes],
> [CPPFLAGS="$OLDCPPFLAGS"
> LDFLAGS="$OLDLDFLAGS"]))
> - if test -n "$with_tokyocabinet" && test "$use_tokyocabinet" != yes
> + if test -n "$with_tokyocabinet" && test "$db_found" != yes
> then
> AC_MSG_ERROR([Tokyo Cabinet could not be used. Check config.log for
> details.])
> fi
> @@ -883,7 +883,7 @@
> [CPPFLAGS="$OLDCPPFLAGS"
> LDFLAGS="$OLDLDFLAGS"])
> LIBS="$saved_LIBS"
> - if test -n "$with_qdbm" && test "$use_qdbm" != yes
> + if test -n "$with_qdbm" && test "$db_found" != yes
> then
> AC_MSG_ERROR([QDBM could not be used. Check config.log for details.])
> fi