Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Summary: Review Request: perl-Data-Report - A flexible plugin-driven reporting framework Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: jvromans@squirrel.nl QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://www.squirrel.nl/pub/xfer/perl-Data-Report.spec SRPM URL: http://www.squirrel.nl/pub/xfer/perl-Data-Report-0.10-1.fc11.src.rpm Description: Data::Report is a framework for report generation.
You define the columns, add the data row by row, and get reports in text, HTML, CSV and so on. Textual ornaments like extra empty lines, dashed lines, and cell lines can be added in a way similar to HTML style sheets.
Last koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1094325
This is one of my first packages, so I'm looking for a sponsor. The spec and srpm are rpmlint free of warnings and errors.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Johan Vromans jvromans@squirrel.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |177841
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Parag AN(पराग) panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |panemade@gmail.com
--- Comment #1 from Parag AN(पराग) panemade@gmail.com 2009-01-31 01:29:10 EDT --- 1)Add following to spec BuildRequires: perl(HTML::Entities) Requires: perl(HTML::Entities)
2) %doc is empty it should be %doc Changes README
3) Not needed to use full path to find command.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Ralf Corsepius rc040203@freenet.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |rc040203@freenet.de
--- Comment #2 from Ralf Corsepius rc040203@freenet.de 2009-01-31 01:51:38 EDT --- I am inclined to think, we first need to have Text::CSV in Fedora, before this package can be added.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #3 from Johan Vromans jvromans@squirrel.nl 2009-01-31 05:37:33 EDT --- @Parag: Thanks for your feedback. 1) HTML::Entities is an optional module and not necessary for building and correct functioning of this package. Do you still need it should be added to the requirements? 2) Fixed. 3) Fixed.
@Ralf: I do not understand. The package can work with either Text::CSV or Text::CSV_XS. The current version mentions Text::CSV in its requirements, and I changed that to Text::CSV_XS mainly because Text::CVS_XS is part of Fedora already.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #4 from Ralf Corsepius rc040203@freenet.de 2009-01-31 06:43:22 EDT --- In my understanding, Text::CSV is an abstraction layer, which may use Text::CVS_XS underneath.
That said, your patch to me is a hack to circumvent Text::CSV.
=> The proper way to package would be to first package Text::CSV, and then Data::Report
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #5 from Johan Vromans jvromans@squirrel.nl 2009-01-31 08:04:39 EDT --- Thanks for your clarification. Not so long ago, Text::CSV and Text::CSV_XS were incompatible implementations with differing APIs. That is why many many tools require Text::CSV_XS directly. And I assume that is also the reason why Text::CSV_XS has been part of Fedora for a long time, while Text::CSV still isn't. Only recently (2007/2008) development of Text::CSV and Text::CSV_XS was coordinated. So I do not share your opinion on my patch being a hack. However, I have contacted the Text::CSV maintainer to ask permission for packaging.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Johan Vromans jvromans@squirrel.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends on| |483406
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Hans de Goede hdegoede@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |hdegoede@redhat.com AssignedTo|nobody@fedoraproject.org |hdegoede@redhat.com Flag| |fedora-review?
--- Comment #6 from Hans de Goede hdegoede@redhat.com 2009-04-03 07:06:34 EDT --- Hi Johan,
As discussed by mail already, I'll review your current 4 package submissions, and once they are all approved, I'll sponsor you.
I've done a full review of this package all in all it looks good, except for a few small things (see below).
MUST FIX -------- * Drop the patch and (Build)Requires perl(Text::CSV) instead of Text::CSV_XS I know you are upstream, but we do not want to deviate from upstream when not necessary and since you've now also packaged TEXT::CSV, there is no reason for the patch anymore. * Add Changes and README to %doc
SHOULD FIX ---------- * Add following to spec: BuildRequires: perl(HTML::Entities) Requires: perl(HTML::Entities)
I know this is optional but in Fedora we always try to enable as much features in packaes as possible, even if this drags in a few additional dependencies.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #7 from Johan Vromans jvromans@squirrel.nl 2009-04-13 10:08:45 EDT --- Spec URL: http://www.squirrel.nl/pub/xfer/perl-Data-Report.spec SRPM URL: http://www.squirrel.nl/pub/xfer/perl-Data-Report-0.10-2.fc10.src.rpm
This version fixes the remarks (MUST FIX and SHOULD FIX) from the reviewers, as summarized in #6.
No koji build, since it now requires perl-Text-CSV which is not yet part of Fedora.
The package and .spec are rpmlint free except for a harmless warning.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Hans de Goede hdegoede@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #8 from Hans de Goede hdegoede@redhat.com 2009-04-16 05:33:57 EDT --- Looks good, approved! Once eekboek also is in good shape I'll sponsor you.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Johan Vromans jvromans@squirrel.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #9 from Johan Vromans jvromans@squirrel.nl 2009-04-22 07:47:43 EDT --- New Package CVS Request ======================= Package Name: perl-Data-Report Short Description: A flexible plugin-driven reporting framework Owners: sciurius Branches: F-10 F-11 InitialCC:
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Kevin Fenzi kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #10 from Kevin Fenzi kevin@tummy.com 2009-04-23 12:25:21 EDT --- cvs done.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #11 from Fedora Update System updates@fedoraproject.org 2009-04-27 15:11:53 EDT --- perl-Data-Report-0.10-1.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/perl-Data-Report-0.10-1.fc10
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #12 from Fedora Update System updates@fedoraproject.org 2009-04-27 15:11:59 EDT --- perl-Data-Report-0.10-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/perl-Data-Report-0.10-1.fc11
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |ON_QA
--- Comment #13 from Fedora Update System updates@fedoraproject.org 2009-04-27 21:22:41 EDT --- perl-Data-Report-0.10-1.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update perl-Data-Report'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-4085
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #14 from Fedora Update System updates@fedoraproject.org 2009-05-09 00:04:17 EDT --- perl-Data-Report-0.10-1.fc11 has been pushed to the Fedora 11 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update perl-Data-Report'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-4421
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #15 from Fedora Update System updates@fedoraproject.org 2009-05-15 19:26:11 EDT --- perl-Data-Report-0.10-1.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |0.10-1.fc11 Resolution| |NEXTRELEASE
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #16 from Fedora Update System updates@fedoraproject.org 2009-05-15 19:26:53 EDT --- perl-Data-Report-0.10-1.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|0.10-1.fc11 |0.10-1.fc10
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Jussi Lehtola jussi.lehtola@iki.fi changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jussi.lehtola@iki.fi Depends on|483406 |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Jussi Lehtola jussi.lehtola@iki.fi changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks|177841(FE-NEEDSPONSOR) |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Michal Ingeli mi@v3.sk changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mi@v3.sk Flag|fedora-cvs+ |fedora-cvs?
--- Comment #17 from Michal Ingeli mi@v3.sk 2009-12-28 07:31:56 EDT --- Package Change Request ====================== Package Name: perl-Data-Report New Branches: EL-5 Owners: sciurius ksyz
The EL-5 branch is needed for MySQL-zrm in EPEL. There is a report (#550192), that MySQL-zrm wont install on el5 and in fact, it misses dependencies like perl(Data::Report) and perl(Text::CSV) that are not branched and build for epel.
I sent notification and request for EL-5 branch to he package owner few day ago. For owner, feel free to take over this branch.
Package builds in mock without problem.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #18 from Johan Vromans jvromans@squirrel.nl 2009-12-28 13:37:38 EDT --- I do not have any particular interest in EPEL. I don't mind building for EPEL either provided someone can tell me what to do (in trivial steps).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #19 from Kevin Fenzi kevin@tummy.com 2009-12-28 22:02:03 EDT --- See: https://fedoraproject.org/wiki/EPEL/GuidelinesAndPolicies
The infrastructure is all the same as the F-* branches... just need to be more conservative pushing updates and new versions.
Shall I process the request from comment 17?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #20 from Michal Ingeli mi@v3.sk 2009-12-29 03:11:44 EDT --- (In reply to comment #18)
I do not have any particular interest in EPEL. I don't mind building for EPEL either provided someone can tell me what to do (in trivial steps).
Packages are handled more sensitively, tested, trying not to break things and change API and so.
I'm willing to maintain the EPEL branch.
(In reply to comment #19)
Shall I process the request from comment 11?
If there is no problem, please proceed.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #21 from Johan Vromans jvromans@squirrel.nl 2009-12-29 04:04:24 EDT --- That's fine with me. It might be a good idea to have a separate maintainer for the EL branches.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Kevin Fenzi kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #22 from Kevin Fenzi kevin@tummy.com 2010-01-02 15:22:15 EDT --- cvs done.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #23 from Fedora Update System updates@fedoraproject.org 2010-01-19 05:53:26 EST --- perl-Data-Report-0.10-4.el5 has been submitted as an update for Fedora EPEL 5. http://admin.fedoraproject.org/updates/perl-Data-Report-0.10-4.el5
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
--- Comment #24 from Fedora Update System updates@fedoraproject.org 2010-02-04 15:43:52 EST --- perl-Data-Report-0.10-4.el5 has been pushed to the Fedora EPEL 5 stable repository. If problems still persist, please make note of it in this bug report.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483286
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|0.10-1.fc10 |0.10-4.el5 Resolution|NEXTRELEASE |ERRATA
package-review@lists.fedoraproject.org