[Bug 699209] Review Request: perl-ORLite-Mirror - Extend ORLite to support remote SQLite databases
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=699209
--- Comment #2 from Iain Arnell 2011-04-25 02:52:58 EDT ---
(In reply to comment #1)
> [x] MUST: The spec file must be written in American English.
> - Please correct the small spelling error.
fixed: readonly -> read-only
> [x] MUST: Packages must not own files or directories already owned by other
> packages.
>
> %{perl_vendorlib}/*
> %{_mandir}/man3/*
>
> Your package owns the folders according to the macros. You should use some
> more
> special folders here, e.g. %{_mandir}/man3/ORLite::Mirror.3pm.*. Use the
> asterisk anyway because we don't refer to a certain compression format here.
This is standard practice for perl modules and mandated in the guidelines:
https://fedoraproject.org/wiki/Packaging:Perl#Directory_Ownership
Spec URL: http://fedorapeople.org/~iarnell/review/perl-ORLite-Mirror.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-ORLite-Mirror-1.20-2.fc16.src.rpm
Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3023633
--
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 593559] Review Request: protobuf-c - C bindings for Google's Protocol Buffers
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=593559 Martin Gieseking changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #25 from Martin Gieseking 2011-04-25 02:30:03 EDT --- OK, the package is ready now. The next step is to request a Git repository with the distro branches you're planning to maintain. See here for further information: https://fedoraproject.org/wiki/PackageMaintainers/CVSAdminProcedure 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 697767] Review Request: rubygem-spruz - Useful tools library in Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=697767 --- Comment #12 from Fedora Update System 2011-04-25 02:08:39 EDT --- rubygem-spruz-0.2.5-3.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/rubygem-spruz-0.2.5-3.fc13 -- 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 697767] Review Request: rubygem-spruz - Useful tools library in Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=697767 --- Comment #10 from Fedora Update System 2011-04-25 02:08:23 EDT --- rubygem-spruz-0.2.5-3.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/rubygem-spruz-0.2.5-3.fc14 -- 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 697767] Review Request: rubygem-spruz - Useful tools library in Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=697767 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 697767] Review Request: rubygem-spruz - Useful tools library in Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=697767 --- Comment #11 from Fedora Update System 2011-04-25 02:08:31 EDT --- rubygem-spruz-0.2.5-3.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/rubygem-spruz-0.2.5-3.fc15 -- 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 635126] Review Request: rubygem-test-unit - Improved version of Test::Unit bundled in Ruby 1.8.x
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=635126 James Ni changed: What|Removed |Added CC||j...@redhat.com --- Comment #9 from James Ni 2011-04-25 02:01:51 EDT --- Hi, Could you help to build it on epel6? or i can help you to build. -- 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 697780] Review Request: rubygem-daemon_controller - A library for implementing daemon management capabilities
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=697780 --- Comment #11 from Fedora Update System 2011-04-25 01:58:33 EDT --- rubygem-daemon_controller-0.2.6-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/rubygem-daemon_controller-0.2.6-1.fc15 -- 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 697780] Review Request: rubygem-daemon_controller - A library for implementing daemon management capabilities
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=697780 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 697780] Review Request: rubygem-daemon_controller - A library for implementing daemon management capabilities
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=697780 --- Comment #12 from Fedora Update System 2011-04-25 01:58:41 EDT --- rubygem-daemon_controller-0.2.6-1.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/rubygem-daemon_controller-0.2.6-1.fc13 -- 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 697780] Review Request: rubygem-daemon_controller - A library for implementing daemon management capabilities
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=697780 --- Comment #10 from Fedora Update System 2011-04-25 01:58:25 EDT --- rubygem-daemon_controller-0.2.6-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/rubygem-daemon_controller-0.2.6-1.fc14 -- 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 690728] Review Request: Nitrate - A test case management system written in Django
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=690728 --- Comment #16 from Yuguang Wang 2011-04-25 01:57:07 EDT --- @Mario, I've read again the changelog guidelines and applied package change infos in spec file, and removed unnecessary release infos: https://fedorahosted.org/nitrate/browser/trunk/nitrate/nitrate.spec Thanks! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 697779] Review Request: rubygem-file-tail - File::Tail for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=697779 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 697779] Review Request: rubygem-file-tail - File::Tail for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=697779 --- Comment #13 from Fedora Update System 2011-04-25 01:08:00 EDT --- rubygem-file-tail-1.0.5-3.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/rubygem-file-tail-1.0.5-3.fc13 -- 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 697779] Review Request: rubygem-file-tail - File::Tail for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=697779 --- Comment #14 from Fedora Update System 2011-04-25 01:08:08 EDT --- rubygem-file-tail-1.0.5-3.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/rubygem-file-tail-1.0.5-3.fc15 -- 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 697779] Review Request: rubygem-file-tail - File::Tail for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=697779 --- Comment #12 from Fedora Update System 2011-04-25 01:07:52 EDT --- rubygem-file-tail-1.0.5-3.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/rubygem-file-tail-1.0.5-3.fc14 -- 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 699316] New: Review Request: inkscape-textext - Textext is an extension for Inkscape that allows adding LaTeX-generated editable text objects to the SVG drawings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: inkscape-textext - Textext is an extension for Inkscape that allows adding LaTeX-generated editable text objects to the SVG drawings https://bugzilla.redhat.com/show_bug.cgi?id=699316 Summary: Review Request: inkscape-textext - Textext is an extension for Inkscape that allows adding LaTeX-generated editable text objects to the SVG drawings Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: wv...@wvega.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://files.wvega.com/fedora/inkscape-textext/inkscape-textext.spec SRPM URL: http://files.wvega.com/fedora/inkscape-textext/inkscape-textext-0.4.4-1.fc14.src.rpm Description: Textext is an extension for Inkscape that allows adding LaTeX-generated text objects to your SVG drawing. These text objects can be edited after creating them. This brings some of the power of TeX typesetting to Inkscape. Notes: - This is my first package and I'm going to need a sponsor. - The package is for version 0.4.4 of the extension which requires Inkscape 0.46 or later. It works for me with inkscape-0.48.1-1.fc14.x86_64 - rpmlint shows no warnings and the package builds in mock. -- 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 699316] Review Request: inkscape-textext - Textext is an extension for Inkscape that allows adding LaTeX-generated editable text objects to the SVG drawings
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=699316 Willington Vega 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 692541] Review Request: rep-gtk - GTK+ binding for librep Lisp environment
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=692541 --- Comment #14 from Fedora Update System 2011-04-24 22:41:13 EDT --- rep-gtk-0.90.5-4.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 678924] Review Request: perl-Math-Random-ISAAC-XS - C implementation of the ISAAC PRNG algorithm
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=678924 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||perl-Math-Random-ISAAC-XS-1 ||.004-2.fc15 Resolution||ERRATA Last Closed||2011-04-24 22:40:20 -- 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 676129] Review Request: qconf - Allows you to have a nice configure script for your qmake-based project
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=676129 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||qconf-1.4-2.fc15 Resolution||ERRATA Last Closed||2011-04-24 22:42:09 -- 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 692541] Review Request: rep-gtk - GTK+ binding for librep Lisp environment
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=692541 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||rep-gtk-0.90.5-4.fc15 Resolution||ERRATA Last Closed||2011-04-24 22:41:18 -- 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 680657] Review Request: mpdas - An MPD audioscrobbling client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=680657 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||mpdas-0.3.0-2.fc15 Resolution||ERRATA Last Closed||2011-04-24 22:43:02 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 678924] Review Request: perl-Math-Random-ISAAC-XS - C implementation of the ISAAC PRNG algorithm
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=678924 --- Comment #10 from Fedora Update System 2011-04-24 22:40:14 EDT --- perl-Math-Random-ISAAC-XS-1.004-2.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676129] Review Request: qconf - Allows you to have a nice configure script for your qmake-based project
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=676129 --- Comment #32 from Fedora Update System 2011-04-24 22:42:04 EDT --- qconf-1.4-2.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 680657] Review Request: mpdas - An MPD audioscrobbling client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=680657 --- Comment #16 from Fedora Update System 2011-04-24 22:42:57 EDT --- mpdas-0.3.0-2.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 633063] Review Request: python-txamqp - A Python library for communicating with AMQP peers and brokers using Twisted
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=633063 Fedora Update System changed: What|Removed |Added Status|NEW |MODIFIED -- 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 633063] Review Request: python-txamqp - A Python library for communicating with AMQP peers and brokers using Twisted
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=633063 --- Comment #4 from Fedora Update System 2011-04-24 22:39:22 EDT --- python-txamqp-0.3-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/python-txamqp-0.3-1.el6 -- 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 688121] Review Request: mod_flvx - FLV progressive download streaming for the Apache HTTP Server
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=688121
Trever Fischer changed:
What|Removed |Added
Flag||needinfo?(redhat-bugzilla@l
||inuxnetz.de)
--- Comment #4 from Trever Fischer 2011-04-24 22:01:44 EDT ---
+ = OK
- = NA
? = issue
+ Package meets naming and packaging guidelines
? Spec file matches base package name.
+ Spec has consistent macro usage.
+ Meets Packaging Guidelines.
+ License
+ License field in spec matches
? License file included in package
+ Spec in American English
+ Spec is legible.
+ Sources match upstream md5sum:
- Package needs ExcludeArch
+ BuildRequires correct
- Spec handles locales/find_lang
- Package is relocatable and has a reason to be.
+ Package has %defattr and permissions on files is good.
+ Package has a correct %clean section.
+ Package has correct buildroot
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
+ Package is code or permissible content.
- Doc subpackage needed/used.
+ Packages %doc files don't affect runtime.
- Headers/static libs in -devel subpackage.
- Spec has needed ldconfig in post and postun
- .pc files in -devel subpackage/requires pkgconfig
- .so files in -devel subpackage.
- -devel package Requires: %{name} = %{version}-%{release}
- .la files are removed.
- Package is a GUI app and has a .desktop file
+ Package compiles and builds on at least one arch.
- Package has no duplicate files in %files.
+ Package doesn't own any directories other packages own.
+ Package owns all the directories it creates.
+ No rpmlint output. (Just a warning about an invalid URL, but github is
strange, I know.)
- final provides and requires are sane:
SHOULD Items:
- Should build in mock.
- Should build on all supported archs
- Should function as described.
- Should have sane scriptlets.
- Should have subpackages require base package with fully versioned depend.
+ Should have dist tag
+ Should package latest version
I'm not entirely familiar with your Requires line. Can you explain to me why
you're executing a command to determine build requirements? No other apache
module package I know of requires this.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 689685] Review Request: Anchorman - The recording-studio-in-a-box
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=689685 --- Comment #11 from Trever Fischer 2011-04-24 21:32:51 EDT --- (In reply to comment #10) > The requested package name does not match the name in the ticket summary. > Please correct whichever is wrong and re-raise the fedora-cvs flag. I'm not sure what you mean. The package name should be 'anchorman', which is the same name that the project uses internally, and in the spec file. Are you referring to the capital "A" in the ticket title? -- 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 629027] Review Request: python-ZODB3 - Zope Object Database: Object Database and Persistence
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=629027 --- Comment #15 from Robin Lee 2011-04-24 21:19:54 EDT --- I am still maintaining the f14 branch so I didn't list it. After all, I was just expecting what you have done. I will do better next time. Thank you! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 688121] Review Request: mod_flvx - FLV progressive download streaming for the Apache HTTP Server
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=688121 Trever Fischer changed: What|Removed |Added CC||wm...@wm161.net AssignedTo|nob...@fedoraproject.org|wm...@wm161.net Flag||fedora-review? --- Comment #3 from Trever Fischer 2011-04-24 21:14:35 EDT --- I'll review 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 689685] Review Request: Anchorman - The recording-studio-in-a-box
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=689685 --- Comment #10 from Jason Tibbitts 2011-04-24 21:10:23 EDT --- The requested package name does not match the name in the ticket summary. Please correct whichever is wrong and re-raise the fedora-cvs flag. -- 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 689685] Review Request: Anchorman - The recording-studio-in-a-box
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=689685 Trever Fischer changed: What|Removed |Added Flag||fedora-cvs? --- Comment #9 from Trever Fischer 2011-04-24 20:55:11 EDT --- New Package SCM Request === Package Name: anchorman Short Description: The recording studio in-a-box Owners: tdfischer Branches: f14 f15 InitialCC: tdfischer -- 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 641572] Review Request: celt071 - Celt version 0.7.1 for mumble compatibility
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=641572 Fedora Update System changed: What|Removed |Added Fixed In Version|celt071-0.7.1-2.fc15|celt071-0.7.1-2.fc14 -- 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 636819] Review Request: gnome-exe-thumbnailer - gnome thumbnailer for exe files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=636819 Fedora Update System changed: What|Removed |Added Fixed In Version|gnome-exe-thumbnailer-0.7-4 |gnome-exe-thumbnailer-0.7-4 |.fc14 |.fc13 -- 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 678196] Review Request: perl-String-Errf - Simple sprintf-like dialect
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=678196 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #7 from Fedora Update System 2011-04-24 19:58:19 EDT --- perl-String-Errf-0.006-1.fc14 has been pushed to the Fedora 14 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 636819] Review Request: gnome-exe-thumbnailer - gnome thumbnailer for exe files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=636819 --- Comment #46 from Fedora Update System 2011-04-24 19:57:33 EDT --- gnome-exe-thumbnailer-0.7-4.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641572] Review Request: celt071 - Celt version 0.7.1 for mumble compatibility
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=641572 --- Comment #21 from Fedora Update System 2011-04-24 19:57:47 EDT --- celt071-0.7.1-2.fc14 has been pushed to the Fedora 14 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 692131] Review Request: sflphone - SIP/IAX2 compatible enterprise-class softphone for multi-way conferencing
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=692131 --- Comment #7 from nucleo 2011-04-24 18:39:40 EDT --- Bundled pjproject also contains code of third party libraries gsm, portaudio, resample, speex and srtp which are packaged 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 435090] Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension 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=435090 Fedora Update System changed: What|Removed |Added Fixed In Version|Cython-0.14.1-3.el5 |Cython-0.14.1-3.el6 -- 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 688264] Review Request: perl-Test-CPAN-Meta-YAML - Validate a META.yml file within a CPAN distribution
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=688264 --- Comment #16 from Fedora Update System 2011-04-24 17:59:18 EDT --- perl-Test-CPAN-Meta-YAML-0.17-2.el5 has been pushed to the Fedora EPEL 5 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 688264] Review Request: perl-Test-CPAN-Meta-YAML - Validate a META.yml file within a CPAN distribution
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=688264 --- Comment #15 from Fedora Update System 2011-04-24 17:58:46 EDT --- perl-Test-CPAN-Meta-YAML-0.17-2.el4 has been pushed to the Fedora EPEL 4 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 435090] Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension 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=435090 Fedora Update System changed: What|Removed |Added Fixed In Version||Cython-0.14.1-3.el5 Resolution|NEXTRELEASE |ERRATA -- 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 435090] Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension 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=435090 --- Comment #20 from Fedora Update System 2011-04-24 17:59:01 EDT --- Cython-0.14.1-3.el6 has been pushed to the Fedora EPEL 6 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 435090] Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension 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=435090 --- Comment #19 from Fedora Update System 2011-04-24 17:58:24 EDT --- Cython-0.14.1-3.el5 has been pushed to the Fedora EPEL 5 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 695281] Review Request: perl-Devel-EnforceEncapsulation - Find access violations to blessed objects
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=695281 --- Comment #5 from Fedora Update System 2011-04-24 17:33:08 EDT --- perl-Devel-EnforceEncapsulation-0.50-3.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/perl-Devel-EnforceEncapsulation-0.50-3.fc15 -- 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 695281] Review Request: perl-Devel-EnforceEncapsulation - Find access violations to blessed objects
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=695281 --- Comment #4 from Fedora Update System 2011-04-24 17:33:00 EDT --- perl-Devel-EnforceEncapsulation-0.50-3.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/perl-Devel-EnforceEncapsulation-0.50-3.el6 -- 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 695281] Review Request: perl-Devel-EnforceEncapsulation - Find access violations to blessed objects
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=695281 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 699168] Review Request: routino - Router for OpenStreetMap Data
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=699168 --- Comment #2 from Kevin Kofler 2011-04-24 17:00:24 EDT --- Spec URL: http://repo.calcforge.org/f16/routino.spec SRPM URL: http://repo.calcforge.org/f16/routino-1.5.1-2.fc14.src.rpm Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3022895 * Sun Apr 24 2011 Kevin Kofler <…> - 1.5.1-2 - Install ChangeLog (The upstream makefiles install all other docs, but not the license or the changelog. This is probably due to Debian policies, which say not to install those files. In Fedora, we want them installed. I had already fixed it for the license in -1, I also added the changelog in -2.) -- 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 699209] Review Request: perl-ORLite-Mirror - Extend ORLite to support remote SQLite databases
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=699209
Mario Blättermann changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||mari...@freenet.de
AssignedTo|nob...@fedoraproject.org|mari...@freenet.de
--- Comment #1 from Mario Blättermann 2011-04-24 15:14:40
EDT ---
$ rpmlint -v perl*
perl-ORLite-Mirror.src: I: checking
perl-ORLite-Mirror.src: W: spelling-error %description -l en_US readonly ->
read only, read-only, readoption
perl-ORLite-Mirror.src: I: checking-url
http://search.cpan.org/dist/ORLite-Mirror/ (timeout 10 seconds)
perl-ORLite-Mirror.src: I: checking-url
http://www.cpan.org/authors/id/A/AD/ADAMK/ORLite-Mirror-1.20.tar.gz (timeout 10
seconds)
perl-ORLite-Mirror.spec: I: checking-url
http://www.cpan.org/authors/id/A/AD/ADAMK/ORLite-Mirror-1.20.tar.gz (timeout 10
seconds)
1 packages and 1 specfiles checked; 0 errors, 1 warnings.
Seems to be all OK. Just a small spelling error. Nice to have cpanspec for such
packages...
OK, here we go:
-
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.
GPL+ or Artistic
[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: The file containing the text of the license(s) for the package must
be included in %doc.
[x] MUST: The spec file must be written in American English.
- Please correct the small spelling error.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source.
$ md5sum OR*
9ebd7c887b7f841348490f0b5b645597 ORLite-Mirror-1.20.tar.gz
9ebd7c887b7f841348490f0b5b645597 ORLite-Mirror-1.20.tar.gz.packaged
[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
- Succesful Koji build available.
[.] MUST: If the package does not successfully compile, build or work on an
architecture, ...
[+] MUST: All build dependencies must be listed in BuildRequires.
[.] 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: Packages must not provide RPM dependency information when that
information is not global in nature, or are otherwise handled.
[.] MUST: When filtering automatically generated RPM dependency information,
the filtering system implemented by Fedora must be used.
[+] 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.
[x] MUST: Packages must not own files or directories already owned by other
packages.
%{perl_vendorlib}/*
%{_mandir}/man3/*
Your package owns the folders according to the macros. You should use some more
special folders here, e.g. %{_mandir}/man3/ORLite::Mirror.3pm.*. Use the
asterisk anyway because we don't refer to a certain compression format here.
[+] 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, the packager SHOULD query upstream...
[+] SHOULD: Timestamps of files should be preserved.
[+] SHOULD: The reviewer should test that the package builds in mock.
See Koji build above (which uses mock anyway)
[+] SHOULD: The reviewer should
[Bug 690038] Review Request: ompl - The Open Motion Planning 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=690038 --- Comment #3 from Rich Mattes 2011-04-24 15:08:08 EDT --- Update: Spec URL: http://rmattes.fedorapeople.org/RPMS/ompl/ompl.spec SRPM URL: http://rmattes.fedorapeople.org/RPMS/ompl/ompl-0.9.2-2.fc15.src.rpm I contacted upstream about the soname, they accidentally omitted it from the release. I've patched in a library version and soversion consistent with what will be added in future releases. The python bindings can't be built without pygccxml and pyplusplus. Neither of these python modules are packaged in Fedora. The file lists have been made more specific, and I've fixed the download path. I've removed the clean section, el5 doesn't have the required dependencies to build this package (cmake 2.8 being one.) I've also added versioning for cmake and boost deps rpmlint: $ rpmlint ompl.spec ../RPMS/x86_64/ompl* 3 packages and 1 specfiles checked; 0 errors, 0 warnings. Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3022644 -- 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 690728] Review Request: Nitrate - A test case management system written in Django
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=690728 --- Comment #15 from Mario Blättermann 2011-04-24 14:50:39 EDT --- (In reply to comment #6) > Who needs all this ChangeLog stuff...? As I already wrote, the package itself > should contain a log for changes in the sources. Our package ChangeLog is for > changes to the package. No one needs overwhelming infos about fixed bugs. Again: The package ChangeLog is for any changes to the package! You don't have to write here all the release infos. They are included in the source package anywhere else. But I can't find anything in your ChangeLog which would mention the changes you did to tweak the package itself. Please read the guidelines again: http://fedoraproject.org/wiki/Packaging/Guidelines#Changelogs -- 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 629027] Review Request: python-ZODB3 - Zope Object Database: Object Database and Persistence
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=629027 Jason Tibbitts changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #14 from Jason Tibbitts 2011-04-24 14:19:00 EDT --- Well, I've unretired those branches and set him as the owner. You didn't tell me what to do with the existing f14 branch, though I assume if you didn't want it you'd either orphan or retire it so I just left it alone. I'm not really sure why you don't seem to want to provide me with the information needed to take care of your request properly, but I've done the best I think I can do. -- 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 600361] Review Request: gobi_loader - Firmware loader for Qualcomm Gobi WWAN devices
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=600361 Felix Kaechele changed: What|Removed |Added Status|ASSIGNED|CLOSED Blocks||201449(FE-DEADREVIEW) Resolution||WONTFIX Flag|fedora-review? | Last Closed||2011-04-24 14:01:17 -- 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 #29 from Richard Shaw 2011-04-24 12:39:42 EDT --- (In reply to comment #28) > 4) And then there is the issue with the unresolved symbols I've run out of > steam (and time) for now to look into that (blergh scons), maybe I'll look > into > it later today. If not feel free to do a new version in the mean time > addressing all the other concerns. I've been working with Dave Platter, the more or less creator of the *nix version of this library and Suse package maintainer, and he's started working on the undefined-non-weak-symbol issues. Apparently Suse doesn't use rpmlint so he wasn't aware of the problem. He fixed two of the errors by updating the SConscript files. I'll attach the patch as a template. Two down, 243 to go. -- 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 #30 from Richard Shaw 2011-04-24 12:40:22 EDT --- Created attachment 494541 --> https://bugzilla.redhat.com/attachment.cgi?id=494541 Patch to fix undefined-non-weak-symbol issues in libbuffer. -- 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 675557] Review Request: matreshka - set of Ada libraries to help to develop information systems
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=675557 --- Comment #12 from Björn Persson 2011-04-24 11:09:00 EDT --- Yikes! There's a "pragma Suppress (All_Checks)" in matreshka_league__release.adc. It's commented out, so it does no harm at this time, but I really hope it won't be turned on in some future release. -- 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 696926] Review Request: rhevsh - A command-line interface to Red Hat Enterprise Virtualization
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=696926 --- Comment #5 from Jason Tibbitts 2011-04-24 10:45:41 EDT --- This request is invalid; the tag is "Owners:", not "Owner:", so the script did not process it. I fixed things up. 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 691096] Review Request: iperf3 - Measurement tool for TCP/UDP bandwidth performance
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=691096 --- Comment #11 from Jason Tibbitts 2011-04-24 10:43:55 EDT --- You requested an invalid branch "e16". I'm not sure if this was a typo for "f16" (in which case it's too early) or for "el6" so I've simply done the f14 and f15 branches. If you did want an el6 branch, feel free to submit a change request. 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 695281] Review Request: perl-Devel-EnforceEncapsulation - Find access violations to blessed objects
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=695281 --- Comment #3 from Jason Tibbitts 2011-04-24 10:44: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 675557] Review Request: matreshka - set of Ada libraries to help to develop information systems
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=675557
--- Comment #11 from Björn Persson 2011-04-24
07:41:59 EDT ---
· I see that you added /usr/include/matreshka to the base package. I think
matreshka-devel is the best package to put this directory in, because only
-devel subpackages have files under /usr/include/matreshka and they all depend
on matreshka-devel (directly or indirectly). The way it is now, there will be
an empty and unnecessary /usr/include/matreshka directory when the compiled
libraries are installed but the -devel packages are not. It's not a big issue
though.
· I had to add quotes in the export command to get the build to work, like
this:
export GPRBUILD_FLAGS="%{GNAT_optflags}"
Without the quotes Bash set GPRBUILD_FLAGS to "-cargs" and then tried to export
variables named "-O2", "-g" and so on.
· Even with the quotes, the parameters from GNAT_optflags weren't passed to GCC
and Gnatlink. I tried making GPRBUILD_FLAGS a parameter to Make instead of an
environment variable, and then the parameters were passed correctly. Apparently
you'll need to make more changes to the makefiles if you want them to pick up
this variable from the environment.
· The build log in Koji shows GPRBUILD_FLAGS being passed to Make as a
parameter. Obviously the package you submitted to Koji was different from the
one you linked to here, despite the version and release numbers being the same.
Don't do that.
· You're neglecting the changelog. Every time you make changes to the package
you should increment the release number and write a new changelog entry.
· The project files have too generic names. Project file names are a shared
namespace that we need to be careful not to pollute. A name like "sql.gpr" is
too prone to collisions. It would be a good name if there were one library that
could claim to be *the* SQL library for Ada, but that's not the case. There are
several database interface libraries and it wouldn't be fair to let one of them
claim the name "SQL". "fastcgi.gpr" is a similar case. Somebody might want to
package another library that also implements the FastCGI interface, and there's
no reason why one library should have a greater right to that name than
another.
To prevent name collisions the Ada packaging policies we wrote state that
either the name of the library must be included in the filenames of the project
files (that is, "matreshka_sql.gpr" or something similar), or else they must be
in a subdirectory named after the library (in which case other projects would
use «with "matreshka/sql.gpr";» et cetera to import them).
Whether you change the names or use a subdirectory, it would be best if this
change were made upstream so that the project files can be referred to in the
same way in all distributions that include Matreshka as well as when users
download the source package and compile it themselves. Otherwise project files
will have to be patched left and right.
--
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 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
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=674008 --- Comment #42 from Rosen Diankov 2011-04-24 06:18:54 EDT --- we just released 0.3.1: https://sourceforge.net/mailarchive/forum.php?thread_name=BANLkTi%3Dibi8J9SXsuSi99R%3DWEiZd8pDx7w%40mail.gmail.com&forum_name=openrave-users we also started a page on how to install openrave on different platforms: http://openrave.programmingvision.com/en/main/install.html it would be great to add Fedora on that list. what do you think about the source rpm proposal? -- 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 699209] New: Review Request: perl-ORLite-Mirror - Extend ORLite to support remote SQLite databases
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-ORLite-Mirror - Extend ORLite to support remote SQLite databases Alias: perl-ORLite-Mirror https://bugzilla.redhat.com/show_bug.cgi?id=699209 Summary: Review Request: perl-ORLite-Mirror - Extend ORLite to support remote SQLite databases Product: Fedora Version: rawhide Platform: Unspecified URL: http://search.cpan.org/dist/ORLite-Mirror/ OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: iarn...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://fedorapeople.org/~iarnell/review/perl-ORLite-Mirror.spec SRPM URL: http://fedorapeople.org/~iarnell/review/perl-ORLite-Mirror-1.20-1.fc16.src.rpm Description: ORLite provides a readonly ORM API when it loads a readonly SQLite database from your local system. By combining this capability with LWP, ORLite::Mirror goes one step better and allows you to load a SQLite database from any arbitrary URI in readonly form as well. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3022230 *rt-0.10_01 -- 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] New: 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. Summary: Review Request: perl-ORLite-Statistics - Statistics enhancement package for ORLite https://bugzilla.redhat.com/show_bug.cgi?id=699208 Summary: Review Request: perl-ORLite-Statistics - Statistics enhancement package for ORLite Product: Fedora Version: rawhide Platform: Unspecified URL: http://search.cpan.org/dist/ORLite-Statistics/ OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: iarn...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://fedorapeople.org/~iarnell/review/perl-ORLite-Statistics.spec SRPM URL: http://fedorapeople.org/~iarnell/review/perl-ORLite-Statistics-0.03-1.fc16.src.rpm Description: This is an enhancement module for ORLite table classes, designed to provide easy integration with the Statistics::Base module. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=308 *rt-0.10_01 -- 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 699205] New: Review Request: perl-Graph-Easy - Convert or render graphs as ASCII, HTML, SVG or via Graphviz
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-Graph-Easy - Convert or render graphs as ASCII, HTML, SVG or via Graphviz Alias: perl-Graph-Easy https://bugzilla.redhat.com/show_bug.cgi?id=699205 Summary: Review Request: perl-Graph-Easy - Convert or render graphs as ASCII, HTML, SVG or via Graphviz Product: Fedora Version: rawhide Platform: Unspecified URL: http://search.cpan.org/dist/Graph-Easy/ OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: iarn...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://fedorapeople.org/~iarnell/review/perl-Graph-Easy.spec SRPM URL: http://fedorapeople.org/~iarnell/review/perl-Graph-Easy-0.70-1.fc16.src.rpm Description: Graph::Easy lets you generate graphs consisting of various shaped nodes connected by edges (with optional labels). It can read and write graphs in a variety of formats, as well as render them via its own grid-based layouter. Since the layouter works on a grid (manhattan layout), the output is most useful for flow charts, network diagrams, or hierarchy trees. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=304 *rt-0.10_01 -- 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
