Fedora Merge Review: perl-HTML-Parser http://cvs.fedora.redhat.com/viewcvs/devel/perl-HTML-Parser/ Initial Owner: rnorwood
Created attachment 147288 [details] Specfile patch Robin, The patch: * updates to version 3.56. * brings specfile closer to the Fedora's Perl template. * converts specfile to UTF-8 (changelog entries). * adds examples and doc files. * adds perl(Test::Pod) to the build requirements list. jpo
Robin, The perl(Test::Pod) BR should be removed from the patch attached to comment #1 while (?) the build system is unable to pull Extras packages. jpo
alright, changes have been incorporated in perl-HTML-Parser-3.56-1.fc7, except for the BuildRequires: perl(Test::Pod) Let me know how it looks to you.
Robin, Looks good. jpo
ok, I think this gets assigned back to me now. (http://www.fedoraproject.org/wiki/WarrenTogami/ReviewWithFlags)
Closing... (merge review with fedora-review+)
Rel-engs, please add `perl-sig' pseudouser watchbugzilla and watchcommits permissions for all perl-HTML-Parser branches.
No SCM request found.
(In reply to Jon Ciesla from comment #8) > No SCM request found. There is no SCM template for adding users to existing branches. I guess you are missing something like this: Package Change Request ====================== Package Name: perl-HTML-Parser Branches: f19 f20 f21 master InitialCC: perl-sig
Complete.