[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 Fedora Update System changed: What|Removed |Added Fixed In Version|perl-App-CSV-0.08-3.fc20|perl-App-CSV-0.08-3.el6 --- Comment #33 from Fedora Update System --- perl-App-CSV-0.08-3.el6 has been pushed to the Fedora EPEL 6 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 Fedora Update System changed: What|Removed |Added Fixed In Version|perl-App-CSV-0.08-3.fc19|perl-App-CSV-0.08-3.fc20 --- Comment #32 from Fedora Update System --- perl-App-CSV-0.08-3.fc20 has been pushed to the Fedora 20 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 Fedora Update System changed: What|Removed |Added Fixed In Version||perl-App-CSV-0.08-3.fc19 Resolution|NEXTRELEASE |ERRATA --- Comment #31 from Fedora Update System --- perl-App-CSV-0.08-3.fc19 has been pushed to the Fedora 19 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 Fabio Alessandro Locati changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |NEXTRELEASE Last Closed||2014-07-30 11:58:51 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 --- Comment #30 from Fedora Update System --- perl-App-CSV-0.08-3.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/perl-App-CSV-0.08-3.el6 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 --- Comment #28 from Fedora Update System --- perl-App-CSV-0.08-3.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/perl-App-CSV-0.08-3.fc20 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 --- Comment #29 from Fedora Update System --- perl-App-CSV-0.08-3.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/perl-App-CSV-0.08-3.fc19 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 --- Comment #27 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 Jon Ciesla changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 Jon Ciesla changed: What|Removed |Added Flags|fedora-review? |fedora-review+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 Fabio Alessandro Locati changed: What|Removed |Added Flags||fedora-cvs? --- Comment #26 from Fabio Alessandro Locati --- New Package SCM Request === Package Name: perl-App-CSV Short Description: App::CSV Perl module Upstream URL: http://search.cpan.org/dist/App-CSV/ Owners: fale Branches: f19 f20 f21 el6 epel7 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 --- Comment #25 from Fabio Alessandro Locati --- Thank you Jon for sponsoring me as a Fedora packager :) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 Jon Ciesla changed: What|Removed |Added Blocks|177841 (FE-NEEDSPONSOR) | --- Comment #24 from Jon Ciesla --- Ok, the package looks good, as does your practice review. Please request sponsorship into the packager group in FAS and I'll sponsor you, then approve this. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 --- Comment #23 from Fabio Alessandro Locati --- I did not wanted to argue, I was only looking for an explanation about a question that was obscure to me. Looking for NEW bugs for 'Package Review' sorted by desc ID, the first one I've found is https://bugzilla.redhat.com/show_bug.cgi?id=1122577, so I've started with this one. I was not able to assign the bug to me nor to mark it as ASSIGNED, probably because I have not enough privileges on bugzilla. Here there is the 0.08-3 version with the suggestion by Christoper :) - SPEC: http://data.fabiolocati.com/fedora/csv/perl-App-CSV.spec - SRPM: http://data.fabiolocati.com/fedora/csv/perl-App-CSV-0.08-3.el7.src.rpm - Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=7187311 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942
--- Comment #22 from Christopher Meng ---
(In reply to Jon Ciesla from comment #21)
> That's a bit harsh, Christopher. It's mostly to provide a way to display an
> unserstanding of the Packaging Guidelines and the review process.
@Jon:
No.
I didn't point to this one. But the fact is, some are sponsored within 3 days,
some need to wait at least 3 months but still get no feedbacks. And actually
some among these 3+3 are not even eligible to being as a packager, they just do
a trivial python module which is not required by any packages else, or a perl
one line script review and pretend to be a guru.
I don't want to start any flamewars here, but Fedora should really revise the
guidelines to attract more devs instead of users to the packager group. Thanks
to the @fedoraproject addr is not restricted like Debian or Gentoo, or more
craps will come.
@Fabio:
Drop find $RPM_BUILD_ROOT -depth -type d -exec rmdir {} 2>/dev/null \;
This has been fixed in MakeMakers for years.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 --- Comment #21 from Jon Ciesla --- That's a bit harsh, Christopher. It's mostly to provide a way to display an unserstanding of the Packaging Guidelines and the review process. https://fedoraproject.org/wiki/Package_Review_Process Find a review or two, go through the process on them but make sure they know you're not sponsored yet. Then post links here, I'll have a look, and once you're sponsored and this is approved you can go back and finish those reviews and approve those packages. And yes, Christopher, the queue is long. I do a review or two when I can. We all should. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 Christopher Meng changed: What|Removed |Added CC||i...@cicku.me --- Comment #20 from Christopher Meng --- (In reply to Fabio Alessandro Locati from comment #19) > (In reply to Jon Ciesla from comment #18) > > Any practice reviews you've worked on? > > What do you mean? You need to prove yourself qualified to maintain the package. Because at first you even didn't name it correctly. So it's better to give you sometime to do some informal package reviews(review others' packages). You should feel lucky comparing to those people still waiting in the queue. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 --- Comment #19 from Fabio Alessandro Locati --- Hi Jon :), I've fixed the permissions of the SPEC file and you can find the koji build at http://koji.fedoraproject.org/koji/taskinfo?taskID=7178191. (In reply to Jon Ciesla from comment #18) > Any practice reviews you've worked on? What do you mean? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 --- Comment #18 from Jon Ciesla --- Gotcha: - rpmlint checks return: perl-App-CSV.src: W: strange-permission perl-App-CSV.spec 0600L A file that you listed to include in your package has strange permissions. Usually, a file should have 0644 permissions. Trivial, but fix. - package meets naming guidelines - package meets packaging guidelines - license ( MIT ) OK, text in %doc, matches source - spec file legible, in am. english - source matches upstream - package compiles on devel (x86) - no missing BR - no unnecessary BR - no locales - not relocatable - owns all directories that it creates - no duplicate files - permissions ok - %clean ok - macro use consistent - code, not content - no need for -docs - nothing in %doc affects runtime - no need for .desktop file So just the spec file perms. Otherwise pretty clean. Any practice reviews you've worked on? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 --- Comment #17 from Fabio Alessandro Locati --- Hi Jon, There is no new SRPM/SPEC incorportating the above Ralf's suggestions, since the #1 is already present in the last SRPM/SPEC and the #2 is about the different GIT branches versions policy (so, it will be applied as soon as a GIT repo will be available) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 --- Comment #16 from Jon Ciesla --- Is there a new SRPM and SPEC incorporating the above? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062942] Review Request: perl-App-CSV - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 Christopher Meng changed: What|Removed |Added Summary|Review Request: csv - The |Review Request: |CSV command line Tool |perl-App-CSV - The CSV ||command line Tool -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
