[Bug 191523] Review Request: perl-XML-Stream

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: perl-XML-Stream


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=191523





------- Additional Comments From tibbs@xxxxxxxxxxx  2006-05-25 23:16 EST -------
Note: test suites should not talk to the network as you don't know that the
build hosts aren't completely firewalled.  In any case, the tests don't work at
all for me in mock, probably due to not having a resolver configuration:

PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0,
'blib/lib', 'blib/arch')" t/*.t
t/buildxml......Cannot resolve compute20.math.uh.edu:  at lib/XML/Stream.pm line
406.
# Looks like you planned 56 tests but only ran 1.
# Looks like your test died just after 1.

My recommendation is to just comment out the "make test" line in %check before
you check in.  I'll assume that's been done for the purposes of this review.

Upstream includes a copy of the LGPL, which you must include in the package.  I
would recommend including the others as well; just do

perldoc perlgpl > LICENSE.GPL
perldoc perlartistic > LICENSE.Artistic

and then include those as %doc along with the included LICENSE.LGPL.

I'd also suggest including any correspondence with the author you have which
might clarify the license issie.

No need to BuildRequires: perl, but not a blocker.

Review:
* package meets naming and packaging guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
O license field matches the actual license (please include the author's
"clarification" of the license issue).
X license is open source-compatible.  License text provided but not included.
* source files match upstream:
   ae09400fac17eaea4c9b12283db06881  XML-Stream-1.22.tar.gz
   ae09400fac17eaea4c9b12283db06881  XML-Stream-1.22.tar.gz-srpm
* latest version is being packaged.
O BuildRequires are proper.  (perl BR is superfluous.)
* package builds in mock (development, x86_64), with test suite disabled.
* rpmlint is silent.
* final provides and requires are sane:
   perl(XML::Stream) = 1.22
   perl(XML::Stream::Namespace) = 1.22
   perl(XML::Stream::Node) = 1.22
   perl(XML::Stream::Parser) = 1.22
   perl(XML::Stream::Parser::DTD) = 1.22
   perl(XML::Stream::Tree) = 1.22
   perl(XML::Stream::XPath) = 1.22
   perl(XML::Stream::XPath::AllOp)
   perl(XML::Stream::XPath::AndOp)
   perl(XML::Stream::XPath::AttributeOp)
   perl(XML::Stream::XPath::ContextOp)
   perl(XML::Stream::XPath::EqualOp)
   perl(XML::Stream::XPath::FunctionOp)
   perl(XML::Stream::XPath::NodeOp)
   perl(XML::Stream::XPath::NotEqualOp)
   perl(XML::Stream::XPath::Op) = 1.22
   perl(XML::Stream::XPath::OrOp)
   perl(XML::Stream::XPath::PositionOp)
   perl(XML::Stream::XPath::Query) = 1.22
   perl(XML::Stream::XPath::Value) = 1.22
   perl-XML-Stream = 1.22-3.fc6
  -
   perl >= 0:5.006_001
   perl(:MODULE_COMPAT_5.8.8)
   perl(Authen::SASL)
   perl(Carp)
   perl(Encode)
   perl(Exporter)
   perl(FileHandle)
   perl(IO::Select)
   perl(IO::Socket)
   perl(MIME::Base64)
   perl(POSIX)
   perl(Sys::Hostname)
   perl(XML::Stream::Namespace)
   perl(XML::Stream::Parser)
   perl(XML::Stream::XPath)
   perl(XML::Stream::XPath::Op)
   perl(XML::Stream::XPath::Query)
   perl(XML::Stream::XPath::Value)
   perl(strict)
   perl(utf8)
   perl(vars)
* no shared libraries are present.
* package is not relocatable.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* %clean is present.
* %check is present but necessarily disabled.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no libtool .la droppings.
* not a GUI app.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]