Skip Menu |

This queue is for tickets about the DBI CPAN distribution.

Report information
The Basics
Id: 24420
Status: resolved
Priority: 0/
Queue: DBI
People
Owner: Nobody in particular
Requestors: gozer [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 1.53
Fixed in: (no value)

History Show all quoted text
  Wed Jan 17 13:42:45 2007 gozer [...] cpan.org - Ticket created
Subject: DBI-1.53's NUM_OF_FIELDS change breaks DBD-Sybase 1.07
This change http://search.cpan.org/diff?from=DBI-1.52&to=DBI-1.53#DBI.xs about "Number of row fields inconsistent with NUM_OF_FIELDS, NUM_OF_FIELDS updated", can cause bugs in existing DBD drivers. For instance, DBD-Sybase 1.07 will break once DBI is upgraded to 1.53. The problem can be seen in fix_fbav() in http://search.cpan.org/src/MEWP/DBD-Sybase-1.07/dbdimp.c Clearly, DBD-Sybase needs to be fixed, but it's also unfortunate that this DBI change will cause an existing DBD-Sybase installation to suddently break. Is there any way this DBI change could be altered not to break existing DBD drivers ? I've also reported a DBD-Sybase bug report: http://rt.cpan.org/Public/Bug/Display.html?id=24289
  Sat Jan 20 05:42:20 2007 TIMB [...] cpan.org - Correspondence added
The message isn't an error, it's a warning. Turning off PrintWarn would silence it.
  Sat Jan 20 05:42:27 2007 The RT System itself - Status changed from 'new' to 'open'
  Sat Jan 20 05:42:43 2007 TIMB [...] cpan.org - Status changed from 'open' to 'rejected'
  Mon Jan 22 16:38:00 2007 gozer [...] cpan.org - Correspondence added
On Sat Jan 20 05:42:20 2007, TIMB wrote: Show quoted text
> The message isn't an error, it's a warning. Turning off PrintWarn > would silence it.
Yes, and that was my original suspicion when I noticed the test failures. However, the test failure is not caused by an extra warning. DBD-Sybase works around changing NUM_OF_FIELDS with it's own little piece of workaround code, and it's that code that _breaks_ against DBI-1.53.
  Mon Jan 22 16:38:01 2007 The RT System itself - Status changed from 'rejected' to 'open'
  Mon Feb 26 04:28:46 2007 TIMB [...] cpan.org - Correspondence added
Please retry with DBI 1.54.
  Mon Apr 02 18:50:25 2007 gozer [...] cpan.org - Correspondence added
From: GOZER [...] cpan.org
On Mon Feb 26 04:28:46 2007, TIMB wrote: Show quoted text
> Please retry with DBI 1.54.
Great, the problem isn't present in DBI-1.54 ;-)
  Tue Apr 03 03:54:30 2007 TIMB [...] cpan.org - Correspondence added
Thanks.
  Tue Apr 03 03:54:33 2007 TIMB [...] cpan.org - Status changed from 'open' to 'resolved'