All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Garzik <jeff@garzik.org>
To: Pete Zaitcev <zaitcev@redhat.com>
Cc: Project Hail List <hail-devel@vger.kernel.org>
Subject: Re: [Patch 1/1] CLD: fix crash in __mutex_get_max (libdb-4.7.so) on F13
Date: Sun, 29 Nov 2009 20:38:45 -0500	[thread overview]
Message-ID: <4B132225.6020505@garzik.org> (raw)
In-Reply-To: <20091129181744.578f4b20@redhat.com>

On 11/29/2009 08:17 PM, Pete Zaitcev wrote:
> Fedora 13 comes with db4.8 and apparently the compat-db4.7 is bust.
> Let us link with 4.8 as a workaround.
>
> Signed-Off-By: Pete Zaitcev<zaitcev@redhat.com>
>
> ---
>   configure.ac |    5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
>
> Not sure how safe or desirable this is, but it seems to work.
>
> commit 77e89dfc8f0a9296ebfb02d5ca34d48f083df50c
> Author: Master<zaitcev@hitlain.zaitcev.lan>
> Date:   Sun Nov 29 18:12:40 2009 -0700
>
>      Fix crash in __mutex_get_max because db4.7 is toast on F13
>      by letting to link against db4.8.

Interesting...  I recall the root cause clearly, now:  /usr/include/db.h 
always refers to the latest installed db4, even if compat-db{,45,46} is 
installed.  Our configure recipe links with the most recent db4 listed 
in configure.ac, combined with the installed /usr/include/db.h.  Thus, 
each new db4 version produces a mismatch.

If we could include /usr/include/db4.6.21/db.h when db-4.6 is present, 
that would be ideal.  But given that db4 lacks anything approaching 
pkgconfig, autodetection becomes quite difficult.

While I would like to see configure.ac have a better test (surely Google 
can find an autoconf recipe for db4?), I think your proposed patch will 
be the best method we can find.

	Jeff


  reply	other threads:[~2009-11-30  1:38 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-30  1:17 [Patch 1/1] CLD: fix crash in __mutex_get_max (libdb-4.7.so) on F13 Pete Zaitcev
2009-11-30  1:38 ` Jeff Garzik [this message]
2009-11-30  2:05   ` Pete Zaitcev
2009-12-02  1:18   ` Pete Zaitcev
2009-12-02 10:40     ` Jeff Garzik
2009-11-30 13:34 ` Jeff Garzik

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4B132225.6020505@garzik.org \
    --to=jeff@garzik.org \
    --cc=hail-devel@vger.kernel.org \
    --cc=zaitcev@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.