Front page | perl.perl5.porters |
Postings from May 2008
Re: [PATCH] Re: [perl #50180] NDBM_File fails to build
Thread Previous
From:
H.Merijn Brand
Date:
May 17, 2008 07:30
Subject:
Re: [PATCH] Re: [perl #50180] NDBM_File fails to build
Message ID:
20080517162950.4233c87a@pc09.procura.nl
On Tue, 13 May 2008 12:28:45 -0400 (EDT), Andy Dougherty
<doughera@lafayette.edu> wrote:
> On Tue, 6 May 2008, Andy Dougherty wrote:
>
> > On Wed, 23 Jan 2008, via RT wrote:
> >
> > > building perl 5.10.0 failed to build/install NDBM_File. i'm runninng
> > > Sidux (Debian Sid/Unstable). located /usr/include/gdbm-ndbm.h and had
> > > to symlink it to /usr/include/ndbm.h to allow NDBM_File to make
> > > properly.
>
> > There appear to be two changes in Debian regarding the ndbm compatibility
> > library that Configure needs to track. First, as you point out, the
> > include file has been moved to <gdbm-ndbm.h>. Second, Configure needs to
> > look in -lgdbm-compat to find the dbm_open() function. (This wasn't an
> > issue for you because the NDBM_File extension does this automatically when
> > not built as part of the perl core.)
> >
> > I'm working on a patch to fix both of these issues, but thought I'd better
> > post for feedback first:
> >
> > We now have three places to look for the "ndbm.h" include file:
> >
> > <ndbm.h> /* traditional */
> > <gdbm/ndbm.h> /* RedHat 7.1, according to the Configure comments */
> > <gdbm-ndbm.h> /* Debian 4.0 */
> >
> > Also, we have at least three different libraries to consider:
> > -lndbm
> > -lgdbm
> > -lgdbm-compat
> >
> > While I'm trying to generalize things, are there any other names or
> > locations Configure should check? (Or any corrections to the comments
> > about who keeps what where?)
>
> I have attached two patches to address this problem. First is a simple
> patch to enable NDBM_File to look in all those different locations, and an
> addition to hints/linux.sh to also look for libgdbm-compat. Second is the
> set of metaconfig patches to implement the appropriate detection.
> (I'd have included the full Configure patch, but it's 143k long, mostly
> due to harmless rearrangement of individual elements within Configure.)
>
> This should be safe for all perl branches.
Thanks, Applied in cluster as change #33845
--
H.Merijn Brand Amsterdam Perl Mongers (http://amsterdam.pm.org/)
using & porting perl 5.6.2, 5.8.x, 5.10.x on HP-UX 10.20, 11.00, 11.11,
& 11.23, SuSE 10.1 & 10.2, AIX 5.2, and Cygwin. http://qa.perl.org
http://mirrors.develooper.com/hpux/ http://www.test-smoke.org
http://www.goldmark.org/jeff/stupid-disclaimers/
Thread Previous