[Bug 497805] Review Request: cppcheck - A tool for static C/C++ code analysis
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=497805 --- Comment #4 from Fedora Update System upda...@fedoraproject.org 2009-04-30 02:04:16 EDT --- cppcheck-1.31-1.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/cppcheck-1.31-1.fc10 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497805] Review Request: cppcheck - A tool for static C/C++ code analysis
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=497805 --- Comment #5 from Fedora Update System upda...@fedoraproject.org 2009-04-30 02:04:57 EDT --- cppcheck-1.31-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/cppcheck-1.31-1.fc11 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 490576] Review Request: bibtex2html - Collection of tools for translating from BibTeX to HTML
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=490576 --- Comment #14 from Jussi Lehtola jussi.leht...@iki.fi 2009-04-30 02:11:29 EDT --- Okay. I can sponsor you, but first I want to see that you know the packaging guidelines. This means you must do a) informal reviews of other packages [you can do real ones after you have been sponsored] and b) submit a few other packages for 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491618] Review Request: mingw32-libsqlite3x - MinGW Windows C++ wrapper for the sqlite database 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=491618 --- Comment #8 from Fedora Update System upda...@fedoraproject.org 2009-04-30 02:13:27 EDT --- mingw32-libsqlite3x-20071018-7.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/mingw32-libsqlite3x-20071018-7.fc11 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498203] Review Request: perl-YAML-LibYAML - YAML::LibYAML Perl module
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=498203 Marcela Maslanova mmasl...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #5 from Marcela Maslanova mmasl...@redhat.com 2009-04-30 02:09:37 EDT --- New Package CVS Request === Package Name: perl-YAML-LibYAML Short Description: YAML Serialization using XS and libyaml Owners: mmaslano Branches: F-11 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498193] Review Request: perl-ExtUtils-InferConfig - Infer Perl Configuration for non-running interpreters
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=498193 --- Comment #2 from Marcela Maslanova mmasl...@redhat.com 2009-04-30 02:19:05 EDT --- http://koji.fedoraproject.org/koji/taskinfo?taskID=1329843 http://mmaslano.fedorapeople.org/review/perl-ExtUtils-InferConfig-1.03-2.fc11.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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491618] Review Request: mingw32-libsqlite3x - MinGW Windows C++ wrapper for the sqlite database 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=491618 --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2009-04-30 02:17:07 EDT --- mingw32-libsqlite3x-20071018-7.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/mingw32-libsqlite3x-20071018-7.fc10 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498371] New: Review Request: fedora-devshell - Fedora Developer's Toolbox
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: fedora-devshell - Fedora Developer's Toolbox https://bugzilla.redhat.com/show_bug.cgi?id=498371 Summary: Review Request: fedora-devshell - Fedora Developer's Toolbox Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: loupgaroubl...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://ynemoy.fedorapeople.org/review/fedora-devshell.spec SRPM URL: http://ynemoy.fedorapeople.org/review/fedora-devshell-0.1.0-1.fc10.src.rpm Description: Fedora Devshell is a developers toolbox for creating packages and developing software for Fedora. It aims to simplify the process of creating and maintaining packages in the Fedora repositories, and simplify the workflow between other Fedora components. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498191] Review Request: perl-ExtUtils-InstallPAR - Install .par's into any installed perl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=498191 --- Comment #2 from Marcela Maslanova mmasl...@redhat.com 2009-04-30 02:19:02 EDT --- http://koji.fedoraproject.org/koji/taskinfo?taskID=1329831 http://mmaslano.fedorapeople.org/review/perl-ExtUtils-InstallPAR-0.03-2.fc11.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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492479] Review Request: razertool - Tool for controlling Razer Copperhead mice
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=492479 --- Comment #19 from Fedora Update System upda...@fedoraproject.org 2009-04-30 02:41:09 EDT --- razertool-0.0.7-5.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/razertool-0.0.7-5.fc10 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492479] Review Request: razertool - Tool for controlling Razer Copperhead mice
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=492479 --- Comment #20 from Fedora Update System upda...@fedoraproject.org 2009-04-30 02:51:26 EDT --- razertool-0.0.7-5.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/razertool-0.0.7-5.fc11 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492130] Review Request: mingw32-gtkmm24 - MinGW Windows C++ interface for GTK2 (a GUI library for 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=492130 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2009-04-30 03:00:33 EDT --- mingw32-gtkmm24-2.14.3-1.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/mingw32-gtkmm24-2.14.3-1.fc10 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492479] Review Request: razertool - Tool for controlling Razer Copperhead mice
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=492479 Andreas Osowski th0...@mkdir.name changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 475017] Review Request: l2fprod-common - In JavaSE missing Swing components, inspired from modern user interfaces
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=475017 --- Comment #21 from Sandro Mathys s...@sandro-mathys.ch 2009-04-30 03:04:11 EDT --- Sure will do, thanks. New Package CVS Request === Package Name: l2fprod-common Short Description: In JavaSE missing Swing components, inspired from modern user interfaces Owners: red Branches: F-9 F-10 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497634] Review Request: perl-App-Daemon - Start an Application as a Daemon
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=497634 --- Comment #7 from Fedora Update System upda...@fedoraproject.org 2009-04-30 03:34:31 EDT --- perl-App-Daemon-0.06-1.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/perl-App-Daemon-0.06-1.fc10 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498216] Review Request: perl-Perl-Tags - Generate Ctags style tags for Perl sourcecode
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=498216 --- Comment #5 from Fedora Update System upda...@fedoraproject.org 2009-04-30 03:35:27 EDT --- perl-Perl-Tags-0.26-1.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/perl-Perl-Tags-0.26-1.fc10 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498216] Review Request: perl-Perl-Tags - Generate Ctags style tags for Perl sourcecode
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=498216 --- Comment #4 from Fedora Update System upda...@fedoraproject.org 2009-04-30 03:35:22 EDT --- perl-Perl-Tags-0.26-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/perl-Perl-Tags-0.26-1.fc11 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498220] Review Request: perl-Module-Used - Find modules loaded by Perl code without running it
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=498220 --- Comment #4 from Fedora Update System upda...@fedoraproject.org 2009-04-30 03:36:11 EDT --- perl-Module-Used-1.2.0-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/perl-Module-Used-1.2.0-1.fc11 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498220] Review Request: perl-Module-Used - Find modules loaded by Perl code without running it
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=498220 --- Comment #5 from Fedora Update System upda...@fedoraproject.org 2009-04-30 03:36:16 EDT --- perl-Module-Used-1.2.0-1.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/perl-Module-Used-1.2.0-1.fc10 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498216] Review Request: perl-Perl-Tags - Generate Ctags style tags for Perl sourcecode
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=498216 Iain Arnell iarn...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #6 from Iain Arnell iarn...@gmail.com 2009-04-30 03:36:52 EDT --- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497634] Review Request: perl-App-Daemon - Start an Application as a Daemon
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=497634 Iain Arnell iarn...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #8 from Iain Arnell iarn...@gmail.com 2009-04-30 03:36:46 EDT --- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498220] Review Request: perl-Module-Used - Find modules loaded by Perl code without running it
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=498220 Iain Arnell iarn...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #6 from Iain Arnell iarn...@gmail.com 2009-04-30 03:36:55 EDT --- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497634] Review Request: perl-App-Daemon - Start an Application as a Daemon
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=497634 --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2009-04-30 03:34:26 EDT --- perl-App-Daemon-0.06-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/perl-App-Daemon-0.06-1.fc11 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 472144] Review Request: tvbrowser - Free EPG for over 500 stations.
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=472144 --- Comment #19 from Sandro Mathys s...@sandro-mathys.ch 2009-04-30 03:44:41 EDT --- Spec URL: http://red.fedorapeople.org/SRPMS/tvbrowser.spec SRPM URL: http://red.fedorapeople.org/SRPMS/tvbrowser-2.7.3-2.fc11.src.rpm Changes in the l2fprod-common package made it necessary to adapt the path of one dependency. --- As an update to the legal blocker: - l2fprod-common now good - skinlf still incompatible ...both have the same upstream and I'm positive that all issues will be fixed (i.e. upstream prefers ASL 2.0 but didn't apply the changes to this new license everywhere yet) - but upstream isn't very responsive and seems not to have enough time to quickly fix 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492259] Review Request: tucan - Manager for downloads and uploads at hosting sites
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=492259 --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2009-04-30 04:04:09 EDT --- tucan-0.3.7-0.1.alpha.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/tucan-0.3.7-0.1.alpha.fc11 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492259] Review Request: tucan - Manager for downloads and uploads at hosting sites
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=492259 --- Comment #15 from Fedora Update System upda...@fedoraproject.org 2009-04-30 04:05:11 EDT --- tucan-0.3.7-0.1.alpha.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/tucan-0.3.7-0.1.alpha.fc10 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497640] Review Request: rubygem-RedCloth - Textile parser 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=497640 --- Comment #1 from Dan Horák d...@danny.cz 2009-04-30 04:36:09 EDT --- first notes: - the license is MIT (see COPYING and README from inside of the gem) - it doesn't build in mock for both F-10 and Rawhide on x86_64 + gem install --local --install-dir /builddir/build/BUILDROOT/rubygem-RedCloth-4.1.9-1.fc11.x86_64/usr/lib/ruby/gems/1.8 --force --rdoc /builddir/build/SOURCES/RedCloth-4.1.9.gem ERROR: Error installing /builddir/build/SOURCES/RedCloth-4.1.9.gem: ERROR: Failed to build gem native extension. /usr/bin/ruby extconf.rb install --local --install-dir /builddir/build/BUILDROOT/rubygem-RedCloth-4.1.9-1.fc11.x86_64/usr/lib/ruby/gems/1.8 --force --rdoc /builddir/build/SOURCES/RedCloth-4.1.9.gem can't find header files for ruby. Gem files will remain installed in /builddir/build/BUILDROOT/rubygem-RedCloth-4.1.9-1.fc11.x86_64/usr/lib/ruby/gems/1.8/gems/RedCloth-4.1.9 for inspection. - is it possible to run the included testsuite during the build (using the %check section)? -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498193] Review Request: perl-ExtUtils-InferConfig - Infer Perl Configuration for non-running interpreters
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=498193 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from Parag AN(पराग) panem...@gmail.com 2009-04-30 04:40:26 EDT --- Review: + package builds in mock (rawhide i586). koji Build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1329843 + rpmlint is silent for SRPM and for RPM. + source files match upstream url bc428af291b36b1c0ffa1e5c96408f72 ExtUtils-InferConfig-1.03.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + make test gave All tests successful. Files=4, Tests=2213, 2 wallclock secs ( 0.45 usr 0.03 sys + 1.45 cusr 0.09 csys = 2.02 CPU) + Package perl-ExtUtils-InferConfig-1.03-2.fc12.noarch = Provides: perl(ExtUtils::InferConfig) = 1.03 Requires: perl(Carp) perl(Config) perl(IPC::Cmd) perl(strict) perl(vars) + Not a GUI application 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498191] Review Request: perl-ExtUtils-InstallPAR - Install .par's into any installed perl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=498191 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from Parag AN(पराग) panem...@gmail.com 2009-04-30 04:40:20 EDT --- Review: + package builds in mock (rawhide i586). koji Build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1329831 + rpmlint is silent for SRPM and for RPM. + source files match upstream url 88832cdff670a78bb44e25cadd700f4d ExtUtils-InstallPAR-0.03.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + make test gave All tests successful. Files=3, Tests=1, 0 wallclock secs ( 0.03 usr 0.00 sys + 0.07 cusr 0.01 csys = 0.11 CPU) + Package perl-ExtUtils-InstallPAR-0.03-2.fc12.noarch = Provides: perl(ExtUtils::InstallPAR) = 0.03 Requires: perl(Carp) perl(Config) perl(Exporter) perl(File::Spec) perl(PAR::Dist) perl(strict) perl(vars) + Not a GUI application 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497663] Review Request: perl-HTML-StripScripts-Parser - XSS filter using HTML::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=497663 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review+ --- Comment #1 from Parag AN(पराग) panem...@gmail.com 2009-04-30 04:43:32 EDT --- Review: + package builds in mock (rawhide i586). koji Build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1330026 + rpmlint is silent for SRPM and for RPM. + source files match upstream url 0fb38dc2c632844883f3e716b056a88a HTML-StripScripts-Parser-1.02.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + make test gave All tests successful. Files=6, Tests=217, 0 wallclock secs ( 0.10 usr 0.02 sys + 0.43 cusr 0.06 csys = 0.61 CPU) + Package perl-HTML-StripScripts-Parser-1.02-1.fc12.noarch = Provides: perl(HTML::StripScripts::Parser) = 1.02 Requires: perl(HTML::Parser) perl(HTML::StripScripts) perl(base) perl(strict) perl(vars) + Not a GUI application 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491650] Review Request: libica - Library for accessing ICA hardware crypto on IBM zSeries
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=491650 --- Comment #2 from Dan Horák d...@danny.cz 2009-04-30 04:49:50 EDT --- (In reply to comment #1) Is the warning regarding no documentation considered fatal? Other than that, this package looks fine to me. I'll approve this and the documentation issue should be fixed or ignored. The warning is harmless here, because there is no documentation other than the AUTHORS and LICENSE files and these are included in the main package that will be always installed (it is the library). New scratch build using the same srpm this time for F-11 is at https://s390.koji.fedoraproject.org/koji/taskinfo?taskID=55252 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498193] Review Request: perl-ExtUtils-InferConfig - Infer Perl Configuration for non-running interpreters
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=498193 Marcela Maslanova mmasl...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Marcela Maslanova mmasl...@redhat.com 2009-04-30 04:52:35 EDT --- New Package CVS Request === Package Name: perl-ExtUtils-InferConfig Short Description: Infer Perl Configuration for non-running interpreter Owners: mmaslano Branches: F-11 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498191] Review Request: perl-ExtUtils-InstallPAR - Install .par's into any installed perl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=498191 Marcela Maslanova mmasl...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Marcela Maslanova mmasl...@redhat.com 2009-04-30 04:53:07 EDT --- New Package CVS Request === Package Name: perl-ExtUtils-InstallPAR Short Description: Install .par's into any installed perl Owners: mmaslano Branches: F-11 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497663] Review Request: perl-HTML-StripScripts-Parser - XSS filter using HTML::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=497663 Xavier Bachelot xav...@bachelot.org changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Xavier Bachelot xav...@bachelot.org 2009-04-30 04:59:49 EDT --- Thanks for yet another review Parag :-) New Package CVS Request === Package Name: perl-HTML-StripScripts-Parser Short Description: XSS filter using HTML::Parser Owners: xavierb Branches: F-9 F-10 F-11 EL-5 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492221] Review Request: dssi-vst - DSSI plugin wrapper for VST plugins
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=492221 --- Comment #1 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-04-30 05:02:20 EDT --- I separated the wine related parts in a -wine subpackage. This subpackage will be built only on ix86 and I will ask releng to push the ix86 package to x86_64 repo. This way we will keep all ix86 and x86_64 users happy. Spec URL: http://oget.fedorapeople.org/review/dssi-vst.spec SRPM URL: http://oget.fedorapeople.org/review/dssi-vst-0.8-2.fc10.src.rpm Changelog: 0.8-2 - Make a ix86 only -wine subpackage Builds fine in koji rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=1330071 Question: Should I change the name of the -wine subpackage to something else (like -wrapper)? -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498390] New: Review Request: rakudo - Rakudo - A Perl compiler for Parrot
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: rakudo - Rakudo - A Perl compiler for Parrot https://bugzilla.redhat.com/show_bug.cgi?id=498390 Summary: Review Request: rakudo - Rakudo - A Perl compiler for Parrot Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: g...@zimt.uni-siegen.de QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: ftp://ftp.uni-siegen.de/pub/parrot.rpms/rakudo/rakudo.spec SRPM URL: ftp://ftp.uni-siegen.de/pub/parrot.rpms/rakudo/rakudo.src.rpm Description: Rakudo Perl is a Perl 6 compiler for the Parrot virtual machine. This is only a placeholder, because Rakudo do not build from an installed parrot. It makes no sense to make a rpm for Rakudo with an integrated parrot. The Parrot community is at the moment activ working that general languages build with an installed parrot. I think it will coming soon so I made already this 'Review Request' but it is only a placeholder until Rakudo build on an installed parrot. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498390] Review Request: rakudo - Rakudo - A Perl compiler for Parrot
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=498390 Gerd Pokorra g...@zimt.uni-siegen.de changed: What|Removed |Added URL||http://rakudo.org -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438588] Review Request: zfstream - C++ iostream like access to compressed 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=438588 Thomas Sailer t.sai...@alumni.ethz.ch changed: What|Removed |Added Alias||zfstream -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 453856] Review Request: globus-gsi-cert-utils - Globus Toolkit - Globus GSI Cert Utils 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=453856 --- Comment #9 from Mattias Ellert mattias.ell...@fysast.uu.se 2009-04-30 05:41:20 EDT --- Package available in devel. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 453857] Review Request: globus-gsi-sysconfig - Globus Toolkit - Globus GSI System Config 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=453857 --- Comment #9 from Mattias Ellert mattias.ell...@fysast.uu.se 2009-04-30 05:41:30 EDT --- Package available in devel. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481224] Review Request: rabbitmq-server - An AMQP server written in Erlang
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=481224
--- Comment #16 from Peter Lemenkov lemen...@gmail.com 2009-04-30 05:39:48
EDT ---
Sorry for the delay.
OK, the spec-file is legible, but I overlooked the naive approach to security
with rabbit_wrapper. I think that this trick is useless, so I advise you just
to remove all %{_rabbit_wrapper}-related stuff.
--
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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491650] Review Request: libica - Library for accessing ICA hardware crypto on IBM zSeries
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=491650 Dan Horák d...@danny.cz changed: What|Removed |Added Flag||fedora-cvs? --- Comment #3 from Dan Horák d...@danny.cz 2009-04-30 05:39:40 EDT --- New Package CVS Request === Package Name: libica Short Description: Library for accessing ICA hardware crypto on IBM zSeries Owners: sharkcz Branches: F-11 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498397] New: Review Request: perl-Devel-REPL - Modern perl interactive shell
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-Devel-REPL - Modern perl interactive shell Alias: perl-Devel-REPL https://bugzilla.redhat.com/show_bug.cgi?id=498397 Summary: Review Request: perl-Devel-REPL - Modern perl interactive shell Product: Fedora Version: rawhide Platform: All URL: http://search.cpan.org/dist/Devel-REPL/ OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: iarn...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fedorapeople.org/~iarnell/review/perl-Devel-REPL.spec SRPM URL: http://fedorapeople.org/~iarnell/review/perl-Devel-REPL-1.003006-1.fc12.src.rpm Description: This is an interactive shell for Perl, commonly known as a REPL - Read, Evaluate, Print, Loop. The shell provides for rapid development or testing of code without the need to create a temporary source code file. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1330112 *rt-0.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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498363] Review Request: wordpress-plugin-add-to-any-subscribe - Add to Any Subscribe Button plugin for WordPress
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=498363 David Nalley da...@gnsa.us changed: What|Removed |Added Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498362] Review Request: wordpress-plugin-add-to-any - Add to Any: Share/Bookmark/Email Button plugin for WordPress
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=498362 David Nalley da...@gnsa.us changed: What|Removed |Added Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497192] Review Request: polkit-qt - Qt bindings for PolicyKit
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=497192 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review+ --- Comment #2 from Parag AN(पराग) panem...@gmail.com 2009-04-30 06:09:02 EDT --- Review: + package builds in mock (rawhide i586). koji Build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1330027 + rpmlint output for SRPM and for RPM. polkit-qt.src: E: description-line-too-long Polkit-qt is a library that lets developers use the PolicyKit API through a nice polkit-qt.src: W: mixed-use-of-spaces-and-tabs (spaces: line 12, tab: line 1) polkit-qt.i586: E: description-line-too-long Polkit-qt is a library that lets developers use the PolicyKit API through a nice polkit-qt.i586: W: shared-lib-calls-exit /usr/lib/libpolkit-qt-core.so.0.9.2 e...@glibc_2.0 polkit-qt-examples.i586: W: non-conffile-in-etc /etc/dbus-1/system.d/org.qt.policykit.examples.conf == plz fix those before cvs import. + source files match upstream url 14cb821f8917c13ed4154a3ab8e596be polkit-qt-0.9.2.tar.bz2 + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + polkit-qt.pc, polkit-qt-core.pc,polkit-qt-gui.pc file present. + -devel subpackage present. + no .la files. + no translations are available + Does owns the directories it creates. + ldconfig scriptlets present. + no duplicates in %files. + file permissions are appropriate. + Package polkit-qt-0.9.2-1.fc12.i586 = Provides: libpolkit-qt-core.so.0 libpolkit-qt-gui.so.0 Requires: libQtCore.so.4 libQtDBus.so.4 libQtGui.so.4 libQtXml.so.4 libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1.3) libdbus-1.so.3 libgcc_s.so.1 libgcc_s.so.1(GCC_3.0) libglib-2.0.so.0 libm.so.6 libpolkit-dbus.so.2 libpolkit-grant.so.2 libpolkit-qt-core.so.0 libpolkit-qt-gui.so.0 libpolkit.so.2 libpthread.so.0 libstdc++.so.6 libstdc++.so.6(CXXABI_1.3) libstdc++.so.6(GLIBCXX_3.4) rtld(GNU_HASH) + Package polkit-qt-devel-0.9.2-1.fc12.i586 = Provides: pkgconfig(polkit-qt) = 0.9.2 pkgconfig(polkit-qt-core) = 0.9.2 pkgconfig(polkit-qt-gui) = 0.9.2 Requires: /usr/bin/pkg-config libpolkit-qt-core.so.0 libpolkit-qt-gui.so.0 pkgconfig(QtCore) pkgconfig(QtGui) pkgconfig(polkit) pkgconfig(polkit-dbus) pkgconfig(polkit-grant) + Package polkit-qt-examples-0.9.2-1.fc12.i586 = Requires: libQtCore.so.4 libQtDBus.so.4 libQtXml.so.4 libc.so.6 libc.so.6(GLIBC_2.0) libdbus-1.so.3 libgcc_s.so.1 libgcc_s.so.1(GCC_3.0) libglib-2.0.so.0 libm.so.6 libpolkit-dbus.so.2 libpolkit-grant.so.2 libpolkit-qt-core.so.0 libpolkit.so.2 libpthread.so.0 libstdc++.so.6 libstdc++.so.6(CXXABI_1.3) libstdc++.so.6(GLIBCXX_3.4) rtld(GNU_HASH) Suggestions: 1) I think polkit-qt-examples should Requires: dbus 2) Should query upstream to change COPYING file with text of GPLv2+. 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497525] Review Request: gnome-applet-bubblemon - Bubbling Load Monitoring Applet for the GNOME Panel
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=497525
--- Comment #8 from Edwin ten Brink ed...@tenbrink-bekkers.nl 2009-04-30
06:14:55 EDT ---
(In reply to comment #7)
- the conversion of the two manpages (languages sv and hu) since they are
already included by the upstream maintainer in the upcoming next release and
these files have a small target audience which should have the used
character
encoding anyway.
Ok, I guess we should trust the translators here. Hope it gets fixed upstream
though.
It currently is, the upstream maintainer assured me of this fact.
(In reply to comment #6)
This is a valid comment. I'm only wondering why the generated template spec
file does not include the part INSTALL=install -p.
Some Makefiles already use the -p option by default, but not this one. Others
don't understand the INSTALL parameter and will fail.
I'll see if I can get that fixed upstream as well. Perhaps it's just a question
of moving to a more recent automake/autoconf.
REVIEW FOR gnome-applet-bubblemon-2.0.13-3.fc10.src.rpm
Issues
- mock build for F-11 and F-12 fails, see
http://koji.fedoraproject.org/koji/taskinfo?taskID=1329436
http://koji.fedoraproject.org/koji/taskinfo?taskID=1329461
The build.log is pretty clear and fortunately identical in F11 and F12. Strange
that it does not happen under F10 or earlier (I used F9 for an earlier version,
and F10 for this submission). I will check myself as well as with the upstream
maintainer what happened.
- SourceURL not found, see
http://fedoraproject.org/wiki/Packaging/SourceURL#Referencing_Source
We need a downloadable URL in the Source0 tag, use spectool to verify.
Sorry, that's my mistake. The download page on upstream's project page embeds
the source directory which is on another server. It should read:
http://savannah.inetbridge.net/bubblemon/bubblemon-%{version}.tar.gz
- Minor: When renaming the package you forgot the comment at the head of the
spec
I see, consider it corrected.
Because of the upcoming release of F11, I suggest not to incorporate this
package until the build issues for F11 and F12 have been resolved. I assume you
agree with me on this.
I will incorporate the above issues and revert as soon as possible. That will
probably be with a new upstream release to fix the build issues.
--
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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497525] Review Request: gnome-applet-bubblemon - Bubbling Load Monitoring Applet for the GNOME Panel
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=497525 --- Comment #9 from Edwin ten Brink ed...@tenbrink-bekkers.nl 2009-04-30 06:32:50 EDT --- (In reply to comment #7) Regarding your sponsorship: Do you have any other packages or have you participated in other reviews? Just yesterday I have submitted another very small package for review (see bug 498218), which has not had a reviewer yet. I have some plans for submitting yet another package (log4cpp, http://sourceforge.net/project/showfiles.php?group_id=15190), after playing around with it a bit more. From the concept it is an interesting package, however does not seem to be very actively maintained at the moment judging from the bug, patch and feature trackers on sourceforge. I'm therefore not sure if this would make a good candidate at the moment. I have not yet participated in any reviews yet. My main activities on Bugzilla so far have been bug reporting (sometimes signalling dupes when I stumbled upon them) and commenting on other bugs which were reported by others before me. I have been actively using RedHat Linux around 1996-2002, and returned to Fedora again about a year ago. I therefore do not possess much recent experience of Fedora packaging and its guidelines, but I am familiar with rpm. Your sponsorship would be very much appreciated. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498407] Review Request: bind-dyndb-ldap - LDAP back-end plug-in for BIND
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=498407 Lubomir Rintel lkund...@v3.sk changed: What|Removed |Added CC||lkund...@v3.sk AssignedTo|nob...@fedoraproject.org|lkund...@v3.sk Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498407] New: Review Request: bind-dyndb-ldap - LDAP back-end plug-in for BIND
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: bind-dyndb-ldap - LDAP back-end plug-in for BIND https://bugzilla.redhat.com/show_bug.cgi?id=498407 Summary: Review Request: bind-dyndb-ldap - LDAP back-end plug-in for BIND Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mn...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://mnagy.fedorapeople.org/bind/bind-dyndb-ldap.spec SRPM URL: http://mnagy.fedorapeople.org/bind/bind-dyndb-ldap-0.1.0-0.1.a1.fc9.src.rpm Description: This package provides an LDAP back-end plug-in for BIND. It features support for dynamic updates and internal caching, to lift the load off of your LDAP server. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder
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=496524 Mattias Ellert mattias.ell...@fysast.uu.se changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from Mattias Ellert mattias.ell...@fysast.uu.se 2009-04-30 07:14:33 EDT --- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 486302] Review Request: parrot - Parrot Virtual Machine
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=486302 --- Comment #55 from Fedora Update System upda...@fedoraproject.org 2009-04-30 07:21:10 EDT --- parrot-1.0.0-6.fc9 has been submitted as an update for Fedora 9. http://admin.fedoraproject.org/updates/parrot-1.0.0-6.fc9 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 486302] Review Request: parrot - Parrot Virtual Machine
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=486302 --- Comment #57 from Fedora Update System upda...@fedoraproject.org 2009-04-30 07:21:26 EDT --- parrot-1.0.0-6.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/parrot-1.0.0-6.fc10 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 486302] Review Request: parrot - Parrot Virtual Machine
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=486302 --- Comment #56 from Fedora Update System upda...@fedoraproject.org 2009-04-30 07:21:19 EDT --- parrot-1.0.0-6.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/parrot-1.0.0-6.fc11 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 494862] Review Request: podofo - Tools and libraries to work with the PDF file format
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=494862 --- Comment #4 from Dan Horák d...@danny.cz 2009-04-30 05:10:50 EDT --- Updated spec URL: http://fedora.danny.cz/podofo.spec Updated SRPM URL: http://fedora.danny.cz/podofo-0.7.0-2.fc11.src.rpm ChangeLog: - removed dependency on openssl, upstream told me, that it is actually not required and when it will become required, they will add the exception clause to the license - added AUTHORS and TODO to the libs subpackage only, because it is always installed -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491317] Review Request: mingw32-gstreamer - MinGW Windows gstreamer 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=491317 --- Comment #10 from Levente Farkas lfar...@lfarkas.org 2009-04-30 07:32:52 EDT --- Richard is there an reason not to 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498407] Review Request: bind-dyndb-ldap - LDAP back-end plug-in for BIND
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=498407 --- Comment #1 from Lubomir Rintel lkund...@v3.sk 2009-04-30 07:34:45 EDT --- - Respects compiler flags - Filelists sane - SPEC file clean and legible, American English used - Provides sane - Uses compiler flags correctly - Uses parallel make correctly - RPMlint happy 1.) Requires You install /usr/lib/bind/ldap.so, but you don't depend on anything that would own /usr/lib/bind. 2.) BuildRequires -BuildRequires: bind-devel = bind-devel-9.6.1-0.3.b1 +BuildRequires: bind-devel = 9.6.1-0.3.b1 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498130] Review Request: lxinput - Keyboard and mouse settings dialog for LXDE
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=498130 --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2009-04-30 07:41:03 EDT --- lxinput-0.1-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/lxinput-0.1-1.fc11 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498125] Review Request: lxde-settings-daemon - XSettings Daemon for LXDE
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=498125 --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2009-04-30 07:39:13 EDT --- lxde-settings-daemon-0.4-2.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/lxde-settings-daemon-0.4-2.fc11 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491317] Review Request: mingw32-gstreamer - MinGW Windows gstreamer 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=491317 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Status|ASSIGNED|NEW Flag|fedora-review? | --- Comment #11 from Richard W.M. Jones rjo...@redhat.com 2009-04-30 07:41:59 EDT --- Probably because I'm super-busy. Unassigning myself from review. If you ask on the fedora-mingw mailing list I'm sure you can get someone to review this. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin
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=498147 --- Comment #7 from Iain Arnell iarn...@gmail.com 2009-04-30 07:38:13 EDT --- My first thought was that it could be a bit confusing for those coming from CPAN to install perl-App-Foo and not find the corresponding foo binary installed. But the more I think about it, I'm starting to like the idea of giving foo a bit more prominence. Since this was reviewed as is, I've checked it in without changes, but offer a split nopaste and perl-App-Nopaste for further consideration before going ahead and arbitrarily making the change. Spec URL: http://iarnell.fedorapeople.org/review/perl-App-Nopaste.spec SRPM URL: http://iarnell.fedorapeople.org/review/perl-App-Nopaste-0.10-2.fc12.src.rpm koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1330196 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498125] Review Request: lxde-settings-daemon - XSettings Daemon for LXDE
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=498125 --- Comment #7 from Fedora Update System upda...@fedoraproject.org 2009-04-30 07:39:29 EDT --- lxde-settings-daemon-0.4-2.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/lxde-settings-daemon-0.4-2.fc10 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498130] Review Request: lxinput - Keyboard and mouse settings dialog for LXDE
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=498130 --- Comment #7 from Fedora Update System upda...@fedoraproject.org 2009-04-30 07:48:59 EDT --- lxinput-0.1-1.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/lxinput-0.1-1.fc10 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226483] Merge Review: tcsh
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=226483 Vitezslav Crhonek vcrho...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Comment #10 from Vitezslav Crhonek vcrho...@redhat.com 2009-04-30 07:49:44 EDT --- Built in tcsh-6.16-1.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488100] Review Request: firebird - Firebird SQL database management system
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=488100
--- Comment #75 from Peter Lemenkov lemen...@gmail.com 2009-04-30 05:17:38
EDT ---
Sorry for the delay.
Still there are some issues with directories owning:
* devel sub-package should own %{fbroot}/include/
* libfbclient and libfbembed both owns %{fbroot}/lib and they both can be
installed.
* server-common sub-package should own %{_localstatedir}/log/%{name}
* server-common sub-package should own %{fbroot}/intl
* server-common sub-package should own %{_sysconfdir}/%{name}
Other things looks sane.
--
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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438588] Review Request: zfstream - C++ iostream like access to compressed 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=438588 Thomas Sailer t.sai...@alumni.ethz.ch changed: What|Removed |Added Blocks||498413(vfrnav) -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498413] New: Review Request: vfrnav - Flight Planning software for flights under Visual Flight Rules (VFR)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: vfrnav - Flight Planning software for flights under Visual Flight Rules (VFR) Alias: vfrnav https://bugzilla.redhat.com/show_bug.cgi?id=498413 Summary: Review Request: vfrnav - Flight Planning software for flights under Visual Flight Rules (VFR) Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: t.sai...@alumni.ethz.ch QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Depends on: 438588 Estimated Hours: 0.0 Classification: Fedora Spec URL: http://sailer.fedorapeople.org/vfrnav.spec SRPM URL: http://sailer.fedorapeople.org/vfrnav-0.3-2.fc11.src.rpm Description: vfrnav is a flight planning and navigation application for flights under visual flight rules. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 453858] Review Request: globus-gsi-callback - Globus Toolkit - Globus GSI Callback 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=453858 Thomas Sailer t.sai...@alumni.ethz.ch changed: What|Removed |Added CC||t.sai...@alumni.ethz.ch Bug 453858 depends on bug 453850, which changed state. Bug 453850 Summary: Review Request: globus-openssl - Openssl Library (virtual GPT glue package) https://bugzilla.redhat.com/show_bug.cgi?id=453850 What|Old Value |New Value Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE Bug 453858 depends on bug 453854, which changed state. Bug 453854 Summary: Review Request: globus-gsi-proxy-ssl - Globus Toolkit - Globus GSI Proxy SSL Library https://bugzilla.redhat.com/show_bug.cgi?id=453854 What|Old Value |New Value Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE Bug 453858 depends on bug 453853, which changed state. Bug 453853 Summary: Review Request: globus-gsi-openssl-error - Globus Toolkit - Globus OpenSSL Error Handling https://bugzilla.redhat.com/show_bug.cgi?id=453853 What|Old Value |New Value Resolution||NEXTRELEASE Status|ASSIGNED|CLOSED --- Comment #5 from Thomas Sailer t.sai...@alumni.ethz.ch 2009-04-30 08:06:46 EDT --- Source matches upstream: $ diff -urN globus_gsi_callback-1.10 globus_gsi_callback-1.10-dl is silent. Scratch Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1330235 Rpmlint: $ rpmlint globus-gsi-callback* globus-gsi-callback.spec globus-gsi-callback-devel.x86_64: W: no-documentation 4 packages and 2 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 453858] Review Request: globus-gsi-callback - Globus Toolkit - Globus GSI Callback 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=453858
Thomas Sailer t.sai...@alumni.ethz.ch changed:
What|Removed |Added
Flag||fedora-review+
--- Comment #6 from Thomas Sailer t.sai...@alumni.ethz.ch 2009-04-30
08:12:08 EDT ---
Fedora review globus-gsi-callback-1.10-1.fc10.src.rpm 2009-04-30
+ OK
! needs attention
+ rpmlint output
The no-documentation warning can be ignored as there is a separate doc
package
+ Package is named according to Fedora Package Naming Guidelines
+ Specfile name matches the package base name
+ Package follows the Fedora packaging guidelines
+ License meets guidelines and is acceptable to Fedora
ASL 2.0
+ License matches the actual package license
+ The package contains the license file
+ Spec file is written in American English
+ Spec file is legible
+ Upstream sources match sources in the srpm
the source repacking makes it impossible to compare md5sum's, thus:
$ diff -urN globus_gsi_callback-1.10 globus_gsi_callback-1.10-dl
reports no differences
+ Package scratch-builds
n/a ExcludeArch bugs filed
+ BuildRequires list all build dependencies
n/a %find_lang instead of %{_datadir}/locale/*
n/a binary RPM with shared library files must call ldconfig in %post and
%postun
+ Does not use Prefix: /usr
+ Package owns all directories it creates
+ No duplicate files in %files
+ %files has %defattr
+ %clean contains rm -rf $RPM_BUILD_ROOT
+ Consistent use of macros
+ Package must contain code or permissible content
+ Large documentation files should go in -doc subpackage
n/a Files marked %doc should not affect package
+ Header files should be in -devel
n/a Static libraries should be in -static
+ Packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'
indirectly, via globus-core
+ libfoo.so must go in -devel
+ -devel must require the fully versioned base
+ Packages should not contain libtool .la files
n/a Packages containing GUI apps must include %{name}.desktop file
+ Packages must not own files or directories owned by other packages
+ %install begins with rm -rf $RPM_BUILD_ROOT
+ Filenames must be valid UTF-8
APPROVED by sailer.
--
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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498407] Review Request: bind-dyndb-ldap - LDAP back-end plug-in for BIND
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=498407 Martin Nagy mn...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Martin Nagy mn...@redhat.com 2009-04-30 08:12:04 EDT --- New Package CVS Request === Package Name: bind-dyndb-ldap Short Description: LDAP back-end plug-in for BIND Owners: mnagy Branches: F-11 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497192] Review Request: polkit-qt - Qt bindings for PolicyKit
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=497192 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Flag||fedora-cvs? --- Comment #3 from Rex Dieter rdie...@math.unl.edu 2009-04-30 08:11:17 EDT --- Thanks, good suggestions, will do. New Package CVS Request === Package Name: polkit-qt Short Description: Qt bindings for PolicyKit Owners: rdieter Branches: F-10 F-11 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484229] Review Request: qbittorrent - A bittorrent 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=484229 --- Comment #53 from leigh scott leigh123li...@googlemail.com 2009-04-30 08:13:04 EDT --- Bump! -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498407] Review Request: bind-dyndb-ldap - LDAP back-end plug-in for BIND
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=498407 --- Comment #2 from Martin Nagy mn...@redhat.com 2009-04-30 07:54:45 EDT --- Done, new spec uploaded. I added a Require for bind = 9.6.1-0.3.b1. I also fixed the BR. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497786] Review Request: perl-JSON-RPC - Perl implementation of JSON-RPC 1.1 protocol
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=497786 Emmanuel Seyman emmanuel.sey...@club-internet.fr changed: What|Removed |Added Flag|fedora-cvs- |fedora-cvs? --- Comment #5 from Emmanuel Seyman emmanuel.sey...@club-internet.fr 2009-04-30 07:57:11 EDT --- (In reply to comment #4) I guess this should be perl-JSON-RPC Yes, you're right. I'm sorry, that was a stupid mistake on my part. New Package CVS Request === Package Name: perl-JSON-RPC Short Description: Perl implementation of JSON-RPC 1.1 protocol-RPC Owners: eseyman Branches: F-9 F-10 F-11 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497661] Review Request: perl-MIME-EncWords - Deal with RFC 2047 encoded words
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=497661 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498407] Review Request: bind-dyndb-ldap - LDAP back-end plug-in for BIND
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=498407 Lubomir Rintel lkund...@v3.sk changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from Lubomir Rintel lkund...@v3.sk 2009-04-30 08:05:16 EDT --- Good work - Requires sane - Builds fine in mock now 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498413] Review Request: vfrnav - Flight Planning software for flights under Visual Flight Rules (VFR)
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=498413
Igor Jurišković juriskovic.i...@gmail.com changed:
What|Removed |Added
CC||juriskovic.i...@gmail.com
--- Comment #1 from Igor Jurišković juriskovic.i...@gmail.com 2009-04-30
08:34:09 EDT ---
Hello Thomas,
this is not official review. I'm in need of sponsor but I can tell you what can
be better or fixed.
- your Release: doesn't match changelog version.
- Group doesn't exist. You should use most appropriate Group available.
less /usr/share/doc/rpm-*/GROUPS* - will give you all possible groups.
- you dont need Require: gypsy. It will get pulled by gypsy-devel
- gcc-c++ is not needed. Read here why:
http://fedoraproject.org/wiki/Packaging/Guidelines
- in %files section you can omit INSTALL. You are making installation so other
users have no need to look into that file.
- NEWS is empty. You can remove it. Same goes for README
- package doesn't own %{_datadir}/%{name} dir. Include it to %files section
with %dir.
- You can speed up packaging by using asterix(*) to own all files in some
directory. You can use this on
%{_datadir}/%{name}/* to own all files inside.
That's what I can see in fast - 10 minute - 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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498413] Review Request: vfrnav - Flight Planning software for flights under Visual Flight Rules (VFR)
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=498413 --- Comment #2 from Igor Jurišković juriskovic.i...@gmail.com 2009-04-30 08:37:00 EDT --- Oops, forgot to tell you about licence. Its wrong. src file says GPLv2 or any later version. So you should change it from GPLv2 to GPLv2+ -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484229] Review Request: qbittorrent - A bittorrent 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=484229 --- Comment #54 from Rex Dieter rdie...@math.unl.edu 2009-04-30 08:43:08 EDT --- Either Version: 1.4.0 Release: 0.1.beta2 or Version: 1.4.0 Release: 0.1.date_stampsvn works, conforming to Naming guidelines: http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Package_Version -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488100] Review Request: firebird - Firebird SQL database management system
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=488100
--- Comment #76 from Philippe Makowski makowski.fed...@gmail.com 2009-04-30
08:47:27 EDT ---
(In reply to comment #75)
* devel sub-package should own %{fbroot}/include/
* server-common sub-package should own %{_localstatedir}/log/%{name}
* server-common sub-package should own %{fbroot}/intl
* server-common sub-package should own %{_sysconfdir}/%{name}
ok easy to fix
but this one :
* libfbclient and libfbembed both owns %{fbroot}/lib and they both can be
installed.
I don't know how to fix the fact that both use %{fbroot}/lib
--
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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488995] Review Request: pidgin-tlen - Tlen IM Pidgin plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488995 --- Comment #6 from Simon Wesp cassmod...@fedoraproject.org 2009-04-30 09:05:18 EDT --- https://bugzilla.redhat.com/show_bug.cgi?id=490140 https://bugzilla.redhat.com/show_bug.cgi?id=495310 I will wait, because this topic is hot! -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 495564] Review Request: libguestfs - Access and modify virtual machine disk images
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=495564 Jim Meyering meyer...@redhat.com changed: What|Removed |Added Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 453858] Review Request: globus-gsi-callback - Globus Toolkit - Globus GSI Callback 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=453858 --- Comment #7 from Mattias Ellert mattias.ell...@fysast.uu.se 2009-04-30 09:32:01 EDT --- Thank you for the review. New Package CVS Request === Package Name: globus-gsi-callback Short Description: Globus Toolkit - Globus GSI Callback Library Owners: ellert Branches: F-9 F-10 F-11 EL-4 EL-5 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 453858] Review Request: globus-gsi-callback - Globus Toolkit - Globus GSI Callback 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=453858 Mattias Ellert mattias.ell...@fysast.uu.se changed: What|Removed |Added Status|NEW |ASSIGNED Flag||fedora-cvs? -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484229] Review Request: qbittorrent - A bittorrent 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=484229 --- Comment #55 from leigh scott leigh123li...@googlemail.com 2009-04-30 09:35:46 EDT --- (In reply to comment #54) Either Version: 1.4.0 Release: 0.1.beta2 or Version: 1.4.0 Release: 0.1.date_stampsvn works, conforming to Naming guidelines: http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Package_Version Thank you, I went with your second example. http://koji.fedoraproject.org/koji/buildinfo?buildID=100393 -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438588] Review Request: zfstream - C++ iostream like access to compressed 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=438588
--- Comment #13 from Thomas Sailer t.sai...@alumni.ethz.ch 2009-04-30
09:45:09 EDT ---
Update
Spec URL: http://sailer.fedorapeople.org/zfstream.spec
SRPM URL: http://sailer.fedorapeople.org/zfstream-20041202-3.fc11.src.rpm
(In reply to comment #12)
The file ChangeLog can go to %doc. I don't think it is worth packaging the
LICENSE file as it doesn't say much about the license.
Done and agreed.
* Patches should be explained and be submitted to upstream. Since the project
seems dead, it won't hurt to bypass the submission part. But please explain in
the SPEC file what the patch does.
I have sent the patch upstream by PM (the project lacks a bug tracker). While
the original author said he integrated the patch into his tree, he apparently
hasn't gotten around to release a new version.
* Similarly, please give the reasoning of Source1.
See comment #9. Basically because I couldn't get toc working, and furthermore
allows autotools to cross-compile the lib (I have also submitted
mingw32-zfstream for review).
* Please remove the duplicate copy of minizip, which we already have in
Fedora,
from Source1 and adjust the BR's. Note that minizip-devel already requires
zlib-devel.
Done.
* The package must be named according to the Package Naming Guidelines. The
upstream tarball has a different name. Why don't we use it?
The project is named just zfstream on the author's homepage. It's just that
the tarball has libs11n_ prefixed. s11n.net is another project by the same
author, so I guess that's where the prefix comes from. zfstream however has
nothing to do with s11n.net, so I found it more natural to just name the
package zfstream.
* No need to explicitly BR automake. libtool does and always will pull that
for
you.
Done
! Please make use of the %{name} macro.
Done.
* The devel package must require bzip2-devel, zlib-devel or minizip-devel
(?), and pkgconfig
Done. I think zlib-devel is enough, as zipstream.hpp does not include anything
from minizip-devel, minizip is only needed for building
? About touch NEWS README AUTHORS: Wasn't there an --add-missing flag to
automake?
Actually, -a (which is already present) or --add-missing (the full name for -a)
only adds missing files _other_ than NEWS, README and AUTHORS. So build will
fail if you remove the touch.
! No need for blank %doc's.
Done.
! Please make the descriptions span 80 columns.
Shortened.
--
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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498194] Review Request: zarafa - Zarafa Outlook Sharing and Open Source Collaboration
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=498194 Tom spot Callaway tcall...@redhat.com changed: What|Removed |Added Blocks||182235(FE-Legal) -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497640] Review Request: rubygem-RedCloth - Textile parser 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=497640 --- Comment #2 from Darryl L. Pierce dpie...@redhat.com 2009-04-30 10:08:45 EDT --- (In reply to comment #1) first notes: - the license is MIT (see COPYING and README from inside of the gem) Fixed. - it doesn't build in mock for both F-10 and Rawhide on x86_64 Fixed that. Added BuildRequires: ruby-devel = 1.8 - is it possible to run the included testsuite during the build (using the %check section)? Running them will require someone to submit echoe http://www.workingwithrails.com/rubygem/4726-echoe as a package for Fedora. If somebody does that, I'll add running the tests to the spec. Updated spec URL: http://mcpierce.fedorapeople.org/rpms/rubygem-RedCloth.spec Updated SRPM URL: http://mcpierce.fedorapeople.org/rpms/rubygem-RedCloth-4.1.9-2.fc10.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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498246] Review Request: towhee - A Monte Carlo molecular simulation code
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=498246
--- Comment #3 from Fabien Georget fabien.geor...@gmail.com 2009-04-30
10:46:00 EDT ---
(In reply to comment #2)
But you don't need to be a sponsor to do reviews, just an approved packager
(you need to be sponsorED).
It was that I meant, I am not an approved packager. Sorry.
- I think that %doc license.gpl Examples/ is not a good thing. Because
Examples/ contains some executable. Install it with the doc flags modify
the rights.
No, it is quite standard to ship executable scripts in %doc, as the program
works without them; they're just examples of use.
If you think people don't have to run them, I think it is good.
- In the file /usr/share/towhee/Forcefields, you remove Makefile but not
Makefile.am and Makefile.in
Nope,
find Examples/ -name Makefile* -exec rm {} \;
also removes Makefile.*
I don't talk about Examples, but about /usr/share/towhee/Forcefields.
(16:21:58)-[man...@teatime ~]$ ls /usr/share/towhee/ForceFields
Makefile.am Makefile.in ...
I mix the two files it's right sorry. You don't remove nothing from this file.
Anyway, Makefile.am and Makefile.in are installed but useless without the
configure script.
- Some macros are available in rpm like %{_cat}, %{_cp} ... for coherence
you should consider to use them.
On the contrary, it is not considered clean to use macros for these sorts of
things. Macros should only be used when absolutely necessary, not for
standard unix commands as cp, mv, rm, mkdir and so on.
That's surprising ... defined but useless macros. But of course, it is not
mandatory. (And I read a new time the packaging guidelines, nothing are
mentionned about this (except for directory). It is not in this document I read
it is better to use macros. So like you want :)
--
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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492927] Review Request: xcowsay - xcowsay displays a cute cow and message
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=492927 --- Comment #7 from Fabien Georget fabien.geor...@gmail.com 2009-04-30 10:56:19 EDT --- (In reply to comment #6) By the way from next time please change the release number every time you modify your spec file to aviod confusion. Of course :/ sorry ! I discover an interesting package in review request, so I start a pre-review : https://bugzilla.redhat.com/show_bug.cgi?id=498246 I think I need more time to read review done by others people to understand some points. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498413] Review Request: vfrnav - Flight Planning software for flights under Visual Flight Rules (VFR)
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=498413 --- Comment #3 from Thomas Sailer t.sai...@alumni.ethz.ch 2009-04-30 11:00:20 EDT --- Thanks, Igor, for your comments! I've updated the package: Spec URL: http://sailer.fedorapeople.org/vfrnav.spec SRPM URL: http://sailer.fedorapeople.org/vfrnav-0.3-3.fc11.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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder
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=496524 Orcan 'oget' Ogetbil oget.fed...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-04-30 11:04:54 EDT --- Thank you for the review! New Package CVS Request === Package Name: jorbis Short Description: Pure Java Ogg Vorbis Decoder Owners: oget Branches: F-9 F-10 F-11 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498397] Review Request: perl-Devel-REPL - Modern perl interactive shell
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=498397 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review+ --- Comment #1 from Parag AN(पराग) panem...@gmail.com 2009-04-30 11:21:38 EDT --- Review: + package builds in mock (rawhide i586). koji build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1330112 + rpmlint is silent for SRPM and for RPM. + source files match upstream url e052cac4b6d55eee9a0ced5a76b5a7be Devel-REPL-1.003006.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + make test outputs Files=1, Tests=17, 1 wallclock secs ( 0.03 usr 0.00 sys + 0.46 cusr 0.02 csys = 0.51 CPU) +Package perl-Devel-REPL-1.003006-1.fc12.noarch = Provides: perl(Devel::REPL) = 1.003006 perl(Devel::REPL::Error) perl(Devel::REPL::Meta::Plugin) perl(Devel::REPL::Plugin) perl(Devel::REPL::Plugin::B::Concise) perl(Devel::REPL::Plugin::Colors) perl(Devel::REPL::Plugin::Commands) perl(Devel::REPL::Plugin::Completion) perl(Devel::REPL::Plugin::CompletionDriver::Globals) perl(Devel::REPL::Plugin::CompletionDriver::INC) perl(Devel::REPL::Plugin::CompletionDriver::Keywords) perl(Devel::REPL::Plugin::CompletionDriver::LexEnv) perl(Devel::REPL::Plugin::CompletionDriver::Methods) perl(Devel::REPL::Plugin::CompletionDriver::Turtles) perl(Devel::REPL::Plugin::DDS) perl(Devel::REPL::Plugin::DumpHistory) perl(Devel::REPL::Plugin::FancyPrompt) perl(Devel::REPL::Plugin::FindVariable) perl(Devel::REPL::Plugin::History) perl(Devel::REPL::Plugin::Interrupt) perl(Devel::REPL::Plugin::LexEnv) perl(Devel::REPL::Plugin::MultiLine::PPI) perl(Devel::REPL::Plugin::NewlineHack) perl(Devel::REPL::Plugin::Nopaste) perl(Devel::REPL::Plugin::OutputCache) perl(Devel::REPL::Plugin::PPI) perl(Devel::REPL::Plugin::Packages) perl(Devel::REPL::Plugin::Packages::DefaultScratchpad) perl(Devel::REPL::Plugin::Peek) perl(Devel::REPL::Plugin::ReadLineHistory) perl(Devel::REPL::Plugin::Refresh) perl(Devel::REPL::Plugin::ShowClass) perl(Devel::REPL::Plugin::Timing) perl(Devel::REPL::Plugin::Turtles) perl(Devel::REPL::Profile) perl(Devel::REPL::Profile::Default) perl(Devel::REPL::Script) Requires: /usr/bin/env perl = 0:5.008001 perl(B::Concise) perl(B::Keywords) perl(Data::Dump::Streamer) perl(Devel::Peek) perl(Devel::REPL) perl(Devel::REPL::Error) perl(Devel::REPL::Meta::Plugin) perl(Devel::REPL::Plugin) perl(Devel::REPL::Script) perl(File::HomeDir) perl(File::Next) perl(File::Spec) perl(Lexical::Persistence) perl(Module::Refresh) perl(Moose) perl(Moose::Role) perl(Moose::Util::TypeConstraints) perl(MooseX::AttributeHelpers) perl(PPI) perl(PPI::Dumper) perl(Scalar::Util) perl(Term::ANSIColor) perl(Term::ReadLine) perl(Time::HiRes) perl(namespace::clean) perl(strict) perl(vars) perl(warnings) suggestions: 1) Not needed BuildRequires: perl = 1:5.8.1 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498397] Review Request: perl-Devel-REPL - Modern perl interactive shell
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=498397 Iain Arnell iarn...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481536] Review Request: enano - Enano CMS, a php-based modular content management system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481536 King InuYasha ngomp...@gmail.com changed: What|Removed |Added Flag|needinfo?(ngomp...@gmail.co | |m) | --- Comment #6 from King InuYasha ngomp...@gmail.com 2009-04-30 11:38:25 EDT --- Which packages in the Enano package are verbatim from another package? Give me a list, and I can check with upstream. If they are truly verbatim, I will start stripping them out and doing to work necessary to symlink in the required parts. However, if they are modified, can I leave them in, or will I have to split them out into an enano-* package? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498397] Review Request: perl-Devel-REPL - Modern perl interactive shell
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=498397 --- Comment #2 from Iain Arnell iarn...@gmail.com 2009-04-30 11:42:01 EDT --- New Package CVS Request === Package Name: perl-Devel-REPL Short Description: Modern perl interactive shell Owners: iarnell Branches: F-10 F-11 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438588] Review Request: zfstream - C++ iostream like access to compressed 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=438588 Orcan 'oget' Ogetbil oget.fed...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #14 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-04-30 11:39:49 EDT --- Thanks for adding the source file explanations to the specfile. Everything looks good, except ! Please make the descriptions span 80 columns. Shortened. I don't see any change. In fact, you shouldn't shorten them. Some lines in the description are even smaller than 70 columns. They need stretched out a little. But this is not a blocker. You can do it before you commit --- This package (zfstream) is APPROVED by oget --- -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 496633] Review Request: monodevelop-debugger-gdb - GDB Debugger Addin for MonoDevelop
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=496633
--- Comment #13 from Mauricio Henriquez buhochil...@gmail.com 2009-04-30
11:43:12 EDT ---
(In reply to comment #12)
(In reply to comment #10)
(In reply to comment #9)
Oh -- and this just a readability nit. I'd move this up to the top:
# rpm does not currently pull debuginfo out of mono packages
%global debug_package %{nil}
Ok, I have to move that to the tip of the file? or just before %description
?
top of the file. It's easiest to spot definitions of global variables when
they're at the top.
Sorry for the delay, to bussy the last ones...Ok, here is the new review:
http://www.ic.uach.cl/mhenriquez/fedora10-monoRPMS/monodevelop-debugger-gdb.spec.review4
I get the Paul answer for the mdb package, so I going to apply what to learn
here...stay tune..
Thanks Toshio
Mauricio
--
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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492927] Review Request: xcowsay - xcowsay displays a cute cow and message
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=492927 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #8 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-04-30 11:49:15 EDT --- Okay. - Your pre-review seems good for initial comments - This package itself is okay. _ This package is approved by mtasaka - \ ^__^ \ (oo)\___ (__)\ )\/\ ||w | || || Please follow the procedure written on: http://fedoraproject.org/wiki/PackageMaintainers/Join from Get a Fedora Account. After you request for sponsorship a mail will be sent to sponsor members automatically (which is invisible for you) which notifies that you need a sponsor. After that, please also write on this bug for confirmation that you requested for sponsorship and your FAS (Fedora Account System) name. Then I will sponsor you. If you want to import this package into Fedora 9/10/11, you also have to look at http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT (after once you rebuilt this package on koji Fedora rebuilding system). If you have questions, please ask me. -- 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438588] Review Request: zfstream - C++ iostream like access to compressed 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=438588 --- Comment #15 from Thomas Sailer t.sai...@alumni.ethz.ch 2009-04-30 11:53:22 EDT --- (In reply to comment #14) I don't see any change. In fact, you shouldn't shorten them. Some lines in the description are even smaller than 70 columns. They need stretched out a little. Ah sorry, I misunderstood what you meant. Will fix before checkin, 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
