[Bug 516522] Review Request: globus-nexus - Globus Toolkit - Nexus 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=516522 --- Comment #5 from Fedora Update System upda...@fedoraproject.org 2009-11-07 03:18:18 EDT --- globus-nexus-6.7-1.el5 has been submitted as an update for Fedora EPEL 5. http://admin.fedoraproject.org/updates/globus-nexus-6.7-1.el5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 516522] Review Request: globus-nexus - Globus Toolkit - Nexus 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=516522 --- Comment #7 from Fedora Update System upda...@fedoraproject.org 2009-11-07 03:18:24 EDT --- globus-nexus-6.7-1.el4 has been submitted as an update for Fedora EPEL 4. http://admin.fedoraproject.org/updates/globus-nexus-6.7-1.el4 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 516527] Review Request: globus-gass-server-ez - Globus Toolkit - Globus Gass Server_ez
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516527 --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2009-11-07 03:18:32 EDT --- globus-gass-server-ez-2.5-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/globus-gass-server-ez-2.5-1.fc11 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 516517] Review Request: globus-gridftp-server - Globus Toolkit - Globus GridFTP Server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516517 --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2009-11-07 03:18:08 EDT --- globus-gridftp-server-3.17-2.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/globus-gridftp-server-3.17-2.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 516517] Review Request: globus-gridftp-server - Globus Toolkit - Globus GridFTP Server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516517 --- Comment #7 from Fedora Update System upda...@fedoraproject.org 2009-11-07 03:18:14 EDT --- globus-gridftp-server-3.17-2.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/globus-gridftp-server-3.17-2.fc11 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 516527] Review Request: globus-gass-server-ez - Globus Toolkit - Globus Gass Server_ez
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516527 --- Comment #7 from Fedora Update System upda...@fedoraproject.org 2009-11-07 03:18:33 EDT --- globus-gass-server-ez-2.5-1.el4 has been submitted as an update for Fedora EPEL 4. http://admin.fedoraproject.org/updates/globus-gass-server-ez-2.5-1.el4 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 516522] Review Request: globus-nexus - Globus Toolkit - Nexus 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=516522 --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2009-11-07 03:18:22 EDT --- globus-nexus-6.7-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/globus-nexus-6.7-1.fc11 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 516522] Review Request: globus-nexus - Globus Toolkit - Nexus 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=516522 --- Comment #4 from Fedora Update System upda...@fedoraproject.org 2009-11-07 03:18:16 EDT --- globus-nexus-6.7-1.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/globus-nexus-6.7-1.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 516527] Review Request: globus-gass-server-ez - Globus Toolkit - Globus Gass Server_ez
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516527 --- Comment #4 from Fedora Update System upda...@fedoraproject.org 2009-11-07 03:18:27 EDT --- globus-gass-server-ez-2.5-1.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/globus-gass-server-ez-2.5-1.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 516527] Review Request: globus-gass-server-ez - Globus Toolkit - Globus Gass Server_ez
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516527 --- Comment #4 from Fedora Update System upda...@fedoraproject.org 2009-11-07 03:18:27 EDT --- globus-gass-server-ez-2.5-1.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/globus-gass-server-ez-2.5-1.fc12 --- Comment #5 from Fedora Update System upda...@fedoraproject.org 2009-11-07 03:18:28 EDT --- globus-gass-server-ez-2.5-1.el5 has been submitted as an update for Fedora EPEL 5. http://admin.fedoraproject.org/updates/globus-gass-server-ez-2.5-1.el5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 532368] Review Request: ns-bola-fonts A Sans Font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532368 --- Comment #2 from TK009 john.brown...@gmail.com 2009-11-07 03:28:32 EDT --- Spec URL: http://linuxkommando.com/dev/ns-bola-fonts.spec SRPM URL: http://linuxkommando.com/dev/ns-bola-fonts-20080203-1.fc12.src.rpm Changed file location. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 530880] Review Request: ns-tiza-fonts - A Slab-Serif Font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530880 Nicolas Mailhot nicolas.mail...@laposte.net changed: What|Removed |Added AssignedTo|nicolas.mail...@laposte.net |john.brown...@gmail.com Flag|fedora-review? |fedora-review+ --- Comment #12 from Nicolas Mailhot nicolas.mail...@laposte.net 2009-11-07 04:30:21 EDT --- Thanks I'm going to approve the package, but please do not forget to reference the licensing file in %doc (don't know if I missed this before or if your removed it since the start of review) ⌚⌚⌚ APPROVED ⌚⌚⌚ I'm going to look at Bola now and sponsor you if Bola's packaging is ok As soon as you're sponsored, you'll be able to continue from: http://fedoraproject.org/wiki/Font_package_lifecycle#3.a I hope the process was pleasant, and that it will inspire you to package a other fonts for Fedora. Please do not hesitate to suggest improvements to our organisation or documentation on the fonts mailing 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 533520] Review Request: ogre-pagedgeometry - Ogre addon for realtime rendering of dense forests
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=533520 Guido Grazioli guido.grazi...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||guido.grazi...@gmail.com AssignedTo|nob...@fedoraproject.org|guido.grazi...@gmail.com Flag||fedora-review? --- Comment #3 from Guido Grazioli guido.grazi...@gmail.com 2009-11-07 04:34:50 EDT --- Specfile is in good shape, taking review of this package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 530880] Review Request: ns-tiza-fonts - A Slab-Serif Font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530880 --- Comment #13 from Nicolas Mailhot nicolas.mail...@laposte.net 2009-11-07 04:34:46 EDT --- (In reply to comment #10) If Serif is not correct for generic family I will change it to Fantasy, however, I am not sure you see Serif in the file can you confirm that is what you see in the review? Again I ask this because I am concerned we are not working with the same file? I'm behind a proxy. If you upload a file on a web server that sets expiry time to a high value I won't necessarily see the changes (a common way to avoid this is to increment the Release number each time you have a change, numbers are cheap and having several files with the same id is always dangerous Lastly, I always work from the spec in the srpm 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 530880] Review Request: ns-tiza-fonts - A Slab-Serif Font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530880 --- Comment #14 from Nicolas Mailhot nicolas.mail...@laposte.net 2009-11-07 04:42:57 EDT --- (In reply to comment #11) C. Changed generic family to 'fantasy'. I am assuming rule #1 overrides all other rules in fontconfig-generics.txt. fontconfig-generics.txt is a decision graph. A font could be fantasy and serif and monospace at the same time, so you need common rules for everyone to chose the same category in the same circumstances -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 533520] Review Request: ogre-pagedgeometry - Ogre addon for realtime rendering of dense forests
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=533520 Dan Horák d...@danny.cz changed: What|Removed |Added CC||d...@danny.cz --- Comment #4 from Dan Horák d...@danny.cz 2009-11-07 04:56:41 EDT --- Setting -DBUILD_SHARED_LIBS:BOOL=ON explicitely on the command line shouldn't be needed, the %cmake macro already contains that. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 508518] Review Request: Meiga - Easy tool for file sharing and content publishing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=508518 Thomas Spura toms...@fedoraproject.org changed: What|Removed |Added Status|NEW |ASSIGNED CC||toms...@fedoraproject.org AssignedTo|nob...@fedoraproject.org|toms...@fedoraproject.org Flag||fedora-review? --- Comment #4 from Thomas Spura toms...@fedoraproject.org 2009-11-07 04:57:14 EDT --- There is a new version, after updating I'll review this this week, at maximum next weekend… -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 530568] Review Request: ghmm - A library with data structures and algorithms for Hidden Markov Models
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530568 --- Comment #9 from Thomas Spura toms...@fedoraproject.org 2009-11-07 05:06:49 EDT --- You told 'make update' the old bug number, and not this one, so this will never get closed on its own... I can't close this right now, because I still can't find this package build in devel... Did you run a 'make build' in the devel directory? It seems, you don't; please do so… -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 533558] New: Review Request: gtkwhiteboard - GTK Wiimote Whiteboard
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: gtkwhiteboard - GTK Wiimote Whiteboard https://bugzilla.redhat.com/show_bug.cgi?id=533558 Summary: Review Request: gtkwhiteboard - GTK Wiimote Whiteboard Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: pro...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://orion.lcg.ufrj.br/RPMS/SPECS/gtkwhiteboard.spec SRPM URL: http://orion.lcg.ufrj.br/RPMS/src/gtkwhiteboard-1.3-2.fc10.src.rpm Description: Wii whiteboard python implementation. This is a linux implementation for Johnny Lee's whiteboard: http://johnnylee.net/projects/wii/ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 530568] Review Request: ghmm - A library with data structures and algorithms for Hidden Markov Models
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530568 --- Comment #10 from D Haley my...@yahoo.com 2009-11-07 05:47:03 EDT --- Did you run a 'make build' in the devel directory? It seems, you don't; please do so… The build was done a week ago: http://koji.fedoraproject.org/koji/buildinfo?buildID=138832 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456353] Review Request: libffado - Free firewire audio driver 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=456353 --- Comment #9 from Alex Kanavin a...@sensi.org 2009-11-07 05:56:28 EDT --- What's the current status? libffado is still not in rawhide as far as I can see. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 530568] Review Request: ghmm - A library with data structures and algorithms for Hidden Markov Models
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530568 --- Comment #11 from Thomas Spura toms...@fedoraproject.org 2009-11-07 06:01:28 EDT --- Strange. It's not in community: https://admin.fedoraproject.org/community/?search=ghmm#search Not on the webservers: http://download.fedora.redhat.com/pub/fedora/linux/development/x86_64/os/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 512170] Review Request: pidgin-sipe - Pidgin plugin for connecting to MS Communications Server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=512170 Rahul Sundaram sunda...@redhat.com changed: What|Removed |Added Flag||fedora-review+ --- Comment #28 from Rahul Sundaram sunda...@redhat.com 2009-11-07 06:00:03 EDT --- Looks good to go except that I am unable to test the functionality since I don't have a Windows box handy. 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 533141] Review Request: perl-CGI-Application-Plugin-DBIx-Class - Access a DBIx::Class Schema from a CGI::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=533141 --- Comment #7 from Fedora Update System upda...@fedoraproject.org 2009-11-07 06:05:06 EDT --- perl-CGI-Application-Plugin-DBIx-Class-0.093011-2.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/perl-CGI-Application-Plugin-DBIx-Class-0.093011-2.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 533141] Review Request: perl-CGI-Application-Plugin-DBIx-Class - Access a DBIx::Class Schema from a CGI::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=533141 --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2009-11-07 06:05:02 EDT --- perl-CGI-Application-Plugin-DBIx-Class-0.093011-2.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/perl-CGI-Application-Plugin-DBIx-Class-0.093011-2.fc11 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 530568] Review Request: ghmm - A library with data structures and algorithms for Hidden Markov Models
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530568 D Haley my...@yahoo.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||ERRATA --- Comment #12 from D Haley my...@yahoo.com 2009-11-07 06:29:12 EDT --- Its odd that it is not on the webserver. I can see it in community, though it has some kind of warning on the page Error loading the data for this page element: https://admin.fedoraproject.org/community/?package=ghmm#package_maintenance -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 526564] Review Request: unittest - C++ unit 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=526564 --- Comment #15 from Fedora Update System upda...@fedoraproject.org 2009-11-07 08:49:28 EDT --- unittest-0.50-62.6.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/unittest-0.50-62.6.fc11 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 532470] Review Request: gstreamer-plugins-bad - GStreamer streaming media framework bad plug-ins
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532470 Hans de Goede hdego...@redhat.com changed: What|Removed |Added CC||hdego...@redhat.com --- Comment #5 from Hans de Goede hdego...@redhat.com 2009-11-07 09:30:02 EDT --- I think it is a very good idea to put the eligible parts of -bad into Fedora and I would like to review this. But I would first like to see a review by Spot of the assertion: The code here shouldn't contain any problematic source code, as this code would usually live in library depedencies [sic]) I find at least some of the muxes worrying. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 530754] Review Request: scour - A script to clean SVG files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530754 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-review? | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 492609] Review Request: hmaccalc - Tools for computing and checking HMAC values for files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=492609 --- Comment #10 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 09:55:47 EDT --- Well, it's up to you; close the ticket out when you no longer want it around to remind you of anything. My job has long since been done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 530880] Review Request: ns-tiza-fonts - A Slab-Serif Font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530880 --- Comment #15 from Nicolas Mailhot nicolas.mail...@laposte.net 2009-11-07 09:55:25 EDT --- (the fontlint warning More points in a glyph than PostScript allows is probably worth looking at upstream too) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 530568] Review Request: ghmm - A library with data structures and algorithms for Hidden Markov Models
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530568 --- Comment #13 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-11-07 10:04:47 EDT --- Note: The devel branch on Fedora CVS and koji (in comment 10) currently points to F-13, which is not available on Fedora mirror servers, while currently rawhide (i.e. development tree on Fedora mirror servers, in comment 11) is for F-12. dist-f12 (on koji) tree is already locked and now you have to submit push request for F-12 on bodhi: https://admin.fedoraproject.org/updates/ and packages for F-12 (currently called as rawhide) won't appear until F-12 is formally 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 533520] Review Request: ogre-pagedgeometry - Ogre addon for realtime rendering of dense forests
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=533520 --- Comment #5 from Bruno Wolff III br...@wolff.to 2009-11-07 10:08:20 EDT --- I'll retest it. That may have been when I was trying to work around them forcing a static library build. I ended up having to use a patch to do that instead. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 501958] Review Request: scotch - Graph, mesh and hypergraph partitioning 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=501958 Deji Akingunola dakin...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #13 from Deji Akingunola dakin...@gmail.com 2009-11-07 10:10:07 EDT --- Imported and built in all branches. Thanks Jason for the review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 445068] Review Request: ocaml-bin-prot - Read and write OCaml values in a type-safe binary protocol
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=445068 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|ti...@math.uh.edu Flag||fedora-review? --- Comment #10 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 10:10:17 EDT --- OK, I'm going to try for a third time to review this. The first time, firefox crashed and I lost the review; the second time, the USB controller on my motherboard fried and I had no keyboard with which to complete the review. Not a good couple of days for me. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 502991] Review Request: erlang-erlsom - Support for XML Schema in Erlang
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=502991 --- Comment #9 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 10:17:03 EDT --- *** Bug 445797 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 533520] Review Request: ogre-pagedgeometry - Ogre addon for realtime rendering of dense forests
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=533520 --- Comment #6 from Bruno Wolff III br...@wolff.to 2009-11-07 10:17:34 EDT --- I rebuilt the src rpm without using -DBUILD_SHARED_LIBS:BOOL=ON and replaced the spec file and added a new src rpm. http://wolff.to/bruno/ogre-pagedgeometry-1.05-4.2721svn.fc12.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 445797] Review Request: erlsom - Support for XML Schema in Erlang
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=445797 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Resolution|NOTABUG |DUPLICATE --- Comment #6 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 10:17:03 EDT --- *** This bug has been marked as a duplicate of 502991 *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 532368] Review Request: ns-bola-fonts A Sans Font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532368 Nicolas Mailhot nicolas.mail...@laposte.net changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|john.brown...@gmail.com Flag||fedora-review+ --- Comment #3 from Nicolas Mailhot nicolas.mail...@laposte.net 2009-11-07 10:16:39 EDT --- Review 1. fontlint states In GID 76 the advance width (2499) is greatert than the stated maximum (999) ⇒ please relay upstream 2. repo-font-audit identifies some scripts the font could easily be extended to cover ⇒ please check and relay upstream 3. you do not need to use Mixed Case In Summary; just the first letter in the sentence will do 4. It supports diacritics ⇒ It includes diacritics ? 5. You've forgotten to add the license file to %doc 6. It would be nice if the font metadata said OFL like the detached license file ⇒ please relay upstream The rest is fine ␆␆␆ APPROVED ␆␆␆ You can now continue from http://fedoraproject.org/wiki/Font_package_lifecycle#3.a I hope the process was pleasant, and that it will inspire you to package a other fonts for Fedora. Please do not hesitate to suggest improvements to our organisation or documentation on the fonts mailing list. Thank you for another contribution to our font package pool. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 532819] Review Request: gdouros-symbola-fonts - A symbol font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532819 Nicolas Mailhot nicolas.mail...@laposte.net changed: What|Removed |Added Flag|fedora-review? |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 532819] Review Request: gdouros-symbola-fonts - A symbol font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532819 Nicolas Mailhot nicolas.mail...@laposte.net changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|ozam...@flukkost.nu Flag||fedora-review? --- Comment #1 from Nicolas Mailhot nicolas.mail...@laposte.net 2009-11-07 10:38:36 EDT --- Review 1. The description is particularly difficult to digest → please try to come up with something better with upstream (not blocking). The specific reference to Unicode 5.2 in particular will force you to change the description each time the font and the standard are updated 2. fontlint notes More points in a glyph than PostScript allows → please relay upstream (not blocking) 3. repo-font-audit notes partial script and unicode coverage → please relay upstream (not blocking) 4. the unicode points are unusual but the style is not exotic, I'd say it's a serif not fantasy font (not blocking) 5. It would be nice if upstream added a .txt licensing file to the zip ₪₪₪ APPROVED ₪₪₪ You can now continue from http://fedoraproject.org/wiki/Font_package_lifecycle#3.a I hope the process was pleasant, and that it will inspire you to package a other fonts for Fedora. Please do not hesitate to suggest improvements to our organisation or documentation on the fonts mailing list. Thank you for another contribution to our font package pool. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 533520] Review Request: ogre-pagedgeometry - Ogre addon for realtime rendering of dense forests
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=533520 --- Comment #7 from Bruno Wolff III br...@wolff.to 2009-11-07 10:49:36 EDT --- I made a new scratch build for -4 to make it easy to review build logs. http://koji.fedoraproject.org/koji/taskinfo?taskID=1794203 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 529441] Review Request: pdfbox - Java PDF 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=529441 --- Comment #4 from Orion Poplawski or...@cora.nwra.com 2009-11-07 11:01:43 EDT --- lucene 2.4.1 on F-12: http://koji.fedoraproject.org/koji/taskinfo?taskID=1794228 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 531051] Review Request: glite-security-util-java - Java Utilities for GSI Credentials
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=531051
Mattias Ellert mattias.ell...@fysast.uu.se changed:
What|Removed |Added
Status|NEW |ASSIGNED
Flag||fedora-review?
--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se 2009-11-07
11:10:50 EDT ---
Fedora review glite-security-util-java 2009-11-07
+ OK
- Needs work
? Needs explanation
$ rpmlint glite-security-util-java-*
glite-security-util-java.src:123: W: libdir-macro-in-noarch-package (main
package) %attr(-,root,root) %{_libdir}/gcj/%{name}/%{name}-%{version}.jar.*
4 packages and 0 specfiles checked; 0 errors, 1 warnings.
+ rpmlint OK
+ Package named according to Java guidelines (name same as jarfile)
+ Specfile named after package
+ License tag ASL 2.0 and EU Datagrid is a Fedora apporved license
+ The tag corresponds to the license in the sources (most is ASL 2.0, but
the code in src/org/glite/security/voms/ac is EU Datagrid
+ LICENSE file is listed as %doc
+ Specfile is written in legible English
+ Sources corresponds to upstream (following the description in the specfile)
+ Package compiles as a mock build
+ Build requirements are sane
- The source tree contains an embedded copy of the source for the
vomsjapi library in src/org/glite/security/voms, bundling copies of
system libraries is not allowed - use the system version instead
This can be achieved by adding the following lines to the %prep section ...
rm -rf src/org/glite/security/voms
sed s/org.glite.security.voms.VOMSValidator/org.glite.voms.VOMSValidator/ \
-i src/org/glite/security/SecurityContext.java
... and adding vomsjapi to BuildRequires, Requires and build-classpath
It might then also be possible to change the License tag to only ASL 2.0
since the code under the other license is then not used.
- Package does not own /usr/lib64/gcj/glite-security-util-java
+ No duplicate files
+ Permissions are sane and %files has %defattr.
Though the %attr(-,root,root) tag for the gcj AOT files is redundant -
covered by the default.
+ %clean clears buildroot
+ Specfile uses macros consistently
+ Package contains code
- The javadoc package does not have a versioned directory and a symlink
+ Package does not own others' directories
+ %install clears buildroot
+ Filenames are UTF8
? axis and servlet are listed as BuildRequires but not as Requires - why?
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 533594] New: Review Request: Pootle - Localization and translation management web application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: Pootle - Localization and translation management web application https://bugzilla.redhat.com/show_bug.cgi?id=533594 Summary: Review Request: Pootle - Localization and translation management web application Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: dwa...@translate.org.za QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://dwayne.fedorapeople.org/pootle-1.3.0-0.3.beta4.spec SRPM URL: http://dwayne.fedorapeople.org/pootle-1.3.0-0.3.beta4.fc11.src.rpm You will need this version of the Translate Toolkit which is currently in F11 updates-testing: https://admin.fedoraproject.org/updates/F11/FEDORA-2009-11035 to test v1.3 of Pootle. Description: Localization and translation management web application Pootle is web application for managing distributed or crowdsourced translation. It's features include:: * Translation of Gettext PO and XLIFF files. * Submitting to remote version control systems (VCS). * Managing groups of translators * Online webbased or offline translation * Quality checks -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 532819] Review Request: gdouros-symbola-fonts - A symbol font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532819 Robin Sonefors ozam...@flukkost.nu changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Robin Sonefors ozam...@flukkost.nu 2009-11-07 11:36:14 EDT --- New Package CVS Request === Package Name: gdouros-symbola-fonts Short Description: A symbol font Owners: ozamosi Branches: F11 F12 InitialCC: fonts-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 532818] Review Request: gdouros-musica-fonts - A font for musical symbols
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532818 Robin Sonefors ozam...@flukkost.nu changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Robin Sonefors ozam...@flukkost.nu 2009-11-07 11:37:10 EDT --- New Package CVS Request === Package Name: gdouros-musica-fonts Short Description: A font for musical symbols Owners: ozamosi Branches: F11 F12 InitialCC: fonts-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 532817] Review Request: gdouros-analecta-fonts - An eccleastic scripts font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532817 Robin Sonefors ozam...@flukkost.nu changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Robin Sonefors ozam...@flukkost.nu 2009-11-07 11:36:44 EDT --- New Package CVS Request === Package Name: gdouros-analecta-fonts Short Description: An eccleastic scripts font Owners: ozamosi Branches: F11 F12 InitialCC: fonts-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 532816] Review Request: gdouros-alexander-fonts - A Greek typeface inspired by Alexander Wilson
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532816 Robin Sonefors ozam...@flukkost.nu changed: What|Removed |Added Flag||fedora-cvs? --- Comment #3 from Robin Sonefors ozam...@flukkost.nu 2009-11-07 11:36:31 EDT --- New Package CVS Request === Package Name: gdouros-alexander-fonts Short Description: A Greek typeface inspired by Alexander Wilson Owners: ozamosi Branches: F11 F12 InitialCC: fonts-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 532306] Review Request: rubygem-ruby-debug - Faster implementation of the standard Debugging
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=532306
--- Comment #5 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-11-07
11:49:20 EDT ---
Well,
- Please package linecache first.
%check fails without linecache and even without %check ruby-debug-base
actually needs linecache (see ruby-debug-base-0.10.3/lib/ruby-debug-base.rb)
http://koji.fedoraproject.org/koji/taskinfo?taskID=1794237
( Does your srpm actually builds for you? )
- head command or so is very dangerous when binary files exist.
Actually
---
61 # Fix anything executable that does not have a shebang
62 for file in `find %{buildroot}/%{geminstdir}
%{buildroot}/%{geminstdir2} -type f -perm /a+x`; do
63 [ -z `head -n 1 $file | grep \^#!/\` ] chmod -v 644 $file
64 done
---
changes the permission of ruby_debug.so to 0644 and currently
debuginfo rpm is not correctly created.
- Executing %check under %buildroot is wrong for this package
because rake test again tries to create ruby_debug.so under
%buildroot%{geminstdir2}/ext/ because we move this file to
%buildroot%ruby_sitearch
---
853 Executing(%check): /bin/sh -e /var/tmp/rpm-tmp.5gRRsq
860 + rake -f ../ruby-debug-0.10.3/Rakefile test
865 gcc -shared -o ruby_debug.so breakpoint.o ruby_debug.o -L. -L/usr/lib
-L. -rdynamic -Wl,-export-dynamic-lruby -lpthread -lrt -ldl -lcrypt -lm
-lc
---
%check must be done under %_builddir for this package.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 528473] Review Request: php-ezc-DatabaseSchema - eZ Components DatabaseSchema
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=528473 --- Comment #2 from Guillaume Kulakowski llaum...@gmail.com 2009-11-07 12:00:13 EDT --- SPEC updated : http://llaumgui.fedorapeople.org/review/ez_components/php-ezc-DatabaseSchema.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 528476] Review Request: php-ezc-PersistentObjectDatabaseSchemaTiein - eZ Components PersistentObjectDatabaseSchemaTiein
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=528476 --- Comment #2 from Guillaume Kulakowski llaum...@gmail.com 2009-11-07 12:03:30 EDT --- SPEC Updated : http://llaumgui.fedorapeople.org/review/ez_components/php-ezc-PersistentObjectDatabaseSchemaTiein.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 530204] Review Request: rubygem-hawler - Hawler, the Ruby HTTP crawler
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530204 --- Comment #10 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-11-07 12:11:39 EDT --- Please submit push request on bodhi. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 445068] Review Request: ocaml-bin-prot - Read and write OCaml values in a type-safe binary protocol
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=445068 --- Comment #11 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 12:15:37 EDT --- OK, this builds fine and rpmlint is silent In multiple licensing scenarios, you'll need to add some indication of which files are under which license. The easiest way is to just add a comment to the spec. http://fedoraproject.org/wiki/Packaging:LicensingGuidelines#Multiple_Licensing_Scenarios You've duplicated the license files between -devel and the main package. This is clearly against the guidelines http://fedoraproject.org/wiki/Packaging:Guidelines#Duplicate_Files but I know we've gone back and forth on this so I've requested that we get an opinion from legal and get the guidelines modified if necessary. The -devel package includes a .ml file; the OCaml guidelines say that these shouldn't be included except in specific circumstances. I can't determine from the information I have whether or not those circumstances apply here, and there's no comment in the spec about it. * source files match upstream. sha256sum: 1a8569a75195b35ec17348046b3aa8823f9f27e451214717692a86075cfbdb9b release-1.2.20.tar.bz2 * package meets naming and versioning guidelines. * specfile is properly named, is cleanly written and uses macros consistently. * summary is OK. * description is OK. * dist tag is present. * build root is OK. * license field matches the actual license. * license is open source-compatible. * license text included in package. * latest version is being packaged. * BuildRequires are proper. * %clean is present. * package builds in mock (rawhide, x86_64). * package installs properly. * rpmlint is silent. * final provides and requires are sane: ocaml-bin-prot-1.2.20-1.fc12.x86_64.rpm dllbin_prot_stubs.so()(64bit) ocaml(Bin_prot) = 68b4c083005ed9d5cde0767b0e6a9782 ocaml(Pa_bin_prot) = d894b67ca8604bc43a9721e774c08303 ocaml-bin-prot = 1.2.20-1.fc12 ocaml-bin-prot(x86-64) = 1.2.20-1.fc12 = ocaml(Arg) = b6513be035dc9c8a458c189cd8841700 ocaml(Array) = 9c9fa5f11e2d6992c427dde4d1168489 ocaml(Bigarray) = fc2b6c88ffd318b9f111abe46ba99902 ocaml(Buffer) = 23af67395823b652b807c4ae0b581211 ocaml(Callback) = 71e1f9b7f211661f1dfeedab5ffae0cc ocaml(CamlinternalLazy) = ed280fb9736e9200aa47db73c5ff077f ocaml(Camlp4) = bb930f7c2bed5d057c794fe07dc8596a ocaml(Camlp4_config) = 80b5d58834366711574a5ec4dfb123fd ocaml(Camlp4_import) = 4d17b58763ba1f0aac92fd5dbb558b59 ocaml(Char) = 3da72249626c7db769beafc97036cb4f ocaml(Complex) = 73899d718b62e5534e8737bb363dbf71 ocaml(Filename) = 7cd172f02b7ee9b8d7bda3bb92144951 ocaml(Format) = b7ba3152a5eec5609d6ab86e6c51eebb ocaml(Hashtbl) = ee2a3220e38a4350c5bc131ce9f3f6ce ocaml(Int32) = b2545c419b6b6a173cac4c0a3e7e0277 ocaml(Int64) = d501d6e89fdce41c79f274fb464995d5 ocaml(Lazy) = 4c7ed568fa7b5f73a2aa02eeb0e5e12b ocaml(Lexing) = 4d17267334f1a6c75730dc3fae21fb9b ocaml(List) = a0e2e49d266ff302f8667651a43f71ba ocaml(Nativeint) = 7233ce5207a538fea4f0c61ed411ea2c ocaml(Obj) = c827f726ce05da709cf7de58fc15e324 ocaml(Parsing) = 29c3f123280f8e6e639cfb025b3c9a3f ocaml(Pa_type_conv) = 917c39ac24d30438f1e78e6e58840e45 ocaml(Pervasives) = 88cb1505c8bdf9a4dcd2cdf3452732b4 ocaml(Printf) = 807ecd3a1538992580464c03462c9964 ocaml(Queue) = 56b5e04dcda600ae0cdf49a37f17fcd9 ocaml(Set) = c4be5d24d30c129dd60d2739e54db7dd ocaml(Stream) = 91a43ea7fb16bf36f3f10c0dc7d08a0e ocaml(String) = ecc403546c1c50056801131811c39017 ocaml(Sys) = 21bf525b2b3f3a46a54b96163adfe387 ocaml(Unix) = 0596a58544f8cd88fed5bf5432a53d43 ocaml(runtime) = 3.11.1 ocaml-bin-prot-devel-1.2.20-1.fc12.x86_64.rpm ocaml-bin-prot-devel = 1.2.20-1.fc12 ocaml-bin-prot-devel(x86-64) = 1.2.20-1.fc12 = ocaml-bin-prot = 1.2.20-1.fc12 * %check is present and all tests pass: Ran: 67 tests in: 0.35 seconds. OK msgs: 10 msg length: 461 write time: 2.548s write rate: 39244.14 msgs/s write throughput: 17.25 MB/s read time: 2.619s read rate: 38181.64 msgs/s read throughput: 16.79 MB/s * owns the directories it creates. * doesn't own any directories it shouldn't. X license files are duplicated. * file permissions are appropriate. * code, not content. * documentation is small, so no -doc subpackage is necessary. * %docs are not necessary for the proper functioning of the package. * .cma, .cmi, .so, .so.owner, META files in the main package. * .a, .cmxa, .cmx and .mli files are in the -devel subpackage. ? a .ml file is included * .so files have no rpath -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the
[Bug 528469] Review Request: php-ezc-EventLogDatabaseTiein - eZ Components EventLogDatabaseTiein
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=528469 --- Comment #1 from Guillaume Kulakowski llaum...@gmail.com 2009-11-07 12:21:19 EDT --- SPEC updated : http://llaumgui.fedorapeople.org/review/ez_components/php-ezc-EventLogDatabaseTiein.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 509856] Review Request: qrencode - The libqrencode library and application encodes QR Code symbols (2d barcodes)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=509856 --- Comment #15 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-11-07 12:32:39 EDT --- I will close this bug as NOTABUG within one week if no response is received from the reporter within ONE WEEK. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 459548] Review Request: php-gtk - GTK PHP extension
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=459548 --- Comment #32 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-11-07 12:35:00 EDT --- I will close this bug as NOTABUG if no response is received from the reporter with ONE WEEK. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 532817] Review Request: gdouros-analecta-fonts - An eccleastic scripts font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532817 --- Comment #3 from Robin Sonefors ozam...@flukkost.nu 2009-11-07 12:45:02 EDT --- Sorry, that description is wrong. Correct description: An ecclesiastic scripts font -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 529542] Review Request: php-phpunit-phpcpd - Copy/Paste Detector (CPD) for PHP code
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=529542 --- Comment #2 from Guillaume Kulakowski llaum...@gmail.com 2009-11-07 13:10:58 EDT --- SPEC updated : http://llaumgui.fedorapeople.org/review/php-phpunit-phpcpd.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487913] Review Request: culmus-fancy-fonts - Fancy fonts for Hebrew
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487913 --- Comment #5 from Nicolas Mailhot nicolas.mail...@laposte.net 2009-11-07 13:25:16 EDT --- (In reply to comment #4) Created an attachment (id=367518) -- (https://bugzilla.redhat.com/attachment.cgi?id=367518) [details] spec with fixes to sever of the comments It might be just about at the packaging standard, but certainly has several of the comments unfixed. Can you post a srpm please? It's a lot easier to review when you can build the package -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 526311] Review Request: mysql-mmm - Multi-Master Replication Manager for MySQL
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=526311 --- Comment #11 from Ruben Kerkhof ru...@rubenkerkhof.com 2009-11-07 13:36:38 EDT --- Since this are all architecture-independent perl libraries, you have to add BuildArch: noarch. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487913] Review Request: culmus-fancy-fonts - Fancy fonts for Hebrew
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487913 Dan Kenigsberg dan...@redhat.com changed: What|Removed |Added Attachment #367518|spec with fixes to sever of |spec with fixes to several description|the comments|of the comments -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487913] Review Request: culmus-fancy-fonts - Fancy fonts for Hebrew
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487913 --- Comment #6 from Dan Kenigsberg dan...@redhat.com 2009-11-07 14:15:07 EDT --- Created an attachment (id=367959) -- (https://bugzilla.redhat.com/attachment.cgi?id=367959) srpm with fixes to several of the problems. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 529441] Review Request: pdfbox - Java PDF 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=529441 Orcan 'oget' Ogetbil oget.fed...@gmail.com changed: What|Removed |Added Depends on||529283 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 529283] Review Request: fontbox - Java library for working with PDF fonts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=529283 Orcan 'oget' Ogetbil oget.fed...@gmail.com changed: What|Removed |Added Blocks||529441 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 529283] Review Request: fontbox - Java library for working with PDF fonts
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=529283
Orcan 'oget' Ogetbil oget.fed...@gmail.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||oget.fed...@gmail.com
AssignedTo|nob...@fedoraproject.org|oget.fed...@gmail.com
Flag||fedora-review+
--- Comment #1 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-11-07
15:43:34 EDT ---
I reviewed this. There are very minor issues that can be fixed before you
import:
* rpmlint says:
W: non-standard-group Development Documentation
This should be just Documentation
W: libdir-macro-in-noarch-package (main package) %attr(-,root,root)
%{_libdir}/gcj/%{name}
rpmlint is wrong. This one can be ignored.
- Patches should be explained and be submitted to upstream. Since this is a
pure Fedora specific patch we can ignore the upstream part.
! %if %{with_gcj}
%attr(-,root,root) %{_libdir}/gcj/%{name}
%endif
Here %attr(-,root,root) is not needed. I know this is in the guidelines but
they have to be corrected. (I proposed this a while ago but things are going a
little slow with updating these guidelines.)
--
This package (fontbox) is APPROVED by oget
--
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 530204] Review Request: rubygem-hawler - Hawler, the Ruby HTTP crawler
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530204 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2009-11-07 17:02:37 EDT --- rubygem-hawler-0.3-5.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/rubygem-hawler-0.3-5.fc11 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 530204] Review Request: rubygem-hawler - Hawler, the Ruby HTTP crawler
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530204 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2009-11-07 17:02:32 EDT --- rubygem-hawler-0.3-5.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/rubygem-hawler-0.3-5.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 532470] Review Request: gstreamer-plugins-bad-free - GStreamer streaming media framework bad plug-ins
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532470 Bastien Nocera bnoc...@redhat.com changed: What|Removed |Added CC||tcall...@redhat.com Summary|Review Request: |Review Request: |gstreamer-plugins-bad - |gstreamer-plugins-bad-free |GStreamer streaming media |- GStreamer streaming media |framework bad plug-ins|framework bad plug-ins --- Comment #6 from Bastien Nocera bnoc...@redhat.com 2009-11-07 19:05:40 EDT --- scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1794399 Spec file: http://people.fedoraproject.org/~hadess/gstreamer-plugins-bad/gstreamer-plugins-bad-free.spec SRPM: http://people.fedoraproject.org/~hadess/gstreamer-plugins-bad/gstreamer-plugins-bad-free-0.10.16-1.fc12.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 529441] Review Request: pdfbox - Java PDF 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=529441
Orcan 'oget' Ogetbil oget.fed...@gmail.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|nob...@fedoraproject.org|oget.fed...@gmail.com
Flag||fedora-review?
--- Comment #5 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-11-07
19:23:58 EDT ---
I made the preliminary review on this package. It needs some work. I also need
to see this building in koji in all archs before approving.
* These need to be added otherwise, package will not build in mock and/or will
have broken deps:
BuildRequires: ant-nodeps
BuildRequires: junit
BuildRequires: jakarta-commons-logging
Requires: jakarta-commons-logging
export CLASSPATH=$( ... jakarta-commons-logging junit)
You don't necessarily need Requires: junit. See below.
- Patches should be explained and be submitted to upstream. Since this is a
pure Fedora specific patch we can ignore the upstream part.
* rpmlint says:
pdfbox.x86_64: W: wrong-file-end-of-line-encoding
/usr/share/doc/pdfbox-0.8.0/RELEASE-NOTES.txt
This can be corrected in %prep easily via sed 's|\r||' or similar.
pdfbox-javadoc.x86_64: W: non-standard-group Development Documentation
Simply use Documentation
pdfbox.src:120: W: libdir-macro-in-noarch-package (main package)
%attr(-,root,root) %{_libdir}/gcj/%{name}
This can be ignored.
* This package includes fonts that are already available in Fedora. This is not
allowed by the guidelines. You will need to patch the source to remove these
fonts from the jar file and use the system ones instead. This will require
adding
Requires: bitstream-vera-sans-fonts
See
http://fedoraproject.org/wiki/Packaging:FontsPolicy
! It would be good to run the supplied tests in %check via ant test. But for
this, you may need to add
BuildRequires(check): ant-junit
and *possibly* add this to classpath before you call ant test
? Test classes are being packaged (they end up in the final jar). Is this
necessary? If yes, then you will need to add
Requires: junit
If not sure, please contact upstream.
! %attr(-,root,root) is not needed as I explained in the previous 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 464757] Review Request: openssl-tpm-engine - OpenSSL engine for TSS 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=464757 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Status Whiteboard||BuildFails --- Comment #9 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 20:27:24 EDT --- Indicating that this doesn't build; please clear the whiteboard when that's been fixed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 475934] Review Request: m6812-elf-binutils - Cross Compiling GNU binutils targeted at m68hc12
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=475934 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Status|NEW |CLOSED Blocks||201449(FE-DEADREVIEW) Resolution||NOTABUG --- Comment #10 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 20:28:41 EDT --- Given nearly four months of silence, I'm going to assume that's a big no and close this out. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 472150] Review Request: coot - crystallographic macromolecular building toolkit
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=472150
--- Comment #53 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 20:24:58
EDT ---
I've had no time for reviews recently, but I have a little now.
This builds find on rawhide, but not F11 because of the clipper-devel
dependency. A version build for F12 will install on F11 but won't run. My
not-quite-F12 boxes are all at work, which makes testing difficult; I can run
it remotely, but it fails with *** Cannot find the double-buffered visual.
Looks like you just can't go GLX over SSH, which I swear used to work OK.
Unfortunately I can't really do a proper review if I can't do any testing.
Anyway, I can make a few comments:
The build is fine; rpmlint has just the expected unused-direct-shlib-dependency
complaints, along with a couple of shared-lib-calls-exit and a single
no-documentation complaint, all of which are OK.
When I installed this I tried to find the icon, only to find that it shows up
in the Graphics category. Unfortunately the desktop file spec doesn't really
have a top-level category that fits this package, but I'd at least expect to
see Graphics;Science;, maybe with Engineering or DataVisualization if they're
appropriate.
Honestly the only real issue I see at this point is the wholesale inclusion of
unrelated upstream projects in the coot/scheme directory. Generally we don't
do that kind of thing; it's basically the situation from
http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries
I didn't really notice before that those were separate projects; I originally
thought they were part of coot but carried different licenses. Now I see that
they are merely prerequisites that shouldn't be part of this package.
Honestly I don't know what to do about that. I guess I'd package guile-gui,
goosh and net-http separately (although probably using a guile- prefix for
the latter two) and then figure out how to make this package use the packaged
versions. Unfortunately I know very little about guile; jugding from the
guile-lib package, it should be as simple as sticking the scheme files in
%{_datadir}/guile/site/goosh or whatever. Getting coot to use those files may
be more difficult; I'm not at all sure.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 466331] Review Request: rcov - ruby code coverage 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=466331 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Status Whiteboard||BuildFails Flag||needinfo?(cdah...@redhat.co ||m) --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 20:54:43 EDT --- Still no buildable package after more than half a year; I guess this should just be closed. I'll do that soon if there's no further response, but in the meantime I'll indicate that this fails to build. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498073] Review Request: python-ekg - Community Health Reporter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=498073 Max Spevack mspev...@redhat.com changed: What|Removed |Added CC||mspev...@redhat.com --- Comment #11 from Max Spevack mspev...@redhat.com 2009-11-07 21:09:15 EDT --- Seeing as this particular code -- python-ekg -- is not being actively worked on, I just wanted those reading this bug to realize that this package review, and our desire to get this code packaged into Fedora's repositories, is not urgent right now. If at some point we have more resources, or folks interested in python-ekg, perhaps we'll ressurect this package review. For the time being, the EKG project is trying to proceed in a different direction. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498073] Review Request: python-ekg - Community Health Reporter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=498073 --- Comment #12 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 21:23:10 EDT --- Max, are you suggesting that this review ticket should be closed out? I suppose you could leave it open if neither Jeroen nor Yaakov mind, but it sounds like there's not much point in proceeding at this point. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 469474] Review Request: sovix - A website revision 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=469474 --- Comment #18 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 21:34:03 EDT --- A few comments: For multiple license scenarios, you need to indicate which parts of the package are under which license. http://fedoraproject.org/wiki/Packaging:LicensingGuidelines#Multiple_Licensing_Scenarios I do not fully understand what requirements the AGPL places on the Fedora package. I don't know if we somehow have to make sure that the source is exposed somehow. I've asked on fedora-legal-list. No supported version of Fedora shipped with a mysql-server older than 5.0.67 or a php older than 5.2.6, so the versioned dependencies are kind of pointless. Even RHEL4 has newer versions than that. Is there any reason why mysql-server would be required? The upstream web page indicates that both mysql and postgres are supported and that use of a database is optional. Not only that, but I can't imagine a situation that would force the database server to be running on the same machine. At worst you'd require the client libraries, and even if you somehow did require the server, you're still missing a dependency on the php interface to the database. Your %description looks like it's been badly word-wrapped. Generally for web applications we provide an apache config file to make the files properly visible to the web. Is there any specific reason for not doing that here? (I realize that makes the package actually require apache itself, but that's now petty much every other packaged webapp does things.) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 469613] Review Request: gnue-appserver - GNU Enterprise Application Server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=469613 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Status Whiteboard||BuildFails --- Comment #2 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 21:37:23 EDT --- Just checking old review tickets. This one fails to build because patch1 fails to apply. Please clear the whiteboard when you have a fixed package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 530772] Review Request: pxe-kexec - Linux boots Linux via network
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530772 --- Comment #2 from Scott Collier boodl...@gmail.com 2009-11-07 21:44:13 EDT --- Koji builds are successful: http://koji.fedoraproject.org/koji/taskinfo?taskID=1794541 http://koji.fedoraproject.org/koji/taskinfo?taskID=1794546 http://koji.fedoraproject.org/koji/taskinfo?taskID=1794551 rpmlint is clean. May I please get a sponsor? I think the package is ready to go. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 469931] Review Request: ipmiutil - IPMI Management Utilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=469931 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Status Whiteboard||BuildFails --- Comment #15 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 21:57:49 EDT --- The package in comment 14 fails to build for me: + cd /builddir/build/BUILDROOT/ipmiutil-2.4.3-1.x86_64/usr/share/man/man8 + gzip -f '*.8' gzip: *.8: No such file or directory I'm not sure what's gone wrong, but do note that you shouldn't try to compress the manpages yourself; rpmbuild will do it automatically. The %ifarch doesn't seem at all relevant to Fedora, and I have to admit that I can't understand why you would define all of those macros that are pretty much the same as existing macros or are longer than the strings they replace. I don't understand the purpose of %pre; packages cannot produce output in scriptlets, so the whole thing seems to be pointless. And the checks for SuSE and Montavista in %post have no place in Fedora. Many other portions of the scriptlets don't seem relevant to Fedora. Please clear the whiteboard when you have a package which builds. I checked the account system and it looks like you are not in the packager group, so I've indicated that you need a sponsor. Please read through http://fedoraproject.org/wiki/Join_the_package_collection_maintainers for more information on becoming a Fedora packager. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 474012] Review Request: qodem - terminal emulator and communications package.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=474012 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Status|NEW |CLOSED Blocks||201449(FE-DEADREVIEW) Resolution||NOTABUG --- Comment #10 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 22:08:52 EDT --- There's been no response in many months; I'm just going to go ahead and close 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 478806] Review Request: simh - A highly portable, multi-system emulator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=478806 --- Comment #11 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 22:44:20 EDT --- The generate-tarball.sh file seems to have nothing to do with this package. It seems that you've done more than just remove files from the tarball. The change to the makefile is the kind of thing that's better done as a patch -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479800] Review Request: hlint - Provides Haskell Source Code Suggestions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=479800 --- Comment #23 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 22:45:46 EDT --- As long as the source itself is unclear, that information needs to be included within the package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 477683] Review Request: fltk2 - C++ user interface toolkit
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=477683 --- Comment #23 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 23:13:02 EDT --- Just FYI, outside of the rpmlint complaints posted in comment 17, there are also a very large number of undefined-non-weak-symbol complaints along with a few unused-direct-shlib-dependency warnings. There are a couple hundred complaints in total; to see them, install the package and run rpmlint fltk2. It is possible that these aren't problematic; the undefined-non-weak-symbol complaints indicate that you can't make use of the library without also linking to the libraries which provide those symbols. Bad practise and good to fix if possible, but probably not a serious issue. The unused-direct-shlib-dependency complaints indicate that the libraries in question are linked against various libraries but don't actually call into them. Again, this may not be problematic; if there aren't any extra dependencies caused by this and the libraries are going to be in memory anyway. You should check those and verify that there aren't any actual problems indicated. The versioning of this package doesn't seem to follow Fedora guidelines, although I can't really tell. What do you expect the actual release version to be? If it's 2.0.0 or something, then note that you'll have to use epoch to keep ordering. because '0' (or indeed any digit) is less than 'x'. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 513779] Review Request: python-simplesettings - Simple settings initialization for Python libraries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=513779 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag||needinfo?(fab...@bernewirel ||ess.net) --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 23:20:42 EDT --- I guess I'll close this soon if there's no response. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 516311] Review Request: ueagle-atm-firmware123 - Firmwares for usb adsl modems based on Eagle I, II and III chipsets
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516311 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Blocks||182235(FE-Legal) --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 23:29:24 EDT --- Without any indication of the license I don't believe that this can be reviewed. I'll block FE-Legal, but there's not much they can do either without someone indicating the actual license on these files. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479800] Review Request: hlint - Provides Haskell Source Code Suggestions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=479800 --- Comment #24 from Conrad Meyer kon...@tylerc.org 2009-11-07 23:29:30 EDT --- Sure, it can be added to a README.Fedora (or LicensingClarification.txt, whatever you would like). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 532739] Review Request: emacs-cedet - Collection of Emacs Development Environment Tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532739 --- Comment #9 from Xavier Maillard x...@gnu.org 2009-11-07 23:49:29 EDT --- As said before M-x semantic-symref did not work due to installation missing files. The current spec is taking care of them: http://xavier.maillard.im/public/emacs-cedet.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 516312] Review Request: ueagle-atm-firmware4 - Firmwares for usb adsl modems based on Eagle IV chipset
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516312 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag||needinfo?(hicham.haou...@gm ||ail.com) --- Comment #4 from Jason Tibbitts ti...@math.uh.edu 2009-11-07 23:52:26 EDT --- So it's been nearly two months since comment 2 and over a month since FE-Legal was lifted; did you intend to comment on the issues raised? Or should this ticket be closed? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 532470] Review Request: gstreamer-plugins-bad-free - GStreamer streaming media framework bad plug-ins
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532470 --- Comment #7 from Hans de Goede hdego...@redhat.com 2009-11-08 02:37:47 EDT --- Spot to be clear, you were added to the CC because of comment #5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
