[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #22 from Kevin Fenzi 2010-01-02 15:22:15 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 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. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 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. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 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? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 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). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 changed: What|Removed |Added CC||m...@v3.sk Flag|fedora-cvs+ |fedora-cvs? --- Comment #17 from Michal Ingeli 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. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 changed: What|Removed |Added CC||jussi.leht...@iki.fi Depends on|483406 | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 changed: What|Removed |Added Fixed In Version|0.10-1.fc11 |0.10-1.fc10 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 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. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 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. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||0.10-1.fc11 Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 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 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 changed: What|Removed |Added Status|ASSIGNED|ON_QA --- Comment #13 from Fedora Update System 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 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 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 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 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 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #10 from Kevin Fenzi 2009-04-23 12:25:21 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 changed: What|Removed |Added Flag||fedora-cvs? --- Comment #9 from Johan Vromans 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: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #8 from Hans de Goede 2009-04-16 05:33:57 EDT --- Looks good, approved! Once eekboek also is in good shape I'll sponsor you. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 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. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 changed: What|Removed |Added Status|NEW |ASSIGNED CC||hdego...@redhat.com AssignedTo|nob...@fedoraproject.org|hdego...@redhat.com Flag||fedora-review? --- Comment #6 from Hans de Goede 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. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 changed: What|Removed |Added Depends on||483406 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 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. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 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 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 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. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 changed: What|Removed |Added CC||rc040...@freenet.de --- Comment #2 from Ralf Corsepius 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. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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(पराग) changed: What|Removed |Added CC||panem...@gmail.com --- Comment #1 from Parag AN(पराग) 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. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework
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 changed: What|Removed |Added Blocks||177841 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review