[Bug 698264] Review Request: perl-TryCatch - First class try catch semantics for Perl, without source filters
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=698264 --- Comment #4 from Marcela Mašláňová mmasl...@redhat.com 2011-04-28 03:02:23 EDT --- (In reply to comment #3) (In reply to comment #2) Rawhide mirrors are lazy. I built requirement week ago. What's the required package, actually? Is it already available from the F15 repos? Otherwise, I should feed Koji again with the package *and* the requirement. It's perl-B-Hooks-OP-PPAddr, which has build here: http://koji.fedoraproject.org/koji/buildinfo?buildID=240288 I don't plan push into F-15. -- 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 700335] New: Review Request: ant-antlr3 - Antlr3 task for Ant
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: ant-antlr3 - Antlr3 task for Ant https://bugzilla.redhat.com/show_bug.cgi?id=700335 Summary: Review Request: ant-antlr3 - Antlr3 task for Ant Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: kk...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://kklic.fedorapeople.org/ant-antlr3.spec SRPM URL: http://kklic.fedorapeople.org/ant-antlr3-20110110-1.fc14.src.rpm Description: Antlr3 task for Ant $ rpmlint ant-antlr3-20110110-1.fc14.src.rpm ant-antlr3.src: W: invalid-url Source0: antlr3-task.zip 1 packages and 0 specfiles checked; 0 errors, 1 warnings. - URL will be included as soon as bug #699529 is fixed in Fedora $ rpmlint ./ant-antlr3-20110110-1.fc14.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint ant-antlr3-javadoc-20110110-1.fc14.noarch.rpm ant-antlr3-javadoc.noarch: W: spelling-error Summary(en_US) Javadocs - Java docs, Java-docs, Javanese 1 packages and 0 specfiles checked; 0 errors, 1 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 700344] New: Review Request: perl-Authen-CAS-Client - Interface for authentication via JA-SIG's Central Authentication Service
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-Authen-CAS-Client - Interface for authentication via JA-SIG's Central Authentication Service https://bugzilla.redhat.com/show_bug.cgi?id=700344 Summary: Review Request: perl-Authen-CAS-Client - Interface for authentication via JA-SIG's Central Authentication Service Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: vano...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://dl.dropbox.com/u/8648526/perl-Authen-CAS-Client.spec SRPM URL: http://dl.dropbox.com/u/8648526/perl-Authen-CAS-Client-0.05-1.fc14.src.rpm Description: The Authen::CAS::Client module provides a simple interface for authenticating users using JA-SIG's CAS protocol. Both CAS v1.0 and v2.0 are supported. This is one of the requirements for Koha. Since I have not yet got any packages in the fedora repository, I am looking for a sponsor -- 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 700345] New: Review Request: com.inamik.utils.tableformatter - A set of Java classes to print text in tabulated form
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: com.inamik.utils.tableformatter - A set of Java classes to print text in tabulated form https://bugzilla.redhat.com/show_bug.cgi?id=700345 Summary: Review Request: com.inamik.utils.tableformatter - A set of Java classes to print text in tabulated form Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: kk...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://kklic.fedorapeople.org/com.inamik.utils.tableformatter.spec SRPM URL: http://kklic.fedorapeople.org/com.inamik.utils.tableformatter-0.96.2-1.fc14.src.rpm Description: A set of Java classes to print text in tabulated form. With these classes you can organize 'cells' of data into rows and columns. Each cell can contain multiple lines of text and may specify a horizontal alignment (left, center, right) and a vertical alignment (top, center, bottom). When you are ready to print the data, these classes will ensure that all cells in a given column have the same width and all cells in a given row have the same height. $ rpmlint ./com.inamik.utils.tableformatter-0.96.2-1.fc14.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint ./com.inamik.utils.tableformatter-javadoc-0.96.2-1.fc14.noarch.rpm com.inamik.utils.tableformatter-javadoc.noarch: W: spelling-error Summary(en_US) tableformatter - table formatter, table-formatter, formatter com.inamik.utils.tableformatter-javadoc.noarch: W: spelling-error %description -l en_US tableformatter - table formatter, table-formatter, formatter 1 packages and 0 specfiles checked; 0 errors, 2 warnings. - tableformatter is in the package name $ rpmlint com.inamik.utils.tableformatter-0.96.2-1.fc14.src.rpm com.inamik.utils.tableformatter.src: W: invalid-url Source0: com.inamik.utils.tableformatter-0.96.2-src.tar.gz 1 packages and 0 specfiles checked; 0 errors, 1 warnings. - Source file cannot be downloaded directly from the website. -- 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 700346] Review Request: perl-Barcode-Code128 - Generate CODE 128 bar codes
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=700346 Nicholas van Oudtshoorn vano...@gmail.com changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- 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 700344] Review Request: perl-Authen-CAS-Client - Interface for authentication via JA-SIG's Central Authentication Service
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=700344 Nicholas van Oudtshoorn vano...@gmail.com changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- 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 700346] New: Review Request: perl-Barcode-Code128 - Generate CODE 128 bar codes
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-Barcode-Code128 - Generate CODE 128 bar codes https://bugzilla.redhat.com/show_bug.cgi?id=700346 Summary: Review Request: perl-Barcode-Code128 - Generate CODE 128 bar codes Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: vano...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://dl.dropbox.com/u/8648526/perl-Barcode-Code128.spec SRPM URL: http://dl.dropbox.com/u/8648526/perl-Barcode-Code128-2.01-1.fc14.src.rpm Description: Barcode::Code128 generates bar codes using the CODE 128 symbology. It can generate images in PNG or GIF format using the GD package, or it can generate a text string representing the barcode that you can render using some other technology if desired. This is one of the requirements for the Koha Library Management System. Since I have not yet got any packages in the fedora repository, I am looking for a sponsor. -- 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 700348] New: Review Request: mingw32-glib-networking - MinGW Windows glib-networking library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: mingw32-glib-networking - MinGW Windows glib-networking library https://bugzilla.redhat.com/show_bug.cgi?id=700348 Summary: Review Request: mingw32-glib-networking - MinGW Windows glib-networking library Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: ka...@smartlink.ee QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://kalev.fedorapeople.org/mingw32-glib-networking.spec SRPM URL: http://kalev.fedorapeople.org/mingw32-glib-networking-2.28.6.1-1.fc15.src.rpm Description: This package contains modules that extend the networking support in GIO. -- 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 700347] New: Review Request: perl-Biblio-EndnoteStyle - Reerence Formatting using Endnote-like templates
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-Biblio-EndnoteStyle - Reerence Formatting using Endnote-like templates https://bugzilla.redhat.com/show_bug.cgi?id=700347 Summary: Review Request: perl-Biblio-EndnoteStyle - Reerence Formatting using Endnote-like templates Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: vano...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://dl.dropbox.com/u/8648526/perl-Biblio-EndnoteStyle.spec SRPM URL: http://dl.dropbox.com/u/8648526/perl-Biblio-EndnoteStyle-0.05-1.fc14.src.rpm Description: This small module provides a way of formatting bibliographic references using style templates similar to those used by the popular reference management software Endnote. The API is embarrassingly simple: a formatter object is made using the class's constructor, the new() method; format() may then be repeatedly called on this object, using the same or different templates. This is one of the requirements for the Koha Library Management System. Since I have not yet got any packages in the fedora repository, I am looking for a sponsor. -- 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 700347] Review Request: perl-Biblio-EndnoteStyle - Reerence Formatting using Endnote-like templates
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=700347 Nicholas van Oudtshoorn vano...@gmail.com changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- 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 700349] New: Review Request: perl-CGI-Session-Driver-memcached - CGI::Session driver for memcached
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-CGI-Session-Driver-memcached - CGI::Session driver for memcached https://bugzilla.redhat.com/show_bug.cgi?id=700349 Summary: Review Request: perl-CGI-Session-Driver-memcached - CGI::Session driver for memcached Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: vano...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://dl.dropbox.com/u/8648526/perl-Biblio-EndnoteStyle.spec SRPM URL: http://dl.dropbox.com/u/8648526/perl-Biblio-EndnoteStyle-0.05-1.fc14.src.rpm Description: This small module stores session data into memcached. This is one of the requirements for the Koha Library Management System. Since I have not yet got any packages in the fedora repository, I am looking for a sponsor. -- 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 700350] New: Review Request: perl-CGI-Session-Driver-memcached - CGI::Session driver for memcached
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-CGI-Session-Driver-memcached - CGI::Session driver for memcached https://bugzilla.redhat.com/show_bug.cgi?id=700350 Summary: Review Request: perl-CGI-Session-Driver-memcached - CGI::Session driver for memcached Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: vano...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://dl.dropbox.com/u/8648526/perl-CGI-Session-Driver-memcached.spec SRPM URL: http://dl.dropbox.com/u/8648526/perl-CGI-Session-Driver-memcached-0.04-1.fc14.src.rpm Description: This small module stores session data into memcached. This is one of the requirements for the Koha Library Management System. Since I have not yet got any packages in the fedora repository, I am looking for a sponsor. -- 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 700353] Review Request: perl-HTTP-OAI - API for the OAI-PMH
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=700353 Nicholas van Oudtshoorn vano...@gmail.com changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- 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 700350] Review Request: perl-CGI-Session-Driver-memcached - CGI::Session driver for memcached
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=700350 Nicholas van Oudtshoorn vano...@gmail.com changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- 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 700352] Review Request: perl-DublinCore-Record - Container for Dublin Core metadata elements
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=700352 Nicholas van Oudtshoorn vano...@gmail.com changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- 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 700352] New: Review Request: perl-DublinCore-Record - Container for Dublin Core metadata elements
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-DublinCore-Record - Container for Dublin Core metadata elements https://bugzilla.redhat.com/show_bug.cgi?id=700352 Summary: Review Request: perl-DublinCore-Record - Container for Dublin Core metadata elements Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: vano...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://dl.dropbox.com/u/8648526/perl-DublinCore-Record.spec SRPM URL: http://dl.dropbox.com/u/8648526/perl-DublinCore-Record-0.03-1.fc14.src.rpm Description: DublinCore::Record is an abstract class for manipulating DublinCore meta-data. The Dublin Core is a small set of meta-data elements for describing information resources. For more information on embedding DublinCore in HTML see RFC 2731 http://www.ietf.org/rfc/rfc2731 or http://www.dublincore.org/documents/dces/ This is one of the requirements for the Koha Library Management System. Since I have not yet got any packages in the fedora repository, I am looking for a sponsor. -- 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 693131] Review Request: sudoku-savant - Solve and generate sudoku puzzles through logical means
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=693131
Martin Gieseking martin.giesek...@uos.de changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|nob...@fedoraproject.org|martin.giesek...@uos.de
Flag||fedora-review?
--- Comment #5 from Martin Gieseking martin.giesek...@uos.de 2011-04-28
04:05:38 EDT ---
OK, the package is almost ready. I had a deeper look into the sources and
wasn't able to find any hint about the GPL version intended. Even on the
website there's no further information. Thus, the package is licensed under
GPL+ since we can't rely on the version number given in COPYING:
http://fedoraproject.org/wiki/Licensing:FAQ#How_do_I_figure_out_what_version_of_the_GPL.2FLGPL_my_package_is_under.3F
Please also send the patch to the developer (if not yet done), and add a
comment with the link pointing to the upstream bug ticket.
(http://fedoraproject.org/wiki/PackagingGuidelines#All_patches_should_have_an_upstream_bug_link_or_comment)
$ rpmlint /var/lib/mock/fedora-14-x86_64/result/*.rpm
sudoku-savant.x86_64: W: no-manual-page-for-binary sudoku-savant
3 packages and 0 specfiles checked; 0 errors, 1 warnings.
-
key:
[+] OK
[.] OK, not applicable
[X] needs work
-
[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}.
[+] MUST: The package must meet the Packaging Guidelines.
[+] MUST: The package must be licensed with a Fedora approved license.
- COPYING contains GPLv3 license text
- src/aboutbox.cpp just mentions GPL but no version
- no further copyright information in the sources nor on
the project website
= GPL+
[X] MUST: The License field in the package spec file must match the actual
license.
- please change the License tag to GPL+ and add a short comment to it
that the sources/docs lack any concrete licensing information
[+] MUST: The file containing the text of the license(s) for the package must
be included in %doc.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source.
$ md5sum sudoku-savant-1.3.tar.bz2*
668e77f35c6e8106da860a88c71f3c73 sudoku-savant-1.3.tar.bz2
668e77f35c6e8106da860a88c71f3c73 sudoku-savant-1.3.tar.bz2.1
[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
[.] MUST: If the package does not successfully compile, build or work on an
architecture, ...
[+] MUST: All build dependencies must be listed in BuildRequires.
[+] MUST: When compiling C, C++, or Fortran files, %{optflags} must be applied.
[+] MUST: The spec file MUST handle locales properly.
[.] MUST: If a package installs files below %{_datadir}/icons, the icon cache
must be updated.
[.] MUST: Packages storing shared library files (not just symlinks) must call
ldconfig in %post and %postun.
[+] MUST: Packages must NOT bundle copies of system libraries.
[.] MUST: If the package is designed to be relocatable, ...
[+] MUST: A package must own all directories that it creates.
[+] MUST: A Fedora package must not list a file more than once in %files.
[+] MUST: Permissions on files must be set properly.
[+] MUST: Each package must consistently use macros.
[+] MUST: The package must contain code, or permissable content.
[.] MUST: Large documentation files must go in a -doc subpackage.
[+] MUST: Files in %doc must not affect the runtime of the application.
[.] MUST: Header files must be in a -devel package.
[.] MUST: Static libraries must be in a -static package.
[.] MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), ...
[.] MUST: devel packages must require the base package using a fully versioned
dependency.
[+] MUST: Packages must NOT contain any .la libtool archives.
[+] MUST: Packages containing GUI applications must include a %{name}.desktop
file.
[+] MUST: .desktop files must be properly installed with desktop-file-install
in the %install section.
[+] MUST: Packages must not own files or directories already owned by other
packages.
[+] MUST: All filenames in rpm packages must be valid UTF-8.
[.] SHOULD: If the source package does not include license text(s) as a
separate file from upstream, ...
[+] SHOULD: The reviewer should test that the package builds in mock.
[+] SHOULD: The reviewer should test that the package functions as described.
[.] SHOULD: If scriptlets are used, those scriptlets must be sane.
[.] SHOULD: Usually, subpackages other than devel should
[Bug 700354] Review Request: perl-Locale-Currency-Format - Perl functions for formatting monetary values
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=700354 Nicholas van Oudtshoorn vano...@gmail.com changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- 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 700353] New: Review Request: perl-HTTP-OAI - API for the OAI-PMH
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-HTTP-OAI - API for the OAI-PMH https://bugzilla.redhat.com/show_bug.cgi?id=700353 Summary: Review Request: perl-HTTP-OAI - API for the OAI-PMH Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: vano...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://dl.dropbox.com/u/8648526/perl-HTTP-OAI.spec SRPM URL: http://dl.dropbox.com/u/8648526/perl-HTTP-OAI-3.24-1.fc14.src.rpm Description: This is one of the requirements for the Koha Library Management System. Since I have not yet got any packages in the fedora repository, I am looking for a sponsor. -- 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 700354] New: Review Request: perl-Locale-Currency-Format - Perl functions for formatting monetary values
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-Locale-Currency-Format - Perl functions for formatting monetary values https://bugzilla.redhat.com/show_bug.cgi?id=700354 Summary: Review Request: perl-Locale-Currency-Format - Perl functions for formatting monetary values Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: vano...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://dl.dropbox.com/u/8648526/perl-Locale-Currency-Format.spec SRPM URL: http://dl.dropbox.com/u/8648526/perl-Locale-Currency-Format-1.28-1.fc14.src.rpm Description: Locale::Currency::Format is a light-weight Perl module that enables Perl code to display monetary values in the formats recognized internationally and/or locally. This is one of the requirements for the Koha Library Management System. Since I have not yet got any packages in the fedora repository, I am looking for a sponsor. -- 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 694287] Review Request: openCOLLADA - 3D import and export libraries
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=694287 Hans de Goede hdego...@redhat.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #52 from Hans de Goede hdego...@redhat.com 2011-04-28 04:31:25 EDT --- Hi, (In reply to comment #51) Everything looks good. The debuginfo file now has some size to it :) New spec and SRC RPM uploaded. SRC RPM: https://docs.google.com/leaf?id=0B4A6of0Rl4nUY2UxZGQ4ZDEtY2Q5MC00MTNmLWExOTEtYWVhNjU4NjFhYzgxhl=en SPEC: https://docs.google.com/leaf?id=0B4A6of0Rl4nUZWVlOWE3ZGEtNDdiMi00MWZmLWE5NmQtOWQ5YTdmYTY5YWFkhl=en Looks good now, approved! One last small thing, but that can be fixed before / after import into git, you copy pasted the Summary for the doc subpackage from the devel doc package and did not adjust it to describe the doc subpackage. So since this package is approved now it is time to sponsor you, next I'll need to know your FAS account username to create one if needed, see: http://fedoraproject.org/wiki/PackageMaintainers/Join#Get_a_Fedora_Account Then I'll add you to the packagers group and sponsor you. Regards, Hans -- 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 700346] Review Request: perl-Barcode-Code128 - Generate CODE 128 bar codes
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=700346 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com --- Comment #1 from Iain Arnell iarn...@gmail.com 2011-04-28 04:45:29 EDT --- Not an official review since I'm not a sponsor, but I hate to see tests simply disabled. There is a patch in https://rt.cpan.org/Public/Bug/Display.html?id=28369 that fixes t/png.t to compare images using gd, so you would only need to remove t/gif.t to have the remaining tests run successfully. -- 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 700349] Review Request: perl-CGI-Session-Driver-memcached - CGI::Session driver for memcached
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=700349 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com --- Comment #1 from Iain Arnell iarn...@gmail.com 2011-04-28 04:56:59 EDT --- Again, not an official review. But why move the endnote-format script to /usr/sbin? It's not a root-only admin thing, so properly belongs in /usr/bin. -- 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 700349] Review Request: perl-Biblio-EndnoteStyle - Reference formatting using Endnote-like templates
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=700349 Iain Arnell iarn...@gmail.com changed: What|Removed |Added Summary|Review Request: |Review Request: |perl-CGI-Session-Driver-mem |perl-Biblio-EndnoteStyle - |cached - CGI::Session |Reference formatting using |driver for memcached|Endnote-like templates --- Comment #2 from Iain Arnell iarn...@gmail.com 2011-04-28 04:59:13 EDT --- (In reply to comment #0) Description: This small module stores session data into memcached. Oh, and seems you pasted the wrong description/summary for this package. I've fixed the latter. -- 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 700346] Review Request: perl-Barcode-Code128 - Generate CODE 128 bar codes
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=700346 --- Comment #2 from Nicholas van Oudtshoorn vano...@gmail.com 2011-04-28 05:00:10 EDT --- Ah! Silly me; don't know what I was thinking. I've updated the spec file and srpm linked above to incorporate that patch and remove t/gif.t Thanks Iaian! -- 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 700349] Review Request: perl-Biblio-EndnoteStyle - Reference formatting using Endnote-like templates
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=700349 --- Comment #3 from Nicholas van Oudtshoorn vano...@gmail.com 2011-04-28 05:05:39 EDT --- Ah! Silly me! Thanks for the heads up about the /usr/bin location. I've updated the spec and srpm linked above to set it back to what it should be. Not sure *why* I changed it in the first place. (Packaging is fairly new to me... my apologies for the noise!) -- 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 682666] Review Request: DeTex - A program to remove TeX constructs from a text file
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=682666 --- Comment #6 from Ankur Sinha sanjay.an...@gmail.com 2011-04-28 05:11:58 EDT --- AH! Sorry, I missed out on the earlier mail. (In reply to comment #4) That looks good :) According to the guidelines, patches in a .spec should have a comment. This concerns also fixes made through sed. I'll add the comments. This will not (only) help the reviewer, but it help you as a maintainer. By the way, why these lines in %prep? sed -i /rm -f xxx\.l/d Makefile sed -i s/mv lex\.yy\.c detex\.c/cp lex\.yy\.c detex\.c/ Makefile The above sed commands are to make some changes in the Makefile to enable proper generation of debuginfo. If these files are removed/moved, debuginfo generation fails. In %description, use « DeTeX » instead of « Detex ». This seems to be the « official » name of the application. Okay. Do I upload a new spec? Or can I make these changes before the scm commit please? (Since only comments etc. are required) Thanks, Ankur -- 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 682666] Review Request: DeTex - A program to remove TeX constructs from a text file
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=682666 --- Comment #7 from Christoph Wickert cwick...@fedoraproject.org 2011-04-28 05:20:53 EDT --- You upload not only a new spec but a new srpm because we are reviewing srpms. -- 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 698953] Review Request: Brazilian Portuguese dictionaries for Aspell
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=698953 Ivana Varekova varek...@redhat.com changed: What|Removed |Added Blocks||674634 -- 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 700345] Review Request: com.inamik.utils.tableformatter - A set of Java classes to print text in tabulated form
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=700345 Alexander Kurtakov akurt...@redhat.com changed: What|Removed |Added CC||akurt...@redhat.com --- Comment #1 from Alexander Kurtakov akurt...@redhat.com 2011-04-28 05:31:18 EDT --- Shouldn't it be called jtable-format ? The current you have chosen looks quite strange for package name. -- 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 682666] Review Request: DeTex - A program to remove TeX constructs from a text file
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=682666 --- Comment #8 from Ankur Sinha sanjay.an...@gmail.com 2011-04-28 05:32:19 EDT --- (In reply to comment #7) You upload not only a new spec but a new srpm because we are reviewing srpms. Well, I've got to keep you happy: http://ankursinha.fedorapeople.org/detex/detex.spec http://ankursinha.fedorapeople.org/detex/detex-2.8-2.fc15.src.rpm Ankur -- 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 682666] Review Request: DeTex - A program to remove TeX constructs from a text file
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=682666
Mohamed El Morabity pikachu.2...@gmail.com changed:
What|Removed |Added
Flag||fedora-review+
--- Comment #9 from Mohamed El Morabity pikachu.2...@gmail.com 2011-04-28
05:41:25 EDT ---
By the way, why these lines in %prep?
sed -i /rm -f xxx\.l/d Makefile
sed -i s/mv lex\.yy\.c detex\.c/cp lex\.yy\.c detex\.c/ Makefile
The above sed commands are to make some changes in the Makefile to enable
proper generation of debuginfo. If these files are removed/moved, debuginfo
generation fails.
OK, good catch :)
You forgot in the last .spec you uploaded to replace « DeTex » by « DeTeX ».
Anyway, it's a minor issue, I trust you to fix it before you upload the
package.
Here is the review:
MUST: rpmlint must be run on every package.
-OK, only false-positive spelling issues
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.
-OK
MUST: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines.
-OK
MUST: The License field in the package spec file must match the actual license.
-OK
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
MUST: The spec file must be written in American English.
-OK
MUST: The spec file for the package MUST be legible.
-OK
MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL.
-OK, md5sum = 7a96b647f43bb077323cde92faa1e893
MUST: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
-OK, see http://koji.fedoraproject.org/koji/taskinfo?taskID=3032154
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.
-N/A
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.
-OK
MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
-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.
-N/A
MUST: Packages must NOT bundle copies of system libraries.
-OK
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.
-N/A
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.
OK
MUST: A Fedora package must not list a file more than once in the spec file's
%files listings.
-OK
MUST: Permissions on files must be set properly. Executables should be set with
executable permissions, for example. Every %files section must include a
%defattr(...) line.
-OK
MUST: Each package must consistently use macros.
-OK
MUST: The package must contain code, or permissable content.
-OK
MUST: Large documentation files must go in a -doc subpackage.
-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.
-OK
MUST: Header files must be in a -devel package.
-N/A
MUST: Static libraries must be in a -static package.
-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.
-N/A
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency: Requires: %{name} =
%{version}-%{release}.
-N/A
MUST: Packages must NOT contain any .la libtool archives, these must be removed
in the spec if they are built.
-OK
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.
[Bug 700335] Review Request: ant-antlr3 - Antlr3 task for Ant
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=700335 Alexander Kurtakov akurt...@redhat.com changed: What|Removed |Added CC||akurt...@redhat.com --- Comment #1 from Alexander Kurtakov akurt...@redhat.com 2011-04-28 05:38:24 EDT --- The package is missing /etc/ant.d integration.Please look at checkstyle for sample usage. Basically it's space separated paths to jar files in %_javadir without the extension. E.g it would be smth like ant/ant-antlr3 antlr3 assuming that you need antlr3 on the classpath to use this task. -- 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 682666] Review Request: DeTex - A program to remove TeX constructs from a text file
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=682666 --- Comment #10 from Christoph Wickert cwick...@fedoraproject.org 2011-04-28 05:52:18 EDT --- There is still a small bug in the spec: the release was not bumped and -2 is in the changelog twie. -- 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 682666] Review Request: DeTex - A program to remove TeX constructs from a text file
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=682666 --- Comment #11 from Ankur Sinha sanjay.an...@gmail.com 2011-04-28 05:58:15 EDT --- (In reply to comment #10) There is still a small bug in the spec: the release was not bumped and -2 is in the changelog twie. That isn't a bug afaics: http://fedoraproject.org/wiki/PackagingGuidelines#Multiple_Changelog_Entries_per_Release Before I do an SCM request, should I change the *entire* package name to DeTeX Mohamed? I mean, change the spec name to DeTeX too, along with the changes in the description? (The final binary, and source is named detex though) And, I did make the change in the description, I changed Detex to DeTex, (forgot the X! XD ) Thanks for the review!! Ankur -- 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 682666] Review Request: DeTex - A program to remove TeX constructs from a text file
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=682666 --- Comment #12 from Christoph Wickert cwick...@fedoraproject.org 2011-04-28 06:15:35 EDT --- (In reply to comment #11) (In reply to comment #10) There is still a small bug in the spec: the release was not bumped and -2 is in the changelog twie. That isn't a bug afaics: http://fedoraproject.org/wiki/PackagingGuidelines#Multiple_Changelog_Entries_per_Release But we should have had two releases actually, even during a review. And if you substitute multiple versions in a changelog entry, it should actually be * Thu Apr 28 2011 Ankur Sinha ankursinha AT fedoraproject DOT org - 2.8-2 - removed patch - renamed man page to .1 instead of .1l - corrected compilation flags - Review ticket #682666 - Added *comments* Highest version/release and latest date contain all changes. I suggest to leave it at -2 and just change the changelog. Before I do an SCM request, should I change the *entire* package name to DeTeX Mohamed? Please don't, leave the spec and the package detex please. LaTeX is also written lowercase in our package names While case sensitivity is not a mandatory requirement, case should only be used where necessary. from http://fedoraproject.org/wiki/PackageNamingGuidelines#Case_Sensitivity -- 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 682666] Review Request: DeTex - A program to remove TeX constructs from a text file
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=682666 Ankur Sinha sanjay.an...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #14 from Ankur Sinha sanjay.an...@gmail.com 2011-04-28 06:27:25 EDT --- New Package SCM Request === Package Name: detex Short Description: A program to remove TeX constructs from a text file Owners: ankursinha Branches: f14 f15 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 682666] Review Request: DeTex - A program to remove TeX constructs from a text file
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=682666 --- Comment #13 from Ankur Sinha sanjay.an...@gmail.com 2011-04-28 06:25:36 EDT --- Hello, (In reply to comment #12) (In reply to comment #11) (In reply to comment #10) There is still a small bug in the spec: the release was not bumped and -2 is in the changelog twie. That isn't a bug afaics: http://fedoraproject.org/wiki/PackagingGuidelines#Multiple_Changelog_Entries_per_Release But we should have had two releases actually, even during a review. And if you substitute multiple versions in a changelog entry, it should actually be * Thu Apr 28 2011 Ankur Sinha ankursinha AT fedoraproject DOT org - 2.8-2 - removed patch - renamed man page to .1 instead of .1l - corrected compilation flags - Review ticket #682666 - Added *comments* Highest version/release and latest date contain all changes. I suggest to leave it at -2 and just change the changelog. Okay, although the link that I posted above gives two ways of doing it (one of which is what I used). Before I do an SCM request, should I change the *entire* package name to DeTeX Mohamed? Please don't, leave the spec and the package detex please. LaTeX is also written lowercase in our package names While case sensitivity is not a mandatory requirement, case should only be used where necessary. from http://fedoraproject.org/wiki/PackageNamingGuidelines#Case_Sensitivity Okay. Thank you for the clarification :) Ankur -- 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 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations
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=697411 Marcela Mašláňová mmasl...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #8 from Marcela Mašláňová mmasl...@redhat.com 2011-04-28 06:50:48 EDT --- New Package SCM Request === Package Name: perl-Data-Alias Short Description: Comprehensive set of aliasing operations Owners: mmaslano psabata ppisar Branches: InitialCC: perl-sig -- 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 700335] Review Request: ant-antlr3 - Antlr3 task for Ant
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=700335 Stanislav Ochotnicky socho...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||socho...@redhat.com AssignedTo|nob...@fedoraproject.org|socho...@redhat.com Flag||fedora-review? --- Comment #2 from Stanislav Ochotnicky socho...@redhat.com 2011-04-28 07:36:03 EDT --- I'll do the whole review, but feel free to create /etc/ant.d file (It would be needed anyway). -- 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 700427] New: Review Request: jopt-simple - A Java command line parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: jopt-simple - A Java command line parser https://bugzilla.redhat.com/show_bug.cgi?id=700427 Summary: Review Request: jopt-simple - A Java command line parser Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: kk...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://kklic.fedorapeople.org/jopt-simple.spec SRPM URL: http://kklic.fedorapeople.org/jopt-simple-3.3-0.git12c0e63.fc14.src.rpm Description: A Java library for parsing command line options. $ rpmlint jopt-simple-3.3-0.git12c0e63.fc14.noarch.rpm jopt-simple.noarch: W: non-conffile-in-etc /etc/maven/fragments/jopt-simple 1 packages and 0 specfiles checked; 0 errors, 1 warnings. $ rpmlint jopt-simple-javadoc-3.3-0.git12c0e63.fc14.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint jopt-simple-3.3-0.git12c0e63.fc14.src.rpm jopt-simple.src: W: invalid-url Source0: pholser-jopt-simple-12c0e63.tar.gz 1 packages and 0 specfiles checked; 0 errors, 1 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 697976] Review Request: pyrenamer - A mass file renamer
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=697976 Jon Ciesla l...@jcomserv.net changed: What|Removed |Added Summary|Review Request: pyrenamer - |Review Request: pyrenamer - |A mass file renamer written |A mass file renamer |in PyGTK| -- 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 700335] Review Request: ant-antlr3 - Antlr3 task for Ant
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=700335
--- Comment #3 from Stanislav Ochotnicky socho...@redhat.com 2011-04-28
08:04:39 EDT ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[x] Rpmlint output:
ant-antlr3-javadoc.noarch: W: spelling-error Summary(en_US) Javadocs - Java
docs, Java-docs, Javanese
false alarm
ant-antlr3.src: W: invalid-url Source0: antlr3-task.zip
explained in spec. It was fixed in upstream, but will the fix be available in
f-13+ rpms? If yes than this can be done properly...
3 packages and 0 specfiles checked; 0 errors, 2 warnings.
[x] Package is named according to the Package Naming Guidelines[1].
[x] Spec file name must match the base package name, in the format
%{name}.spec.
[x] Package meets the Packaging Guidelines[2].
[x] Package successfully compiles and builds into binary rpms.
[x] Buildroot definition is not present
[x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[x] License field in the package spec file matches the actual license.
License type: ASL 2.0
[-] 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.
It would be best if you contacted upstream and asked them to include LICENSE
file in the tarball/zip. And while you are at it...ask them to provide
versioned source zip in one of standard directories.
http://antlr.org/share/1169924912745/antlr3-task.zip is weird because it
doesn't contain any date/version information.
[-] All independent sub-packages have license of their own
[x] Spec file is legible and written in American English.
[-] Sources used to build the package matches the upstream source, as provided
in the spec URL.
Regenerated from upstream sources, contents match
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines[5].
[x] Package must own all directories that it creates.
[x] Package requires other packages for directories it uses.
[x] Package does not contain duplicates in %files.
[x] Permissions on files are set properly.
[x] Package does NOT have a %clean section which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). (not needed anymore)
[x] Package consistently uses macros (no %{buildroot} and $RPM_BUILD_ROOT
mixing)
[x] Package contains code, or permissable content.
[x] Fully versioned dependency in subpackages, if present.
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x] Package does not own files or directories owned by other packages.
[x] Javadoc documentation files are generated and included in -javadoc
subpackage
[x] Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlinks)
[x] Packages have proper BuildRequires/Requires on jpackage-utils
[!] Javadoc subpackages have Require: jpackage-utils
Javadocs should Require jpackage-utils for directory
[-] Package uses %global not %define
[-] If package uses tarball from VCS include comment how to re-create that
tarball (svn export URL, git clone URL, ...)
[!] If source tarball includes bundled jar/class files these need to be
removed prior to building
Zipball contains ant-antlr3.jar that is not used during build, but we normally
remove *jar and *class files in %prep to be sure and not to confuse reviewers
:-)
[x] All filenames in rpm packages must be valid UTF-8.
[x] Jar files are installed to %{_javadir}/%{name}.jar (see [6] for details)
[-] If package contains pom.xml files install it (including depmaps) even when
building with ant
[-] pom files has correct add_to_maven_depmap call which resolves to the pom
file (use JPP. and JPP- correctly)
=== Other suggestions ===
[x] If possible use upstream build method (maven/ant/javac)
[x] Avoid having BuildRequires on exact NVR unless necessary
[x] Package has BuildArch: noarch (if possible)
[?] Latest version is packaged.
[x] Reviewer should test that the package builds in mock.
Tested on: fedora-rawhide-x86_64
=== Issues ===
1. javadoc requires on jpackage-utils
2. source url...is it going to be fixed for fedora rpms?
3. inclusion license file in zipball and versioned zipball releases (upstream)
4. remove bundled jar for the sake of consistency :-)
5. include already mentioned ant.d file
[1] https://fedoraproject.org/wiki/Packaging:NamingGuidelines
[2] https://fedoraproject.org/wiki/Packaging:Guidelines
[3] https://fedoraproject.org/wiki/Packaging:LicensingGuidelines
[4] https://fedoraproject.org/wiki/Licensing:Main
[5] https://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2
[6] https://fedoraproject.org/wiki/Packaging:Java#Filenames
--
[Bug 697976] Review Request: pyrenamer - A mass file renamer written in PyGTK
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=697976 Jon Ciesla l...@jcomserv.net changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Jon Ciesla l...@jcomserv.net 2011-04-28 08:02:03 EDT --- Thanks for the review! I'll fix the manpage compression before import. New Package SCM Request === Package Name: pyrenamer Short Description: A mass file renamer Owners: limb Branches: f13 f14 f15 el6 el5 InitialCC: Note for cvsadmin, this package was formerly in Fedora, orphaned, last built for f12. -- 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 678855] Review Request: rpyc - A Transparent, Symmetrical Python Library for Distributed-Computing
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=678855 Dan Kenigsberg dan...@redhat.com changed: What|Removed |Added CC||dan...@redhat.com Flag||fedora-review?, ||needinfo?(lut...@redhat.com ||) --- Comment #5 from Dan Kenigsberg dan...@redhat.com 2011-04-28 08:06:37 EDT --- We are unable to set fedora‑cvs?. Should we ask for fedora‑review+ first? -- 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 678855] Review Request: rpyc - A Transparent, Symmetrical Python Library for Distributed-Computing
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=678855 Ankur Sinha sanjay.an...@gmail.com changed: What|Removed |Added CC||sanjay.an...@gmail.com --- Comment #6 from Ankur Sinha sanjay.an...@gmail.com 2011-04-28 08:17:40 EDT --- Yep! When approved, the review flag must be set to +. -- 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 698952] Review Request: European Portuguese dictionaries for Aspell
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=698952 Ivana Varekova varek...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Last Closed||2011-04-28 08:49:30 --- Comment #7 from Ivana Varekova varek...@redhat.com 2011-04-28 08:49:30 EDT --- Thanks, fixed the dot after the description, removed BuildRoot tag and clean part. -- 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 694287] Review Request: openCOLLADA - 3D import and export libraries
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=694287 Richard Shaw hobbes1...@gmail.com changed: What|Removed |Added Flag|fedora-review+ |fedora-review? --- Comment #53 from Richard Shaw hobbes1...@gmail.com 2011-04-28 08:52:41 EDT --- Woops. Fixed the description, forgot the summary. FAS account name is hobbes1069. -- 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 694287] Review Request: openCOLLADA - 3D import and export libraries
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=694287 --- Comment #54 from Hans de Goede hdego...@redhat.com 2011-04-28 09:03:52 EDT --- (In reply to comment #53) Woops. Fixed the description, forgot the summary. FAS account name is hobbes1069. Ok, you've been added to the packagers group and sponsored, next step is to do a scm admin request to get a module created in git and bugzilla for openCOLLADA, as described here: http://fedoraproject.org/wiki/Package_SCM_admin_requests Note that it may take up to an hour for your new packager right to propagate, so if you get an error when you try to set the cvs flag, wait a bit :) -- 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 700348] Review Request: mingw32-glib-networking - MinGW Windows glib-networking 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=700348 Erik van Pienbroek erik-fed...@vanpienbroek.nl changed: What|Removed |Added Status|NEW |ASSIGNED CC||erik-fed...@vanpienbroek.nl AssignedTo|nob...@fedoraproject.org|erik-fed...@vanpienbroek.nl Flag||fedora-review? --- Comment #1 from Erik van Pienbroek erik-fed...@vanpienbroek.nl 2011-04-28 09:08:17 EDT --- The package is pretty straightforward. Thanks to this review request, some unowned dirs were found in the mingw32-glib2 package which have just been fixed in rawhide and F15. The Requires: pkgconfig can be dropped as no pkgconfig files are used -- 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 694287] Review Request: openCOLLADA - 3D import and export libraries
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=694287 --- Comment #55 from Richard Shaw hobbes1...@gmail.com 2011-04-28 09:17:04 EDT --- I'm assuming we are not worried about branching for F13, just F14 and F15? -- 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 694287] Review Request: openCOLLADA - 3D import and export libraries
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=694287 --- Comment #56 from Hans de Goede hdego...@redhat.com 2011-04-28 09:29:08 EDT --- (In reply to comment #55) I'm assuming we are not worried about branching for F13, just F14 and F15? Yes, I don't think it is likely we will see blender update using this for F13, so no need for an F13 branch. -- 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 700348] Review Request: mingw32-glib-networking - MinGW Windows glib-networking 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=700348 --- Comment #2 from Kalev Lember ka...@smartlink.ee 2011-04-28 09:29:29 EDT --- Thanks for fixing mingw32-glib2. Spec URL: http://kalev.fedorapeople.org/mingw32-glib-networking.spec SRPM URL: http://kalev.fedorapeople.org/mingw32-glib-networking-2.28.6.1-2.fc15.src.rpm * Thu Apr 28 2011 Kalev Lember ka...@smartlink.ee - 2.28.6.1-2 - Dropped Requires: pkgconfig (#700348) -- 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 694287] Review Request: openCOLLADA - 3D import and export libraries
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=694287 Richard Shaw hobbes1...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #57 from Richard Shaw hobbes1...@gmail.com 2011-04-28 09:30:58 EDT --- New Package SCM Request === Package Name: openCOLLADA Short Description: Collada 3D import and export libraries Owners: hobbes1069 Branches: f14 f15 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 700348] Review Request: mingw32-glib-networking - MinGW Windows glib-networking 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=700348 Erik van Pienbroek erik-fed...@vanpienbroek.nl changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from Erik van Pienbroek erik-fed...@vanpienbroek.nl 2011-04-28 09:34:39 EDT --- Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3032727 $ rpmlint mingw32-glib-networking.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint mingw32-glib-networking-2.28.6.1-1.fc15.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint mingw32-glib-networking-2.28.6.1-1.fc15.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpm --query --requires mingw32-glib-networking rpmlib(PayloadFilesHavePrefix) = 4.0-1 rpmlib(CompressedFileNames) = 3.0.4-1 mingw32(kernel32.dll) mingw32(libgcrypt-11.dll) mingw32(libgio-2.0-0.dll) mingw32(libglib-2.0-0.dll) mingw32(libgnutls-26.dll) mingw32(libgobject-2.0-0.dll) mingw32(msvcrt.dll) rpmlib(PayloadIsXz) = 5.2-1 $ rpm --query --provides mingw32-glib-networking mingw32(libgiognutls.dll) mingw32-glib-networking = 2.28.6.1-1.fc15 $ rpm --query --fileprovide mingw32-glib-networking /usr/i686-pc-mingw32/sys-root/mingw/lib/gio/modules/libgiognutls.dll /usr/i686-pc-mingw32/sys-root/mingw/share/locale/as/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/bg/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/bn_IN/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/ca/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/cs/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/da/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/de/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/el/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/en_CA/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/en_GB/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/es/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/et/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/eu/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/fr/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/gl/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/gu/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/he/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/hi/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/hu/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/id/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/it/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/ja/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/kn/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/ko/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/lv/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/mr/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/nb/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/nl/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/or/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/pa/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/pl/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/pt/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/pt_BR/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/ro/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/ru/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/sl/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/sr/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/sr@latin/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/sv/LC_MESSAGES/glib-networking.mo /usr/i686-pc-mingw32/sys-root/mingw/share/locale/tr/LC_MESSAGES/glib-networking.mo
[Bug 700335] Review Request: ant-antlr3 - Antlr3 task for Ant
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=700335 --- Comment #4 from Karel Klíč kk...@redhat.com 2011-04-28 09:33:27 EDT --- Spec URL: http://kklic.fedorapeople.org/ant-antlr3.spec SRPM URL: http://kklic.fedorapeople.org/ant-antlr3-20110110-2.fc14.src.rpm * Thu Apr 28 2011 Karel Klíč kk...@redhat.com - 20110110-2 - Added /etc/ant.d integration - Require jpackage-utils by -javadoc package (needed for directory) - Remove bundled jar in %%prep 1. javadoc requires on jpackage-utils Fixed. 2. source url...is it going to be fixed for fedora rpms? RPM developers decided to fix it in upstream only. I'll revisit the issue every time the package is updated (inevitably, as the upstream file needs to be repackaged until the bug is fixed). I think this is not a blocker. 3. inclusion license file in zipball and versioned zipball releases (upstream) I'm going to write an email to the upstream developer right now and ask him for this. This should not block the review process. 4. remove bundled jar for the sake of consistency :-) Done 5. include already mentioned ant.d file Done Thank you for the 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 700348] Review Request: mingw32-glib-networking - MinGW Windows glib-networking 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=700348 Kalev Lember ka...@smartlink.ee changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Kalev Lember ka...@smartlink.ee 2011-04-28 09:45:55 EDT --- Thanks for the review, Erik! New Package SCM Request === Package Name: mingw32-glib-networking Short Description: MinGW Windows glib-networking library Owners: kalev epienbro Branches: f15 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 693131] Review Request: sudoku-savant - Solve and generate sudoku puzzles through logical means
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=693131 --- Comment #6 from Mario Blättermann mari...@freenet.de 2011-04-28 09:50:18 EDT --- Spec URL: http://dl.dropbox.com/u/19373040/Fedora/SPECS/sudoku-savant.spec SRPM URL: http://dl.dropbox.com/u/19373040/Fedora/sudoku-savant-1.3-3.fc14.src.rpm I've added some comments regarding the license and build problems, including links to the appropriate upstream bugs. -- 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 695745] Review Request: wso2-wsf-cpp - Web Services Framework for C++
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=695745 Robert Rati rr...@redhat.com changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution||NEXTRELEASE Flag|fedora-cvs+ |fedora-cvs? Last Closed||2011-04-28 10:03:15 -- 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 700335] Review Request: ant-antlr3 - Antlr3 task for Ant
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=700335
--- Comment #5 from Stanislav Ochotnicky socho...@redhat.com 2011-04-28
10:01:35 EDT ---
I am almost sure this is wrong:
cat $RPM_BUILD_ROOT%{_sysconfdir}/ant.d/ant-antlr3 EOF
ant-antlr3 antlr3
EOF
Shouldn't it be?:
cat $RPM_BUILD_ROOT%{_sysconfdir}/ant.d/ant-antlr3 EOF
ant/ant-antlr3 antlr3
EOF
Other than package seems good now, but I'd advise you to actually use it when
building something (nudge nudge, wink wink - stanse) to see that it actually
works.
--
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 693131] Review Request: sudoku-savant - Solve and generate sudoku puzzles through logical means
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=693131 Martin Gieseking martin.giesek...@uos.de changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #7 from Martin Gieseking martin.giesek...@uos.de 2011-04-28 10:10:03 EDT --- OK, the package looks good now. There's a small typo in the German %description you can fix before checking the package into Git (no need to bump the Release number for that, though): Das manuelle Lösen... (capital L) Package APPROVED -- 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 699586] Review Request: tabish-eeyek-fonts - Font for Meetei Mayek script
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=699586 --- Comment #4 from Mario Blättermann mari...@freenet.de 2011-04-28 10:12:29 EDT --- Created attachment 495552 -- https://bugzilla.redhat.com/attachment.cgi?id=495552 Change the FSF postal address to the current one. -- 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 699586] Review Request: tabish-eeyek-fonts - Font for Meetei Mayek script
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=699586 --- Comment #5 from Mario Blättermann mari...@freenet.de 2011-04-28 10:13:12 EDT --- $ rpmlint -v * ... tabish-eeyek-fonts.noarch: E: incorrect-fsf-address /usr/share/doc/tabish-eeyek-fonts-1.0/copying.txt ... 2 packages and 1 specfiles checked; 1 errors, 0 warnings. The license blurb still contains the old FSF address, the new one is: Free Software Foundation 51 Franklin Street, Suite 500 Boston, MA 02110-1335 USA Apply the attached patch please. -- 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 693131] Review Request: sudoku-savant - Solve and generate sudoku puzzles through logical means
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=693131 --- Comment #8 from Mario Blättermann mari...@freenet.de 2011-04-28 10:20:38 EDT --- Thanks for your review. The spelling error in the German description is already changed in the Dropbox 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 672629] Review Request: perl-Perl-Critic-Tics - Policies for things that make me wince
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=672629 --- Comment #9 from Petr Pisar ppi...@redhat.com 2011-04-28 10:24:17 EDT --- Package installing into vendor can be found on following locators: Spec URL: http://ppisar.fedorapeople.org/perl-Perl-Critic-Tics/perl-Perl-Critic-Tics.spec SRPM URL: http://ppisar.fedorapeople.org/perl-Perl-Critic-Tics/perl-Perl-Critic-Tics-0.006-1.fc16.src.rpm -- 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 693131] Review Request: sudoku-savant - Solve and generate sudoku puzzles through logical means
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=693131 Mario Blättermann mari...@freenet.de changed: What|Removed |Added Flag||fedora-cvs? --- Comment #9 from Mario Blättermann mari...@freenet.de 2011-04-28 10:22:43 EDT --- New Package SCM Request === Package Name: sudoku-savant Short Description: Solve and generate sudoku puzzles through logical means Owners: mariobl Branches: f14 f15 -- 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 699586] Review Request: tabish-eeyek-fonts - Font for Meetei Mayek script
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=699586 Martin Gieseking martin.giesek...@uos.de changed: What|Removed |Added CC||martin.giesek...@uos.de --- Comment #6 from Martin Gieseking martin.giesek...@uos.de 2011-04-28 10:22:18 EDT --- No, please don't change any files if it's not required to fix build problems. Instead, ask upstream to apply the patch. Until then, just add the original license file with the old address. -- 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 591150] Review Request: ansifilter - ANSI escape code processor and converter
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=591150 Kalev Lember ka...@smartlink.ee changed: What|Removed |Added Status|NEW |CLOSED Resolution||NOTABUG Last Closed||2011-04-28 10:30:54 --- Comment #1 from Kalev Lember ka...@smartlink.ee 2011-04-28 10:30:54 EDT --- I am no longer interested in this package, closing the ticket. -- 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 226352] Merge Review: quagga
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=226352 Jiri Skala jsk...@redhat.com changed: What|Removed |Added Flag|needinfo?(jsk...@redhat.com | |) | --- Comment #6 from Jiri Skala jsk...@redhat.com 2011-04-28 10:38:31 EDT --- I've added requirement for /sbin/chkconfig in post and preun sections and corrected section for requires /sbin/install-info. Commited in rawhide. -- 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 226352] Merge Review: quagga
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=226352 Itamar Reis Peixoto ita...@ispbrasil.com.br changed: What|Removed |Added CC||ita...@ispbrasil.com.br --- Comment #7 from Itamar Reis Peixoto ita...@ispbrasil.com.br 2011-04-28 10:45:39 EDT --- why you don't build quaga for fedora-epel ? -- 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 678855] Review Request: rpyc - A Transparent, Symmetrical Python Library for Distributed-Computing
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=678855 David Lutterkort lut...@redhat.com changed: What|Removed |Added Flag|fedora-review?, |fedora-review+ |needinfo?(lut...@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 226352] Merge Review: quagga
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=226352 Vojtech Vitek vvi...@redhat.com changed: What|Removed |Added Status|NEW |CLOSED Resolution||RAWHIDE Last Closed||2011-04-28 10:54:57 --- Comment #8 from Vojtech Vitek vvi...@redhat.com 2011-04-28 10:54:57 EDT --- review+ -- 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 645857] Review Request: xorg-x11-drv-omapfb - Xorg X11 omap frame buffer 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=645857 Dennis Gilmore den...@ausil.us changed: What|Removed |Added Flag|needinfo?(den...@ausil.us) | --- Comment #7 from Dennis Gilmore den...@ausil.us 2011-04-28 11:13:26 EDT --- SRPM: http://ausil.us/packages/xorg-x11-drv-omapfb-0.1.1-1.git20110428.fc13.src.rpm SPEC: http://ausil.us/packages/xorg-x11-drv-omapfb.spec scratch build http://arm.koji.fedoraproject.org/koji/taskinfo?taskID=93433 i switched to just pulling a git snapshot -- 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 645857] Review Request: xorg-x11-drv-omapfb - Xorg X11 omap frame buffer 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=645857 Peter Robinson pbrobin...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #8 from Peter Robinson pbrobin...@gmail.com 2011-04-28 11:45:48 EDT --- Looks good. APPROVED! -- 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 645857] Review Request: xorg-x11-drv-omapfb - Xorg X11 omap frame buffer 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=645857 Dennis Gilmore den...@ausil.us changed: What|Removed |Added Flag||fedora-cvs? --- Comment #9 from Dennis Gilmore den...@ausil.us 2011-04-28 11:49:38 EDT --- New Package SCM Request === Package Name: xoeg-x11-drv-omapfb Short Description: X.Org X11 omap frame buffer driver Owners: ausil Branches: f13 f14 f15 devel 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 700537] New: Review Request: eclipse-mylyn-builds - Eclipse Mylyn Builds
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: eclipse-mylyn-builds - Eclipse Mylyn Builds https://bugzilla.redhat.com/show_bug.cgi?id=700537 Summary: Review Request: eclipse-mylyn-builds - Eclipse Mylyn Builds Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: sgehw...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://fedorapeople.org/~jerboaa/rpm/mylyn/eclipse-mylyn-builds.spec SRPM URL: http://fedorapeople.org/~jerboaa/rpm/mylyn/eclipse-mylyn-builds-3.5.0-2.fc15.src.rpm Description: Provides a common framework to interact with continuous integration build providers using Eclipse Mylyn. Thanks for the 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 700345] Review Request: com.inamik.utils.tableformatter - A set of Java classes to print text in tabulated form
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=700345
--- Comment #2 from Karel Klíč kk...@redhat.com 2011-04-28 12:04:23 EDT ---
Yes, the name is ugly. I have followed the tarball name but it feels wrong.
If we rename the package to jtable-format, it will contain a jar file not
consistent with it:
/usr/share/java/com.inamik.utils.tableformatter.jar
The packaging guidelines: When naming a package, the name should match the
upstream tarball or project name from which this software came.
The tarball is com.inamik.utils.tableformatter-%{version}-src.tar.gz
The project names used on the website are:
- iNamik Text Table Formatter for Java
- TableFormatter
- Table Formatter
- iNamik TableFormatter
- the URL of the upstream website contains jtable_format
So what about the inamik-tableformatter name? It looks closer to the upstream
project name and to the included jar file name.
What do you think?
--
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 629324] Review Request: python-zc-buildout - System for managing development buildouts
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=629324 --- Comment #7 from Dennis Gilmore den...@ausil.us 2011-04-28 12:07:01 EDT --- 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 645857] Review Request: xorg-x11-drv-omapfb - Xorg X11 omap frame buffer 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=645857 --- Comment #10 from Dennis Gilmore den...@ausil.us 2011-04-28 12:14:08 EDT --- 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 700551] New: Review Request: xwax - Open source vinyl emulation software for Linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: xwax - Open source vinyl emulation software for Linux https://bugzilla.redhat.com/show_bug.cgi?id=700551 Summary: Review Request: xwax - Open source vinyl emulation software for Linux Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: jbr...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://people.redhat.com/~jbrier/rpms/xwax.spec SRPM URL: http://people.redhat.com/~jbrier/rpms/xwax-0.9-2.fc14.src.rpm Description: xwax is open-source vinyl emulation software for Linux. It allows DJs and turntablists to playback digital audio files (MP3, Ogg Vorbis, FLAC, AAC and more), controlled using a normal pair of turntables via timecoded vinyls. It's designed for both beat mixing and scratch mixing. Needle drops, pitch changes, scratching, spinbacks and rewinds are all supported, and feel just like the audio is pressed onto the vinyl itself. The focus is on an accurate vinyl feel which is efficient, stable and fast. --- This is my first package and I need a sponsor. Note this package can be enhanced by mpg123 and it could be considered not useful without external bits. I talked to FESCo about this and got them to allow it: https://fedorahosted.org/fesco/ticket/583 -- 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 668240] Review Request: clapham - Railroad diagram generator
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=668240 --- Comment #10 from Dennis Gilmore den...@ausil.us 2011-04-28 12:19:30 EDT --- 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 700335] Review Request: ant-antlr3 - Antlr3 task for Ant
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=700335
--- Comment #6 from Karel Klíč kk...@redhat.com 2011-04-28 12:27:11 EDT ---
Spec URL: http://kklic.fedorapeople.org/ant-antlr3.spec
SRPM URL: http://kklic.fedorapeople.org/ant-antlr3-20110110-3.fc14.src.rpm
* Thu Apr 28 2011 Karel Klíč kk...@redhat.com - 20110110-3
- Fixed the name in /etc/ant.d/ant-antlr3
Shouldn't it be?:
cat $RPM_BUILD_ROOT%{_sysconfdir}/ant.d/ant-antlr3 EOF
ant/ant-antlr3 antlr3
EOF
Yes, sorry.
I'd advise you to actually use it when
building something (nudge nudge, wink wink - stanse) to see that it actually
works.
I use it for building stanse and it seems to work well.
I guess the /etc/ant.d/ant-antlr3 file makes it possible to use ant-antlr3 task
without adding ant-antlr3 and antlr3 jars to classpath. In stanse, the jar
files are added to classpath when building it.
--
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 672951] Review Request: python-elixir - A declarative mapper for SQLAlchemy
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=672951 --- Comment #17 from Dennis Gilmore den...@ausil.us 2011-04-28 12:26:22 EDT --- this already exists in fedora -- 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 678699] Review Request: vdrsymbol-fonts - Font for use with VDR plugins and patches
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=678699 --- Comment #3 from Dennis Gilmore den...@ausil.us 2011-04-28 12:32:07 EDT --- 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 682666] Review Request: DeTex - A program to remove TeX constructs from a text file
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=682666 --- Comment #15 from Dennis Gilmore den...@ausil.us 2011-04-28 12:39:13 EDT --- 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 693131] Review Request: sudoku-savant - Solve and generate sudoku puzzles through logical means
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=693131 --- Comment #10 from Dennis Gilmore den...@ausil.us 2011-04-28 12:42:23 EDT --- Mario you need to have the fas email match the one in bugzilla -- 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 694287] Review Request: openCOLLADA - 3D import and export libraries
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=694287 --- Comment #58 from Dennis Gilmore den...@ausil.us 2011-04-28 12:42:53 EDT --- 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 700427] Review Request: jopt-simple - A Java command line parser
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=700427 Karel Klíč kk...@redhat.com changed: What|Removed |Added Blocks||700566 -- 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 700335] Review Request: ant-antlr3 - Antlr3 task for Ant
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=700335 Karel Klíč kk...@redhat.com changed: What|Removed |Added Blocks||700566 -- 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 700566] Review Request: stanse - An automatic bug-finding tool for C
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=700566 Karel Klíč kk...@redhat.com changed: What|Removed |Added Depends on||700427, 700345, 700335 -- 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 700566] New: Review Request: stanse - An automatic bug-finding tool for C
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: stanse - An automatic bug-finding tool for C https://bugzilla.redhat.com/show_bug.cgi?id=700566 Summary: Review Request: stanse - An automatic bug-finding tool for C Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: kk...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://kklic.fedorapeople.org/stanse.spec SRPM URL: http://kklic.fedorapeople.org/stanse-1.2.0-1.fc14.src.rpm Description: An implementation of modern approaches to static analysis and automatic bug finding for programs written in C. Stanse can automatically discover some types of bugs in real-life sized projects. $ rpmlint stanse-1.2.0-1.fc14.src.rpm stanse.src: W: invalid-url Source0: stanse-1.2.0.tar.bz2 1 packages and 0 specfiles checked; 0 errors, 1 warnings. $ rpmlint stanse-1.2.0-1.fc14.i386.rpm stanse.i386: W: no-manual-page-for-binary st_xml_bugcount stanse.i386: W: no-manual-page-for-binary stcc stanse.i386: W: no-manual-page-for-binary stpreproc stanse.i386: W: no-manual-page-for-binary stanse stanse.i386: W: no-manual-page-for-binary stcparser-c stanse.i386: W: no-manual-page-for-binary stanse-make 1 packages and 0 specfiles checked; 0 errors, 6 warnings. $ rpmlint stanse-javadoc-1.2.0-1.fc14.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint stanse-web-1.2.0-1.fc14.noarch.rpm stanse-web.noarch: W: no-documentation stanse-web.noarch: W: no-manual-page-for-binary st_xml2sqlite3 1 packages and 0 specfiles checked; 0 errors, 2 warnings. $ rpmlint stanse-debuginfo-1.2.0-1.fc14.i386.rpm stanse-debuginfo.i386: E: debuginfo-without-sources 1 packages and 0 specfiles checked; 1 errors, 0 warnings. - debuginfo needs fixing -- 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 700345] Review Request: com.inamik.utils.tableformatter - A set of Java classes to print text in tabulated form
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=700345 Karel Klíč kk...@redhat.com changed: What|Removed |Added Blocks||700566 -- 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 672951] Review Request: python-elixir - A declarative mapper for SQLAlchemy
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=672951 Jeffrey Ness jeffrey.n...@rackspace.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||WONTFIX Last Closed||2011-04-28 12:54:14 --- Comment #18 from Jeffrey Ness jeffrey.n...@rackspace.com 2011-04-28 12:54:14 EDT --- Seems with all the python26 packages we are porting to EPEL (and making the needed byte-code compile changes to the root python26 package) this one got mixed in.. My apologizes this should of been one of the packages we contacted EPEL python-elixir maintainer for. I'm going to close out this bug and work with EPEL maintainer. -- 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 629324] Review Request: python-zc-buildout - System for managing development buildouts
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=629324 Robin Lee robinlee.s...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Fixed In Version||python-zc-buildout-1.5.2-1. ||fc16 Resolution||RAWHIDE Last Closed||2011-04-28 12:57:24 -- 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 698681] Review Request: grilo - Content discovery 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=698681 --- Comment #8 from Dennis Gilmore den...@ausil.us 2011-04-28 13:05:36 EDT --- 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 699208] Review Request: perl-ORLite-Statistics - Statistics enhancement package for ORLite
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=699208 --- Comment #3 from Dennis Gilmore den...@ausil.us 2011-04-28 13:06:42 EDT --- 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 700348] Review Request: mingw32-glib-networking - MinGW Windows glib-networking 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=700348 --- Comment #5 from Dennis Gilmore den...@ausil.us 2011-04-28 13:07:51 EDT --- 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 700571] New: Review Request: spindown - Daemon that can spindown idle disks
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: spindown - Daemon that can spindown idle disks https://bugzilla.redhat.com/show_bug.cgi?id=700571 Summary: Review Request: spindown - Daemon that can spindown idle disks Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mcer...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://www.physics.muni.cz/~cermak/spindown/spindown.spec SRPM URL: http://www.physics.muni.cz/~cermak/spindown/spindown-0.4.0-1.fc14.src.rpm Patch0 URL: http://www.physics.muni.cz/~cermak/spindown/spindown-0.4.0-initscript.patch Patch1 URL: http://www.physics.muni.cz/~cermak/spindown/spindown-0.4.0-Makefile.patch Description: Hi! I had a look at the Package maintainers wishlist [1] and decided to get the spindown daemon packaged for Fedora. I would appreciate a review. + rpmlint spindown.spec spindown.spec: W: invalid-url Source0: http://spindown.googlecode.com/files/spindown-0.4.0.tar.gz HTTP Error 404: Not Found 0 packages and 1 specfiles checked; 0 errors, 1 warnings. + rpmlint spindown-0.4.0-1.fc14.src.rpm spindown.src: W: name-repeated-in-summary C spindown spindown.src: W: spelling-error %description -l en_US sg - SG, chg, cig spindown.src: W: spelling-error %description -l en_US swappable - trappable, mappable, unflappable spindown.src: W: spelling-error %description -l en_US hda - had, hd, ha spindown.src: W: spelling-error %description -l en_US sdb - sd, sb, db spindown.src: W: invalid-url Source0: http://spindown.googlecode.com/files/spindown-0.4.0.tar.gz HTTP Error 404: Not Found 1 packages and 0 specfiles checked; 0 errors, 6 warnings. + rpmlint spindown-0.4.0-1.fc14.x86_64.rpm spindown.x86_64: W: name-repeated-in-summary C spindown spindown.x86_64: W: spelling-error %description -l en_US sg - SG, chg, cig spindown.x86_64: W: spelling-error %description -l en_US swappable - trappable, mappable, unflappable spindown.x86_64: W: spelling-error %description -l en_US hda - had, hd, ha spindown.x86_64: W: spelling-error %description -l en_US sdb - sd, sb, db spindown.x86_64: W: no-documentation spindown.x86_64: W: no-manual-page-for-binary spindownd 1 packages and 0 specfiles checked; 0 errors, 7 warnings. --- [1] http://fedoraproject.org/wiki/PackageMaintainers/WishList -- 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 700571] Review Request: spindown - Daemon that can spindown idle disks
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=700571 Martin Cermak mcer...@redhat.com changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- 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 672246] Review Request: perl-Statistics-Basic - A collection of very basic statistics modules
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=672246 Iain Arnell iarn...@gmail.com changed: What|Removed |Added CC||iarn...@gmail.com --- Comment #11 from Iain Arnell iarn...@gmail.com 2011-04-28 13:29:17 EDT --- Petr, would you mind branching this for f14 as well? It's a dependency of perl-ORLite-Statistics. -- 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
