Bug 175898 (perl-Text-CSV_XS) - Review Request: perl-Text-CSV_XS - Comma-separated values manipulation routines
Summary: Review Request: perl-Text-CSV_XS - Comma-separated values manipulation routines
Alias: perl-Text-CSV_XS
Product: Fedora
Classification: Fedora
Component: Package Review   
(Show other bugs)
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Paul Howarth
QA Contact: David Lawrence
Keywords: Reopened
Depends On:
Blocks: 175899
TreeView+ depends on / blocked
Reported: 2005-12-16 00:19 UTC by Jose Pedro Oliveira
Modified: 2008-06-03 15:44 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-06-03 15:44:25 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
tibbs: fedora-review+
kevin: fedora-cvs+

Attachments (Terms of Use)

Description Jose Pedro Oliveira 2005-12-16 00:19:22 UTC
Spec Name or Url:

SRPM Name or Url:

Text::CSV provides facilities for the composition and decomposition of
comma-separated values.  An instance of the Text::CSV class can combine
fields into a CSV string and parse a CSV string into fields.

Comment 1 Paul Howarth 2006-01-05 16:16:01 UTC

- rpmlint clean
- package and spec naming OK
- package meets guidelines
- license is same as perl, matches spec
- spec file written in English and is legible
- sources match upstream
- package builds OK on FC4 (i386) and in mock for rawhide (i386)
- no explicit BR's needed nor present
- no locales, libraries, pkgconfigs, or subpackages to worry about
- not relocatable
- no directory ownership or permissions issues
- no duplicate files
- %clean section present and correct
- macro usage is consistent
- code, not content
- no large docs
- docs don't affect runtime
- no desktop entry needed
- no scriptlets


- The compiler flags are specified twice:

    CFLAGS="$RPM_OPT_FLAGS" %{__perl} Makefile.PL INSTALLDIRS=vendor
    make %{?_smp_mflags} OPTIMIZE="$RPM_OPT_FLAGS"

  It'd be slightly neater to do it just once:

    %{__perl} Makefile.PL INSTALLDIRS=vendor OPTIMIZE="$RPM_OPT_FLAGS"
    make %{?_smp_mflags}

  Just a cosmetic thing really though.


Comment 2 Jose Pedro Oliveira 2006-01-06 12:40:59 UTC

* RPM_OPT_FLAGS: specfile updated (and my specfile generator script).
* Imported and build for FC-4 and devel.


Comment 3 Marcela Mašláňová 2008-05-22 07:47:47 UTC
Package Change Request
Package Name: perl-Text-CSV_XS
New Branches: EL-4 EL-5

Comment 4 Xavier Bachelot 2008-05-22 16:43:58 UTC
Package Change Request
Package Name: perl-Text-CSV_XS
New Branches: EL-4 EL-5
Updated EPEL Owners: kasal xavierb
Updated EPEL CC: perl-sig

Comment 5 Kevin Fenzi 2008-05-22 18:24:58 UTC
Can you guys coordinate here? Marcela and Štěpán are listed as co-owners of the
fedora package. Do all 3 of you want to maintain for EPEL?

Comment 6 Xavier Bachelot 2008-05-23 09:38:49 UTC
Not sure what Stepan and Marcela really want. I and Chris Weyl separately asked
to have this package for EPEL (RHBZ 441646 and 237025). Since the bug I opened
wasn't moving after some time, I offered to do the build myself or even
co-maintain. Marcela answered me to either take ownership or comaintain. I'd
rather co-maintain. As the EPEL owners weren't specified in the first branch
request, I updated it with the primary owner and myself. I don't know if it was
the proper way to deal with the issue, nor do I know exactly what Stepan and
Marcela want to do.

I added everyone involved to the CC: so everyone can comment.

Comment 7 Jason Tibbitts 2008-05-30 15:47:55 UTC
Setting fedora-review flag properly so this doesn't show up in the
"to-be-reviewed" list.

Comment 8 Xavier Bachelot 2008-05-30 16:04:01 UTC
Stepan, Marcela, could you please comment ? Chris, if you wish too...

Comment 9 Kevin Fenzi 2008-06-03 14:22:04 UTC
Sorry for the delay here... 

I just copied over all the maintainers/co-maintainers/cc's to the epel branches
and added Xavier as a co-maintainer of them. 

Hope that works for everyone. 

Comment 10 Xavier Bachelot 2008-06-03 15:44:25 UTC
Built for EL-4 and EL-5.

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