[Bug 501223] Review Request: tuxmath - Educational math tutor for children
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=501223 --- Comment #9 from Fedora Update System 2009-06-12 02:30:14 EDT --- tuxmath-1.7.2-4.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/tuxmath-1.7.2-4.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 501223] Review Request: tuxmath - Educational math tutor for children
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=501223 --- Comment #10 from Fedora Update System 2009-06-12 02:30:19 EDT --- tuxmath-1.7.2-4.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/tuxmath-1.7.2-4.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 482216] Review Request: Mayavi - The Mayavi scientific data 3-dimensional visualizer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482216 Orcan 'oget' Ogetbil changed: What|Removed |Added CC||oget.fed...@gmail.com AssignedTo|nob...@fedoraproject.org|oget.fed...@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 502525] Review Request: lazygal - static photo gallery generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=502525 --- Comment #10 from Mamoru Tasaka 2009-06-12 01:47:42 EDT --- No need to fix as CVS admin will understand what you mean. By the way please set the fedora-cvs flag to ? . -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 499579] Review Request: libxdg-basedir - Implementation of the XDG Base Directory Specifications
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=499579 Kevin Fenzi changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #9 from Kevin Fenzi 2009-06-12 00:07:29 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500476] Review Request: cups-bjnp - cups backend for Canon bjnp (USB over IP) printers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=500476 Kevin Fenzi changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #16 from Kevin Fenzi 2009-06-12 00:13:50 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505448] Review Request: perl-Pod-Xhtml - Generate well-formed XHTML documents from POD format documentation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505448 Parag AN(पराग) 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(पराग) 2009-06-12 00:55:10 EDT --- Review: + package builds in mock (rawhide i586). koji Build =>http://koji.fedoraproject.org/koji/taskinfo?taskID=1407297 + rpmlint is silent for SRPM and for RPM. + source files match upstream url fcd2b4ff57da57b04c34fcfd62f43f06d5f6f0a9 Pod-Xhtml-1.59.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=5, Tests=44, 1 wallclock secs ( 0.06 usr 0.02 sys + 0.43 cusr 0.06 csys = 0.57 CPU) + Package perl-Pod-Xhtml-1.59-1.fc10 => Provides: perl(Pod::Hyperlink::BounceURL) = 1.7 perl(Pod::Xhtml) Requires: /usr/bin/perl perl(Carp) perl(Getopt::Long) perl(Pod::ParseUtils) perl(Pod::Parser) perl(Pod::Usage) perl(Pod::Xhtml) perl(URI::Escape) perl(constant) 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 505447] Review Request: perl-HTML-FillInForm - Populates HTML Forms with data
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505447 Parag AN(पराग) 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(पराग) 2009-06-12 00:55:07 EDT --- Review: + package builds in mock (rawhide i586). koji Build =>http://koji.fedoraproject.org/koji/taskinfo?taskID=1407299 + rpmlint is silent for SRPM and for RPM. + source files match upstream url 9d107115e4f9e9c8e7aba53183e75744f2a8a6c4 HTML-FillInForm-2.00.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=23, Tests=104, 2 wallclock secs ( 0.14 usr 0.05 sys + 0.97 cusr 0.19 csys = 1.35 CPU) + Package perl-HTML-FillInForm-2.00-1.fc12.noarch=> Provides: perl(HTML::FillInForm) = 2.00 Requires: perl >= 0:5.005 perl(Carp) perl(HTML::Parser) >= 3.26 perl(integer) 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 505260] Review Request: perl-Exception-Class-TryCatch - Syntactic try/catch sugar for use with Exception::Class
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505260 Kevin Fenzi changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Kevin Fenzi 2009-06-12 00:33:24 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505155] Review Request: libcap-ng - An alternative posix capabilities 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=505155 Kevin Fenzi changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #17 from Kevin Fenzi 2009-06-12 00:32:25 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 502065] Review Request: slashem - Super Lotsa Added Stuff Hack - Extended Magic
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=502065 Kevin Fenzi changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #10 from Kevin Fenzi 2009-06-12 00:20:12 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505046] Review Request: perl-Config-Auto - Magical config file 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=505046 Kevin Fenzi changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Kevin Fenzi 2009-06-12 00:29:35 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 502525] Review Request: lazygal - static photo gallery generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=502525 --- Comment #9 from Byron Clark 2009-06-12 00:51:59 EDT --- I misunderstood the instructions for the CVS Request and included F-12 when it is not necessary. Is there a way to fix that request or will it go through without changes? -- Configure 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 502525] Review Request: lazygal - static photo gallery generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=502525 --- Comment #8 from Byron Clark 2009-06-12 00:36:43 EDT --- New Package CVS Request === Package Name: lazygal Short Description: A static web gallery generator Owners: byronclark Branches: F-11 F-12 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 451744] Review Request: root - The CERN analyzer for high to medium energy physics
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=451744 --- Comment #28 from Suvayu 2009-06-11 23:33:09 EDT --- Hi, I have been following this bug but I have no experience in packaging. So I posted to the project mailing list. The developers replied back about some of the issues discussed here (e.g. CINT). They also mentioned they will be able to help if someone takes this up with the developer team at root...@root.cern.ch or on their bug-tracker at https://savannah.cern.ch/projects/savroot/ . I am quoting the email below. Maybe this will help someone. - Suvayu *** quote begins *** Hi Suvayu, a few commentes: * CINT needs the include files in cint/cint/include at runtime. They are not build time include files (those are in cint/cint/inc) - think of them as something to be put in /usr/share/root. * ROOT does not use the stand-alone CINT binary; we split them intentionally also to allow package maintainers to produce CINT and ROOT packages without overlap. * ROOT is part of Debian, so you might want to contact "them" for the design they used. They managed to create ROOT modules like this (output from apt-get install root-, so some might be bogus): root-db-clientroot-plugin-glroot-plugin-netx root-plugin-ruby root-system-bin root-file-server root-plugin-hbook root-plugin-odbc root-plugin-sql root-system-common root-fitter root-plugin-krb5 root-plugin-peac root-plugin-tmva root-system-doc root-glviewer root-plugin-ldap root-plugin-pgsql root-plugin-unuranroot-system-proofd root-plugin-asimage root-plugin-mathmore root-plugin-proof root-plugin-xml root-system-rootd root-plugin-clarens root-plugin-minuitroot-plugin-python root-plugin-xproofroot-system-xrootd root-plugin-eve root-plugin-minuit2 root-plugin-qt root-proofd root-tail root-plugin-fftw3 root-plugin-mlp root-plugin-quadproot-ttf root-plugin-fumiliroot-plugin-mysql root-plugin-roofit root-system * ROOT in Debian is a proof that there are no licensing issues ;-) * You won't have fun with your build result if you use the fedora default libAfterImage. We use the bleeding edge one all the time, basically because we are one of its main drivers for development. You have been warned :-) * Christian Holm Christensen is our packaging guru, as you were probably able to tell looking at $ROOTSYS/build/package/rpm/spec.in. Cheers, Axel. -- Configure 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 502614] Review Request: stfl - STFL implements a curses-based widget set for text terminals
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=502614 --- Comment #10 from Mamoru Tasaka 2009-06-11 23:08:23 EDT --- Note that currently I sponsor Byron. -- Configure 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 502525] Review Request: lazygal - static photo gallery generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=502525 Mamoru Tasaka changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | --- Comment #7 from Mamoru Tasaka 2009-06-11 23:06:44 EDT --- Okay, now I am sponsoring you. Please follow "Join" wiki again. -- Configure 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 426750] Review Request: ghc-utf8-string - Support reading and writing UTF8 Strings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=426750 --- Comment #19 from Jens Petersen 2009-06-11 23:01:04 EDT --- I think you can also change it when you come to import to cvs, but I would suggest simply: Summary: Haskell UTF8 layer for IO and Strings -- Configure 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 482216] Review Request: Mayavi - The Mayavi scientific data 3-dimensional visualizer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482216 --- Comment #9 from Rakesh Pandit 2009-06-11 22:50:01 EDT --- Please use all dependencies from latest builds in koji (as I have just pushed them). These packages are: python-EnvisagePlugins-3.1.0 python-Traits-3.1.0 python-AppTools-3.2.0 python-Traits-3.1.0 python-EnthoughtBase-3.0.2 python-TraitsBackendQt-3.1.0 python-EnvisageCore-3.1.0 python-TraitsGUI-3.0.4 for all releases. -- Configure 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 482216] Review Request: Mayavi - The Mayavi scientific data 3-dimensional visualizer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482216 --- Comment #8 from Rakesh Pandit 2009-06-11 22:46:40 EDT --- http://rakesh.fedorapeople.org/spec/Mayavi.spec http://rakesh.fedorapeople.org/srpm/Mayavi-3.2.0-2.fc11.src.rpm Updated. -- Configure 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 484423] Review Request: netbook-launcher - A clutter-based desktop launcher
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=484423 --- Comment #14 from Christoph Wickert 2009-06-11 22:10:06 EDT --- OK, just a short message I'm still working on this. From a packaging POV most things look good (but there are some problems), but the app itself needs some love, especially the patch: - Logout dialog is huge, at least when using German localization - The dialog is not closed after switching user or locking screen - Icons standby and hibernate are not the right ones, should be the ones from g-p-m - Icon for reboot should be xfsm-reboot from xfce-session I guess - The Ubuntu artwork should be replaced with Fedora or removed - Change the colors to something more Fedora-like? I will look into these this weekend. BTW: The package does not build in F-12 due to missing BRs. I guess this a problem with pkg-config evaluating Requires.private in a pkgconfig file that is missing a requirement, but I haven't looked what is actually missing. -- Configure 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 426750] Review Request: ghc-utf8-string - Support reading and writing UTF8 Strings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=426750 --- Comment #18 from Zach Oglesby 2009-06-11 22:06:06 EDT --- Here you go hopefully this will do it http://zoglesby.fedorapeople.org/ghc-utf8-string.spec http://zoglesby.fedorapeople.org/ghc-utf8-string-0.3.5-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 505155] Review Request: libcap-ng - An alternative posix capabilities 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=505155 Steve Grubb changed: What|Removed |Added Flag||fedora-cvs? --- Comment #16 from Steve Grubb 2009-06-11 22:04:40 EDT --- New Package CVS Request === Package Name: libcap-ng Short Description: An alternative posix capabilities library Owners: sgrubb Branches: F-11 F10 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 482216] Review Request: Mayavi - The Mayavi scientific data 3-dimensional visualizer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482216 --- Comment #7 from Rakesh Pandit 2009-06-11 21:48:03 EDT --- http://rakesh.fedorapeople.org/misc/Mayavi.spec http://rakesh.fedorapeople.org/misc/Mayavi-3.2.0-1.fc11.src.rpm Updated. -- Configure 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 426750] Review Request: ghc-utf8-string - Support reading and writing UTF8 Strings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=426750 --- Comment #17 from Jens Petersen 2009-06-11 21:47:25 EDT --- Zach, please apply for the packager group in FAS and I will sponsor you upon successful completion of this review. :) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 426750] Review Request: ghc-utf8-string - Support reading and writing UTF8 Strings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=426750 --- Comment #16 from Jens Petersen 2009-06-11 21:43:20 EDT --- Created an attachment (id=347505) --> (https://bugzilla.redhat.com/attachment.cgi?id=347505) ghc-utf8-string.spec-1.patch Basically likes fine to me now. I suggest this little patch to describe the package more. -- Configure 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 426750] Review Request: ghc-utf8-string - Support reading and writing UTF8 Strings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=426750 --- Comment #15 from Jens Petersen 2009-06-11 21:41:56 EDT --- I sent a mail to the maintainer asking to add license info to the above files. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 496635] Review Request: monodevelop-debugger-mdb - Mono 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=496635 --- Comment #46 from Toshio Ernie Kuratomi 2009-06-11 21:19:40 EDT --- Excellent. Mauricio, please go to the account system and apply for the packager group: https://admin.fedoraproject.org/accounts/ I'll sponsor you in and then you can import and build your packages. Paul has agreed to guide you in your work (since I don't do very much with mono packaging) so when you do your cvs request you can put him on the package as well. Note that once I've sponsored you, you'll be on step #4 here: http://fedoraproject.org/wiki/Package_Review_Process -- Configure 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 Toshio Ernie Kuratomi changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #16 from Toshio Ernie Kuratomi 2009-06-11 21:15:46 EDT --- Okay, so Paul's finished the review of the other package. This package is 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 502525] Review Request: lazygal - static photo gallery generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=502525 --- Comment #6 from Byron Clark 2009-06-11 21:00:12 EDT --- I applied for the packager group. FAS Username: byronclark -- Configure 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 426750] Review Request: ghc-utf8-string - Support reading and writing UTF8 Strings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=426750 --- Comment #14 from Zach Oglesby 2009-06-11 20:33:36 EDT --- Here is the updated 0.3.5 package http://zoglesby.fedorapeople.org/ghc-utf8-string.spec http://zoglesby.fedorapeople.org/ghc-utf8-string-0.3.5-1.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 426750] Review Request: ghc-utf8-string - Support reading and writing UTF8 Strings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=426750 --- Comment #13 from Jens Petersen 2009-06-11 20:08:28 EDT --- I think it is not a blocker but I note that the following src files are missing license info/headers: System/Environment/UTF8.hs Data/ByteString/UTF8.hs Data/ByteString/Lazy/UTF8.hs Data/String/UTF8.hs Codec/Binary/UTF8/Generic.hs It would be good to point this out to the maintainer so that they can hopefully add headers to those files. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 495902] Review Request: olpc-kbdshim - grab key and better rotation support for the XO laptop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=495902 --- Comment #24 from Christoph Wickert 2009-06-11 20:00:00 EDT --- (In reply to comment #22) > Yeah, F-9 branches are no longer done. > You can ask for an exception, I guess from FESCo? Ticket filed as https://fedorahosted.org/fesco/ticket/163 -- Configure 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 467655] Review Request: yafaray - a raytracer for Blender.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=467655 --- Comment #51 from Paulo Roma Cavalcanti 2009-06-11 19:53:09 EDT --- (In reply to comment #50) > libsunsky.so from yafaray-blender requires text relocation. > http://people.redhat.com/drepper/textrelocs.html > This requirement have to be either removed, or handled by the appropriate > selinux-policy-targeted package (you have to submit a bug against the said > package). > I always deactivate selinux on my systems. That is why I have never noticed the problem. > > Unfortunately I was still not able to have a working yafaray rendering with > blender. I was testing it with a newly installed F-10 i686 with flgrx. I was > not able to test with F-11 because some artifacts with the radeon driver... Why not? Because of the selinux problem or because of the ATI driver? I only have nvidia cards running the proprietary driver. > > Given that blender 2.48 seems to be the lastest version to support yafray. I > would request blender to be keepted at 2.48a for F-10. I am running blender 2.49 just fine with yafaray on F10 x86_64, but I cannot say if version 2.49 is stable enough (despite of yafaray). -- Configure 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 426750] Review Request: ghc-utf8-string - Support reading and writing UTF8 Strings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=426750 --- Comment #12 from Jens Petersen 2009-06-11 19:45:02 EDT --- Thanks: convention is to have a newline between changelog entries. Also there is a newer version on hackage: http://hackage.haskell.org/cgi-bin/hackage-scripts/package/utf8-string Also please update the srpm too when updating the spec file. :) There are a few rpmlint warnings from the binary packages: ghc-utf8-string-doc.x86_64: E: description-line-too-long This package contains development documentation files for the ghc-utf8-string library. 1 packages and 0 specfiles checked; 1 errors, 0 warnings. Would be good to fix. ghc-utf8-string-prof.x86_64: E: devel-dependency ghc-utf8-string-devel ghc-utf8-string-prof.x86_64: W: no-documentation ghc-utf8-string-prof.x86_64: W: devel-file-in-non-devel-package /usr/lib64/ghc-6.10.3/utf8-string-0.3.5/libHSutf8-string-0.3.5_p.a 1 packages and 0 specfiles checked; 1 errors, 2 warnings. Normal and can be waived. -- Configure 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 467655] Review Request: yafaray - a raytracer for Blender.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=467655 --- Comment #50 from Nicolas Chauvet (kwizart) 2009-06-11 19:03:38 EDT --- libsunsky.so from yafaray-blender requires text relocation. http://people.redhat.com/drepper/textrelocs.html This requirement have to be either removed, or handled by the appropriate selinux-policy-targeted package (you have to submit a bug against the said package). Unfornately I was still not able to have a working yafaray rendering with blender. I was testing it with a newly installed F-10 i686 with flgrx. I was not able to test with F-11 because some artifacts with the radeon driver... Given that blender 2.48 seems to be the lastest version to support yafray. I would request blender to be keepted at 2.48a for F-10. -- Configure 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 505448] New: Review Request: perl-Pod-Xhtml - Generate well-formed XHTML documents from POD format documentation
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-Pod-Xhtml - Generate well-formed XHTML documents from POD format documentation https://bugzilla.redhat.com/show_bug.cgi?id=505448 Summary: Review Request: perl-Pod-Xhtml - Generate well-formed XHTML documents from POD format documentation Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: emmanuel.sey...@club-internet.fr QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://people.parinux.org/~seyman/fedora/perl-Pod-Xhtml/perl-Pod-Xhtml.spec SRPM URL: http://people.parinux.org/~seyman/fedora/perl-Pod-Xhtml/perl-Pod-Xhtml-1.59-1.fc11.src.rpm Description: This module files containing POD content and genrates well-formed XHTML documents from it. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505155] Review Request: libcap-ng - An alternative posix capabilities 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=505155 --- Comment #15 from Tomas Mraz 2009-06-11 18:35:08 EDT --- OK -- Configure 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 505447] New: Review Request: perl-HTML-FillInForm - Populates HTML Forms with data
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-HTML-FillInForm - Populates HTML Forms with data https://bugzilla.redhat.com/show_bug.cgi?id=505447 Summary: Review Request: perl-HTML-FillInForm - Populates HTML Forms with data Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: emmanuel.sey...@club-internet.fr QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://people.parinux.org/~seyman/fedora/perl-HTML-FillInForm/perl-HTML-FillInForm.spec SRPM URL: http://people.parinux.org/~seyman/fedora/perl-HTML-FillInForm/perl-HTML-FillInForm-2.00-1.fc11.src.rpm Description: This module fills in an HTML form with data from a Perl data structure, allowing you to keep the HTML and Perl separate. -- Configure 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 499579] Review Request: libxdg-basedir - Implementation of the XDG Base Directory Specifications
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=499579 Michal Nowak changed: What|Removed |Added Flag||fedora-cvs? --- Comment #8 from Michal Nowak 2009-06-11 18:09:04 EDT --- New Package CVS Request === Package Name: libxdg-basedir Short Description: Implementation of the XDG Base Directory Specifications Owners: mnowak Branches: F-10 F-11 InitialCC: mnowak -- Configure 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 499579] Review Request: libxdg-basedir - Implementation of the XDG Base Directory Specifications
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=499579 --- Comment #7 from Michal Nowak 2009-06-11 18:03:18 EDT --- Thanks a lot, Christian! -- Configure 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 505046] Review Request: perl-Config-Auto - Magical config file 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=505046 Emmanuel Seyman changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Emmanuel Seyman 2009-06-11 17:38:54 EDT --- (In reply to comment #1) > > Should: > 1) Drop following Requires as yum will going to resolve it automatically. > Requires: perl(Config::IniFiles) > Requires: perl(Test::More) Done. I've also parsed all my .spec files so that this doesn't happen again. Again, thanks for the review. New Package CVS Request === Package Name: perl-Config-Auto Short Description: Magical config file parser Owners: eseyman Branches: F-11 F-10 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 505155] Review Request: libcap-ng - An alternative posix capabilities 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=505155 --- Comment #14 from Steve Grubb 2009-06-11 17:37:18 EDT --- I must have had a typo when I tried it. It works. New spec file and srpm uploaded. -- Configure 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 505260] Review Request: perl-Exception-Class-TryCatch - Syntactic try/catch sugar for use with Exception::Class
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505260 Emmanuel Seyman changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Emmanuel Seyman 2009-06-11 17:31:46 EDT --- (In reply to comment #1) > > Should: > 1) Remove unnecessary > BuildRequires: perl >= 0:5.005 Done. Thanks, Parag. New Package CVS Request === Package Name: perl-Exception-Class-TryCatch Short Description: Syntactic try/catch sugar for use with Exception::Class Owners: eseyman Branches: F-11 F-10 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 505155] Review Request: libcap-ng - An alternative posix capabilities 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=505155 --- Comment #13 from Jussi Lehtola 2009-06-11 17:24:41 EDT --- (In reply to comment #12) > But wrt to item 1, the only complication is that its saving the cwd and using > it later. Its impossible to make a relative link without being in the > directory > so that the kernel sees both ends connected. You cannot make an absolute link > since this is in the build root. So, I have updated the spec file with only > the > libdir fix. AFAIK there's nothing to prevent it. You will just get a dangling-symlink warning from rpmlint if the file you link to is not in the same rpm 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 499579] Review Request: libxdg-basedir - Implementation of the XDG Base Directory Specifications
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=499579
Christian Krause changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #6 from Christian Krause 2009-06-11 17:21:30
EDT ---
Hello Michal,
(In reply to comment #5)
> (In reply to comment #4)
> Both issues fixed:
>
> * Tue Jun 9 2009 Michal Nowak - 1.0.1-2
> - removed bogus ownership of %%{_libdir}/pkgconfig/
> - "docs" sub-package renamed to "doc"
>
>
> http://mnowak.fedorapeople.org/libxdg-basedir/libxdg-basedir.spec
> http://mnowak.fedorapeople.org/libxdg-basedir/libxdg-basedir-1.0.1-2.fc11.src.rpm
>
I've reviewed the new packages and all reported issues are fixed.
-> APPROVED.
Christian
--
Configure 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 488096] Review Request: rygel - A UPnP v2 Media Server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488096 Peter Robinson changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Comment #24 from Peter Robinson 2009-06-11 17:17:11 EDT --- Added to F11 as an update https://admin.fedoraproject.org/updates/rygel-0.3-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 505155] Review Request: libcap-ng - An alternative posix capabilities 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=505155
--- Comment #12 from Steve Grubb 2009-06-11 17:08:00 EDT
---
In response to comment #11, I can remove the '/' from %{_libdir} even though it
hurts nothing.
But wrt to item 1, the only complication is that its saving the cwd and using
it later. Its impossible to make a relative link without being in the directory
so that the kernel sees both ends connected. You cannot make an absolute link
since this is in the build root. So, I have updated the spec file with only the
libdir fix.
--
Configure 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 495902] Review Request: olpc-kbdshim - grab key and better rotation support for the XO laptop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=495902 --- Comment #23 from Fedora Update System 2009-06-11 16:40:13 EDT --- olpc-kbdshim-6-4.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/olpc-kbdshim-6-4.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 496635] Review Request: monodevelop-debugger-mdb - Mono 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=496635
--- Comment #45 from Paul Lange 2009-06-11 16:33:13 EDT ---
make
The %{name}-devel package contains development files for %{name}.
to
The %{name}-devel package contains development files for
%{name}.
%{name} is replaced with monodevelop-debugger-mdb and so the line gets longer
than 80 characters which we want to avoid.
I'll write Toshio a mail.
--
Configure 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 450527] Review Request: libkni3 - C++ library for the Katana robot arm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=450527 Tim Niemueller changed: What|Removed |Added Summary|Review Request: libkni -|Review Request: libkni3 - |C++ library for the Katana |C++ library for the Katana |robot arm |robot arm --- Comment #17 from Tim Niemueller 2009-06-11 16:27:41 EDT --- I'm back, thesis handed in and now there is more time again... - I have checked the library problems and fixed them with another patch. I have used this version with our arm and software and it works just fine. - I have renamed the package to libkni3. I have checked KNI4. But for older arms (the previous version, just out of production for a year) is not supported in version 4. Therefore version 3 will be necessary for some time to come (I'll contact Neuronics about this issue). I will file another review request for a libkni package, will track up2date upstream (i.e. v4.2 atm). Thanks for your patience, Jason. With your help we should be through with this real soon now. Sorry for the previous half-assed work, was distracted... New package is at http://fedorapeople.org/~timn/robotics/libkni3-3.9.2-9.fc10.src.rpm, spec at http://fedorapeople.org/~timn/robotics/libkni3.spec, successful scratch build is at http://koji.fedoraproject.org/koji/taskinfo?taskID=1406033 -- Configure 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 505374] Review Request: nilfs-utils - Utilities for managing NILFS v2 filesystems
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505374 --- Comment #3 from Eric Sandeen 2009-06-11 16:11:54 EDT --- Shoot, I hadn't even noticed the chmod u+s on some of these files, that strikes me as pretty odd, I'll have to investigate more. -- Configure 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 505374] Review Request: nilfs-utils - Utilities for managing NILFS v2 filesystems
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505374 --- Comment #2 from Eric Sandeen 2009-06-11 15:58:23 EDT --- wow, that was a bit messier than I expected ;) Ok, thanks for the review, will fix up. Heh, I got the original tarball from their original src.rpm which doesn't match their released tarball, sigh. I'll fix that too. Thanks! -Eric -- Configure 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 504891] Review Request: trove - Efficient Java collections
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504891 --- Comment #2 from Jerry James 2009-06-11 15:56:31 EDT --- I don't know anything about that other Trove project, but it sure appears to be dead. It's developer mailing list doesn't even exist any more. However, I'm all for avoiding name conflicts. Should I call this submission "trove4j", do you think? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505374] Review Request: nilfs-utils - Utilities for managing NILFS v2 filesystems
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=505374
--- Comment #1 from Jussi Lehtola 2009-06-11 15:54:38
EDT ---
- nilfs_cleanerd links against libnilfs.so.0 so the libraries should go in
%{_lib} as for other packages of this type.
rpmlint output:
nilfs-utils.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/lssu
['/usr/lib64']
nilfs-utils.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/mkcp
['/usr/lib64']
nilfs-utils.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/rmcp
['/usr/lib64']
nilfs-utils.x86_64: E: binary-or-shlib-defines-rpath /sbin/nilfs_cleanerd
['/usr/lib64']
nilfs-utils.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/lscp
['/usr/lib64']
nilfs-utils.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/dumpseg
['/usr/lib64']
nilfs-utils.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/chcp
['/usr/lib64']
nilfs-utils.x86_64: W: devel-file-in-non-devel-package /usr/lib64/libnilfs.so
nilfs-utils.x86_64: E: setuid-binary /sbin/mkfs.nilfs2 root 04755
nilfs-utils.x86_64: E: non-standard-executable-perm /sbin/mkfs.nilfs2 04755
nilfs-utils.x86_64: E: setuid-binary /sbin/nilfs_cleanerd root 04755
nilfs-utils.x86_64: E: non-standard-executable-perm /sbin/nilfs_cleanerd 04755
nilfs-utils.x86_64: E: setuid-binary /sbin/mount.nilfs2 root 04755
nilfs-utils.x86_64: E: non-standard-executable-perm /sbin/mount.nilfs2 04755
nilfs-utils.x86_64: E: setuid-binary /sbin/umount.nilfs2 root 04755
nilfs-utils.x86_64: E: non-standard-executable-perm /sbin/umount.nilfs2 04755
nilfs-utils.x86_64: W: one-line-command-in-%post /sbin/ldconfig
nilfs-utils.x86_64: W: one-line-command-in-%postun /sbin/ldconfig
nilfs-utils-devel.x86_64: W: no-documentation
4 packages and 0 specfiles checked; 15 errors, 4 warnings.
- Get rid of rpath.
- The .so file goes in -devel.
- Are the permissions as wanted? I have to check other file system tools.
- Use
%post -p /sbin/ldconfig
%postun -p /sbin/ldconfig
MUST: The spec file for the package is legible and macros are used
consistently. ~OK
- I'd use a macro for /sbin, say
%global _sbin /sbin
MUST: The package must be named according to the Package Naming Guidelines. OK
MUST: The spec file name must match the base package %{name}. OK
MUST: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines. OK
MUST: The License field in the package spec file must match the actual license.
OK
MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. NEEDSWORK
2e056e7979ed77727a7798d79de5188f nilfs-utils-2.0.12.tar.bz2
6819db4b59f9504abe68ebc7818fd6ae ../SOURCES/nilfs-utils-2.0.12.tar.bz2
MUST: The package MUST successfully compile and build into binary rpms. OK
MUST: The spec file MUST handle locales properly. N/A
MUST: Optflags are used and time stamps preserved. NEEDSWORK
- Time stamps are not preserved,
make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p"
should do the trick.
MUST: Packages containing shared library files must call ldconfig. OK
- See comments on rpmlint warnings.
MUST: A package must own all directories that it creates or require the package
that owns the directory. OK
MUST: Files only listed once in %files listings. OK
MUST: Debuginfo package is complete. OK
MUST: Permissions on files must be set properly. OK
MUST: Clean section exists. OK
MUST: Large documentation files must go in a -doc subpackage. N/A
MUST: All relevant items are included in %doc. Items in %doc do not affect
runtime of application. OK
- You might want to add
[ -s AUTHORS ] && exit 1
[ -s NEWS ] && exit 1
[ -s README ] && exit 1
to the %setup phase so you get notified if these acquire content.
MUST: Header files must be in a -devel package. OK
MUST: Static libraries must be in a -static package. N/A
MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'. N/A
MUST: If a package contains library files with a suffix then library files
ending in .so must go in a -devel package. NEEDSWORK
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency. OK
- You might use just
Requires: %{name} = %{version}-%{release}
to be clearer.
MUST: Packages does not contain any .la libtool archives. OK
MUST: Desktop files are installed properly. N/A
MUST: No file conflicts with other packages and no general names. OK
MUST: Buildroot cleaned before install. OK
SHOULD: %{?dist} tag is used in release. OK
SHOULD: If the package does not include license text(s) as separate files from
upstream, the packager should query upstream to include it. OK
SHOULD: The package builds in mock. OK
--
Configure 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 lis
[Bug 485652] Review Request: navit - Car navigation system with routing engine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=485652 --- Comment #11 from Adam Williamson 2009-06-11 15:52:19 EDT --- I hadn't tried it in mock, so it's possible, but it builds on Mandriva's buildsystem (which uses clean chroots), so... just a sec, I'll try it in mock here. Hmm, you're right. Will investigate. -- Fedora Bugzappers volunteer triage team https://fedoraproject.org/wiki/BugZappers -- Configure 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 504891] Review Request: trove - Efficient Java collections
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504891 --- Comment #1 from Jason Tibbitts 2009-06-11 15:51:59 EDT --- Any comments on the conflict between this and ESR's Trove project? http://catb.org/~esr/trove/ Is the other project sufficiently dead that this isn't an issue? Even this project's upstream seems to use "trove4j" in places. -- Configure 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 503847] Review Request: paperbox - A GTK tracker based document browser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=503847 --- Comment #7 from Mamoru Tasaka 2009-06-11 15:39:18 EDT --- (Note that I don't intend to do full-review on this package, I am just trying to check if this package correctly builds) 0.4.2-2.fc10 won't build on F-12 because of another issue: http://mtasaka.fedorapeople.org/Review_request/paperbox/MOCK-paperbox-x86_64-2.fc10.log With the following patch, at least on F-12 i386/x86_64 this package builds (again I just checked if this package builds or not) http://mtasaka.fedorapeople.org/Review_request/paperbox/paperbox-0.4.2-g++44.patch http://mtasaka.fedorapeople.org/Review_request/paperbox/MOCK-paperbox-i586-modified-2.fc10.log http://mtasaka.fedorapeople.org/Review_request/paperbox/MOCK-paperbox-x86_64-modified-fc10.log -- Configure 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 505374] Review Request: nilfs-utils - Utilities for managing NILFS v2 filesystems
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505374 Jussi Lehtola changed: What|Removed |Added Status|NEW |ASSIGNED CC||jussi.leht...@iki.fi AssignedTo|nob...@fedoraproject.org|jussi.leht...@iki.fi 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 503013] Review Request: gnaughty - Downloader for adult content
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=503013 --- Comment #17 from Simon Wesp 2009-06-11 15:32:58 EDT --- Tom, is there a decision of 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 505406] New: Review Request: rubygem-rake-compiler - Rake-based Ruby C Extension task generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: rubygem-rake-compiler - Rake-based Ruby C Extension task generator https://bugzilla.redhat.com/show_bug.cgi?id=505406 Summary: Review Request: rubygem-rake-compiler - Rake-based Ruby C Extension task generator Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mtas...@ioa.s.u-tokyo.ac.jp QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://mtasaka.fedorapeople.org/Review_request/rubygem-rake-compiler/rubygem-rake-compiler.spec SRPM URL: http://mtasaka.fedorapeople.org/Review_request/rubygem-rake-compiler/rubygem-rake-compiler-0.5.0-1.fc.src.rpm Description: rake-compiler aims to help Gem developers while dealing with Ruby C extensions, simplifiying the code and reducing the duplication. It follows *convention over configuration* and set an standarized structure to build and package C extensions in your gems. This is the result of expriences dealing with several Gems that required native extensions across platforms and different user configurations where details like portability and clarity of code were lacking. Koji build - For F-12: http://koji.fedoraproject.org/koji/taskinfo?taskID=1405896 - For F-11: http://koji.fedoraproject.org/koji/taskinfo?taskID=1405894 -- Configure 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 505155] Review Request: libcap-ng - An alternative posix capabilities 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=505155
Tomas Mraz changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #11 from Tomas Mraz 2009-06-11 15:23:12 EDT ---
I have only two nitpicks:
1. the script to create the new symlink is overly complicated why not do it
this way:
rm -f $RPM_BUILD_ROOT/%{_lib}/%{name}.so
mkdir -p $RPM_BUILD_ROOT/%{_libdir}
VLIBNAME=$(ls $RPM_BUILD_ROOT/%{_lib}/%{name}.so.*.*.*)
LIBNAME=$(basename $VLIBNAME)
ln -s ../../%{_lib}/$LIBNAME $RPM_BUILD_ROOT/%{_libdir}/%{name}.so
2. there is unnecessary / before %{_libdir} on the symlink line in %files
devel.
The rpmlint is still silent:
$ rpmlint -v libcap-ng-0.4.1-1.x86_64.rpm
libcap-ng.x86_64: I: checking
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
$ rpmlint -v libcap-ng-debuginfo-0.4.1-1.x86_64.rpm
libcap-ng-debuginfo.x86_64: I: checking
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
$ rpmlint -v libcap-ng-devel-0.4.1-1.x86_64.rpm
libcap-ng-devel.x86_64: I: checking
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
$ rpmlint -v libcap-ng-0.4.1-1.src.rpm
libcap-ng.src: I: checking
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
$ rpmlint -v libcap-ng-utils-0.4.1-1.x86_64.rpm
libcap-ng-utils.x86_64: I: checking
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
The nitpicks are no showstoppers so the package is ACCEPTED.
--
Configure 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 503256] Review Request: gtkmm-utils - C++ utility and widget library based on glibmm and gtkmm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=503256 --- Comment #42 from Jussi Lehtola 2009-06-11 15:24:23 EDT --- (In reply to comment #41) > By the way "BR: graphviz" may be missing as build.log contains > may lines like: Yes, this was noted already in #33. Thanks for enlightening on the rpath issue. John: please go through all comments and check that you've addressed them properly. -- Configure 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 478759] Review Request: perl-SystemPerl - SystemPerl 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=478759 --- Comment #31 from Fedora Update System 2009-06-11 15:20:56 EDT --- perl-SystemPerl-1.321-1.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/perl-SystemPerl-1.321-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 478759] Review Request: perl-SystemPerl - SystemPerl 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=478759 --- Comment #32 from Fedora Update System 2009-06-11 15:21:33 EDT --- perl-SystemPerl-1.321-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/perl-SystemPerl-1.321-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 503256] Review Request: gtkmm-utils - C++ utility and widget library based on glibmm and gtkmm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=503256 --- Comment #41 from Mamoru Tasaka 2009-06-11 15:15:39 EDT --- Well, binary-or-shlib-defines-rpath rpmlint issue is completely different from unstripped-binary-or-object rpmlint issue. While the latter one can usually be fixed by installing redhat-rpm-config (as I said before), the formar one usually needs some code fix. Note that in many cases binary-or-shlib-defines-rpath rpmlint warnings can occur on x86_64/ppc64 but not on i386/ppc. When you add "/usr/lib/rpm/check-rpaths" at the end of %install, the result is: - i586: http://koji.fedoraproject.org/koji/taskinfo?taskID=1405628 - x86_64: http://koji.fedoraproject.org/koji/taskinfo?taskID=1405624 Here actually only on x86_64 libgtkmm-utils.so.2.0.0 has rpath. There are several ways to fix this issue. A. https://fedoraproject.org/wiki/Packaging:Guidelines#Removing_Rpath B. I usually do -- %prep %setup -q sed -i.rpath -e \ '/sys_lib_dlsearch_path_spec/s|/usr/lib |/usr/lib /usr/lib64 /lib /lib64 |' \ configure -- The result of B is: http://koji.fedoraproject.org/koji/taskinfo?taskID=1405800 By the way "BR: graphviz" may be missing as build.log contains may lines like: -- Building full member lists recursively... Adding membsh: dot: command not found Problems running dot: exit code=127, command='dot', arguments='"classGtk_1_1EntryCompletion__inherit__graph.dot" -Tpng -o "classGtk_1_1EntryCompletion__inherit__graph.png"' sh: dot: command not found Problems running dot: exit code=127, command='dot', arguments='"classGtk_1_1EventBox__inherit__graph.dot" -Tpng -o "classGtk_1_1EventBox__inherit__graph.png"' -- -- Configure 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 448661] Review Request: u-boot-tools - contains mkimage a tool to create kernel bootable images for u-boot
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=448661 Peter Lemenkov changed: What|Removed |Added CC||lemen...@gmail.com --- Comment #17 from Peter Lemenkov 2009-06-11 15:02:34 EDT --- Ping. Any news here? -- Configure 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 505155] Review Request: libcap-ng - An alternative posix capabilities 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=505155 --- Comment #10 from Steve Grubb 2009-06-11 14:46:22 EDT --- New srpm has been uploaded to: http://people.redhat.com/sgrubb/libcap-ng/libcap-ng-0.4.1-1.src.rpm Please let me know if there are any other 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 467655] Review Request: yafaray - a raytracer for Blender.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=467655 --- Comment #49 from Paulo Roma Cavalcanti 2009-06-11 14:15:48 EDT --- Thanks. I have built it for F10. yafaray is still running fine with blender 2.49. At least one user acknowledged that yafaray rpms are working: http://www.yafaray.org/community/forum/viewtopic.php?f=12&t=2257 -- Configure 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 500476] Review Request: cups-bjnp - cups backend for Canon bjnp (USB over IP) printers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=500476 --- Comment #15 from Louis Lagendijk 2009-06-11 13:47:25 EDT --- New Package CVS Request === Package Name: cups-bjnp Short Description: CUPS backend for the Canon BJNP network printers Owners: llagendijk Branches: F-10 F-11 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 500476] Review Request: cups-bjnp - cups backend for Canon bjnp (USB over IP) printers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=500476 Louis Lagendijk 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 504076] Review Request: libiodbc - iODBC Driver Manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504076 --- Comment #12 from Fedora Update System 2009-06-11 13:42:51 EDT --- libiodbc-3.52.6-4.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/libiodbc-3.52.6-4.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 504076] Review Request: libiodbc - iODBC Driver Manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504076 --- Comment #13 from Fedora Update System 2009-06-11 13:43:29 EDT --- libiodbc-3.52.6-4.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/libiodbc-3.52.6-4.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 504076] Review Request: libiodbc - iODBC Driver Manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504076 Rex Dieter changed: What|Removed |Added Alias||libiodbc -- Configure 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 502525] Review Request: lazygal - static photo gallery generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=502525 Mamoru Tasaka changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp Flag||fedora-review+ --- Comment #5 from Mamoru Tasaka 2009-06-11 13:31:30 EDT --- Okay. - This package itself is now okay. - Fedora requests a people who needs to be sponsored to either submit another review request or do a pre-review of other person's review request to "show that you have an understanding of the process and of the packaging guidelines" as is described on : http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored Now you already submitted several other review request and from a quick look they are well packaged to some extent. This package (lazygal) is APPROVED by mtasaka 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 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 485652] Review Request: navit - Car navigation system with routing engine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=485652 Patrick Laughton changed: What|Removed |Added CC||j...@jima.tk --- Comment #10 from Patrick Laughton 2009-06-11 13:29:50 EDT --- Is my environment screwed up, or is autopoint looking for `cvs` during the build process? I've tried both of your SRPMs to the same result. Adding a BR leads to some sort of automake fun. Am I missing something? -- Configure 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 505374] New: Review Request: nilfs-utils - Utilities for managing NILFS v2 filesystems
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: nilfs-utils - Utilities for managing NILFS v2 filesystems https://bugzilla.redhat.com/show_bug.cgi?id=505374 Summary: Review Request: nilfs-utils - Utilities for managing NILFS v2 filesystems Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: esand...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec & src.rpm at: http://sandeen.fedorapeople.org/nilfs-utils/ Description: Userspace utilities for creating and mounting NILFS v2 filesystems. (nilfs is a new upstream filesystem, enabled in rawhide, having userspace around might be nice) -- Configure 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 505293] Review Request: kobo - python modules for tools development
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505293 Jason Tibbitts changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #6 from Jason Tibbitts 2009-06-11 13:05:22 EDT --- CVS done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 502795] Review Request: mcu8051ide - IDE for MCS-51 based microcontrollers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=502795 Jason Tibbitts changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #14 from Jason Tibbitts 2009-06-11 13:04:37 EDT --- CVS done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505138] Review Request: trac-peerreview-plugin - Framework for realtime code review within Trac
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505138 Jason Tibbitts changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Jason Tibbitts 2009-06-11 13:03:47 EDT --- Please stick to the CVS request format; it saves manual work by your friendly CVS admin. CVS done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478759] Review Request: perl-SystemPerl - SystemPerl 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=478759 Jason Tibbitts changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #30 from Jason Tibbitts 2009-06-11 13:02:14 EDT --- CVS done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 504748] Review Request: perl-PDF-Reuse - Reuse and mass produce PDF documents
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504748 Jason Tibbitts changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #7 from Jason Tibbitts 2009-06-11 13:01:18 EDT --- CVS done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481523] Review Request: rubygem-trollop - A command-line option parsing library 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=481523 --- Comment #2 from Wesley Tanaka 2009-06-11 12:55:31 EDT --- Created an attachment (id=347439) --> (https://bugzilla.redhat.com/attachment.cgi?id=347439) add rubygem-hoe dependency rpmbuild --rebuild rubygem-trollop-1.10.2-0.fc10.src.rpm (and also on rubygem-ditz-0.5-0.fc10.src.rpm) produces this error: rake aborted! no such file to load -- hoe The error (both for trollop and ditz) went away after running yum install rubygem-hoe so I think this change might be needed to the .spec file -- Configure 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 471231] Review Request: WebCalendar - Single/multi-user web-based calendar application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=471231 --- Comment #5 from Patrick Monnerat 2009-06-11 12:52:34 EDT --- New SRPM: http://monnerat.fedorapeople.org/WebCalendar-1.2.0-5.fc12.src.rpm _ hkit unbundled. php-hkit (review request 505358) required _ PHPMailer unbundled. php-PHPMailer (review request 505356) required _ captcha unbundled. php-captchaphp (review request 505354) required _ JSCookMenu unbundled. JSCookMenu (review request 505360) required koji scratch build at http://koji.fedoraproject.org/koji/taskinfo?taskID=1405564 rpmlint output: WebCalendar.noarch: W: dangling-symlink /usr/share/WebCalendar/includes/classes/phpmailer/class.smtp.php /usr/share/php/class.smtp.php WebCalendar.noarch: W: non-standard-gid /etc/WebCalendar/settings.php apache WebCalendar.noarch: E: non-readable /etc/WebCalendar/settings.php 0660 WebCalendar.noarch: W: dangling-symlink /usr/share/WebCalendar/includes/classes/hKit/hkit.class.php /usr/share/php/hkit.class.php WebCalendar.noarch: W: non-standard-gid /etc/WebCalendar apache WebCalendar.noarch: E: non-standard-dir-perm /etc/WebCalendar 0775 WebCalendar.noarch: W: dangling-symlink /usr/share/WebCalendar/includes/classes/phpmailer/class.phpmailer.php /usr/share/php/class.phpmailer.php New warnings (i.e.: dangling-symlinks) are symbolic links to files in the required packages mentioned above. _ I did not contact Ray Jones, since its captcha updates are now deleted from the package. _ There are still files in the menu subdir: these are WebCalendar-specific menu themes for JSCookMenu. _ Everaldo's icons are retained for now: they are used by the specific menu themes. I left a question at WebCalendar's forum about icons licensing (https://sourceforge.net/forum/forum.php?thread_id=3299158&forum_id=11587), but I have no answer yet. _ There is still a module I have tried to unbundle: popups.php. It uses some Javascript code (infobox.js) from Klaus Knopper's web site (http://www.knopper.net), but although this original code is GPL2, it has never been publicly released (there is no home page for this script: it is only accessible as a part of Knopper's site). In addition, both versions have been changed in incompatible ways. Thus unbundling this code and making it a maintainable package is hardly feasible: I suggest we live with it as now. _ Unbundling captchaphp was a pain the the ass, because it was not initially intended to be used this way ! -- Configure 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 491430] Review Request: sslogger - A keystroke logging utility for privileged user escalation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491430 --- Comment #26 from Mamoru Tasaka 2009-06-11 12:35:59 EDT --- (In reply to comment #25) > Any one care to suggest a new name for sl before i pick one out of the hat > > -Ed Maybe %_bindir/slog or so? -- Configure 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 493250] Review Request: perl-Goo-Canvas -- Goo::Canvas Perl interface to the GooCanvas
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=493250
--- Comment #32 from Mamoru Tasaka 2009-06-11
12:28:22 EDT ---
For 0.05-6:
- The directory %{perl_vendorarch}/Goo/ should be owned by
this package as currently no package owns this directory.
- By the way it seems that latest version is 0.06. Would you
upgrade your srpm?
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505354] Review Request: php-captchaphp - PHP very user-friendly CAPTCHA solution
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505354 --- Comment #1 from Patrick Monnerat 2009-06-11 12:16:40 EDT --- Please note that the original (public domain) tarball includes a font with an incompatible license: the package removes this font, requires the dejavu fonts and uses those as a default replacement. -- Configure 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 505360] New: Review Request: JSCookMenu - Javascript GUI-like web menus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: JSCookMenu - Javascript GUI-like web menus https://bugzilla.redhat.com/show_bug.cgi?id=505360 Summary: Review Request: JSCookMenu - Javascript GUI-like web menus Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: p...@datasphere.ch QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://monnerat.fedorapeople.org/JSCookMenu.spec SRPM URL: http://monnerat.fedorapeople.org/JSCookMenu-2.0.4-1.fc12.src.rpm Description: JSCookMenu is a powerful menu script written in JavaScript that can mimic complex menus found in popular GUI Applications. It is relatively simple and easy to use. Creating a new theme requires some patience, but rarely does one has to write one since some good ones are provided. The following features are implemented: * Supports both horizontal and vertical menus. * Supports relative positioning. * Supports different menus with different themes in the same web page. * Eases the menu creation process with a menu builder. * Special effects such as sliding and fading in/out is available. * APIs for JavaScript developers. rpmlint output: JSCookMenu.noarch: W: no-documentation --> No documentation files provided in the upstream tarball. The MIT-compatible license text is in script comment. koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1405548 -- Configure 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 505358] New: Review Request: php-hkit - Simple PHP5 API for extracting common microformats from a page
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: php-hkit - Simple PHP5 API for extracting common microformats from a page https://bugzilla.redhat.com/show_bug.cgi?id=505358 Summary: Review Request: php-hkit - Simple PHP5 API for extracting common microformats from a page Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: p...@datasphere.ch QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://monnerat.fedorapeople.org/php-hkit.spec SRPM URL: http://monnerat.fedorapeople.org/php-hkit-0.5-1.fc12.src.rpm Description: hKit is a simple toolkit for extracting common microformats from a page. The page can be presented as a string or a URL, and the result is handed back as a standard PHP array structure. hKit uses SimpleXML for parsing, and therefore requires PHP5. Designed for humans first and machines second, microformats are a set of simple, open data formats built upon existing and widely adopted standards. The only microformat module currently supported by hKit is hCard. However, hKit makes possible for a user to write its own microformat module. rpmlint output: none koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1405534 -- Configure 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 505356] New: Review Request: php-PHPMailer - PHP email transport class with a lot of features
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: php-PHPMailer - PHP email transport class with a lot of features https://bugzilla.redhat.com/show_bug.cgi?id=505356 Summary: Review Request: php-PHPMailer - PHP email transport class with a lot of features Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: p...@datasphere.ch QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://monnerat.fedorapeople.org/php-PHPMailer.spec SRPM URL: http://monnerat.fedorapeople.org/php-PHPMailer-5.0.2-1.fc12.src.rpm Description: PHP email transport class featuring file attachments, SMTP servers, CCs, BCCs, HTML messages, word wrap, and more. Sends email via sendmail, PHP mail(), QMail, or with SMTP. rpmlint output: none koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1405528 -- Configure 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 505354] New: Review Request: php-captchaphp - PHP very user-friendly CAPTCHA solution
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: php-captchaphp - PHP very user-friendly CAPTCHA solution https://bugzilla.redhat.com/show_bug.cgi?id=505354 Summary: Review Request: php-captchaphp - PHP very user-friendly CAPTCHA solution Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: p...@datasphere.ch QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://monnerat.fedorapeople.org/php-captchaphp.spec SRPM URL: http://monnerat.fedorapeople.org/php-captchaphp-2.0-1.fc12.src.rpm Description: This PHP script provides a very user-friendly CAPTCHA solution. You can easily embed it into your generation scripts to prevent spambot access. It strives to be accessible and implements an arithmetic riddle as alternative for visually impaired users. It does not require cookies, but makes use of "AJAX" to give users visual feedback for solving the CAPTCHA. It grants access fuzzily (when single letters were misguessed) instead of frustrating people. And it is rather customizable. rpmlint output: none koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1405513 -- Configure 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 226117] Merge Review: mailman
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=226117
--- Comment #8 from Jon Ciesla 2009-06-11 11:46:08 EDT ---
Better. I still get
mailman.src:46: E: hardcoded-library-path in /usr/lib/%{name}
A library path is hardcoded to one of the following paths: /lib, /usr/lib. It
should be replaced by something like /%{_lib} or %{_libdir}.
mailman.src:131: W: configure-without-libdir-spec
A configure script is run without specifying the libdir. configure options
must be augmented with something like --libdir=%{_libdir} whenever the script
supports it.
mailman.src: W: mixed-use-of-spaces-and-tabs (spaces: line 63, tab: line 132)
The specfile mixes use of spaces and tabs for indentation, which is a cosmetic
annoyance. Use either spaces or tabs for indentation, not both.
On the SRPM.
There's a *TON* of rpmlint errors on the RPM, mostly perm and gid issues. Some
are not. The only ones that concern me are these:
Lots of:
mailman.i386: E: file-in-usr-marked-as-conffile
/usr/lib/mailman/templates/zh_TW/userpass.txt
A file in /usr is marked as being a configuration file. Store your conf files
in /etc/ instead.
mailman.i386: W: dangerous-command-in-%pre rm
mailman.i386: W: dangerous-command-in-%post rm
mailman.i386: W: incoherent-subsys /etc/rc.d/init.d/mailman $prog
The filename of your lock file in /var/lock/subsys/ is incoherent with your
actual init script name. For example, if your script name is httpd, you have
to use 'httpd' as the filename in your subsys directory. It is also possible
that rpmlint gets this wrong, especially if the init script contains
nontrivial shell variables and/or assignments. These cases usually manifest
themselves when rpmlint reports that the subsys name starts a with '$'; in
these cases a warning instead of an error is reported and you should check the
script manually.
mailman.i386: W: incoherent-subsys /etc/rc.d/init.d/mailman $prog
The filename of your lock file in /var/lock/subsys/ is incoherent with your
actual init script name. For example, if your script name is httpd, you have
to use 'httpd' as the filename in your subsys directory. It is also possible
that rpmlint gets this wrong, especially if the init script contains
nontrivial shell variables and/or assignments. These cases usually manifest
themselves when rpmlint reports that the subsys name starts a with '$'; in
these cases a warning instead of an error is reported and you should check the
script manually.
mailman.i386: W: service-default-enabled /etc/rc.d/init.d/mailman
The service is enabled by default after "chkconfig --add"; for security
reasons, most services should not be. Use "-" as the default runlevel in the
init script's "chkconfig:" line and/or remove the "Default-Start:" LSB keyword
to fix this if appropriate for this service.
All need fixing.
Still present from #3.
--
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 426750] Review Request: ghc-utf8-string - Support reading and writing UTF8 Strings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=426750 --- Comment #11 from Zach Oglesby 2009-06-11 11:16:48 EDT --- Spec files have been fixed. I do need a sponsor as well. -- Configure 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 494219] Review Request: trac-accountmanager-plugin - Account manager plugin for trac
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=494219 --- Comment #2 from Paul Howarth 2009-06-11 11:09:33 EDT --- Review Guidelines = - rpmlint clean - package name is OK and is consistent with existing trac plugin packages in Fedora - spec file name consistent with package name - package meets guidelines - Beerware license is OK for Fedora - correct "Copyright only" License field used in spec - no detached license file to include - spec file written in English and is legible - source pulling instructions are missing "-r" option to set svn revision - sources from SRPM match sources from upstream VCS - package builds OK in mock for Fedora 10 (x86_64) - buildreqs fine - no locale data, libs, or devel files to worry about - not relocatable - directory ownership is fine - no duplicate entries in %files list - %defattr(...) and %clean present and correct - macro usage is consistent - code, not content - no large docs (in fact hardly any docs at all!) - docs don't affect runtime - not a GUI app, no .desktop file needed - %install section properly cleans buildroot - filenames all valid UTF-8 - no scriptlets needed - package appears to work as advertised - no subpackages used - no file dependencies Notes = Perhaps add comment that python-setuptools runtime dependency is needed for pkg_resources import? Please include contrib/sessionstore_convert.py as %doc (see http://www.mailinglistarchive.com/trac-us...@googlegroups.com/msg03696.html) Please add -r $svnrel to svn export instructions -- Configure 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 505155] Review Request: libcap-ng - An alternative posix capabilities 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=505155 --- Comment #9 from Steve Grubb 2009-06-11 10:58:54 EDT --- ok, new spec file uploaded to same place. Wrt to timestamp issue, the suggested fix doesn't work. The symlink was moved. The typos are fixed. -- Configure 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 460044] Review Request: pssh - Parallel SSH tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=460044 Patrick Laughton changed: What|Removed |Added CC||j...@jima.tk --- Comment #7 from Patrick Laughton 2009-06-11 10:50:12 EDT --- [x] Package does not own files or directories owned by other packages. I take it you overlooked /usr/bin/pscp owned by putty for five years prior? BZ#485969 BZ#472617 -- Configure 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 494219] Review Request: trac-accountmanager-plugin - Account manager plugin for trac
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=494219 Paul Howarth changed: What|Removed |Added Status|NEW |ASSIGNED CC||p...@city-fan.org AssignedTo|nob...@fedoraproject.org|p...@city-fan.org Flag||fedora-review? --- Comment #1 from Paul Howarth 2009-06-11 09:51:42 EDT --- Thanks for packaging this up. I've used this package to set up my own trac instance using mod_fcgid and it all seems to be working fine. I've documented the process here: http://www.city-fan.org/tips/TracWithFastCGIonFedora Anyway, I'll now return the favour by reviewing this package for you. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 495692] Review Request: tslib - Touchscreen Access 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=495692 Nicolas Chauvet (kwizart) changed: What|Removed |Added Status|NEW |CLOSED Resolution||NEXTRELEASE --- Comment #12 from Nicolas Chauvet (kwizart) 2009-06-11 09:51:32 EDT --- Thx 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 226117] Merge Review: mailman
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=226117
--- Comment #7 from Daniel Novotny 2009-06-11 09:31:33
EDT ---
hello,
I worked on the .spec file to adress these issues.
see
http://people.fedoraproject.org/~dnovotny/review/mailman.spec
these are my changes:
* hardcoded library path changed to %{_libdir}
* mixed use of spaces and tabs fixed
* added --libdir=%{_libdir} to configure
* fixed URL to tarball
* permission of source files changed to 0644 (no strange-permission warnings on
.src.rpm)
* got rid of "file listed twice" warnings: listing the files explicitly
the source files permission changes can't be seen in the .spec, I had to do it
in the cvs checkout
as for the permission and setgid warnings, I guess mailman needs them that way.
Also the incoherent-subsys warnings can be ignored because
"It is also possible that rpmlint gets this wrong, especially if the init
script contains nontrivial shell variables and/or assignments. These cases
usually manifest themselves when rpmlint reports that the subsys name starts a
with '$'"
which is exactly the case.
Do you think that's all, or do I have to fix something more?
--
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 505155] Review Request: libcap-ng - An alternative posix capabilities 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=505155 --- Comment #8 from Jussi Lehtola 2009-06-11 09:29:11 EDT --- Oh, the permission lines %defattr(-,root,root) should AFAIK be %defattr(-,root,root,-) as per http://fedoraproject.org/wiki/Packaging/Guidelines#File_Permissions -- Configure 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
