[Bug 497140] Review Request: php-IDNA_Convert - Internationalized domain name to UTF-8 converter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497140 --- Comment #3 from Jussi Lehtola jussi.leht...@iki.fi 2009-04-23 02:05:53 EDT --- (In reply to comment #2) (In reply to comment #1) - Does the package work with php4? Maybe a version could be added to Requires: php =5. This does work with php 4.3 and later per the internal comments. I'll be happy to add the requires with a version though since for current version of Fedora php5 is the only option. Okay, then it's no problem. MUST: The package must be licensed with a Fedora approved license and meet the Licensing Guidelines. NEEDSFIX - According to idna_convert.class.php license is LGPLv2+ not LGPLv2. Other php files do not contain any license specifications. The 'LICENCE' file included with the software distribution is LGPLv2, and given the lack of licensing for the rest of the .php files this leads me to believe that these are LGPLv2 and that thusly the package should be LGPLv2 instead of LGPLv2+. I am open to correction on this point, though, but this is my understanding. See https://fedoraproject.org/wiki/Licensing/FAQ#How_do_I_figure_out_what_version_of_the_GPL.2FLGPL_my_package_is_under.3F I think it has been mentioned also somewhere more visible in the guidelines, but I didn't find it in a couple of minutes. -- Configure 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 495577] Review Request: xsw - A simple slideshow producer and viewer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=495577 --- Comment #9 from Jussi Lehtola jussi.leht...@iki.fi 2009-04-23 02:10:40 EDT --- (In reply to comment #8) MUST: Optflags are used and time stamps preserved. NEEDSFIX - Time stamps are not preserved, e.g. /bin/sh /builddir/build/BUILD/xsw-0.2.3/install-sh -c -m 644 'intro/bloc.bmp' '/builddir/build/BUILDROOT/xsw-0.2.3-1.fc10.x86_64/usr/share/xsw/intro/bloc.bmp' The man page is ok. But you are right about the rest of the files. I will see what I can do to preserve the time stamps. You probably have to patch the makefile. I'll have a new look once you have fixed the time stamp problem. -- Configure 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 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Depends on||497274 -- Configure 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 #10 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-04-23 02:25:56 EDT --- (In reply to comment #9) perltetris.pl definitely does not work in rawhide (x86_64): [s...@velociraptor ~]$ /usr/bin/perltetris.pl GLib-GObject-WARNING **: cannot derive `GooCanvasWidgetAccessible' from non-fundamental parent type `GooCanvasItem' at /usr/bin/perltetris.pl line 761, DATA line 35. GLib-CRITICAL **: g_once_init_leave: assertion `initialization_value != 0' failed at /usr/bin/perltetris.pl line 761, DATA line 35. GLib-GObject-CRITICAL **: g_object_new: assertion `G_TYPE_IS_OBJECT (object_type)' failed at /usr/bin/perltetris.pl line 761, DATA line 35. CRITICAL **: atk_object_initialize: assertion `ATK_IS_OBJECT (accessible)' failed at /usr/bin/perltetris.pl line 761, DATA line 35. Well, - Confirmed that perltetris.pl works with F-10 goocanvas, does not work with F-11 goocanvas (I am using i586) - Even demo/widgets-demo in goocanvas 0.13 tarball does not work and this shows the same warnings, where widgets-demo in goocanvas 0.10 tarball (used on F-10) works. Filed against goocanvas (bug 497274) -- Configure 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 476524] Review Request: python-zope-testing - Zope Testing Framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=476524 --- Comment #2 from Conrad Meyer kon...@tylerc.org 2009-04-23 02:30:52 EDT --- Bumped to 3.7.3, added the two files. New URLs: http://konradm.fedorapeople.org/fedora/SPECS/python-zope-testing.spec http://konradm.fedorapeople.org/fedora/SRPMS/python-zope-testing-3.7.3-1.fc10.src.rpm Thanks for taking a look at this :). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 476528] Review Request: python-ZConfig - Structured Configuration 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=476528 Conrad Meyer kon...@tylerc.org changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Conrad Meyer kon...@tylerc.org 2009-04-23 02:34:13 EDT --- It needs python-zope-testing to run the tests, but because that hasn't been approved yet I had disabled them. I will enable them once python-zope-testing is included in Fedora. Thanks! New Package CVS Request === Package Name: python-ZConfig Short Description: Structured Configuration Library Owners: konradm Branches: F-11 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497251] Review Request: libstdc++-docs - Documentation in html for libstdc++
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497251 Jussi Lehtola jussi.leht...@iki.fi 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 497251] Review Request: libstdc++-docs - Documentation in html for libstdc++
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=497251
--- Comment #2 from Jussi Lehtola jussi.leht...@iki.fi 2009-04-23 02:38:29
EDT ---
- The lines
Obsoletes: libstdc++-docs = 4.3.0
Provides: libstdc++-docs = %{version}-%{release}
since they are redundant.
- Please remove the empty %check section.
- Use %{version} in the Source lines and in the %{build} phase.
Once you have added the man pages, I will perform 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 497251] Review Request: libstdc++-docs - Documentation in html for libstdc++
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497251 --- Comment #3 from Jakub Jelinek ja...@redhat.com 2009-04-23 02:43:20 EDT --- Isn't all this generated from gcc sources? What is needed to generate it? Just doxygen? If yes I'd say it would be better to ship this as subpackages of gcc, guarded by some spec macro that those who initially bootstrap gcc in a rpm based distro could turn off, so that gcc build doesn't require doxyen. -- Configure 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 477542] Review Request: mpdscribble - A mpd client which submits information about tracks being played to Last.fm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=477542 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225769] Merge Review: freeradius
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225769 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are 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 487148] Review Request: gearmand - A distributed job system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487148 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 495949] Review Request: kinterbasdb - A Python DB-API 2.0 compliant interface to Firebird
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=495949 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492950] Review Request: lv2-vocoder-plugins - Add a robotic effect to vocals
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=492950 Mattias Ellert mattias.ell...@fysast.uu.se changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|mattias.ell...@fysast.uu.se 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 492945] Review Request: lv2-swh-plugins - LV2 ports of LADSPA swh plugins
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=492945 Mattias Ellert mattias.ell...@fysast.uu.se changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|mattias.ell...@fysast.uu.se 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 492971] Review Request: lv2-zynadd-plugins - LV2 port of the ZynAddSubFX 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=492971 Mattias Ellert mattias.ell...@fysast.uu.se changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|mattias.ell...@fysast.uu.se 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 478929] Review Request: globus-usage - Globus Toolkit - Usage 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=478929 --- Comment #5 from Mattias Ellert mattias.ell...@fysast.uu.se 2009-04-23 03:04:11 EDT --- (In reply to comment #4) Everything seems good, except: * Please add some more information to the description because Usage Library is too general to use for a description. Please do that before you commit. I'll dig out some additional information from somewhere before committing. I also have a stack of packages that depend on each other. The ultimate goal is to get frinika approved: https://bugzilla.redhat.com/show_bug.cgi?id=492203 Could you help me with that whenever you have time? Thank you for your review. I have built the your three lv2-*-plugins packages yesterday and will start reviewing them soon. I'll grab them in bugzilla. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478929] Review Request: globus-usage - Globus Toolkit - Usage 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=478929 Mattias Ellert mattias.ell...@fysast.uu.se changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Mattias Ellert mattias.ell...@fysast.uu.se 2009-04-23 03:06:16 EDT --- New Package CVS Request === Package Name: globus-usage Short Description: Globus Toolkit - Usage Library Owners: ellert Branches: F-9 F-10 F-11 EL-4 EL-5 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497251] Review Request: libstdc++-docs - Documentation in html for libstdc++
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497251 --- Comment #4 from Jussi Lehtola jussi.leht...@iki.fi 2009-04-23 03:23:03 EDT --- (In reply to comment #3) Isn't all this generated from gcc sources? What is needed to generate it? Just doxygen? If yes I'd say it would be better to ship this as subpackages of gcc, guarded by some spec macro that those who initially bootstrap gcc in a rpm based distro could turn off, so that gcc build doesn't require doxyen. Yeah, this package should be part of the gcc package, since libstdc++ is also generated in the gcc 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 497283] New: Review Request: perl-Syntax-Highlight-Engine-Kate - Port to Perl of the syntax highlight engine of the Kate texteditor
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-Syntax-Highlight-Engine-Kate - Port to Perl of the syntax highlight engine of the Kate texteditor https://bugzilla.redhat.com/show_bug.cgi?id=497283 Summary: Review Request: perl-Syntax-Highlight-Engine-Kate - Port to Perl of the syntax highlight engine of the Kate texteditor Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mmasl...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://mmaslano.fedorapeople.org/review/perl-Syntax-Highlight-Engine-Kate.spec SRPM URL:http://mmaslano.fedorapeople.org/review/perl-Syntax-Highlight-Engine-Kate-0.04-1.fc11.src.rpm Description: Syntax::Highlight::Engine::Kate is a port to perl of the syntax highlight engine of the Kate text editor. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1316294 Problems: - strange description - can't rid of rpmlint warning. This file is here probably twice, but if I remove it, the test are complaining about missing file. perl-Syntax-Highlight-Engine-Kate.noarch: E: useless-provides perl(Syntax::Highlight::Engine::Kate::Alerts) -- Configure 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 497138] Review Request: perl-Text-VimColor - Syntax color text in HTML or XML using Vim
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497138 Iain Arnell iarn...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|iarn...@gmail.com Flag||fedora-review+ --- Comment #3 from Iain Arnell iarn...@gmail.com 2009-04-23 04:12:55 EDT --- Great - issues addressed - 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 497138] Review Request: perl-Text-VimColor - Syntax color text in HTML or XML using Vim
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497138 Marcela Maslanova mmasl...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Marcela Maslanova mmasl...@redhat.com 2009-04-23 04:42:56 EDT --- New Package CVS Request === Package Name: perl-Text-VimColor Short Description: Syntax color text in HTML or XML using Vim Owners: mmaslano Branches: F-11 InitialCC: perl-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 486302] Review Request: parrot - Parrot Virtual Machine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=486302 --- Comment #36 from Gerd Pokorra g...@zimt.uni-siegen.de 2009-04-23 04:46:31 EDT --- Thank you for the very helpful list of points to have fixed. All points are fixed now. I hope the comments in the spec file are sufficient. URLs: SRPM-file: ftp://ftp.uni-siegen.de/pub/parrot.rpms/fedora/SRPMS/parrot-1.0.0.src.rpm SPEC-file: ftp://ftp.uni-siegen.de/pub/parrot.rpms/fedora/specs/parrot.spec Scratch-Build-f10: http://koji.fedoraproject.org/koji/taskinfo?taskID=1316317 rpmlint-output: ftp://ftp.uni-siegen.de/pub/parrot.rpms/fedora/rpmlint -- Configure 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 476404] Review Request: bullet - 3D Collision Detection and Rigid Body Dynamics 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=476404 --- Comment #32 from Bruno Mahe br...@gnoll.org 2009-04-23 04:57:48 EDT --- I don't have enough time right now. So I give up and someone else can take over the package. Thanks a lot for your reviews and patience! -- Configure 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 496477] Review Request: perl-WWW-Pastebin-PastebinCom-Create - Paste to http://pastebin.com from Perl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=496477 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 496473] Review Request: perl-Mixin-Linewise - Write your linewise code for handles; this does the rest
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=496473 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 496475] Review Request: perl-Lexical-Persistence - Persistent lexical variable values for arbitrary calls
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=496475 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 496117] Review Request: xpyb - Python bindings for XCB
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=496117 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492971] Review Request: lv2-zynadd-plugins - LV2 port of the ZynAddSubFX 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=492971 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 496476] Review Request: perl-Clipboard - Copy and paste with any OS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=496476 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 496478] Review Request: perl-WWW-Pastebin-RafbNet-Create - Create new pastes on http://rafb.net/
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=496478 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492950] Review Request: lv2-vocoder-plugins - Add a robotic effect to vocals
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=492950 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497295] New: Review Request: main package name here - short summary here
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: main package name here - short summary here https://bugzilla.redhat.com/show_bug.cgi?id=497295 Summary: Review Request: main package name here - short summary here 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-Email-MIME-Encodings/perl-Email-MIME-Encodings.spec SRPM URL: http://people.parinux.org/~seyman/fedora/perl-Email-MIME-Encodings/perl-Email-MIME-Encodings-1.313-1.fc10.src.rpm Description: This module simply wraps MIME::Base64 and MIME::QuotedPrint so that you can throw the contents of a Content-Transfer-Encoding header at some text and have the right thing happen. -- Configure 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 492945] Review Request: lv2-swh-plugins - LV2 ports of LADSPA swh plugins
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=492945 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 495563] Review Request: febootstrap - Bootstrap a new Fedora system (like debootstrap)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=495563 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484547] Review Request: pyfacebook - Python wrapper for Facebook's API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=484547 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492197] Review Request: toot2 - Java models and frameworks for Audio/MIDI
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=492197 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ 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 #11 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-04-23 05:30:01 EDT --- Again: (In reply to comment #9) perltetris.pl definitely does not work in rawhide (x86_64): Would you test if perltetris.pl works if you have Assistive Technologies disabled? (although I don't know what at-spi exactuly does, disabling Assistive Technologies works). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456383] Review Request: python-plotkit - PlotKit Javascript Chart Plotting packed as TurboGears Widget
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=456383 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added CC||lemen...@gmail.com Flag||needinfo?(pavel.khardi...@g ||mail.com) --- Comment #3 from Peter Lemenkov lemen...@gmail.com 2009-04-23 05:33:59 EDT --- 404 while retrieving packages. Pavel, are you still interested in packaging this? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 361701] Review Request: konserve - System tray application that performs periodic backups
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=361701 Till Maas opensou...@till.name changed: What|Removed |Added CC||opensou...@till.name Summary|Package review: konserve - |Review Request: konserve - |System tray application |System tray application |that performs periodic |that performs periodic |backups |backups -- Configure 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 476404] Review Request: bullet - 3D Collision Detection and Rigid Body Dynamics 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=476404 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Status|ASSIGNED|CLOSED Blocks|177841(FE-NEEDSPONSOR) |201449(FE-DEADREVIEW) Resolution||NOTABUG 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 476404] Review Request: bullet - 3D Collision Detection and Rigid Body Dynamics 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=476404 --- Comment #33 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-04-23 05:42:44 EDT --- No worry. Now I once close this bug. If you can have some time for review request again, please feel free to open a new review request and mark this bug as a duplicate of the new one. Thank you anyway. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 437785] Review Request: php-ZendFramework-api-doc - Zend Framework API 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=437785 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497295] Review Request: perl-Email-MIME-Encodings - Unified interface to MIME encoding and decoding
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497295 Emmanuel Seyman emmanuel.sey...@club-internet.fr changed: What|Removed |Added CC||ita...@ispbrasil.com.br Summary|Review Request: main |Review Request: |package name here - short |perl-Email-MIME-Encodings - |summary here |Unified interface to MIME ||encoding and decoding --- Comment #1 from Emmanuel Seyman emmanuel.sey...@club-internet.fr 2009-04-23 05:18:41 EDT --- This package became one of Bugzilla's dependencies a few days ago. -- Configure 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 496492] Review Request: sing - Sends fully customized ICMP packets from command line
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=496492 Robert Scheck red...@linuxnetz.de changed: What|Removed |Added Flag||fedora-cvs? --- Comment #10 from Robert Scheck red...@linuxnetz.de 2009-04-23 06:10:25 EDT --- Christoph, I didn't question that I need to apply your suggestions and enhancements. I just wanted to make sure, that I don't need to create a new srpm for these minor changes. Sorry, if I was unclear. I'll have a look into the Packaging Tricks as well, before importing into CVS. But I am now requesting CVS branching, because it also takes some time - at least usually. New Package CVS Request === Package Name: sing Short Description: Sends fully customized ICMP packets from command line Owners: robert Branches: EL-4 EL-5 F-9 F-10 F-11 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497140] Review Request: php-IDNA_Convert - Internationalized domain name to UTF-8 converter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497140 --- Comment #4 from David Nalley da...@gnsa.us 2009-04-23 06:17:27 EDT --- Thanks for the clarification, please find corrected spec and srpm: Spec URL: http://ke4qqq.fedorapeople.org/php-IDNA_Convert.spec SRPM URL: http://ke4qqq.fedorapeople.org/php-IDNA_Convert-0.6.3-2.fc10.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483406] Review Request: perl-Text-CSV - Comma-separated values manipulator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483406 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 437801] Review Request: php-ZendFramework-manual-en - Documentation for the Zend Framework in English
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=437801 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 496492] Review Request: sing - Sends fully customized ICMP packets from command line
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=496492 --- Comment #11 from Christoph Wickert fed...@christoph-wickert.de 2009-04-23 06:23:40 EDT --- (In reply to comment #10) I'll have a look into the Packaging Tricks as well, before importing into CVS. No, _after_ import. Please import the package as is and then make the changes, so the community can follow them on the commits list. -- Configure 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 497310] New: Review Request: php-simplepie - Simple RSS Library in PHP
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-simplepie - Simple RSS Library in PHP https://bugzilla.redhat.com/show_bug.cgi?id=497310 Summary: Review Request: php-simplepie - Simple RSS Library in PHP Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: da...@gnsa.us QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://ke4qqq.fedorapeople.org/php-simplepie.spec SRPM URL: http://ke4qqq.fedorapeople.org/php-simplepie-1.1.3-1.fc10.src.rpm Description: SimplePie is a very fast and easy-to-use class, written in PHP, that puts the 'simple' back into 'really simple syndication'. Flexible enough to suit beginners and veterans alike, SimplePie is focused on speed, ease of use, compatibility and standards compliance. rpmlint output: [ke4...@nalleyt61 SPECS]$ rpmlint ./php-simplepie.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. [ke4...@nalleyt61 SPECS]$ rpmlint ../SRPMS/php-simplepie-1.1.3-1.fc10.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [ke4...@nalleyt61 SPECS]$ rpmlint ../RPMS/noarch/php-simplepie-1.1.3-1.fc10.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 495942] Review Request: guimup - A GTKmm based drag--drop oriented client for MPD
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=495942 --- Comment #2 from Andreas Osowski th0...@mkdir.name 2009-04-23 06:27:09 EDT --- Hello, I guess I fixed everything now. Regarding the Source0: The url supplied in the spec file is valid and working... only the link on the upstream homepage is bad. Spec URL: http://fedora.mkdir.name/packages/guimup-0.1.4/guimup.spec SRPM URL: http://fedora.mkdir.name/packages/guimup-0.1.4/guimup-0.1.4-2.fc10.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 495942] Review Request: guimup - A GTKmm based drag--drop oriented client for MPD
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=495942
--- Comment #3 from Christoph Wickert fed...@christoph-wickert.de 2009-04-23
06:37:03 EDT ---
(In reply to comment #2)
Hello, I guess I fixed everything now.
Thanks, I will look into this tonight.
The url supplied in the spec file is valid and working... only the link on the
upstream homepage is bad.
http://www.coonsden.com/dl0ads/%{srcname}-%{version}src.tar.gz resolves to
http://www.coonsden.com/dl0ads/Guimup-0.1.4src.tar.gz
$ wget http://www.coonsden.com/dl0ads/Guimup-0.1.4src.tar.gz
--2009-04-23 12:33:48-- http://www.coonsden.com/dl0ads/Guimup-0.1.4src.tar.gz
Auflösen des Hostnamen »www.coonsden.com« 83.137.194.17
Verbindungsaufbau zu www.coonsden.com|83.137.194.17|:80... verbunden.
HTTP Anforderung gesendet, warte auf Antwort... 403 Forbidden
2009-04-23 12:33:48 FEHLER 403: Forbidden.
$ wget http://www.coonsden.com/cgi-bin/ShotGun.cgi?ammo=Guimup-stable-src
--2009-04-23 12:35:49--
http://www.coonsden.com/cgi-bin/ShotGun.cgi?ammo=Guimup-stable-src
Auflösen des Hostnamen »www.coonsden.com« 83.137.194.17
Verbindungsaufbau zu www.coonsden.com|83.137.194.17|:80... verbunden.
HTTP Anforderung gesendet, warte auf Antwort... 200 OK
Länge: nicht spezifiziert [text/html]
»Last-modified«-Kopfzeile fehlt -- Zeitstempel abgeschaltet.
--2009-04-23 12:35:49--
http://www.coonsden.com/cgi-bin/ShotGun.cgi?ammo=Guimup-stable-src
Verbindungsaufbau zu www.coonsden.com|83.137.194.17|:80... verbunden.
HTTP Anforderung gesendet, warte auf Antwort... 200 OK
Länge: nicht spezifiziert [text/html]
In »ShotGun.cgi?ammo=Guimup-stable-src« speichern.
[ = ] 387 --.-K/s in 0s
2009-04-23 12:35:49 (21,9 MB/s) - »ShotGun.cgi?ammo=Guimup-stable-src«
gespeichert [387]
$ cat ShotGun.cgi\?ammo\=Guimup-stable-src
htmlheadtitleError/titlelink rel=stylesheet type=text/css
href=http://www.coonsden.com/scripts/template.css;/headbodyspan
class=titleTextError/spanblockquote
Please eitherbrbr1. Disable your download managerbr2. Left click the file
you are attempting to downloadbrbrorbrbr3. Enable your browser to send
HTTP Referrers.
/blockquote/body/html
--
Configure 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 497283] Review Request: perl-Syntax-Highlight-Engine-Kate - Port to Perl of the syntax highlight engine of the Kate texteditor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497283 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 495942] Review Request: guimup - A GTKmm based drag--drop oriented client for MPD
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=495942 --- Comment #4 from Andreas Osowski th0...@mkdir.name 2009-04-23 06:51:24 EDT --- Hey, I'm sorry for that. As I only tried accessing the download URL via Firefox, I didn't notice that they are checking for the user agent... if you execute wget http://www.coonsden.com/dl0ads/Guimup-0.1.4src.tar.gz; -U Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.8) Gecko/2009032713 Fedora/3.0.8-1.fc10 Firefox/3.0.8 It works fine. Anyway, I'll contact upstream on this issue. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481536] Review Request: enano - Enano CMS, a php-based modular content management system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481536 David Nalley da...@gnsa.us changed: What|Removed |Added CC||da...@gnsa.us AssignedTo|nob...@fedoraproject.org|da...@gnsa.us 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 483847] Review Request: gstreamer-rtsp - GStreamer RTSP server 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=483847 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added CC||lemen...@gmail.com --- Comment #1 from Peter Lemenkov lemen...@gmail.com 2009-04-23 06:52:07 EDT --- Failed to build on F-11, x86_64 http://koji.fedoraproject.org/koji/taskinfo?taskID=1316484 -- Configure 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 497283] Review Request: perl-Syntax-Highlight-Engine-Kate - Port to Perl of the syntax highlight engine of the Kate texteditor
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=497283
--- Comment #1 from Parag AN(पराग) panem...@gmail.com 2009-04-23 06:59:52
EDT ---
Review:
+ package builds in mock (rawhide i586).
koji Build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1316294
+ rpmlint output is for SRPM and for RPM.
perl-Syntax-Highlight-Engine-Kate.noarch: E: useless-provides
perl(Syntax::Highlight::Engine::Kate::Alerts)
== you should filter this
+ source files match upstream url
7f94ab9430a4c8504bb93c7974bb7493 Syntax-Highlight-Engine-Kate-0.04.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=1, Tests=528, 99 wallclock secs ( 0.21 usr 0.01 sys + 96.32 cusr 0.21
csys = 96.75 CPU)
Suggestions:
1) you should remove
OPTIMIZE=$RPM_OPT_FLAGS
And
find $RPM_BUILD_ROOT -type f -name '*.bs' -size 0 -exec rm -f {} \;
from SPEC as this is noarch package.
2) You must replace
%attr(0644,root,root)
/usr/lib/perl5/vendor_perl/5.10.0/Syntax/Highlight/Engine/Kate.pm
%attr(0644,root,root)
/usr/lib/perl5/vendor_perl/5.10.0/Syntax/Highlight/Engine/Kate.pod
%attr(0644,root,root)
/usr/lib/perl5/vendor_perl/5.10.0/Syntax/Highlight/Engine/Kate/*
to
%{perl_vendorlib}/*
3) When I filtered either one provides from
perl(Syntax::Highlight::Engine::Kate::Alerts) = 0.01
perl(Syntax::Highlight::Engine::Kate::Alerts) = 0.03
Got
Unknown arguments to TAP::Harness::new (ignore_exit) at
/usr/lib/perl5/5.10.0/Test/Harness.pm line 255
maybe Ralf can help to fix this 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 497295] Review Request: perl-Email-MIME-Encodings - Unified interface to MIME encoding and decoding
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497295 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added CC||panem...@gmail.com --- Comment #2 from Parag AN(पराग) panem...@gmail.com 2009-04-23 07:03:46 EDT --- This package already in Fedora and I see only new version of this package here. You should instead report bug against perl-Email-MIME-Encodings to update to latest upstream release. I did mock build for you. http://koji.fedoraproject.org/koji/taskinfo?taskID=1316543 -- Configure 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 495942] Review Request: guimup - A GTKmm based drag--drop oriented client for MPD
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=495942 --- Comment #5 from Christoph Wickert fed...@christoph-wickert.de 2009-04-23 07:02:58 EDT --- (In reply to comment #4) Hey, I'm sorry for that. No need to feel sorry. :) As I only tried accessing the download URL via Firefox Don't use firefox because of https://fedoraproject.org/wiki/Packaging/Guidelines#Timestamps we need a downloadable URL. -- Configure 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 497295] Review Request: perl-Email-MIME-Encodings - Unified interface to MIME encoding and decoding
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497295 Emmanuel Seyman emmanuel.sey...@club-internet.fr changed: What|Removed |Added Status|NEW |CLOSED Resolution||NOTABUG --- Comment #3 from Emmanuel Seyman emmanuel.sey...@club-internet.fr 2009-04-23 07:23:54 EDT --- I'm an idiot. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481536] Review Request: enano - Enano CMS, a php-based modular content management system
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=481536
David Nalley da...@gnsa.us changed:
What|Removed |Added
Flag||needinfo?(ngomp...@gmail.co
||m)
--- Comment #2 from David Nalley da...@gnsa.us 2009-04-23 07:35:57 EDT ---
Enano Review
FIX: rpmlint must be run on every package. The output should be posted in the
review.
[ke4...@nalleyt61 rpmbuild]$ rpmlint SPECS/enano.spec
SPECS/enano.spec: W: no-%build-section
0 packages and 1 specfiles checked; 0 errors, 1 warnings.
[ke4...@nalleyt61 rpmbuild]$ rpmlint
SRPMS/enano-1.1.6-2.20090301hge1ce6a91469b.fc10.src.rpm
enano.src: W: name-repeated-in-summary Enano
enano.src: W: no-%build-section
1 packages and 0 specfiles checked; 0 errors, 2 warnings.
[ke4...@nalleyt61 rpmbuild]$ rpmlint
RPMS/noarch/enano-1.1.6-2.20090301hge1ce6a91469b.fc10.noarch.rpm
enano.noarch: W: name-repeated-in-summary Enano
enano.noarch: W: incoherent-version-in-changelog 1.1.6-2
['1.1.6-2.20090301hge1ce6a91469b.fc10', '1.1.6-2.20090301hge1ce6a91469b']
enano.noarch: E: htaccess-file /usr/share/enano/cache/.htaccess
enano.noarch: W: hidden-file-or-dir /usr/share/enano/.htaccess.new
enano.noarch: E: htaccess-file /usr/share/enano/themes/.htaccess
enano.noarch: E: non-executable-script
/usr/share/enano/images/smilies/convert.sh 0644
enano.noarch: E: htaccess-file /usr/share/enano/install/.htaccess
enano.noarch: E: script-without-shebang
/usr/share/enano/install/includes/cli-core.php
enano.noarch: E: htaccess-file /usr/share/enano/includes/clientside/.htaccess
enano.noarch: E: htaccess-file /usr/share/enano/files/.htaccess
enano.noarch: E: htaccess-file /usr/share/enano/images/.htaccess
enano.noarch: E: htaccess-file /usr/share/enano/files/avatars/.htaccess
1 packages and 0 specfiles checked; 9 errors, 3 warnings.
So a few comments from rpmlint:
Since you a providing a conf file for /etc/httpd/ you should push all of the
.htaccess content to that and purge the .htaccess files.
Put a comment in the build section and rpmlint will quit complaining about
that.
The rest should be relatively obvious.
OK: The package must be named according to the Package Naming Guidelines .
Though you may find it easier to use MMDDhg or something similar and note
the change number in a comment.
OK: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
OK: The package must meet the Packaging Guidelines .
OK: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
FIX: The License field in the package spec file must match the actual license.
-
code and readme indicate this is GPLv2+ not GPLv2
OK: If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package must be included in %doc.
OK: The spec file must be written in American English.
OK: The spec file for the package MUST be legible.
FIX: The sources used to build the package must match the upstream source, as
provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
Using method in spec to grab source doesn't agree with what's packaged in the
SRPM
[ke4...@nalleyt61 tmp]$ md5sum ~/rpmbuild/SOURCES/enano-1.1.6-20090301.tar.gz
52cdd1476c9d63238e487d05d437b724
/home/ke4qqq/rpmbuild/SOURCES/enano-1.1.6-20090301.tar.gz
[ke4...@nalleyt61 tmp]$ md5sum ./enano-1.1.6-20090301.tar.gz
6450ad3e971d54680a07f69eb15ea7c6 ./enano-1.1.6-20090301.tar.gz
OK: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
NA: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.
OK: All build dependencies must be listed in BuildRequires, except for any that
are listed in the exceptions section of the Packaging Guidelines ; inclusion of
those as BuildRequires is optional. Apply common sense.
NA: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
NA: Every binary RPM package (or subpackage) which stores shared library files
(not just symlinks) in any of the dynamic linker's default paths, must call
ldconfig in %post and %postun.
NA: If the
[Bug 496824] Review Request: ipa-gothic-fonts - Japanese Gothic-typeface OpenType font by IPA
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=496824 Akira TAGOH ta...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #6 from Akira TAGOH ta...@redhat.com 2009-04-23 07:38:37 EDT --- available on koji now. thanks. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 496826] Review Request: ipa-mincho-fonts - Japanese Mincho-typeface OpenType font by IPA
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=496826 Akira TAGOH ta...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #5 from Akira TAGOH ta...@redhat.com 2009-04-23 07:39:28 EDT --- available on koji now. thanks. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 496825] Review Request: ipa-pgothic-fonts - Japanese Proportional Gothic-typeface OpenType font by IPA
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=496825 Akira TAGOH ta...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #6 from Akira TAGOH ta...@redhat.com 2009-04-23 07:39:01 EDT --- available on koji now. thanks. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 496827] Review Request: ipa-pmincho-fonts - Japanese Proportional Mincho-typeface OpenType font by IPA
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=496827 Akira TAGOH ta...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #5 from Akira TAGOH ta...@redhat.com 2009-04-23 07:39:54 EDT --- available on koji now. thanks. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 437785] Review Request: php-ZendFramework-api-doc - Zend Framework API 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=437785 --- Comment #4 from Alexander Kahl ak...@iconmobile.com 2009-04-23 07:28:15 EDT --- Oh.. seems like I have missed the approval.. thank you for pinging David, want me to update to upstream's latest? -- Configure 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 495942] Review Request: guimup - A GTKmm based drag--drop oriented client for MPD
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=495942 --- Comment #6 from Andreas Osowski th0...@mkdir.name 2009-04-23 08:11:24 EDT --- I just talked to upstream, he gave me links to the source on SF. Spec URL: http://fedora.mkdir.name/packages/guimup-0.1.4/guimup.spec SRPM URL: http://fedora.mkdir.name/packages/guimup-0.1.4/guimup-0.1.4-3.fc10.src.rpm * Thu Apr 23 2009 Andreas Osowski th0...@mkdir.name 0.1.4-3 - Updated Source0 -- Configure 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 494726] Review Request: Gnote - Note Taking 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=494726 --- Comment #24 from Fedora Update System upda...@fedoraproject.org 2009-04-23 08:34:24 EDT --- gnote-0.2.0-1.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/gnote-0.2.0-1.fc10 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492395] Review Request: php-geshi - Generic syntax highlighter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=492395 David Nalley da...@gnsa.us changed: What|Removed |Added Status|NEW |ASSIGNED CC||da...@gnsa.us AssignedTo|nob...@fedoraproject.org|da...@gnsa.us 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 497283] Review Request: perl-Syntax-Highlight-Engine-Kate - Port to Perl of the syntax highlight engine of the Kate texteditor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497283 --- Comment #2 from Marcela Maslanova mmasl...@redhat.com 2009-04-23 08:41:14 EDT --- The spec was recreated because it is not noarch. The problem in test, which I see is: PERL_DL_NONLAZY=1 /usr/bin/perl -MExtUtils::Command::MM -e test_harness(0, 'blib/lib', 'blib/arch') t/*.t t/Kate.t .. 448/? Use of uninitialized value $ch in concatenation (.) or string at /home/marca/rpmbuild/BUILD/Syntax-Highlight-Engine-Kate-0.04/blib/lib/Syntax/Highlight/Engine/Kate/Template.pm line 534. Use of uninitialized value $ch in string ne at /home/marca/rpmbuild/BUILD/Syntax-Highlight-Engine-Kate-0.04/blib/lib/Syntax/Highlight/Engine/Kate/Template.pm line 535. Use of uninitialized value $ch in concatenation (.) or string at /home/marca/rpmbuild/BUILD/Syntax-Highlight-Engine-Kate-0.04/blib/lib/Syntax/Highlight/Engine/Kate/Template.pm line 534. This looks more like problem of wrong test, which didn't count with empty variable. I can't still get rid off double perl(Syntax::Highlight::Engine::Kate::Alerts). -- Configure 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 489564] Review Request: Blueman - Bluetooth 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=489564 Lubomir Rintel lkund...@v3.sk changed: What|Removed |Added CC||lkund...@v3.sk AssignedTo|nob...@fedoraproject.org|lkund...@v3.sk Flag|needinfo?(extras...@fedorap |fedora-review? |roject.org) | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497338] New: Review Request: python-cclib - A library for processing results of computational chemistry packages
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: python-cclib - A library for processing results of computational chemistry packages https://bugzilla.redhat.com/show_bug.cgi?id=497338 Summary: Review Request: python-cclib - A library for processing results of computational chemistry packages Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: jussi.leht...@iki.fi QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://theory.physics.helsinki.fi/~jzlehtol/rpms/python-cclib.spec SRPM URL: http://theory.physics.helsinki.fi/~jzlehtol/rpms/python-cclib-0.91-1.fc10.src.rpm Upstream URL: http://cclib.sourceforge.net/ Description: cclib is an open source library, written in Python, for parsing and interpreting the results of computational chemistry packages. The current version, cclib 0.9, parses output files from ADF, GAMESS (US), GAMESS-UK, Gaussian, Jaguar, Molpro, ORCA and PC GAMESS. rpmlint output is clean. -- Configure 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 497339] Review Request: qmforge - Analysis tools for quantum mechanical calculations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497339 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added Depends on||497338 -- Configure 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 497338] Review Request: python-cclib - A library for processing results of computational chemistry packages
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497338 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added Blocks||497339 -- Configure 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 497339] Review Request: qmforge - Analysis tools for quantum mechanical calculations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497339 --- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi 2009-04-23 08:56:33 EDT --- Even though this package contains Python modules, IMHO the name is correct since the primary use of the package is to act as a GUI. Which reminds me, I have to make a desktop 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 497339] New: Review Request: qmforge - Analysis tools for quantum mechanical calculations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: qmforge - Analysis tools for quantum mechanical calculations https://bugzilla.redhat.com/show_bug.cgi?id=497339 Summary: Review Request: qmforge - Analysis tools for quantum mechanical calculations Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: jussi.leht...@iki.fi QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://theory.physics.helsinki.fi/~jzlehtol/rpms/qmforge.spec SRPM URL: http://theory.physics.helsinki.fi/~jzlehtol/rpms/qmforge-2.1-1.fc10.src.rpm Upstream URL: http://qmforge.sourceforge.net/ Description: QMForge reads output from various quantum mechanical calculations and provides tools for analyzing these results. Specifically, population (Mulliken and C-squared), fragment, and charge decomposition analyses are available. rpmlint output: qmforge.noarch: E: explicit-lib-dependency python-cclib 2 packages and 0 specfiles checked; 1 errors, 0 warnings. This is not a problem, it's just caused by the name of cclib. -- Configure 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 492395] Review Request: php-geshi - Generic syntax highlighter
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=492395
David Nalley da...@gnsa.us changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #3 from David Nalley da...@gnsa.us 2009-04-23 09:05:43 EDT ---
OK: rpmlint must be run on every package. The output should be posted in the
review.
[ke4...@nalleyt61 SPECS]$ rpmlint ./php-geshi.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[ke4...@nalleyt61 SPECS]$ rpmlint ../SRPMS/php-geshi-1.0.8.3-1.fc10.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[ke4...@nalleyt61 SPECS]$ rpmlint
../RPMS/noarch/php-geshi-1.0.8.3-1.fc10.noarch.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
OK: The package must be named according to the Package Naming Guidelines .
OK: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
OK: The package must meet the Packaging Guidelines .
OK: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
OK: The License field in the package spec file must match the actual license.
OK: If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package must be included in %doc.[4]
OK: The spec file must be written in American English. [5]
OK: The spec file for the package MUST be legible. [6]
OK: The sources used to build the package must match the upstream source, as
provided in the spec URL.
[ke4...@nalleyt61 SOURCES]$ md5sum GeSHi-1.0.8.3.tar.gz
fa7ad8e9f5d2573cfca58dd3d5945712 GeSHi-1.0.8.3.tar.gz
[ke4...@nalleyt61 SOURCES]$ md5sum GeSHi-1.0.8.3.tar.gz.1
fa7ad8e9f5d2573cfca58dd3d5945712 GeSHi-1.0.8.3.tar.gz.1
OK: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
Works on at least x86_64 - not that there is a real build here.
NA: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.
OK: All build dependencies must be listed in BuildRequires, except for any that
are listed in the exceptions section of the Packaging Guidelines ; inclusion of
those as BuildRequires is optional. Apply common sense.
NA: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.[9]
NA: Every binary RPM package (or subpackage) which stores shared library files
(not just symlinks) in any of the dynamic linker's default paths, must call
ldconfig in %post and %postun.
NA: If the package is designed to be relocatable, the packager must state this
fact in the request for review, along with the rationalization for relocation
of that specific package. Without this, use of Prefix: /usr is considered a
blocker.
OK: A package must own all directories that it creates. If it does not create a
directory that it uses, then it should require a package which does create that
directory.
OK: A Fedora package must not list a file more than once in the spec file's
%files listings.
OK: Permissions on files must be set properly. Executables should be set with
executable permissions, for example. Every %files section must include a
%defattr(...) line.
OK: Each package must have a %clean section, which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT).
OK: Each package must consistently use macros.
OK: The package must contain code, or permissable content.
NA: Large documentation files must go in a -doc subpackage.
OK: If a package includes something as %doc, it must not affect the runtime of
the application. To summarize: If it is in %doc, the program must run properly
if it is not present.
NA: Header files must be in a -devel package.
NA: Static libraries must be in a -static package.
NA: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig' (for
directory ownership and usability).
NA: If a package contains library files with a suffix (e.g. libfoo.so.1.1),
then library files that end in .so (without suffix) must go in a -devel
package.
NA: In the vast majority of cases, devel packages must require the base package
using a fully versioned dependency: Requires: %{name} = %{version}-%{release}
NA: Packages must NOT contain any .la libtool archives, these must be removed
in the spec if they are built.
NA: Packages containing GUI applications must include a %{name}.desktop
[Bug 483847] Review Request: gstreamer-rtsp - GStreamer RTSP server 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=483847 --- Comment #2 from Bastien Nocera bnoc...@redhat.com 2009-04-23 09:09:52 EDT --- Python requires missing it seems. -- Configure 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 497339] Review Request: qmforge - Analysis tools for quantum mechanical calculations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497339 --- Comment #2 from Jussi Lehtola jussi.leht...@iki.fi 2009-04-23 09:13:55 EDT --- Although, there's no icon available for the application, and I'm not going to make one [not with my graphical skills!]. I'm going to invoke the Review guidelines: If you feel that your packaged GUI application does not need a .desktop file, you must put a comment in the spec file with your explanation. -- Configure 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 483847] Review Request: gstreamer-rtsp - GStreamer RTSP server 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=483847 --- Comment #3 from Bastien Nocera bnoc...@redhat.com 2009-04-23 09:20:37 EDT --- Fixed in: http://people.fedoraproject.org/~hadess/gstreamer-rtsp/gstreamer-rtsp-0.10.1.0-2.fc10.src.rpm http://people.fedoraproject.org/~hadess/gstreamer-rtsp/gstreamer-rtsp.spec Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1316764 -- Configure 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 492395] Review Request: php-geshi - Generic syntax highlighter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=492395 Xavier Bachelot xav...@bachelot.org changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Xavier Bachelot xav...@bachelot.org 2009-04-23 09:23:25 EDT --- Thanks for the review David. New Package CVS Request === Package Name: php-geshi Short Description: Generic syntax highlighter Owners: xavierb Branches: F-9 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 476524] Review Request: python-zope-testing - Zope Testing Framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=476524 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from Fabian Affolter fab...@bernewireless.net 2009-04-23 09:30:47 EDT --- Thanks, package APPROVED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483847] Review Request: gstreamer-rtsp - GStreamer RTSP server 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=483847 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|lemen...@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 487097] Review Request: ReviewBoard - web based code review tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487097 --- Comment #5 from Fabian Affolter fab...@bernewireless.net 2009-04-23 09:35:15 EDT --- AUTHORS, COPYING, NEWS, and README should be placed in %doc. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487098] Review Request: Djblets - A collection of useful classes and functions for Django
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487098 --- Comment #4 from Fabian Affolter fab...@bernewireless.net 2009-04-23 09:37:47 EDT --- NEWS should go into %doc. 0.5beta1 was released. -- Configure 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 495399] Review Request: NotCourier-Sans-sfd-fonts - NotCourier Sans is a re-interpretation of Nimbus Mono.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=495399 --- Comment #1 from Ankur Sinha sanjay.an...@gmail.com 2009-04-23 09:51:13 EDT --- hi, sorry for this.. pingy :) -- Configure 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 483847] Review Request: gstreamer-rtsp - GStreamer RTSP server 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=483847
--- Comment #4 from Peter Lemenkov lemen...@gmail.com 2009-04-23 10:03:25
EDT ---
REVIEW:
- rpmlint is not silent:
[pe...@host-12-116 Desktop]$ rpmlint gstreamer-rtsp-*
gstreamer-rtsp.i586: E: library-without-ldconfig-postin
/usr/lib/libgstrtspserver-0.10.so.0.0.0
gstreamer-rtsp.i586: E: library-without-ldconfig-postun
/usr/lib/libgstrtspserver-0.10.so.0.0.0
gstreamer-rtsp-devel.i586: W: no-documentation
gstreamer-rtsp-python.i586: W: no-documentation
gstreamer-rtsp-vala.i586: W: no-documentation
5 packages and 0 specfiles checked; 2 errors, 3 warnings.
[pe...@host-12-116 Desktop]$
Regarding the first messages - please, add
%post -p /sbin/ldconfig
%postun -p /sbin/ldconfig
Other messages may be ignored.
+ The package is named according to the Package Naming Guidelines.
+ The spec file name matches the base package %{name}, in the format
%{name}.spec.
+ The package meets the Packaging Guidelines.
+ The package is licensed with a Fedora approved license and meets the
Licensing Guidelines .
+ The License field in the package spec file must matches the actual license.
+ The file, containing the text of the license(s) for the package, is included
in %doc.
+ The spec file is written in American English.
+ The spec file for the package is legible.
+ The sources used to build the package matches the upstream source:
[pe...@sulaco SOURCES]$ md5sum gst-rtsp-0.10.1.0.tar.bz2*
67133d588a5153d80a4f8828aad2ec83 gst-rtsp-0.10.1.0.tar.bz2
67133d588a5153d80a4f8828aad2ec83 gst-rtsp-0.10.1.0.tar.bz2.1
[pe...@sulaco SOURCES]$
+ The package successfully compiles and builds into binary rpms on at least one
primary architecture (see koji log above).
+ All build dependencies are listed in BuildRequires.
- Every binary RPM package (or subpackage) which stores shared library files
(not just symlinks) in any of the dynamic linker's default paths, must call
ldconfig in %post and %postun. See rpmlint messages above.
- A package must own all directories that it creates. Unfortunately,
%{_datadir}/gst-rtsp in devel-subpackage is not owned. Please, add it as %dir.
+ The package does not list a file more than once in the spec file's %files
listings.
+ Permissions on files are set properly.
+ The package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ The package consistently uses macros.
+ The package contains code, or permissable content.
+ No large documentation files.
+ Everything, the package includes as %doc, does not affect the runtime of the
application.
+ Header files are in a -devel package.
+ No static libraries.
- The devel sub-package (containing pkgconfig(.pc) file) MUST have 'Requires:
pkgconfig' (for directory ownership and usability).
+ The library files that ends in .so (without suffix) are in a -devel package.
+ The devel sub-package requires the base package using a fully versioned
dependency: Requires: %{name} = %{version}-%{release}
+ The package does NOT contain any .la libtool archives.
+ Not a GUI application.
+ The package does not own files or directories already owned by other
packages.
+ At the beginning of %install, the package runs rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ All filenames in rpm packages must be valid UTF-8.
- Missing Requires:
* gstreamer-devel for devel sub-package (%{_includedir}/gstreamer-0.10/gst/)
* pkgconfig for devel (see note above, %{_libdir}/pkgconfig/)
* gstreamer-python for python syb-package
(%{_libdir}/python*/site-packages/gst-0.10/gst)
* vala for vala sub-package (%{_datadir}/vala/vapi/)
--
Configure 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 497338] Review Request: python-cclib - A library for processing results of computational chemistry packages
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497338 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||lemen...@gmail.com AssignedTo|nob...@fedoraproject.org|lemen...@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 492945] Review Request: lv2-swh-plugins - LV2 ports of LADSPA swh plugins
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=492945 --- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2009-04-23 10:10:32 EDT --- Fedora review lv2-swh-plugins-1.0.15-1.fc10.src.rpm 2008-04-23 rpmlint output: 3 packages and 1 specfiles checked; 0 errors, 0 warnings. * OK ! needs attention ? needs clarification * Package is named according to guidelines. * Spec file is named after the package. * The package is licensed with a Fedora approved license (GPLv2+). * There are parts of the sources in the tarfile under other licenses, but as far as I can tell all files in the binary RPM have at least one source file under GPLv2+, so GPLv2+ is correct for the full binary RPM. * The sources has no (relevant) license file, and the binary package doesn't either. (There is a util/gsm/COPYRIGHT file, but the code in the util/gsm directory is not compiled during the build.) * The specfile is written in legible English The answer to the comment FIXME: Fix weird permissions. How can we handle this in %%prep? is to use install -m 644 instead of install when installing the .ttl files. But then the command must be split in two because the .so files installed in the same command should not have 644 permission, but the default 755. It might be easier to do it the way it is currently done, and ask upstream to fix it for a later version. * Sources matches upstream and is the latest version: c78f42c36d7bf2fb5b17f795ef9636d1 swh-lv2-1.0.15.tar.gz c78f42c36d7bf2fb5b17f795ef9636d1 SRPM/swh-lv2-1.0.15.tar.gz * Package compiles in mock (Fedora 10) ! Some of the plugins have undefined symbols. This is OK if the undefined symbols are available in the application that loads the plugins, but it might be safer to link the plugins to the libraries providing the missing symbols if this can not always be guaranteed. 62 of the 91 plugins have missing symbols from libm (sin, cos, tan, exp, sqrt, log, pow, etc.) In addition the following undefined symbols are present: undefined symbol: shm_open (/usr/lib64/lv2/analogue_osc-swh.lv2/plugin-Linux.so) undefined symbol: shm_open (/usr/lib64/lv2/fm_osc-swh.lv2/plugin-Linux.so) undefined symbol: shm_open (/usr/lib64/lv2/hermes_filter-swh.lv2/plugin-Linux.so) undefined symbol: fftwf_plan_r2r_1d (/usr/lib64/lv2/mbeq-swh.lv2/plugin-Linux.so) undefined symbol: fftwf_execute (/usr/lib64/lv2/mbeq-swh.lv2/plugin-Linux.so) undefined symbol: fftwf_destroy_plan (/usr/lib64/lv2/mbeq-swh.lv2/plugin-Linux.so) undefined symbol: pitch_scale (/usr/lib64/lv2/pitch_scale-swh.lv2/plugin-Linux.so) At least the last one is potentially tricky since the header file that provides this symbol is within the source tarball, but the corresponding implementation file is not compiled: util/pitchscale.h and util/pitchscale.c. Or is there a different implementation of this function somewhere that is used to resolve the missing symbol when the plugin is loaded. The other undefined symbols in the list above are from librt and libfftw3f - are the applications loading the plugins always linked to these libraries? The plugins in the other two packages have no undefined symbols. * BuildRequires are sane. * No shared libraries in the default library path. * Owns or depends on packages that own it directories * No duplicate files * Permissions are sane and %files has %defattr * %clean clears buildroot * macros are used consistently * Contains code * %doc is not essential at runtime * Package does not own other's directories * %install clears buildroot * Installed filenames are valid UTF-8 ? The plugins from this package are all labelled with a -Linux suffix, while this is not the case for the plugins from the other two plugin packages. What is the reason for not being consistent among the different packages? -- Configure 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 492971] Review Request: lv2-zynadd-plugins - LV2 port of the ZynAddSubFX 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=492971
--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2009-04-23
10:10:41 EDT ---
Fedora review lv2-zynadd-plugins-1-2.fc10.src.rpm 2009-04-23
* OK
! Needs attention
? Needs clarification
rpmlint output:
lv2-zynadd-plugins.spec: W: mixed-use-of-spaces-and-tabs (spaces: line 12, tab:
line 3)
lv2-zynadd-plugins.src: W: mixed-use-of-spaces-and-tabs (spaces: line 12, tab:
line 3)
3 packages and 1 specfiles checked; 0 errors, 2 warnings.
! rpmlint warning is fixable (replace the space on line 12 with a tab)
* Package is named according to guidelines
* Specfile is named after the package
* The package is licensed as GPLv2 which is a Fedora approved license
* The license matches the license statements in the source
* The license file in the sources (gpl.txt) is packaged as %doc
* The spec file is written in legible English
* The source matches upstream and is the latest version
ead0a8db5440e480c8a23441195a76df zyn-1.tar.bz2
ead0a8db5440e480c8a23441195a76df SRPM/zyn-1.tar.bz2
* Package builds in mock (Fedora 10)
? BuildRequires on jack-audio-connection-kit-devel - is it needed?
* No shared libraries in the default library path
! Neither the package nor any of its Requires own %{_libdir}/lv2
(the other two plugin packages have Requires: lv2core)
* No duplicate files
* Permissions are sane and %files has %defattr
* %clean clear buildroot
* Macros are used consistently
* Contains code
* %doc is not essential at runtime
* Package does not own other's directories
* %install clears buildroot
* Installed filenames are valid UTF-8
--
Configure 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 492950] Review Request: lv2-vocoder-plugins - Add a robotic effect to vocals
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=492950 Mattias Ellert mattias.ell...@fysast.uu.se changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2009-04-23 10:10:35 EDT --- Fedora review lv2-vocoder-plugins-1-1.fc10.src.rpm 2009-04-23 rpmlint output: 3 packages and 1 specfiles checked; 0 errors, 0 warnings. * Package is named according to guidelines * The spec file is named after the package * The package is licensed as GPLv2, which is a Fedora approved license * The license matches the stated license in the sources * The license file in the sources (gpl.txt) is installed as %doc * The specfile is written in legible English * Sources matches upstream and is the latest version 8c0096978e9c58a252f247e1f7f7ed63 lv2vocoder-1.tar.bz2 8c0096978e9c58a252f247e1f7f7ed63 SRPM/lv2vocoder-1.tar.bz2 * Package compiles in mock (Fedora 10) * BuildRequires are sane * No shared libraries in default library path * Package owns or depend on packages that own its directories * No duplicate files * Permission are sane and %files has %defattr * %clean clears buildroot * Contains code * Uses macros consistently * %doc is not essential at runtime * Package does not own other's directories * %install clears buildroot * installed filenames are valid UTF-8 Package approved. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497310] Review Request: php-simplepie - Simple RSS Library in PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497310 Jussi Lehtola jussi.leht...@iki.fi 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 483847] Review Request: gstreamer-rtsp - GStreamer RTSP server 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=483847 --- Comment #5 from Bastien Nocera bnoc...@redhat.com 2009-04-23 10:29:23 EDT --- Fixed in: http://people.fedoraproject.org/~hadess/gstreamer-rtsp/gstreamer-rtsp-0.10.1.0-3.fc10.src.rpm http://people.fedoraproject.org/~hadess/gstreamer-rtsp/gstreamer-rtsp.spec Let me know if I missed anything. -- Configure 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 497310] Review Request: php-simplepie - Simple RSS Library in PHP
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=497310
--- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi 2009-04-23 10:31:42
EDT ---
- No need to Requires: php since it is already pulled in by php-IDNA_Convert.
- Why do you rm -rf compatibility_test?
rpmlint output is clean.
MUST: The package does not yet exist in Fedora. The Review Request is not a
duplicate. OK
MUST: The spec file for the package is legible and macros are used
consistently. NEEDSFIX
- Don't mix $RPM_BUILD_ROOT and %{buildroot}. Choose one and stick with it.
- You could clean up the install phase by replacing
%{__mkdir} -p %{buildroot}/%{_datadir}/php/%{name}
cp -a create.php %{buildroot}/%{_datadir}/php/%{name}/
cp -a simplepie.inc %{buildroot}/%{_datadir}/php/%{name}/
with
install -D -p -m 644 create.php
%{buildroot}/%{_datadir}/php/%{name}/create.php
install -D -p -m 644 simplepie.php
%{buildroot}/%{_datadir}/php/%{name}/simplepie.php
MUST: The package must be named according to the Package Naming Guidelines. OK
- No PECL, PEAR or CHANNEL.
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
- simplepie.inc contains BSD headers.
MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. OK
MUST: Optflags are used and time stamps preserved. OK
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: Permissions on files must be set properly. OK
MUST: Clean section exists. OK
MUST: Large documentation files must go in a -doc subpackage. OK
MUST: All relevant items are included in %doc. Items in %doc do not affect
runtime of application. OK
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 list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497338] Review Request: python-cclib - A library for processing results of computational chemistry packages
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=497338
--- Comment #1 from Peter Lemenkov lemen...@gmail.com 2009-04-23 10:41:27
EDT ---
Missing BR: numpy.
OK, let's assume that you added this (koji scratchbuild with numpy -
http://koji.fedoraproject.org/koji/taskinfo?taskID=1316865 )
REVIEW:
+ rpmlint is silent.
+ The package is named according to the Package Naming Guidelines.
+ The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. [2] .
+ The package meets the Packaging Guidelines .
+ The package is licensed with a Fedora approved license and meets the
Licensing Guidelines .
+ The License field in the package spec file matches the actual license.
+ The file, containing the text of the license(s) for the package, is included
in %doc.
+ The spec file is written in American English.
+ The spec file for the package is legible.
+ The sources used to build the package matches the upstream source:
[pe...@sulaco SOURCES]$ md5sum cclib-0.91.tar.gz*
64e2cf1ab50cf8cb7f2882c88025 cclib-0.91.tar.gz
64e2cf1ab50cf8cb7f2882c88025 cclib-0.91.tar.gz.1
[pe...@sulaco SOURCES]$
+/- The package successfully compiles and builds into binary rpms on at least
one primary architecture. But only after adding numpy as BR:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1316865
- All build dependencies must be listed in BuildRequires, so, please add numpy.
Definitely there are more missing runtime python dependencies - here is an
output of find -name *py -exec grep import {} \; | sort|uniq:
http://peter.fedorapeople.org/cclib_imports.txt
Please, add all necessary runtime python dependencies.
+ The package owns all directories that it creates.
+ The package does not list a file more than once in the spec file's %files
listings.
+ Permissions on files are set properly.
+ The package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ The package consistently uses macros.
+ The package must contains code, or permissible content.
+ No large documentation files
+ Everytnong, the package includes as %doc, does not affect the runtime of the
application.
+ Not a GUI application.
+ The package does not own files or directories already owned by other
packages.
+ At the beginning of %install, the package runs rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ All filenames in rpm packages must be valid UTF-8.
So, please add the only missign BuildRequire and all missing Requires.
Also, I'm not sure, that we should ship *.pyc-files in %doc, however this is
not a blocker.
--
Configure 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 497310] Review Request: php-simplepie - Simple RSS Library in PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497310 --- Comment #2 from David Nalley da...@gnsa.us 2009-04-23 10:42:58 EDT --- compatibility_tests is a php page designed to display via a webserver whether or not your machine has the prerequisites (ie php 4.3 or greater) to use SimplePie. If you think it more appropriate, I can move it to %doc I'll remove the extra requires and push a new spec/srpm up shortly. -- Configure 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 476524] Review Request: python-zope-testing - Zope Testing Framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=476524 Conrad Meyer kon...@tylerc.org changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Conrad Meyer kon...@tylerc.org 2009-04-23 10:47:10 EDT --- New Package CVS Request === Package Name: python-zope-testing Short Description: Zope Testing Framework Owners: konradm Branches: F-11 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 497310] Review Request: php-simplepie - Simple RSS Library in PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497310 --- Comment #3 from David Nalley da...@gnsa.us 2009-04-23 10:47:57 EDT --- Spec URL: http://ke4qqq.fedorapeople.org/php-simplepie.spec SRPM URL: http://ke4qqq.fedorapeople.org/php-simplepie-1.1.3-2.fc10.src.rpm Extra requires stripped new spec and srpm above. -- Configure 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 483847] Review Request: gstreamer-rtsp - GStreamer RTSP server 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=483847 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #6 from Peter Lemenkov lemen...@gmail.com 2009-04-23 10:48:58 EDT --- I don't see any other issues here, so 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 497310] Review Request: php-simplepie - Simple RSS Library in PHP
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=497310
--- Comment #4 from Jussi Lehtola jussi.leht...@iki.fi 2009-04-23 10:57:34
EDT ---
(In reply to comment #2)
compatibility_tests is a php page designed to display via a webserver whether
or not your machine has the prerequisites (ie php 4.3 or greater) to use
SimplePie.
If you think it more appropriate, I can move it to %doc
OK, no need to ship it then. But no need to delete it either!
I'll remove the extra requires and push a new spec/srpm up shortly.
OK.
- Use %{version} in Source0 line
- You are still mixing $RPM_BUILD_ROOT and %{buildroot}. This is a blocker.
- Don't use a macro for mkdir, it's silly and makes the spec file harder to
read.
- Move the chmod's to the setup phase, since they're not related to the install
phase
- Don't remove compatibility_test, since it's not going anywhere from the
builddir.
[- You can copy both create.php and simplepie.inc in a single line.]
--
Configure 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
