[Bug 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-03-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718

Sven Nierlein  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)




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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-03-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718

Christopher Meng  changed:

   What|Removed |Added

 Blocks||1069988




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1069988
[Bug 1069988] Review Request: naemon - Open Source Host, Service And
Network Monitoring Program
-- 
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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-04-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718



--- Comment #1 from Sven Nierlein  ---
Spec file update according to comments of related bugs and uploaded
new files to:

http://nierlein.com/fedora/2014-04-06/perl-Catalyst-Plugin-Compress.spec
http://nierlein.com/fedora/2014-04-06/perl-Catalyst-Plugin-Compress-0.005-1.fc21.noarch.rpm

-- 
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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-04-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718



--- Comment #2 from Sven Nierlein  ---
Source RPM:
http://nierlein.com/fedora/2014-04-06/perl-Catalyst-Plugin-Compress-0.005-1.fc21.src.rpm

-- 
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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-04-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718

Christopher Meng  changed:

   What|Removed |Added

  Alias||perl-Catalyst-Plugin-Compre
   ||ss



-- 
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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-04-21 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718



--- Comment #3 from Sven Nierlein  ---
new spec file:
http://nierlein.com/fedora/2014-04-21/perl-Catalyst-Plugin-Compress.spec

the only thing i changed is the release number.

-- 
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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-04-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718

Ralf Corsepius  changed:

   What|Removed |Added

 CC||rc040...@freenet.de



--- Comment #4 from Ralf Corsepius  ---
Missing [MUSTFIX]: 
Requires:  perl(:MODULE_COMPAT_%(eval "`perl -V:version`"; echo $version))

-- 
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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-05-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718



--- Comment #5 from Sven Nierlein  ---
Sorry for the delay, i wanted to get through the review process with one module
first and then work on the remaining review requests.

So here is the updated spec file:
http://nierlein.com/fedora/2014-05-20/perl-Catalyst-Plugin-Compress.spec
http://nierlein.com/fedora/2014-05-20/perl-Catalyst-Plugin-Compress-0.005-3.fc21.src.rpm

-- 
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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-05-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718

Sven Nierlein  changed:

   What|Removed |Added

 Blocks|177841 (FE-NEEDSPONSOR) |




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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-05-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718

David Dick  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||dd...@cpan.org
   Assignee|nob...@fedoraproject.org|dd...@cpan.org
  Flags||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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-05-21 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718



--- Comment #6 from David Dick  ---
Hi Sven,

The following BRs MUST be fixed.

--- perl-Catalyst-Plugin-Compress.old   2014-05-20 23:51:55.0 +1000
+++ perl-Catalyst-Plugin-Compress.spec  2014-05-21 21:02:00.988340126 +1000
@@ -8,16 +8,28 @@
 Source0:   
http://www.cpan.org/modules/by-module/Catalyst/Catalyst-Plugin-Compress-%{version}.tar.gz
 BuildArch:  noarch
 BuildRequires:  perl
+BuildRequires:  perl(Catalyst::Utils)
+BuildRequires:  perl(Config)
+BuildRequires:  perl(Cwd)
+BuildRequires:  perl(Fcntl)
+BuildRequires:  perl(File::Find)
+BuildRequires:  perl(File::Path)
+BuildRequires:  perl(File::Spec)
+BuildRequires:  perl(FindBin)
+BuildRequires:  perl(List::Util)
 BuildRequires:  perl(Test::More)
 BuildRequires:  perl(Catalyst::Runtime) >= 5.80001
 BuildRequires:  perl(Compress::Bzip2) >= 2
 BuildRequires:  perl(Compress::Zlib)
 BuildRequires:  perl(ExtUtils::MakeMaker)
+BuildRequires:  perl(ExtUtils::MM_Unix)
 BuildRequires:  perl(MRO::Compat)
+BuildRequires:  perl(strict)
+BuildRequires:  perl(vars)
+BuildRequires:  perl(warnings)
 Requires:   perl(Catalyst::Runtime) >= 5.80001
 Requires:   perl(Compress::Bzip2) >= 2
 Requires:   perl(Compress::Zlib)
-Requires:   perl(MRO::Compat)
 Requires:   perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo
$version))

 %description

The runtime "Requires: perl(MRO::Compat)" was removed as it is auto-detected. 
The other requirements cannot be auto-detected, so they were left.  Once you
apply these changes, this package should be good to go.

-- 
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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-05-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718



--- Comment #7 from Sven Nierlein  ---
Ok, thanks. I applied your patch and uploaded new files here:
http://nierlein.com/fedora/2014-05-24/perl-Catalyst-Plugin-Compress.spec
http://nierlein.com/fedora/2014-05-24/perl-Catalyst-Plugin-Compress-0.005-4.fc21.src.rpm

-- 
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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-05-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718

David Dick  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #8 from David Dick  ---
koji builds ok at http://koji.fedoraproject.org/koji/taskinfo?taskID=6887280

License ok.

Build/Requires ok.

rpmlint is fine (incorrect spelling errors are the only output)

pure perl library.  no C deps

fedora-review did not produce any errors/warnings

package APPROVED!

-- 
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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-05-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718

Sven Nierlein  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #9 from Sven Nierlein  ---
New Package SCM Request
===
Package Name: perl-Catalyst-Plugin-Compress
Short Description: Compress response
Owners: sni
Branches: f21
InitialCC: perl-sig

-- 
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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718

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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718



--- Comment #10 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 1079718] Review Request: perl-Catalyst-Plugin-Compress - Compress Catalyst response

2014-06-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079718

Sven Nierlein  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||0.005-4
 Resolution|--- |RAWHIDE
Last Closed||2014-06-05 14:37:01



--- Comment #11 from Sven Nierlein  ---
build and upload ready

-- 
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