Bug 602674 - Review Request: perl-POE-Filter-HTTP-Parser - HTTP POE filter for HTTP clients or servers
Summary: Review Request: perl-POE-Filter-HTTP-Parser - HTTP POE filter for HTTP client...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Marcela Mašláňová
QA Contact: Fedora Extras Quality Assurance
URL: http://search.cpan.org/~bingos/POE-Fi...
Whiteboard:
Depends On: 602659 602703
Blocks: 602616
TreeView+ depends on / blocked
 
Reported: 2010-06-10 13:07 UTC by Petr Pisar
Modified: 2013-05-07 13:01 UTC (History)
4 users (show)

Fixed In Version: perl-POE-Filter-HTTP-Parser-1.02-1.fc14
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-06-23 08:29:18 UTC
Type: ---
Embargoed:
mmaslano: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Petr Pisar 2010-06-10 13:07:08 UTC
New package perl-POE-Filter-HTTP-Parser is required to upgrade perl-POE-Component-Server-SimpleHTTP to 2.04 version.

As current lastest upstream version of perl-POE-Filter-HTTP-Parser requires perl(HTTP::Parser) >= 0.04 that is not in Fedora yet, this bug report should be considered as placeholder for review request that will come after chain of prerequisites is fulfilled.

Comment 1 Petr Pisar 2010-06-10 13:13:06 UTC
This package requires following packages that are not yet in Fedora:

perl(HTTP::Parser) >= 0.04
perl(Test::POE::Client::TCP) >= 0.1

Comment 2 Petr Pisar 2010-06-10 13:14:17 UTC
Review Request (bug #602659) for perl-Test-POE-Client-TCP must be completed
before this bug can be resolved.

Comment 3 Petr Pisar 2010-06-10 14:20:14 UTC
Review Request (bug #602703) for HTTP-Parser must be completed before this bug can be resolved.

Comment 4 Petr Pisar 2010-06-18 13:22:16 UTC
Spec URL: http://ppisar.fedorapeople.org/perl-POE-Filter-HTTP-Parser/perl-POE-Filter-HTTP-Parser.spec
SRPM URL: http://ppisar.fedorapeople.org/perl-POE-Filter-HTTP-Parser/perl-POE-Filter-HTTP-Parser-1.02-1.fc13.src.rpm
Description:
POE::Filter::HTTP::Parser is a POE::Filter for HTTP which is based on HTTP::Parser.

This new package is required for further versions of perl-POE-Component-Server-SimpleHTTP.

This package compiles and tests cleanly only of Fedora 14 where required
dependencies exist
(http://koji.fedoraproject.org/koji/taskinfo?taskID=2257045). This package is
not intended to emerge in F-13 or lower distributions.

Comment 5 Marcela Mašláňová 2010-06-21 10:16:23 UTC
- rpmlint checks return: OK

- package meets naming guidelines OK
- package meets packaging guidelines OK
- license (GPL+ or Artistic = same as Perl) OK, text in %doc, matches source OK
- spec file legible, in am. english OK
- source matches upstream OK cfaba167c59c51823353939b2241d407
- package compiles on devel (x86) OK
- no missing BR OK
- no unnecessary BR NOT OK
- no locales OK
- not relocatable OK
- owns all directories that it creates OK
- no duplicate files OK
- permissions ok OK
- %clean ok OK
- macro use consistent OK
- code, not content OK
- no need for -docs OK
- nothing in %doc affects runtime OK
- no need for .desktop file  OK

Please remove BR: perl, because that's not needed. Except this is package review ACCEPTED.

Comment 6 Petr Pisar 2010-06-21 10:54:03 UTC
New Package CVS Request
=======================
Package Name: perl-POE-Filter-HTTP-Parser
Short Description: HTTP POE filter for HTTP clients or servers
Owners: ppisar mmaslano psabata
Branches: F-14 only
InitialCC: perl-sig

Please add this package to CVS. devel branch is enough.

Comment 7 Kevin Fenzi 2010-06-23 01:51:30 UTC
CVS done (by process-cvs-requests.py).

Comment 8 Petr Pisar 2010-06-23 08:29:18 UTC
Built for F14. Thanks for review and CVS access.

Comment 9 Andrea Veri 2013-05-07 11:56:08 UTC
Package Change Request
======================
Package Name: perl-POE-Filter-HTTP-Parser
New Branches: el6
Owners: averi psabata
InitialCC: perl-sig

Comment 10 Gwyn Ciesla 2013-05-07 13:01:58 UTC
Git done (by process-git-requests).


Note You need to log in before you can comment on or make changes to this bug.