Bug 232619 - Review Request: perl-XML-SAX-Writer - SAX2 Writer
Summary: Review Request: perl-XML-SAX-Writer - SAX2 Writer
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Ville Skyttä
QA Contact: Fedora Package Reviews List
Depends On: 232618
Blocks: 232620
TreeView+ depends on / blocked
Reported: 2007-03-16 13:26 UTC by Andreas Thienemann
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2007-03-23 10:18:20 UTC
Type: ---
ville.skytta: fedora-review+
petersen: fedora-cvs+

Attachments (Terms of Use)

Description Andreas Thienemann 2007-03-16 13:26:13 UTC
Spec URL: http://home.bawue.de/~ixs/perl-XML-SAX-Writer/perl-XML-SAX-Writer.spec
SRPM URL: http://home.bawue.de/~ixs/perl-XML-SAX-Writer/perl-XML-SAX-Writer-0.50-1.src.rpm

A new XML Writer to match the SAX2 effort.

Comment 1 Ville Skyttä 2007-03-17 20:13:03 UTC
* These explicit dependencies are dupes of (unversioned) autogenerated ones,
just remove them?  The versions seem to come from Makefile.PL but upstream has
chose to not require any particular version of things at runtime.
perl(Text::Iconv) >= 1.2
perl(XML::Filter::BufferText) >= 1.00
perl(XML::NamespaceSupport) >= 1.00
perl(XML::SAX::Exception) >= 1.01

* Doing the iconv-to-UTF-8 stuff for the man page only doesn't help perldoc. 
Doing it for Writer.pm in %prep would work for both man and perldoc.

Comment 2 Andreas Thienemann 2007-03-17 22:24:52 UTC

new spec, fixing these issues.

Comment 3 Ville Skyttä 2007-03-18 10:35:14 UTC

Comment 4 Andreas Thienemann 2007-03-18 11:53:30 UTC
New Package CVS Request
Package Name: perl-XML-SAX-Writer
Short Description: SAX2 Writer
Owners: andreas@bawue.net
Branches: FC-5, FC-6, devel
InitialCC: fedora-perl-devel-list@redhat.com

Comment 5 Jens Petersen 2007-03-20 00:06:31 UTC

Comment 6 Andreas Thienemann 2007-03-23 10:18:20 UTC
Thanks for the review. Package has been built for FC-5, FC-6 and rawhide.

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