Skip Menu |

This queue is for tickets about the CDB_File CPAN distribution.

Report information
The Basics
Id: 23997
Status: resolved
Priority: 0/
Queue: CDB_File
People
Owner: Nobody in particular
Requestors: ANDK [...] cpan.org
Cc: SREZIC [...] cpan.org
AdminCc:
Bug Information
Severity: Important
Broken in: (no value)
Fixed in: 0.96

History Show all quoted text
  Tue Dec 19 05:03:02 2006 ANDK [...] cpan.org - Ticket created
CC: srezic [...] cpan.org
Subject: bleadperl failing compilation
Patch 24817 to bleadperl made sv_upgrade a void function. Since then gcc does not compile CDB_File. I made a patch available on CPAN as ANDK/patches/CDB_File-0.95-ANDK-01.patch.gz that makes it compile with bleadperl@2481[67]. As it is short, I paste it here as well: --- CDB_File-0.95-4Bn4X7/CDB_File.xs~ 2006-07-28 04:34:14.000000000 +0200 +++ CDB_File-0.95-4Bn4X7/CDB_File.xs 2006-12-19 10:42:14.000000000 +0100 @@ -475,7 +475,8 @@ cdb_FETCH(this, k) if ((found != 0) && (found != 1)) readerror(); } ST(0) = sv_newmortal(); - if (found && sv_upgrade(ST(0), SVt_PV)) { + if (found) { + sv_upgrade(ST(0), SVt_PV); U32 dlen = cdb_datalen(this); (void)SvPOK_only(ST(0)); Thanks!
  Sun Feb 03 06:09:16 2008 tokuhirom+cpan [...] gmail.com - Correspondence added
Subject: perl5.10 failing compilation
Perl5.10 is released at last year. Please support Perl 5.10. I want to use CDB_File with Perl5.10. On Tue Dec 19 05:03:02 2006, ANDK wrote: Show quoted text
> Patch 24817 to bleadperl made sv_upgrade a void function. Since then gcc > does not compile CDB_File. > > I made a patch available on CPAN as > > ANDK/patches/CDB_File-0.95-ANDK-01.patch.gz > > that makes it compile with bleadperl@2481[67]. As it is short, I paste > it here as well: > > --- CDB_File-0.95-4Bn4X7/CDB_File.xs~ 2006-07-28 04:34:14.000000000
+0200 Show quoted text
> +++ CDB_File-0.95-4Bn4X7/CDB_File.xs 2006-12-19 10:42:14.000000000
+0100 Show quoted text
> @@ -475,7 +475,8 @@ cdb_FETCH(this, k) > if ((found != 0) && (found != 1)) readerror(); > } > ST(0) = sv_newmortal(); > - if (found && sv_upgrade(ST(0), SVt_PV)) { > + if (found) { > + sv_upgrade(ST(0), SVt_PV); > U32 dlen = cdb_datalen(this); > > (void)SvPOK_only(ST(0)); > > > > Thanks!
  Sun Feb 03 06:09:26 2008 The RT System itself - Status changed from 'new' to 'open'
  Tue Feb 19 03:56:34 2008 leon [...] astray.com - Correspondence added
From: leon [...] astray.com
On Sun Feb 03 06:09:16 2008, TOKUHIROM wrote: Show quoted text
> Perl5.10 is released at last year.
Ah, I just made this patch too. Bump! Leon
  Wed Apr 08 21:18:13 2009 tokuhirom+cpan [...] gmail.com - Correspondence added
note. fixed in 0.96
  Sat Feb 13 01:39:09 2010 ANDK [...] cpan.org - Status changed from 'open' to 'resolved'
  Sat Feb 13 01:39:09 2010 ANDK [...] cpan.org - Fixed in 0.96 added