[Bug 781793] New: Review Request: perl-String-ToIdentifier-EN - Convert Strings to English Program Identifiers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-String-ToIdentifier-EN - Convert Strings to English Program Identifiers https://bugzilla.redhat.com/show_bug.cgi?id=781793 Summary: Review Request: perl-String-ToIdentifier-EN - Convert Strings to English Program Identifiers Product: Fedora Version: rawhide Platform: Unspecified URL: http://search.cpan.org/dist/String-ToIdentifier-EN/ OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: iarn...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://fedorapeople.org/~iarnell/review/perl-String-ToIdentifier-EN.spec SRPM URL: http://fedorapeople.org/~iarnell/review/perl-String-ToIdentifier-EN-0.06-1.fc17.src.rpm Description: This module provides a utility method, to_identifier for converting an arbitrary string into a readable representation using the ASCII subset of \w for use as an identifier in a computer program. The intent is to make unique identifier names from which the content of the original string can be easily inferred by a human just by reading the identifier. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3696115 *rt-0.10_02 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781795] New: Review Request: perl-DBIx-Class-IntrospectableM2M - Introspect many-to-many shortcuts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-DBIx-Class-IntrospectableM2M - Introspect many-to-many shortcuts https://bugzilla.redhat.com/show_bug.cgi?id=781795 Summary: Review Request: perl-DBIx-Class-IntrospectableM2M - Introspect many-to-many shortcuts Product: Fedora Version: rawhide Platform: Unspecified URL: http://search.cpan.org/dist/DBIx-Class-IntrospectableM 2M/ OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: iarn...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://fedorapeople.org/~iarnell/review/perl-DBIx-Class-IntrospectableM2M.spec SRPM URL: http://fedorapeople.org/~iarnell/review/perl-DBIx-Class-IntrospectableM2M-0.001001-1.fc17.src.rpm Description: Because the many-to-many relationships are not real relationships, they can not be introspected with DBIx::Class. Many-to-many relationships are actually just a collection of convenience methods installed to bridge two relationships. This DBIx::Class component can be used to store all relevant information about these non-relationships so they can later be introspected and examined. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3696128 *rt-0.10_02 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 658234] Review Request: gdal-grass - Standalone GRASS 6 Drivers for GDAL and OGR
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=658234 --- Comment #7 from Volker Fröhlich volke...@gmx.at 2012-01-15 04:01:23 EST --- I take over this package. I built a version from the GDAL 1.7.3 sources, which is also a lot simpler than the previous approach. It requires a patched GDAL though, because of http://trac.osgeo.org/gdal/ticket/ I'm not sure about all the details, as you can read in the comments. http://www.geofrogger.net/review/gdal-grass.spec http://www.geofrogger.net/review/gdal-grass-1.7.3-1.fc16.src.rpm It is not necessarily ready for a review, but I'd be happy if you commented on it. I also contacted the Debian maintainer on most of my questions. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 476160] Review Request: perl-Directory-Scratch - Self-cleaning scratch space for tests
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=476160 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #5 from Iain Arnell iarn...@gmail.com 2012-01-15 04:17:01 EST --- Package Change Request == Package Name: perl-Directory-Scratch InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 482863] Review Request: perl-MooseX-Daemonize - Role for daemonizing your Moose based application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482863 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #10 from Iain Arnell iarn...@gmail.com 2012-01-15 04:19:28 EST --- Package Change Request == Package Name: perl-MooseX-Daemonize InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 481681] Review Request: perl-MooseX-ClassAttribute - Declare class attributes Moose-style
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481681 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #4 from Iain Arnell iarn...@gmail.com 2012-01-15 04:18:40 EST --- Package Change Request == Package Name: perl-MooseX-ClassAttribute InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 483459] Review Request: perl-MooseX-Iterator - Iterate over collections
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483459 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #11 from Iain Arnell iarn...@gmail.com 2012-01-15 04:20:13 EST --- Package Change Request == Package Name: perl-MooseX-Iterator InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 483258] Review Request: perl-MooseX-LogDispatch - Logging Role for Moose
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483258 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #4 from Iain Arnell iarn...@gmail.com 2012-01-15 04:21:56 EST --- Package Change Request == Package Name: perl-MooseX-LogDispatch InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 482856] Review Request: perl-MooseX-POE - Moose wrapper around a POE::Session
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482856 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #13 from Iain Arnell iarn...@gmail.com 2012-01-15 04:22:33 EST --- Package Change Request == Package Name: perl-MooseX-POE InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 483461] Review Request: perl-MooseX-Param - Simple role to provide a standard param method
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483461 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #4 from Iain Arnell iarn...@gmail.com 2012-01-15 04:23:24 EST --- Package Change Request == Package Name: perl-MooseX-Param InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 485154] Review Request: perl-MooseX-Role-Cmd - Wrap system command binaries the Moose way
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=485154 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #4 from Iain Arnell iarn...@gmail.com 2012-01-15 04:24:06 EST --- Package Change Request == Package Name: perl-MooseX-Role-Cmd InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 485967] Review Request: perl-MooseX-LazyLogDispatch - Logging Role for Moose
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=485967 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #4 from Iain Arnell iarn...@gmail.com 2012-01-15 04:21:05 EST --- Package Change Request == Package Name: perl-MooseX-LazyLogDispatch InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 483557] Review Request: perl-MooseX-SimpleConfig - Moose role for setting attributes from a simple configfile
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483557 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #4 from Iain Arnell iarn...@gmail.com 2012-01-15 04:24:49 EST --- Package Change Request == Package Name: perl-MooseX-SimpleConfig InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 239087] Review Request: perl-Nmap-Parser - Parse nmap scan data with perl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=239087 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #10 from Iain Arnell iarn...@gmail.com 2012-01-15 04:28:12 EST --- Package Change Request == Package Name: perl-Nmap-Parser InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 483569] Review Request: perl-MooseX-Types-Set-Object - Set::Object type with coercions and stuff
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483569 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #4 from Iain Arnell iarn...@gmail.com 2012-01-15 04:25:33 EST --- Package Change Request == Package Name: perl-MooseX-Types-Set-Object InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 483933] Review Request: perl-autobox - Call methods on native types
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483933 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #5 from Iain Arnell iarn...@gmail.com 2012-01-15 04:29:01 EST --- Package Change Request == Package Name: perl-autobox InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 239157] Review Request: perl-Net-Pcap - Interface to pcap LBL packet capture library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=239157 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #9 from Iain Arnell iarn...@gmail.com 2012-01-15 04:27:13 EST --- Package Change Request == Package Name: perl-Net-Pcap InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 483649] Review Request: perl-MooseX-Workers - Provides a simple sub-process management for asynchronous tasks
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483649 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #4 from Iain Arnell iarn...@gmail.com 2012-01-15 04:26:19 EST --- Package Change Request == Package Name: perl-MooseX-Workers InitialCC: perl-sig Please add perl-sig to this package with watchbugzilla and watchcommits bits on all fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 768174] Review Request: srm-ifce - Storage Resources Manager client implementation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=768174
--- Comment #16 from Pierre-YvesChibon pin...@pingoured.fr 2012-01-15
06:44:13 EST ---
Package Review
==
Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated
C/C++
[x]: MUST Header files in -devel subpackage, if present.
[x]: MUST ldconfig called in %post and %postun if required.
[x]: MUST Package does not contain any libtool archives (.la)
[x]: MUST Package does not contain kernel modules.
[x]: MUST Package contains no static executables.
[x]: MUST Rpath absent or only used for internal libs.
[x]: MUST Package is not relocatable.
[x]: MUST Development .so files in -devel subpackage, if present.
Generic
[x]: MUST Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
least one supported primary architecture.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[!]: MUST Buildroot is not present
Note: Invalid buildroot found: %{_tmppath}/%{name}-%{version}-%{release}
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
$RPM_BUILD_ROOT)
Note: Clean would be needed if support for EPEL is required
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Each %files section contains %defattr if rpm 4.4
Note: Note: defattr macros not found. They would be needed for EPEL5
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
Note: rm -rf would be needed if support for EPEL5 is required
[x]: MUST If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
[!]: MUST License field in the package spec file matches the actual license.
[x]: MUST License file installed when any subpackage combination is installed.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
names).
[x]: MUST Package meets the Packaging Guidelines.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generates any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint output is silent.
rpmlint srm-ifce-1.12-4.src.rpm
srm-ifce.src: W: invalid-url Source0: srm-ifce-1.12.tar.gz
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
rpmlint srm-ifce-devel-1.12-4.i686.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
rpmlint srm-ifce-1.12-4.i686.rpm
srm-ifce.i686: W: no-manual-page-for-binary gfal_srm_ifce_version
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
rpmlint srm-ifce-debuginfo-1.12-4.i686.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[x]: MUST Sources used to build the package match the upstream source, as
provided in the spec URL.
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[-]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[x]: SHOULD If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to
include it.
[!]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
/usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
--requires).
[x]: SHOULD Package functions as described.
[x]: SHOULD Package does not include license text files separate from
upstream.
[x]: SHOULD Scriptlets must be sane, if used.
[x]: SHOULD SourceX is a working URL.
[x]: SHOULD Description and summary sections in the package spec file contains
[Bug 768183] Review Request: is-interface - library for the information system in wlcg
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=768183 --- Comment #4 from Pierre-YvesChibon pin...@pingoured.fr 2012-01-15 06:46:10 EST --- Wrong url for the srpm. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 768310] Review Request: gridftp-ifce - GridFTP abstraction layer for wlcg.
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=768310
--- Comment #4 from Pierre-YvesChibon pin...@pingoured.fr 2012-01-15 06:58:28
EST ---
Package Review
==
Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated
C/C++
[x]: MUST Header files in -devel subpackage, if present.
[x]: MUST ldconfig called in %post and %postun if required.
[x]: MUST Package does not contain any libtool archives (.la)
[x]: MUST Package does not contain kernel modules.
[x]: MUST Package contains no static executables.
[x]: MUST Rpath absent or only used for internal libs.
[x]: MUST Package is not relocatable.
[x]: MUST Development .so files in -devel subpackage, if present.
Generic
[x]: MUST Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
least one supported primary architecture.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[!]: MUST Buildroot is not present
Note: Invalid buildroot found: %{_tmppath}/%{name}-%{version}-%{release}
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
$RPM_BUILD_ROOT)
Note: Clean would be needed if support for EPEL is required
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Each %files section contains %defattr if rpm 4.4
Note: Note: defattr macros not found. They would be needed for EPEL5
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
Note: rm -rf would be needed if support for EPEL5 is required
[!]: MUST If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
[ ]: MUST License field in the package spec file matches the actual license.
[x]: MUST License file installed when any subpackage combination is installed.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
names).
[x]: MUST Package meets the Packaging Guidelines.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generates any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[!]: MUST Rpmlint output is silent.
rpmlint gridftp-ifce-devel-2.1.3-2.i686.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
rpmlint gridftp-ifce-debuginfo-2.1.3-2.i686.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
rpmlint gridftp-ifce-2.1.3-2.src.rpm
gridftp-ifce.src: W: spelling-error Summary(en_US) lcgutil - guiltily
gridftp-ifce.src: W: spelling-error %description -l en_US globus - globs,
globes, glob's
gridftp-ifce.src: W: invalid-url Source0: gridftp-ifce-2.1.3.tar.gz
1 packages and 0 specfiles checked; 0 errors, 3 warnings.
rpmlint gridftp-ifce-2.1.3-2.i686.rpm
gridftp-ifce.i686: W: spelling-error Summary(en_US) lcgutil - guiltily
gridftp-ifce.i686: W: spelling-error %description -l en_US globus - globs,
globes, glob's
1 packages and 0 specfiles checked; 0 errors, 2 warnings.
[x]: MUST Sources used to build the package match the upstream source, as
provided in the spec URL.
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[-]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[-]: SHOULD If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to
include it.
[!]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
/usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
--requires).
[-]:
[Bug 716469] Review Request: rubygem-rhc - Openshift Express Client Tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=716469 Guillermo Gómez guillermo.go...@gmail.com changed: What|Removed |Added Resolution|ERRATA |CURRENTRELEASE Flag|fedora-cvs+ |fedora-cvs? --- Comment #16 from Guillermo Gómez guillermo.go...@gmail.com 2012-01-15 07:45:11 EST --- Package Change Request == Package Name: rubygem-rhc New Branches: el5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781823] New: Review Request: easyplay - An easy to use categories- and playlists-based music player
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: easyplay - An easy to use categories- and playlists-based music player https://bugzilla.redhat.com/show_bug.cgi?id=781823 Summary: Review Request: easyplay - An easy to use categories- and playlists-based music player Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: johannes.l...@googlemail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://hannes.fedorapeople.org/easyplay.spec SRPM URL: http://hannes.fedorapeople.org/easyplay-1.0.8-1.fc16.src.rpm Description: EP is ideal for small- to medium-sized music collections (hundreds to thousands of tracks). EP remembers the current category, every category's current playlist, and every playlist's current track. This makes it easy to listen to one playlist, then switch to another and then go back to the original playlist and continue. Just a small python package which should be fairly easy to review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781831] New: Review Request: python-nova-adminclient - Nova administration Python API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: python-nova-adminclient - Nova administration Python API https://bugzilla.redhat.com/show_bug.cgi?id=781831 Summary: Review Request: python-nova-adminclient - Nova administration Python API Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rbry...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://fedorapeople.org/~russellb/python-nova-adminclient/python-nova-adminclient.spec SRPM URL: http://fedorapeople.org/~russellb/python-nova-adminclient/python-nova-adminclient-0.1.8-1.fc16.src.rpm Description: Client library for administering OpenStack Nova -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771230] Review Request: gemrb - GemRB is a port of BioWare's Infinity Engine-based games, fine RPGs like Baldur's Gate, Baldur's Gate II: Shadows of Amn, Icewind Dale, Icewind Dale II and Planesc
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=771230 Mattia Verga mattia.ve...@tiscali.it changed: What|Removed |Added CC||mattia.ve...@tiscali.it --- Comment #2 from Mattia Verga mattia.ve...@tiscali.it 2012-01-15 11:02:51 EST --- URLs provided are not working. Also, you must fill the FE-NEEDSPONSOR flag into the blocks field and not just put it in the description. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=771254 Mattia Verga mattia.ve...@tiscali.it changed: What|Removed |Added CC||mattia.ve...@tiscali.it --- Comment #1 from Mattia Verga mattia.ve...@tiscali.it 2012-01-15 11:08:30 EST --- Seems to miss %post and %postun sections required for libraries: https://fedoraproject.org/wiki/Packaging:Guidelines#Shared_Libraries -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771462] Review Request: Wnotes - Graphical text notes for X Window System display
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=771462 Mattia Verga mattia.ve...@tiscali.it changed: What|Removed |Added CC||mattia.ve...@tiscali.it --- Comment #1 from Mattia Verga mattia.ve...@tiscali.it 2012-01-15 11:15:13 EST --- Hello Soumya. You must provide the 'Group' of the package. After that I'm offering to review this. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781831] Review Request: python-nova-adminclient - Nova administration Python API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=781831 Steven Dake sd...@redhat.com changed: What|Removed |Added CC||sd...@redhat.com AssignedTo|nob...@fedoraproject.org|sd...@redhat.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781831] Review Request: python-nova-adminclient - Nova administration Python API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=781831 --- Comment #1 from Steven Dake sd...@redhat.com 2012-01-15 12:31:14 EST --- Russell, I had originally intended to review your package, but you need a sponsor since your not part of the fedora packager group. I am not (yet) a sponsor. Regards -steve -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781831] Review Request: python-nova-adminclient - Nova administration Python API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=781831 Steven Dake sd...@redhat.com changed: What|Removed |Added AssignedTo|sd...@redhat.com|nob...@fedoraproject.org -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781831] Review Request: python-nova-adminclient - Nova administration Python API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=781831 Steven Dake sd...@redhat.com changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|sd...@redhat.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781831] Review Request: python-nova-adminclient - Nova administration Python API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=781831 --- Comment #2 from Steven Dake sd...@redhat.com 2012-01-15 12:38:36 EST --- Ignore comment #1. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781831] Review Request: python-nova-adminclient - Nova administration Python API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=781831 Steven Dake sd...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781831] Review Request: python-nova-adminclient - Nova administration Python API
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=781831
--- Comment #3 from Steven Dake sd...@redhat.com 2012-01-15 12:58:44 EST ---
[PASS] MUST: rpmlint must be run on the source rpm and all binary rpms the
build produces. The output should be posted in the review.[1]
[sdake@beast noarch]$ rpmlint python-nova-adminclient-0.1.8-1.fc16.noarch.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[sdake@beast noarch]$ rpmlint python-nova-adminclient*
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[PASS] MUST: The package must be named according to the Package Naming
Guidelines .
[PASS] MUST: The spec file name must match the base package %{name}, in the
format %{name}.spec unless your package has an exemption. [2] .
[PASS] MUST: The package must meet the Packaging Guidelines .
[PASS] MUST: The package must be licensed with a Fedora approved license and
meet the Licensing Guidelines .
[PASS] MUST: The License field in the package spec file must match the actual
license. [3]
[PASS] MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.[4]
[PASS] MUST: The spec file must be written in American English. [5]
[PASS] MUST: The spec file for the package MUST be legible. [6]
[PASS] MUST: The sources used to build the package must match the upstream
source, as provided in the spec URL. Reviewers should use md5sum for this task.
If no upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
[PASS] MUST: The package MUST successfully compile and build into binary rpms
on at least one primary architecture. [7]
[PASS] MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line. [8]
[PASS] MUST: All build dependencies must be listed in BuildRequires, except for
any that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.
[N/A] MUST: The spec file MUST handle locales properly. This is done by using
the %find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.[9]
[N/A] MUST: Every binary RPM package (or subpackage) which stores shared
library files (not just symlinks) in any of the dynamic linker's default paths,
must call ldconfig in %post and %postun. [10]
[N/A] MUST: Packages must NOT bundle copies of system libraries.[11]
[N/A] MUST: If the package is designed to be relocatable, the packager must
state this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker. [12]
[PASS] MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory. [13]
[PASS] MUST: A Fedora package must not list a file more than once in the spec
file's %files listings. (Notable exception: license texts in specific
situations)[14]
MUST: Permissions on files must be set properly. Executables should be set with
executable permissions, for example. [15]
[PASS] MUST: Each package must consistently use macros. [16]
[PASS] MUST: The package must contain code, or permissable content. [17]
[N/A] MUST: Large documentation files must go in a -doc subpackage. (The
definition of large is left up to the packager's best judgement, but is not
restricted to size. Large can refer to either size or quantity). [18]
[N/A] MUST: If a package includes something as %doc, it must not affect the
runtime of the application. To summarize: If it is in %doc, the program must
run properly if it is not present. [18]
[N/A] MUST: Header files must be in a -devel package. [19]
[N/A] MUST: Static libraries must be in a -static package. [20]
[N/A] MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), then library files that end in .so (without suffix) must go in
a -devel package. [19]
[N/A] MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency: Requires: %{name}%{?_isa} =
%{version}-%{release} [21]
[N/A] MUST: Packages must NOT contain any .la libtool archives, these must be
removed in the spec if they are built.[20]
[N/A] MUST: Packages containing GUI applications must include a %{name}.desktop
file, and that file must be properly installed with desktop-file-install in the
%install section. If you feel
[Bug 664912] Review Request: perl-HTML-TreeBuilder-LibXML - HTML::TreeBuilder and XPath compatible interface with libxml
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=664912 --- Comment #11 from Jóhann B. Guðmundsson johan...@hi.is 2012-01-15 12:56:23 EST --- Created attachment 555367 -- https://bugzilla.redhat.com/attachment.cgi?id=555367 The spec file containing Requires: perl(HTML::TreeBuilder::XPath) Added the reviewers reguested one liner to the spec file... Are you guys telling me that we have been holding back introducing RT-4 for ca 8 - 9 months due to a one liner? Ralf just remove it once requires generator has been fixed. Petr I assume that the attached spec file passes your requirement? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 772616] Review Request: epstool - A utility to create or extract preview images in EPS files
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=772616
Jussi Lehtola jussi.leht...@iki.fi changed:
What|Removed |Added
Flag||fedora-cvs?
--- Comment #4 from Jussi Lehtola jussi.leht...@iki.fi 2012-01-15 12:59:14
EST ---
Thanks for the review. I'll switch to using %{version} on the Source0 line
before git import.
New Package SCM Request
===
Package Name: epstool
Short Description: A utility to create or extract preview images in EPS files
Owners: jussilehtola
Branches: f15 f16 el5 el6
InitialCC:
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781831] Review Request: python-nova-adminclient - Nova administration Python API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=781831 --- Comment #4 from Steven Dake sd...@redhat.com 2012-01-15 13:04:30 EST --- Python review: From: http://fedoraproject.org/wiki/Packaging:Python To build a package containing python2 files, you need to have: BuildRequires: python2-devel [BLOCKER] python2-devel not a BR [BLOCKER] The python_sitelib definition is not needed - it is already defined by the build system. [PASS] Must: Python eggs must be built from source. They cannot simply drop an egg from upstream into the proper directory. (See prebuilt binaries Guidelines for details) [PASS] Must: Python eggs must not download any dependencies during the build process. [N/A] Must: When building a compat package, it must install using easy_install -m so it won't conflict with the main package. [N/A] Must: When building multiple versions (for a compat package) one of the packages must contain a default version that is usable via import MODULE with no prior setup. [PASS] Should: A package which is used by another package via an egg interface should provide egg info. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781831] Review Request: python-nova-adminclient - Nova administration Python API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=781831 --- Comment #5 from Steven Dake sd...@redhat.com 2012-01-15 13:06:01 EST --- I would recommend making the %description section a little more verbose or at least terminating it with a period. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781858] New: Review Request: openstack-tempo - OpenStack RESTful API to cron
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: openstack-tempo - OpenStack RESTful API to cron https://bugzilla.redhat.com/show_bug.cgi?id=781858 Summary: Review Request: openstack-tempo - OpenStack RESTful API to cron Product: Fedora Version: rawhide Platform: Unspecified OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: matt_dom...@dell.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://mdomsch.fedorapeople.org//openstack-tempo.spec SRPM URL: http://mdomsch.fedorapeople.org//openstack-tempo-0-0.1.20110909git940d9dba.fc17.src.rpm Description: OpenStack RESTful API to cron. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781858] Review Request: openstack-tempo - OpenStack RESTful API to cron
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=781858 --- Comment #1 from Matt Domsch matt_dom...@dell.com 2012-01-15 13:10:13 EST --- This package built on koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3697109 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 591190] Review Request: debhelper - Helper programs for debian/rules
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=591190 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added CC||rdie...@math.unl.edu Flag||needinfo?(supercyper1@gmail ||.com) --- Comment #14 from Rex Dieter rdie...@math.unl.edu 2012-01-15 13:30:08 EST --- Chen, as current reviewer, I haven't seen any feedback or comment here from you for awhile. are you still interested in helping to finish this review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711229] Review Request: ruby-spqr - easy QMF agent framework for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711229 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2012-01-15 14:57:44 EST --- ruby-spqr-0.3.5-1.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 702143] Review Request: wallaby - configuration service for Condor pools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=702143 --- Comment #19 from Fedora Update System upda...@fedoraproject.org 2012-01-15 14:59:17 EST --- wallaby-0.12.4-1.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711229] Review Request: ruby-spqr - easy QMF agent framework for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711229 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|ruby-spqr-0.3.5-1.fc15 |ruby-spqr-0.3.5-1.fc16 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2012-01-15 14:59:39 EST --- ruby-spqr-0.3.5-1.fc16 has been pushed to the Fedora 16 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 754004] Review Request: lv2-abGate - an LV2 Noise Gate plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=754004 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2012-01-15 14:59:51 EST --- Package lv2-abGate-1.1.3-2.fc16: * should fix your issue, * was pushed to the Fedora 16 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing lv2-abGate-1.1.3-2.fc16' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2012-0505/lv2-abGate-1.1.3-2.fc16 then log in and leave karma (feedback). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 768545] Review Request: blktap - blktap user space utilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=768545 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2012-01-15 15:02:08 EST --- blktap-2.0.90-3.git20111216.62de90d.fc16 has been pushed to the Fedora 16 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771485] Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=771485 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2012-01-15 15:05:21 EST --- libcxgb4-1.2.0-1.fc16 has been pushed to the Fedora 16 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 772582] Review Request: python-rest-client - A REST Client for use in python, using httplib2 and urllib2
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=772582 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2012-01-15 15:02:18 EST --- python-rest-client-0.3-1.fc16 has been pushed to the Fedora 16 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711230] Review Request: ruby-rhubarb - simple versioned object-graph persistence for ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711230 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|ruby-rhubarb-0.4.1-1.fc15 |ruby-rhubarb-0.4.1-1.fc16 --- Comment #13 from Fedora Update System upda...@fedoraproject.org 2012-01-15 15:02:44 EST --- ruby-rhubarb-0.4.1-1.fc16 has been pushed to the Fedora 16 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 718502] Review Request: Temperature.app - Window Maker applet which fetches local temperature information
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=718502 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #17 from Fedora Update System upda...@fedoraproject.org 2012-01-15 15:05:08 EST --- Temperature.app-1.5-8.fc16 has been pushed to the Fedora 16 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 772288] Review Request: infiniband-diags - various InfiniBand fabric test utilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=772288 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2012-01-15 15:08:27 EST --- infiniband-diags-1.5.12-1.fc15 has been pushed to the Fedora 15 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711230] Review Request: ruby-rhubarb - simple versioned object-graph persistence for ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711230 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|ruby-rhubarb-0.4.1-1.fc16 |ruby-rhubarb-0.4.1-1.fc15 --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2012-01-15 15:09:02 EST --- ruby-rhubarb-0.4.1-1.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771233] Review Request: rubygem-rack-protection - Ruby gem that protects against typical web attacks
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=771233 --- Comment #13 from Fedora Update System upda...@fedoraproject.org 2012-01-15 15:10:32 EST --- rubygem-rack-protection-1.2.0-2.fc16 has been pushed to the Fedora 16 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 727672] Review Request: metasploit-4.0 - The Metasploit Framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727672 Scott Tsai scottt...@gmail.com changed: What|Removed |Added CC||scottt...@gmail.com --- Comment #4 from Scott Tsai scottt...@gmail.com 2012-01-15 15:13:24 EST --- I'm getting HTTP 404 from the SPEC file link. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781870] New: Review Request: sord - A lightweight C library for storing RDF in memory
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: sord - A lightweight C library for storing RDF in memory https://bugzilla.redhat.com/show_bug.cgi?id=781870 Summary: Review Request: sord - A lightweight C library for storing RDF in memory Product: Fedora Version: 16 Platform: Unspecified OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: brendan.jones...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- sord is a lightweight C library for storing RDF data in memory. It depends on serd (bug 781685). These lightweight libraries are intended for use in real-time/embedded applications (such as the LV2 audio framework). SRPM: http://bsjones.fedorapeople.org/lv2/sord-0.5.0-1.fc16.src.rpm SPEC: http://bsjones.fedorapeople.org/lv2/sord.spec fedora16:~ $ rpmlint /home/bsjones/rpmbuild/SRPMS/sord-0.5.0-1.fc16.src.rpm /home/bsjones/rpmbuild/RPMS/x86_64/sord* 4 packages and 0 specfiles checked; 0 errors, 0 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 773419] Review Request: warmux - 2D turn-based artillery game
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=773419
Karel Volný kvo...@redhat.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
URL||http://www.wormux.org/
CC||kvo...@redhat.com
AssignedTo|nob...@fedoraproject.org|kvo...@redhat.com
Flag||fedora-review?
--- Comment #1 from Karel Volný kvo...@redhat.com 2012-01-15 15:11:54 EST ---
1) package renaming - FAIL
Obsoletes: wormux 0.9.2.1-7
Obsoletes: wormux-data 0.9.2.1-7
- this is ok (the latest available version is 0.9.2.1-6), but there is missing
Provides: wormux = %{version}-%{release}
Provides: wormux-data = %{version}-%{release}
so the package does NOT provide a clean update path, see
https://fedoraproject.org/wiki/Packaging:Guidelines#Renaming.2FReplacing_Existing_Packages
note especially Provides should be assumed to be deprecated and short lived
and removed in the distro release after the next one ... and the distro version
where it is planned to be dropped documented in a comment in the specfile -
such comment is also missing
2) must items
* MUST: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review.
$ rpmlint -v SPECS/warmux.spec SRPMS/warmux-11.04.1-1.fc16.src.rpm
RPMS/x86_64/warmux-11.04.1-1.fc16.x86_64.rpm
RPMS/noarch/warmux-data-11.04.1-1.fc16.noarch.rpm
SPECS/warmux.spec: I: checking-url
http://download.gna.org/warmux/warmux-11.04.1.tar.bz2 (timeout 10 seconds)
warmux.src: I: checking
warmux.src: W: spelling-error %description -l en_US toon - tun, too, ton
- http://en.wikipedia.org/wiki/Toon
warmux.src: W: spelling-error %description -l en_US firefox - Firefox,
firebox, fire fox
warmux.src: W: spelling-error %description -l en_US wilber - wilier, wilder,
Wilbert
- ok, these are names of the characters - not a single person name which would
be written with a capital letter
warmux.src: W: spelling-error %description -l en_US pre - per, ore, pee
- http://www.thefreedictionary.com/pre-eminently
warmux.src: I: checking-url http://www.wormux.org (timeout 10 seconds)
warmux.src: I: checking-url
http://download.gna.org/warmux/warmux-11.04.1.tar.bz2 (timeout 10 seconds)
warmux.x86_64: I: checking
warmux.x86_64: W: spelling-error %description -l en_US toon - tun, too, ton
warmux.x86_64: W: spelling-error %description -l en_US firefox - Firefox,
firebox, fire fox
warmux.x86_64: W: spelling-error %description -l en_US wilber - wilier,
wilder, Wilbert
warmux.x86_64: W: spelling-error %description -l en_US pre - per, ore, pee
- same as above
warmux.x86_64: I: checking-url http://www.wormux.org (timeout 10 seconds)
warmux.x86_64: W: obsolete-not-provided wormux
- see above the renaming review
warmux.x86_64: E: incorrect-fsf-address /usr/share/doc/warmux-11.04.1/COPYING
- please report upstream (note that it needs to be changed in source files too)
warmux.x86_64: W: no-manual-page-for-binary warmux-list-games
- not a problem, it is a helper utility
warmux-data.noarch: I: checking
warmux-data.noarch: I: checking-url http://www.wormux.org (timeout 10 seconds)
warmux-data.noarch: W: obsolete-not-provided wormux-data
- see above the renaming review
warmux-data.noarch: W: no-documentation
- it is part of the main package
3 packages and 1 specfiles checked; 1 errors, 12 warnings.
* MUST: The package must be named according to the Package Naming Guidelines.
- ok
* MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
- ok
* MUST: The package must meet the Packaging Guidelines.
- fail ... the specific problems are described under other items of this review
* MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines.
* MUST: The License field in the package spec file must match the actual
license.
- both okay
... except that xml_document.cpp itself doesn't mention libxml/BSD license -
report upstream?
* MUST: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s) for the
package must be included in %doc.
- ok, %doc AUTHORS COPYING ChangeLog
* MUST: The spec file must be written in American English.
- ok
* MUST: The spec file for the package MUST be legible.
- ok; it'be nice to have more consistent indentation ...
* MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL. Reviewers should use md5sum for this task.
- ok, 26ff65c43a9bb61a3f0529c98b943e35 matches my download (sum not provided by
upstream)
[Bug 772217] Review Request: libyuv - YUV conversion and scaling functionality library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=772217 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2012-01-15 15:11:02 EST --- libyuv-0-0.3.20120109svn128.fc16 has been pushed to the Fedora 16 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 702143] Review Request: wallaby - configuration service for Condor pools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=702143 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|wallaby-0.12.4-1.fc15 |wallaby-0.12.4-1.fc16 --- Comment #20 from Fedora Update System upda...@fedoraproject.org 2012-01-15 15:05:38 EST --- wallaby-0.12.4-1.fc16 has been pushed to the Fedora 16 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 658234] Review Request: gdal-grass - Standalone GRASS 6 Drivers for GDAL and OGR
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=658234 Volker Fröhlich volke...@gmx.at changed: What|Removed |Added Status|ASSIGNED|NEW -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 766154] Review Request: lv2-kn0ck0ut - An LV2 spectral subtraction plugin (linux audio)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=766154 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2012-01-15 15:06:27 EST --- Package lv2-kn0ck0ut-1.1-0.3.git60421a3.fc16: * should fix your issue, * was pushed to the Fedora 16 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing lv2-kn0ck0ut-1.1-0.3.git60421a3.fc16' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2012-0449/lv2-kn0ck0ut-1.1-0.3.git60421a3.fc16 then log in and leave karma (feedback). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 769450] Review Request: gap-pkg-tomlib - GAP Table of Marks package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=769450 Scott Tsai scottt...@gmail.com changed: What|Removed |Added CC||scottt...@gmail.com AssignedTo|nob...@fedoraproject.org|scottt...@gmail.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 675234] Review Request: duply - Wrapper for duplicity
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=675234 Mario Santagiuliana fed...@marionline.it changed: What|Removed |Added CC||fed...@marionline.it --- Comment #8 from Mario Santagiuliana fed...@marionline.it 2012-01-15 16:22:39 EST --- I never review a package, but this could be my first...could I help you? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 768174] Review Request: srm-ifce - Storage Resources Manager client implementation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=768174
--- Comment #17 from adev ade...@gmail.com 2012-01-15 16:29:53 EST ---
[!]: MUST Buildroot is not present
Note: Invalid buildroot found: %{_tmppath}/%{name}-%{version}-%{release}
see http://fedoraproject.org/wiki/EPEL/GuidelinesAndPolicies#BuildRoot_tag
- Corrected set to %(mktemp -ud
%{_tmppath}/%{name}-%{version}-%{release}-XX)
[!]: MUST If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
- Corrected, %doc added to the LICENSE path
[!]: SHOULD Dist tag is present.
- tag Added
[!]: MUST License field in the package spec file matches the actual license.
Some code is licensed under the BSD license (see under cmake/module/)
- cmake is under BSD License and normally not directly linked with the script
file.
Spec URL : http://firwen.org/home/specs/srm-ifce.spec
SRPM URL : http://firwen.org/home/specs/srm-ifce-1.12-5.el5.centos.src.rpm
rpmlint :
srm-ifce.src: W: invalid-url Source0: srm-ifce-1.12.tar.gz
srm-ifce.x86_64: W: incoherent-version-in-changelog 1.12-5
['1.12-5.el5.centos', '1.12-5.centos']
4 packages and 0 specfiles checked; 0 errors, 2 warnings.
Koji : http://koji.fedoraproject.org/koji/taskinfo?taskID=3699592
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 768174] Review Request: srm-ifce - Storage Resources Manager client implementation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=768174
--- Comment #19 from adev ade...@gmail.com 2012-01-15 16:39:31 EST ---
[!]: MUST Buildroot is not present
Note: Invalid buildroot found:
%{_tmppath}/%{name}-%{version}-%{release}
see
http://fedoraproject.org/wiki/EPEL/GuidelinesAndPolicies#BuildRoot_tag
- Corrected set to %(mktemp -ud
%{_tmppath}/%{name}-%{version}-%{release}-XX)
[!]: SHOULD Dist tag is present.
- tag Added
[!]: MUST License field in the package spec file matches the actual
license.
Some code is licensed under the BSD license (see under cmake/module/)
- cmake is under BSD License and normally not directly linked with the
script
file.
Spec URL : http://firwen.org/home/specs/srm-ifce.spec
SRPM URL : http://firwen.org/home/specs/srm-ifce-1.12-5.el5.centos.src.rpm
rpmlint :
srm-ifce.src: W: invalid-url Source0: srm-ifce-1.12.tar.gz
srm-ifce.x86_64: W: incoherent-version-in-changelog 1.12-5
['1.12-5.el5.centos', '1.12-5.centos']
4 packages and 0 specfiles checked; 0 errors, 2 warnings.
Koji : http://koji.fedoraproject.org/koji/taskinfo?taskID=3699592
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 768174] Review Request: srm-ifce - Storage Resources Manager client implementation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=768174
--- Comment #18 from Pierre-YvesChibon pin...@pingoured.fr 2012-01-15
16:36:29 EST ---
(In reply to comment #17)
[!]: MUST If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
- Corrected, %doc added to the LICENSE path
This is actually a mistake on my side, I corrected the output above but missed
in the Issues section. Since the LICENSE file is in the %{_docdir} it does
not have to be tagged as %doc.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771070] Review Request: nwipe - Securely erase disks using a variety of recognized methods
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=771070 Michal Ambroz re...@seznam.cz changed: What|Removed |Added Flag||fedora-cvs? --- Comment #22 from Michal Ambroz re...@seznam.cz 2012-01-15 16:47:34 EST --- New Package SCM Request === Package Name: nwipe Short Description: Securely erase disks using a variety of recognized methods Owners: rebus Branches: f17 f16 el6 el5 el4 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 239087] Review Request: perl-Nmap-Parser - Parse nmap scan data with perl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=239087 --- Comment #11 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:04:52 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 239157] Review Request: perl-Net-Pcap - Interface to pcap LBL packet capture library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=239157 --- Comment #10 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:05:58 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 476160] Review Request: perl-Directory-Scratch - Self-cleaning scratch space for tests
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=476160 --- Comment #6 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:07:19 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 481681] Review Request: perl-MooseX-ClassAttribute - Declare class attributes Moose-style
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481681 --- Comment #5 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:08:25 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 482856] Review Request: perl-MooseX-POE - Moose wrapper around a POE::Session
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482856 --- Comment #14 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:09:50 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 483258] Review Request: perl-MooseX-LogDispatch - Logging Role for Moose
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483258 --- Comment #5 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:12:28 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 482863] Review Request: perl-MooseX-Daemonize - Role for daemonizing your Moose based application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482863 --- Comment #11 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:11:27 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 483459] Review Request: perl-MooseX-Iterator - Iterate over collections
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483459 --- Comment #12 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:14:00 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 483461] Review Request: perl-MooseX-Param - Simple role to provide a standard param method
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483461 --- Comment #5 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:15:39 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 483933] Review Request: perl-autobox - Call methods on native types
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483933 --- Comment #6 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:19:57 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 483569] Review Request: perl-MooseX-Types-Set-Object - Set::Object type with coercions and stuff
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483569 --- Comment #5 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:17:50 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 483649] Review Request: perl-MooseX-Workers - Provides a simple sub-process management for asynchronous tasks
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483649 --- Comment #5 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:18:21 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781260] Review Request: leechcraft - A Free Open Source Cross-Platform Modular Internet-Client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=781260 --- Comment #6 from Minh Ngo nlmin...@gmail.com 2012-01-15 17:18:00 EST --- 0.4.98-2 SPEC: https://raw.github.com/Ignotus/leechcraft-fedora/2f22b10881f0ad359f8571c25a234113c1289078/leechcraft.spec SRPM: https://github.com/Ignotus/leechcraft-fedora/blob/2f22b10881f0ad359f8571c25a234113c1289078/leechcraft-0.4.98-2.fc16.src.rpm?raw=true Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3701112 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 483557] Review Request: perl-MooseX-SimpleConfig - Moose role for setting attributes from a simple configfile
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483557 --- Comment #5 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:17:27 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781858] Review Request: openstack-tempo - OpenStack RESTful API to cron
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=781858 Peter Robinson pbrobin...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||pbrobin...@gmail.com AssignedTo|nob...@fedoraproject.org|pbrobin...@gmail.com Flag||fedora-review? --- Comment #2 from Peter Robinson pbrobin...@gmail.com 2012-01-15 17:16:23 EST --- I'll review -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 485967] Review Request: perl-MooseX-LazyLogDispatch - Logging Role for Moose
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=485967 --- Comment #5 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:21:01 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 485154] Review Request: perl-MooseX-Role-Cmd - Wrap system command binaries the Moose way
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=485154 --- Comment #5 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:20:32 EST --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 716469] Review Request: rubygem-rhc - Openshift Express Client Tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=716469 --- Comment #17 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:21:42 EST --- No owner specified. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 737286] Review Request: salt - A parallel remote execution system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=737286 --- Comment #20 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:22:25 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 772709] Review Request: TeXamator - Helping you making your exercise sheets
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=772709 --- Comment #16 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:25:29 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 772616] Review Request: epstool - A utility to create or extract preview images in EPS files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=772616 --- Comment #5 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:24:58 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771070] Review Request: nwipe - Securely erase disks using a variety of recognized methods
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=771070 --- Comment #23 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:24:26 EST --- Git done (by process-git-requests). Ivan, please take ownership of review BZs. Michal, since we're not branched yet, you don't need to request f17 as it ==devel. Also, be aware that EL-4 is nearly EOL. Thanks! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 773502] Review Request: perl-Spreadsheet-read - universal API to read any spreadsheet
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=773502 Michael Schwendt mschwe...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|mschwe...@gmail.com Flag||fedora-review+ --- Comment #3 from Michael Schwendt mschwe...@gmail.com 2012-01-15 17:24:47 EST --- AUTOMATED_TESTING=1 Confirmed. No questions about the example scripts are asked anymore. Requires: perl(IO::Scalar) +Requires: perl(Test::More) = 0.98 http://fedoraproject.org/wiki/Packaging:Guidelines#Explicit_Requires Even if it may seem that it is specific to libraries only, it covers explicit Requires in general. Comments in the spec file, which explain the explicit Requires, are good packaging practice. * Please keep the spec %changelog accurate. Your updates to the spec invalidated the only %changelog entry about cpanspec usage. * A few more packages for the tests are available in Fedora already, btw, just in case you consider the tests worthwhile: perl-Text-CSV perl-Text-CSV_XS perl-Test-Pod perl-Test-Pod-Coverage perl-Spreadsheet-ParseExcel t/00_pod.t . ok t/01_pod.t . ok t/10_basics.t .. ok t/11_call.t ok t/20_csv.t . # Parser: Text::CSV_XS-0.82 t/20_csv.t . ok t/21_csv.t . ok t/22_csv.t . ok t/23_csv.t . ok t/24_csv.t . ok t/30_xls.t . # Parser: Spreadsheet::ParseExcel-0.59 t/30_xls.t . ok t/31_clr.t . ok t/32_fmt.t . ok t/33_misc.t ok t/34_dates.t ... ok t/35_perc.t ok t/36_xls.t . ok t/40_sxc.t . skipped: No SXC parser found t/45_ods.t . skipped: No SXC parser found t/46_clr.t . skipped: No OpenOffice ODS parser found t/50_sc.t .. # Parser: Spreadsheet::Read-0.45 t/50_sc.t .. ok t/51_sc.t .. ok t/60_xlsx.t skipped: No M$-Excel parser found t/61_clr.t . skipped: No M$-Excel parser found t/62_fmt.t . skipped: No M$-Excel parser found t/63_misc.t skipped: No M$-Excel parser found t/64_dates.t ... skipped: No M$-Excel parser found t/65_perc.t skipped: No M$-Excel parser found All tests successful. * You get an 'APPROVED' from me, provided that you update the %changelog within pkg git and add comments to the explicit Requires. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 591190] Review Request: debhelper - Helper programs for debian/rules
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=591190 --- Comment #15 from Oron Peled o...@actcom.co.il 2012-01-15 17:28:35 EST --- Continuing comment 14 above, is Jeroen van Meeuwen still interested? For the record, since I want to have the debian build tools on Fedora, I'm willing to help with any related package -- this means I'll be happy to maintain/co-maintain/review (whatever helps most) this package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781858] Review Request: openstack-tempo - OpenStack RESTful API to cron
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=781858
Peter Robinson pbrobin...@gmail.com changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #3 from Peter Robinson pbrobin...@gmail.com 2012-01-15 17:27:59
EST ---
Looks fine. APPROVED
+ rpmlint output
$ rpmlint openstack-tempo.spec
openstack-tempo-0-0.1.20110909git940d9dba.fc16.src.rpm
openstack-tempo-0-0.1.20110909git940d9dba.fc16.noarch.rpm
openstack-tempo.spec: W: invalid-url Source0: openstack-tempo-940d9dba.tar.bz2
openstack-tempo.src: W: spelling-error Summary(en_US) cron - corn, con, crone
openstack-tempo.src: W: spelling-error %description -l en_US cron - corn, con,
crone
openstack-tempo.src: W: invalid-url Source0: openstack-tempo-940d9dba.tar.bz2
openstack-tempo.noarch: W: spelling-error Summary(en_US) cron - corn, con,
crone
openstack-tempo.noarch: W: spelling-error %description -l en_US cron - corn,
con, crone
openstack-tempo.noarch: W: no-manual-page-for-binary tempo-cron-snapshot
openstack-tempo.noarch: W: no-manual-page-for-binary tempo-backup
openstack-tempo.noarch: W: no-manual-page-for-binary nova-snapshot.sh
openstack-tempo.noarch: W: no-manual-page-for-binary tempo-api
openstack-tempo.noarch: W: no-manual-page-for-binary nova-backup.sh
openstack-tempo.noarch: W: no-manual-page-for-binary tempo-snapshot
openstack-tempo.noarch: W: no-manual-page-for-binary tempo-cron-backup
2 packages and 1 specfiles checked; 0 errors, 13 warnings.
+ package name satisfies the packaging naming guidelines
+ specfile name matches the package base name
+ package should satisfy packaging guidelines
+ license meets guidelines and is acceptable to Fedora
+ license matches the actual package license
+ latest version packaged
+ %doc includes license file
+ spec file written in American English
+ spec file is legible
+ upstream sources match sources in the srpm
f8bee95fd00c14f5d4fa3d0610a5167b openstack-tempo-940d9dba.tar.bz2
+ package successfully builds on at least one architecture
+ tested using koji scratch build
+ BuildRequires list all build dependencies
n/a %find_lang instead of %{_datadir}/locale/*
n/a binary RPM with shared library files must call ldconfig in %post and
%postun+ does not use Prefix: /usr
+ package owns all directories it creates
n/a no duplicate files in %files
+ Package perserves timestamps on install
+ Permissions on files must be set properly
+ %defattr line
+ consistent use of macros
+ package must contain code or permissible content
n/a large documentation files should go in -doc subpackage
+ files marked %doc should not affect package runtime
n/a header files should be in -devel
n/a static libraries should be in -static
n/a packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'
n/a libfoo.so must go in -devel
n/a devel must require the fully versioned base
+ packages should not contain libtool .la files
n/a packages containing GUI apps must include %{name}.desktop file
+ packages must not own files or directories owned by other packages
+ filenames must be valid UTF-8
Optional:
n/a if there is no license file, packager should query upstream to include it
n/a translations of description and summary for non-English languages, if
available
+ reviewer should build the package in mock/koji
n/a the package should build into binary RPMs on all supported architectures
n/a review should test the package functions as described
+ scriptlets should be sane
n/a non -devel packages should require fully versioned base
n/a pkgconfig files should go in -devel
+ shouldn't have file dependencies outside /etc /bin /sbin /usr/bin or
/usr/sbin
- Package should have man files
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781685] Review Request: serd - A lightweight C library for RDF syntax
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=781685 --- Comment #8 from Jon Ciesla limburg...@gmail.com 2012-01-15 17:25:59 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781260] Review Request: leechcraft - A Free Open Source Cross-Platform Modular Internet-Client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=781260 --- Comment #7 from Minh Ngo nlmin...@gmail.com 2012-01-15 17:34:34 EST --- leechcraft-full metapackage fix SPEC: https://raw.github.com/Ignotus/leechcraft-fedora/f6f532d51edb582e7949a9d7d16b7aa9ab056209/leechcraft.spec SRPM: https://github.com/Ignotus/leechcraft-fedora/blob/f6f532d51edb582e7949a9d7d16b7aa9ab056209/leechcraft-0.4.98-2.fc16.src.rpm?raw=true -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771254] Review Request: libva-vdpau-driver - HW video decode support for VDPAU platforms
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=771254 --- Comment #2 from Nicolas Chauvet (kwizart) kwiz...@gmail.com 2012-01-15 17:48:16 EST --- Hello Mattia and thanks for your interest in this review. This is a good remark, but actually this is a special case here. This library is not intended to be used 'system wide' but only from a dedicated library (libva), which last is installed with ldconfig. So libva is expected to 'dlopen' the right backend %name_drv_video.so from the special directory ( %_libdir/dri ) given the current driver used by the Xorg server. For the same reason, this is usually expected to have the shared object un-versioned in this case. So to sum-up, you can consider this library as a module or a plugin like the one provided by mesa-dri-drivers from the mesa case. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 768310] Review Request: gridftp-ifce - GridFTP abstraction layer for wlcg.
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=768310
--- Comment #5 from adev ade...@gmail.com 2012-01-15 17:53:52 EST ---
Spec URL : http://firwen.org/home/specs/gridftp-ifce.spec
SRPM URL : http://firwen.org/home/specs/gridftp-ifce-2.1.3-3.src.rpm
rpmlint
4 packages and 0 specfiles checked; 0 errors, 0 warnings.
Koji : http://koji.fedoraproject.org/koji/taskinfo?taskID=3702540
[!]: MUST Buildroot is not present
Note: Invalid buildroot found: %{_tmppath}/%{name}-%{version}-%{release}
- corrected
[!]: MUST If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
- done with the %{_docdir} macro
Also, seems you seem to be close to upstream, I would strongly encourage him to
actually do tarball release.
- done
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 781884] New: Review Request: rubygem-raindrops - Real-time stats for preforking Rack servers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: rubygem-raindrops - Real-time stats for preforking Rack servers https://bugzilla.redhat.com/show_bug.cgi?id=781884 Summary: Review Request: rubygem-raindrops - Real-time stats for preforking Rack servers Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: guillermo.go...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://gomix.fedorapeople.org/rubygem-raindrops/rubygem-raindrops.spec SRPM URL: http://gomix.fedorapeople.org/rubygem-raindrops/rubygem-raindrops-0.8.0-1.fc16.src.rpm Description: Raindrops is a real-time stats toolkit to show statistics for Rack HTTP servers. It is designed for preforking servers such as Rainbows! and Unicorn, but should support any Rack HTTP server under Ruby 1.9, 1.8 and Rubinius on platforms supporting POSIX shared memory. It may also be used as a generic scoreboard for sharing atomic counters across multiple proc $ rpmlint -v SPECS/rubygem-raindrops.spec SPECS/rubygem-raindrops.spec: I: checking-url http://rubygems.org/gems/raindrops-0.8.0.gem (timeout 10 seconds) 0 packages and 1 specfiles checked; 0 errors, 0 warnings.esses. $ rpmlint -v RPMS/x86_64/rubygem-raindrops-0.8.0-1.fc16.x86_64.rpm rubygem-raindrops.x86_64: I: checking rubygem-raindrops.x86_64: W: spelling-error Summary(en_US) preforking - preforming, preferring rubygem-raindrops.x86_64: W: spelling-error %description -l en_US preforking - preforming, preferring rubygem-raindrops.x86_64: I: checking-url http://raindrops.bogomips.org/ (timeout 10 seconds) rubygem-raindrops.x86_64: W: no-soname /usr/lib/ruby/gems/1.8/gems/raindrops-0.8.0/lib/raindrops_ext.so 1 packages and 0 specfiles checked; 0 errors, 3 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 770134] Review Request: trac-workflowadmin-plugin - Web interface for workflow administration for Trac
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=770134 --- Comment #8 from Fedora Update System upda...@fedoraproject.org 2012-01-15 18:20:48 EST --- trac-workflowadmin-plugin-0.12-20111223svn11062.el6 has been pushed to the Fedora EPEL 6 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 770034] Review Request: drupal7-rules - It allows site administrators to define conditionally executed actions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=770034 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|drupal7-rules-2.0-1.fc16|drupal7-rules-2.0-1.el5 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2012-01-15 18:25:26 EST --- drupal7-rules-2.0-1.el5 has been pushed to the Fedora EPEL 5 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 770129] Review Request: drupal7-workbench - Provides overall improvements for managing content
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=770129 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|drupal7-workbench-1.1-1.fc1 |drupal7-workbench-1.1-1.el5 |5 | --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2012-01-15 18:21:24 EST --- drupal7-workbench-1.1-1.el5 has been pushed to the Fedora EPEL 5 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 770136] Review Request: drupal7-mediawiki_api - Provides a MediaWikisyntax to html conversion input filter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=770136 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|drupal7-mediawiki_api-1.0-0 |drupal7-mediawiki_api-1.0-0 |.1.alpha1.fc16 |.1.alpha1.el6 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2012-01-15 18:23:52 EST --- drupal7-mediawiki_api-1.0-0.1.alpha1.el6 has been pushed to the Fedora EPEL 6 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
