Fedora Account System
Red Hat Associate
Red Hat Customer
Spec Name or Url: http://gsd.di.uminho.pt/jpo/software/fedora/perl-DBD-XBase.spec SRPM Name or Url: http://gsd.di.uminho.pt/jpo/software/fedora/perl-DBD-XBase-0.241-1.src.rpm Description: This module can read and write XBase database files, known as dbf in dBase and FoxPro world. It also transparently reads memo fields from the dbt, fpt and smt files and works with index files (ndx, ntx, mdx, idx, cdx and SDBM). This module XBase.pm provides simple native interface to XBase files. For DBI compliant database access, see DBD::XBase and DBI modules and their man pages. Note: This is a perl-SQL-Statement BR
Basically the package seems OK to me, but I am seeing 2 minor issues with it: 1. The package contains /usr/bin/indexdump. This name ("indexdump") seems too general/generic to me for this specialized tools to have it in /usr/bin. May-be one should consider to split the applications out of the perl-module package into a separate application package. I am not sure. 2. Something it broken with the Makefile. The ./bin/indexdump being generated inside when running "make" contains two "#!/usr/bin/perl -w" shebangs instead of one. Anyway, APPROVED
(In reply to comment #1) > Basically the package seems OK to me, but I am seeing 2 minor issues with it: > > 1. The package contains /usr/bin/indexdump. > This name ("indexdump") seems too general/generic to me for this specialized > tools to have it in /usr/bin. May-be one should consider to split the > applications out of the perl-module package into a separate application package. > I am not sure. Maybe renaming it to dbfindexdump in case of conflict? The patch would have to modified the Makefile.PL, some parts of the documentation, and at least a test file. > 2. Something it broken with the Makefile. > The ./bin/indexdump being generated inside when running "make" contains > two "#!/usr/bin/perl -w" shebangs instead of one. Corrected. Patched added to CVS (devel branch - tagged and built as release 2). > Anyway, APPROVED Thanks. PS - Still waiting for the FC-4 CVS branch creation.
FC-4 build completed.
*** Bug 177272 has been marked as a duplicate of this bug. ***