[Bug 785057] Review Request: perl-Net-Server-Coro - Co-operative multithreaded server using Coro

2012-02-05 Thread bugzilla
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=785057

Mathieu Bridon  changed:

   What|Removed |Added

 Resolution|NOTABUG |NEXTRELEASE

--- Comment #6 from Mathieu Bridon  2012-02-05 
23:13:42 EST ---
Thanks Petr, good catch!

I've fixed this, it's building and I'll obsolete the packages I had already
submitted to testing.

(Note: I see this bug is fixed as NOTABUG rather than NEXTRELEASE. I guess I
chose the wrong one when I closed it, so I'm changing it now, in case that
matters to somebody)

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785057] Review Request: perl-Net-Server-Coro - Co-operative multithreaded server using Coro

2012-02-03 Thread bugzilla
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=785057

Petr Pisar  changed:

   What|Removed |Added

 CC||ppi...@redhat.com

--- Comment #5 from Petr Pisar  2012-02-03 03:30:08 EST ---
Excuse me, the Makefile.PL uses inc::Module::Install, which is provided by
separate package perl-Module-Install (in F17). I think this package should
build-require `perl(inc::Module::Install)'.

Also it would be good to delete `inc' subdirectory before building to remove
bundled Module::Install.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785057] Review Request: perl-Net-Server-Coro - Co-operative multithreaded server using Coro

2012-02-03 Thread bugzilla
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=785057

Mathieu Bridon  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NOTABUG
Last Closed||2012-02-03 03:06:10

--- Comment #4 from Mathieu Bridon  2012-02-03 
03:06:10 EST ---
Thank you Jon.

Package built in all branches, closing the ticket.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785057] Review Request: perl-Net-Server-Coro - Co-operative multithreaded server using Coro

2012-02-01 Thread bugzilla
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=785057

--- Comment #3 from Jon Ciesla  2012-02-02 02:48:29 EST 
---
Git done (by process-git-requests).

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785057] Review Request: perl-Net-Server-Coro - Co-operative multithreaded server using Coro

2012-02-01 Thread bugzilla
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=785057

Mathieu Bridon  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #2 from Mathieu Bridon  2012-02-02 
01:27:47 EST ---
Thanks Parag!

New Package CVS Request
===
Package Name: perl-Net-Server-Coro
Short Description: Co-operative multithreaded server using Coro
Owners: bochecha
Branches: f16 el6
InitialCC: perl-sig

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785057] Review Request: perl-Net-Server-Coro - Co-operative multithreaded server using Coro

2012-02-01 Thread bugzilla
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=785057

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||panem...@gmail.com
 AssignedTo|nob...@fedoraproject.org|panem...@gmail.com
   Flag||fedora-review+

--- Comment #1 from Parag AN(पराग)  2012-02-02 01:20:37 EST 
---
Review:-

+ koji build (f17) ->
http://koji.fedoraproject.org/koji/taskinfo?taskID=3754354

+ rpmlint on rpms gave
perl-Net-Server-Coro.src: W: spelling-error Summary(en_US) multithreaded ->
multicolored
perl-Net-Server-Coro.src: W: spelling-error %description -l en_US multithreaded
-> multicolored
perl-Net-Server-Coro.noarch: W: spelling-error Summary(en_US) multithreaded ->
multicolored
perl-Net-Server-Coro.noarch: W: spelling-error %description -l en_US
multithreaded -> multicolored
2 packages and 0 specfiles checked; 0 errors, 4 warnings.

+ License is valid

+ source verified with upstream as (sha1sum)
0d33fa51f9ad50dd22a03d1c5b46407f750836e6  Net-Server-Coro-1.2.tar.gz
0d33fa51f9ad50dd22a03d1c5b46407f750836e6  ../SOURCES/Net-Server-Coro-1.2.tar.gz


+ Package perl-Net-Server-Coro-1.2-1.fc17.noarch =>
Provides: perl(Net::Server::Coro) = 1.2 perl(Net::Server::Proto::Coro)
perl(Net::Server::Proto::Coro::FH)

Requires: perl(Coro) perl(Coro::Handle::FH) perl(Coro::Socket)
perl(Coro::Specific) perl(Coro::Util) perl(EV) perl(Net::Server)
perl(Net::Server::Proto::Coro) perl(Net::Server::Proto::TCP) perl(Socket)
perl(base) perl(constant) perl(strict) perl(vars) perl(warnings)

+ Follows packaging guidelines.

APPROVED.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review