Bug 441830 (perl-DBD-ODBC) - Review Request: perl-DBD-ODBC - ODBC Driver for DBI
Summary: Review Request: perl-DBD-ODBC - ODBC Driver for DBI
Keywords:
Status: CLOSED NOTABUG
Alias: perl-DBD-ODBC
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Chris Weyl
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FE-DEADREVIEW
TreeView+ depends on / blocked
 
Reported: 2008-04-10 13:38 UTC by Xavier Bachelot
Modified: 2008-07-01 03:04 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-07-01 03:04:25 UTC
Type: ---
Embargoed:
cweyl: fedora-review-


Attachments (Terms of Use)

Description Xavier Bachelot 2008-04-10 13:38:58 UTC
Spec URL: http://washington.kelkoo.net/fedora/SPECS/perl-DBD-ODBC.spec
SRPM URL: http://washington.kelkoo.net/fedora/SRPMS/perl-DBD-ODBC-1.15-1.fc8.src.rpm
Description: 
Perl ODBC Driver for DBI. This module is needed to access ODBC databases from within Perl. The module uses the unixODBC manager to connect to the database.

Comment 1 Peter Lemenkov 2008-04-12 09:14:39 UTC
Typo in changelog - s/Xvaier/Xavier/g

Comment 2 Xavier Bachelot 2008-04-12 11:18:46 UTC
(In reply to comment #1)
> Typo in changelog - s/Xvaier/Xavier/g

Thx Peter, will fix during review.

Comment 3 Chris Weyl 2008-04-13 06:47:19 UTC
scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=563988

Almost all tests are skipped.  I'm not sure what can currently be done to enable
them, aside from also packaging SQLiteODBC[1] and rigging a little test
environment... I'd encourage this; we certainly already require (as a matter of
course) the packaging of any additional cpan/perl deps required to fully enable
a test suite.

[1] http://www.ch-werner.de/sqliteodbc/html/

Comment 4 Xavier Bachelot 2008-04-13 17:13:35 UTC
(In reply to comment #3)
> scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=563988
> 
> Almost all tests are skipped.  I'm not sure what can currently be done to enable
> them, aside from also packaging SQLiteODBC[1] and rigging a little test
> environment... I'd encourage this; we certainly already require (as a matter of
> course) the packaging of any additional cpan/perl deps required to fully enable
> a test suite.
> 
> [1] http://www.ch-werner.de/sqliteodbc/html/

Thanks for the suggestion Chris. I've packaged sqliteodbc and will submit it for
review once perl-DBD-ODBC is properly modified to use it.

http://washington.kelkoo.net/fedora/SPECS/sqliteodbc.spec
http://washington.kelkoo.net/fedora/SRPMS/sqliteodbc-0.77-1.fc8.src.rpm

Comment 5 Xavier Bachelot 2008-04-14 20:31:42 UTC
Chris, I'm not sure how to build an sqlite test env. I've looked at it a bit,
but I'm not sure how to proceed. The flu that is hammering me down for 3 days
probably doesn't help. Any hint or suggestion ?

Comment 6 Chris Weyl 2008-04-14 21:40:24 UTC
Heh... I'm dealing with a head-cold myself.  Ugh. :)

Let me take a more in depth look at DBI-ODBC and see what we need to do in order
to create a small testing environment...

Comment 7 Chris Weyl 2008-04-20 16:01:29 UTC
Hm.  If you set up dummy odbcinst.ini and odbc.ini's along the lines the sqlite
homepage suggests, you can cause the tests to use it by running them along the
lines of:

 ODBCSYSINI=. ODBCINI=odbc.ini DBI_DSN=dbi:ODBC:mysqlitedb make test

where odbcinst.ini is:
[SQLite]
Description=SQLite ODBC Driver
Driver=/usr/lib64/libsqlite3odbc-0.77.so
Setup=/usr/lib64/libsqlite3odbc-0.77.so
Threading=2

And odbc.ini is:
[mysqlitedb]
Description=My SQLite test database
Driver=SQLite
Database=/tmp/mytest-foo.db
# optional lock timeout in milliseconds
Timeout=2000

Unfortunately, most of the tests fail:
Failed Test   Stat Wstat Total Fail  Failed  List of Failed
-------------------------------------------------------------------------------
t/02simple.t   255 65280    62   98 158.06%  14-62
t/03dbatt.t      7  1792    24    7  29.17%  5 7 15-18 23
t/05meth.t     255 65280    13   17 130.77%  3 6-13
t/07bind.t     255 65280    11   20 181.82%  2-11
t/08bind2.t      3   768     5    3  60.00%  3-5
t/10handler.t    1   256    10    1  10.00%  10
156 subtests skipped.
Failed 6/12 test scripts, 50.00% okay. 79/295 subtests failed, 73.22% okay.

Fixing the first failed test in 02simple.t cuts the number of failed tests
significantly:

Failed Test   Stat Wstat Total Fail  Failed  List of Failed
-------------------------------------------------------------------------------
t/02simple.t     7  1792    62    7  11.29%  34 36-39 43 53
t/03dbatt.t      1   256    24    1   4.17%  16
t/10handler.t    1   256    10    1  10.00%  10
156 subtests skipped.
Failed 3/12 test scripts, 75.00% okay. 9/295 subtests failed, 96.95% okay.

Looking at the test suite, it looks like the test suite can't handle sqlite for
some reason.  It might be a good idea to file a bug upstream asking about the
test suite and using an sqlite backend...

Comment 8 Jason Tibbitts 2008-06-18 20:27:38 UTC
I'm just fixing up the flags here, but I note that it's been two months without
any response from Xavier.

Comment 9 Xavier Bachelot 2008-06-23 15:48:35 UTC
yup, I'm sorry about my unresponsiveness, but I was not prepared for this amount
of work and got dragged into other things anyway. I've not lost interest in this
package and will get back to it as time permit. I've just noticed there is a
1.16 version out, I'll give it a try. Again, sorry for the delay.

Comment 10 Chris Weyl 2008-07-01 03:04:25 UTC
Marking as deadreview :(


Note You need to log in before you can comment on or make changes to this bug.