Bug 1020435
Summary: | Review Request: perl-jmx4perl - JSON-HTTP based acess to a remote JMX agent | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Kostas Georgiou <k.georgiou> |
Component: | Package Review | Assignee: | Petr Pisar <ppisar> |
Status: | CLOSED NOTABUG | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | rawhide | CC: | k.georgiou, oliver, package-review, ppisar |
Target Milestone: | --- | Flags: | ppisar:
fedora-review-
|
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2015-08-25 11:12:16 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 |
Description
Kostas Georgiou
2013-10-17 16:18:08 UTC
Updated spec and srpm Spec URL: http://georgiou.fedorapeople.org//perl-jmx4perl.spec SRPM URL: http://georgiou.fedorapeople.org//perl-jmx4perl-1.07-4.fc19.src.rpm URL is usable. Ok. Source0 is valid. Ok. Source tar ball is original (SHA-256: efbc2dce1fffbf651d098500aedf53065cdb3cc308784f94135f77d93e111124). Ok. Summary is Ok. Description verified from README. Ok. inc/* files are perl-licensed but they don't go into binary package. Their license is not significant for License tag. License verified from LICENSE, REAMDE, lib/JMX/Jmx4Perl/Manual.pod. Ok. No XS code, noarch BuildArch is Ok. FIX: The dependency filtering macros used are obsolete. Use __requires_exclude instead <https://fedoraproject.org/wiki/Packaging:AutoProvidesAndRequiresFiltering>. Bundled Module::Build has been removed. Ok. TODO: Do not package META.json. It does not bring any additional documentation. FIX: Build-require `perl(constant)' (lib/JMX/Jmx4Perl/Request.pm:141). FIX: Build-require `perl(HTTP::Request)' (lib/JMX/Jmx4Perl/Agent.pm:6). FIX: Build-require `perl(Nagios::Plugin::Functions)' (lib/JMX/Jmx4Perl/Nagios/CheckJmx4Perl.pm:11). TODO: Build-require `perl(overload)' (lib/JMX/Jmx4Perl/Alias/Object.pm:46). FIX: Build-require `perl(URI::Escape)' (lib/JMX/Jmx4Perl/Agent.pm:5). TODO: Build-require `perl(FindBin)' (t/10_handler.t:5). TODO: Build-require `perl(lib)' (t/10_handler.t:6). FIX: The Build.PL is interactive. Redirect /dev/null to stdin. All tests pass. Ok. $ rpmlint perl-jmx4perl.spec ../SRPMS/perl-jmx4perl-1.07-4.fc21.src.rpm ../RPMS/noarch/perl-jmx4perl-1.07-4.fc21.noarch.rpm perl-jmx4perl.noarch: W: conffile-without-noreplace-flag /etc/jmx4perl/jetty.cfg perl-jmx4perl.noarch: W: conffile-without-noreplace-flag /etc/jmx4perl/common.cfg perl-jmx4perl.noarch: W: conffile-without-noreplace-flag /etc/jmx4perl/memory.cfg perl-jmx4perl.noarch: W: conffile-without-noreplace-flag /etc/jmx4perl/glassfish.cfg perl-jmx4perl.noarch: W: conffile-without-noreplace-flag /etc/jmx4perl/metrics.cfg perl-jmx4perl.noarch: W: conffile-without-noreplace-flag /etc/jmx4perl/threads.cfg perl-jmx4perl.noarch: W: conffile-without-noreplace-flag /etc/jmx4perl/jboss.cfg perl-jmx4perl.noarch: W: conffile-without-noreplace-flag /etc/jmx4perl/tomcat.cfg perl-jmx4perl.noarch: W: conffile-without-noreplace-flag /etc/jmx4perl/weblogic.cfg perl-jmx4perl.noarch: E: incorrect-fsf-address /usr/share/doc/perl-jmx4perl/LICENSE 2 packages and 1 specfiles checked; 1 errors, 9 warnings. FIX: Mark those configuration files with %config(noreplace) macro in the %files section <https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#Configuration_files>. File layout and permissions are Ok. $ rpm -q --requires -p ../RPMS/noarch/perl-jmx4perl-1.07-4.fc21.noarch.rpm | sort -i | uniq -c 1 gnupg2 1 perl(Archive::Zip) 1 perl(base) 2 perl(Carp) 1 perl(Config::General) 1 perl(Config::General) >= 2.34 1 perl(constant) 1 perl(Crypt::Blowfish_PP) 1 perl(Cwd) 2 perl(Data::Dumper) 2 perl(Digest::MD5) 2 perl(Digest::SHA1) 1 perl(Exporter) 1 perl(File::SearchPath) 1 perl(File::Spec) 1 perl(File::Temp) 1 perl(FindBin) 1 perl(Getopt::Long) 1 perl(HTTP::Request) 1 perl(JMX::Jmx4Perl) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::ArtifactHandler) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::DownloadAgent) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Logger) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Meta) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Verifier) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Verifier::ChecksumVerifier) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Verifier::PGPKey) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::WebXmlHandler) 1 perl(JMX::Jmx4Perl::Agent::UserAgent) 1 perl(JMX::Jmx4Perl::Alias) 1 perl(JMX::Jmx4Perl::Alias::Object) 1 perl(JMX::Jmx4Perl::Config) 1 perl(JMX::Jmx4Perl::J4psh) 1 perl(JMX::Jmx4Perl::J4psh::Command) 1 perl(JMX::Jmx4Perl::J4psh::CommandHandler) 1 perl(JMX::Jmx4Perl::J4psh::CompletionHandler) 1 perl(JMX::Jmx4Perl::J4psh::ServerHandler) 1 perl(JMX::Jmx4Perl::J4psh::Shell) 1 perl(JMX::Jmx4Perl::Nagios::CactiJmx4Perl) 1 perl(JMX::Jmx4Perl::Nagios::CheckJmx4Perl) 1 perl(JMX::Jmx4Perl::Nagios::SingleCheck) 1 perl(JMX::Jmx4Perl::Product::BaseHandler) 1 perl(JMX::Jmx4Perl::Request) 1 perl(JMX::Jmx4Perl::Response) 1 perl(JMX::Jmx4Perl::Util) 1 perl(JSON) 1 perl(JSON) >= 2.12 1 perl(lib) 2 perl(LWP::UserAgent) 1 perl(:MODULE_COMPAT_5.18.1) 2 perl(Module::Find) 1 perl(Nagios::Plugin) 1 perl(Nagios::Plugin) >= 0.27 1 perl(Nagios::Plugin::Functions) 1 perl(Pod::Usage) 1 perl(POSIX) 2 perl(Scalar::Util) 1 perl(strict) 2 perl(Sys::SigAction) 1 perl(TAP::Harness) 1 perl(Term::ANSIColor) 2 perl(Term::Clui) 1 perl(Term::ProgressBar) 2 perl(Term::ShellUI) 1 perl(Term::Size) >= 0.207 2 perl(Text::ParseWords) 1 perl(Time::HiRes) 1 perl(URI) >= 1.35 1 perl(URI::Escape) 1 perl(vars) 1 perl(warnings) 1 perl(XML::LibXML) 1 perl(XML::Twig) 1 rpmlib(CompressedFileNames) <= 3.0.4-1 1 rpmlib(FileDigests) <= 4.6.0-1 1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 1 rpmlib(PayloadIsXz) <= 5.2-1 1 /usr/bin/perl TODO: Do not run-require these modules explicitly, they are autodetected: 2 perl(Carp) 2 perl(Data::Dumper) 2 perl(Digest::MD5) 2 perl(Digest::SHA1) 2 perl(LWP::UserAgent) 2 perl(Module::Find) 2 perl(Scalar::Util) 2 perl(Sys::SigAction) 2 perl(Term::Clui) 2 perl(Term::ShellUI) 2 perl(Text::ParseWords) TODO: Filter under-specified dependencies for these unversioned modules: perl(Config::General), perl(JSON), perl(Nagios::Plugin). $ rpm -q --provides -p ../RPMS/noarch/perl-jmx4perl-1.07-4.fc21.noarch.rpm | sort -i | uniq -c 1 perl-jmx4perl = 1.07-4.fc21 1 perl(JMX::Jmx4Perl) = 1.07 1 perl(JMX::Jmx4Perl::Agent) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::ArtifactHandler) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::DownloadAgent) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Logger) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Logger::None) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Meta) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Verifier) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Verifier::ChecksumVerifier) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Verifier::GnuPGVerifier) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Verifier::MD5Verifier) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Verifier::OpenPGPVerifier) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Verifier::PGPKey) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::Verifier::SHA1Verifier) 1 perl(JMX::Jmx4Perl::Agent::Jolokia::WebXmlHandler) 1 perl(JMX::Jmx4Perl::Agent::UserAgent) 1 perl(JMX::Jmx4Perl::Alias) 1 perl(JMX::Jmx4Perl::Alias::Object) 1 perl(JMX::Jmx4Perl::Config) 1 perl(JMX::Jmx4Perl::J4psh) 1 perl(JMX::Jmx4Perl::J4psh::Command) 1 perl(JMX::Jmx4Perl::J4psh::Command::Global) 1 perl(JMX::Jmx4Perl::J4psh::CommandHandler) 1 perl(JMX::Jmx4Perl::J4psh::Command::MBean) 1 perl(JMX::Jmx4Perl::J4psh::Command::Server) 1 perl(JMX::Jmx4Perl::J4psh::CompletionHandler) 1 perl(JMX::Jmx4Perl::J4psh::ServerHandler) 1 perl(JMX::Jmx4Perl::J4psh::Shell) 1 perl(JMX::Jmx4Perl::Nagios::CactiJmx4Perl) 1 perl(JMX::Jmx4Perl::Nagios::CheckJmx4Perl) 1 perl(JMX::Jmx4Perl::Nagios::SingleCheck) 1 perl(JMX::Jmx4Perl::Product::ActiveMQ) 1 perl(JMX::Jmx4Perl::Product::BaseHandler) 1 perl(JMX::Jmx4Perl::Product::Geronimo) 1 perl(JMX::Jmx4Perl::Product::Glassfish) 1 perl(JMX::Jmx4Perl::Product::Hadoop) 1 perl(JMX::Jmx4Perl::Product::JBoss) 1 perl(JMX::Jmx4Perl::Product::Jetty) 1 perl(JMX::Jmx4Perl::Product::Jonas) 1 perl(JMX::Jmx4Perl::Product::Resin) 1 perl(JMX::Jmx4Perl::Product::SpringDM) 1 perl(JMX::Jmx4Perl::Product::Terracotta) 1 perl(JMX::Jmx4Perl::Product::Tomcat) 1 perl(JMX::Jmx4Perl::Product::Unknown) 1 perl(JMX::Jmx4Perl::Product::Weblogic) 1 perl(JMX::Jmx4Perl::Product::Websphere) 1 perl(JMX::Jmx4Perl::Request) 1 perl(JMX::Jmx4Perl::Response) 1 perl(JMX::Jmx4Perl::Util) Binary provides are Ok. $ resolvedeps rawhide ../RPMS/noarch/perl-jmx4perl-1.07-4.fc21.noarch.rpm Binary dependencies resolvable. Ok. Package builds in F21 (http://koji.fedoraproject.org/koji/taskinfo?taskID=6094373). OK. Otherwise the package is in line with Fedora and Perl packaging guidelines. Please correct all `FIX' issues, consider fixing `TODO' items, and provide new spec file. Resolution: Package NOT approved. Any progress? Long time ago... Is this ever going to happen? I closed this review request by rejecting it because the submitter is unresponsive. Anybody willing to maintain this package can start a new review. |