Bug 244484

Summary:	Review Request: perl-XML-SAX-ExpatXS - Perl SAX 2 XS extension to Expat parser
Product:	[Fedora] Fedora	Reporter:	Chris Weyl <cweyl>
Component:	Package Review	Assignee:	Nobody's working on this, feel free to take it <nobody>
Status:	CLOSED DUPLICATE	QA Contact:	Fedora Package Reviews List <fedora-package-review>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	mhroncok
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
URL:	http://search.cpan.org/dist/XML-SAX-ExpatXS/
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2007-11-25 21:05:33 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	201449, 244485

Description Chris Weyl 2007-06-15 23:58:36 UTC

SRPM URL: http://home.comcast.net/~ckweyl/perl-XML-SAX-ExpatXS-1.20-1.fc6.src.rpm
SPEC URL: http://home.comcast.net/~ckweyl/perl-XML-SAX-ExpatXS.spec

Description:
XML::SAX::ExpatXS is a direct XS extension to Expat XML parser. It
implements Perl SAX 2.1 interface.  Any deviations from the Perl
SAX 2.1 specification are considered as bugs.

See http://perl-xml.sourceforge.net/perl-sax for the Perl SAX API
description.

Comment 1 Ville Skyttä 2007-06-16 09:41:07 UTC

Not taking the review because I wouldn't be approving it as long as t/* are
shipped in the main package, but here's a couple of other comments:

XML::SAX parser registration/unregistration missing.  See perl-XML-SAX and
perl-XML-LibXML for examples how to do it (or just copy-paste from
http://cachalot.mine.nu/7/SRPMS/perl-XML-SAX-ExpatXS-1.20-0.4.cmn7.src.rpm).

BRs on perl(Devel::Peek) and perl(XML::Parser) seem superfluous.

Comment 2 Chris Weyl 2007-11-25 21:05:33 UTC

Not actually using at $work + lack of time == FE-DEADREVIEW.

Please feel free to resurrect :)

Comment 3 Miro Hrončok 2012-12-30 10:07:18 UTC


*** This bug has been marked as a duplicate of bug 890806 ***