[Bug 663384] Review Request: scap-workbench - GUI tool that provides scanning, tailoring, editing and validation functionality for SCAP content
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=663384 --- Comment #11 from Peter Vrabec pvra...@redhat.com 2011-02-09 03:32:59 EST --- New Package SCM Request === Package Name: scap-workbench Short Description: GUI tool that provides scanning, tailoring, editing and validation functionality for SCAP content Owners: mbarabas pvrabec Branches: f14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 Tim Niemueller t...@niemueller.de changed: What|Removed |Added Depends on||676159 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676159] Review Request: crlibm - Correctly Rounded mathematical 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=676159 Tim Niemueller t...@niemueller.de changed: What|Removed |Added Blocks||674008 --- Comment #2 from Tim Niemueller t...@niemueller.de 2011-02-09 03:44:03 EST --- Pavel, it's ok to add a tracker bug, but why remove the existing dependency!? Re-adding. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 675705] Review Request: rubygem-tilt - Generic interface to multiple Ruby template engines
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=675705 Michal Fojtik mfoj...@redhat.com changed: What|Removed |Added CC||mfoj...@redhat.com AssignedTo|nob...@fedoraproject.org|mfoj...@redhat.com --- Comment #1 from Michal Fojtik mfoj...@redhat.com 2011-02-09 07:02:00 EST --- I'll do review 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676308] New: Review Request: rubygem-net-scp - A pure Ruby implementation of the SCP client protocol
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: rubygem-net-scp - A pure Ruby implementation of the SCP client protocol https://bugzilla.redhat.com/show_bug.cgi?id=676308 Summary: Review Request: rubygem-net-scp - A pure Ruby implementation of the SCP client protocol Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: vondr...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://people.redhat.com/vondruch/rubygem-net-scp.spec SRPM URL: http://people.redhat.com/vondruch/rubygem-net-scp-1.0.4-1.fc14.src.rpm Description: A pure Ruby implementation of the SCP client protocol Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2818302 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 427121] Review Request: grib_api - ECMWF encoding/decoding GRIB software
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=427121 --- Comment #20 from Patrice Dumas pertu...@free.fr 2011-02-09 08:51:30 EST --- Sorry, I didn't read my bug for some time. Of course Orion can own grib_api for whatever branch he wants. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674008 --- Comment #8 from Rosen Diankov rosen.dian...@gmail.com 2011-02-09 09:00:56 EST --- i agree with you. in fact, an official tar ball is the number one item on every openrave user's list ;0) we're currently developing a proper testing framework that can track regressions, once this is done we'll immediately start publishing tarballs and releases. For now, I hope you can live with just the change log. and 0.2.19 was just released (r1996). the highlight of this release is ikfast being able to handle arbitrarily complex manipulator kinematics! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674006] Review Request: openni - Library for human-machine Natural Interaction
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674006 Rich Mattes richmat...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #5 from Rich Mattes richmat...@gmail.com 2011-02-09 09:29:18 EST --- Alright it looks like you've taken care of everything. You might want to consider using the %doc macro instead of manually installing the docs in the doc package, e.g. %doc Source/DoxyGen/html/* which will place them in /usr/share/doc. It makes the file install path consistent with other *-doc packages. I can't find an active upstream for the glh stuff anywhere. It’s just used for the example apps and isn’t exposed as an API anywhere, so I think it's safe to just leave it as is. rpmlint is still complaining about the changelogs because it expects a . in front of git4c9ff978 These last comments aren't blockers, just suggestions. So, APPROVED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 669146] Review Request: gnumed-server - medical practice management - 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=669146 --- Comment #3 from Susmit thinklinux@gmail.com 2011-02-09 09:36:21 EST --- Thanks for the comments. Please find the inline comments. A few comments; too many things wrong to do a full review at this time. I sure hope this isn't a copy of the package upstream is currently offering, because I'm afraid that making an acceptable package is going to make it difficult to keep much compatibility with that one. No, my packaging skills are not world class yet. ;) The Source0: URL is invalid; upstream only seems to have 14.5 there. Given that, I probably shouldn't build this, but what the heck, it's a disposable build machine. They did release a 14.6, but for some reason it is not in the directory. I shall be working with upstream to get it resolved. rpmlint says: gnumed-server.noarch: W: only-non-binary-in-usr-lib This is very problematic. The package references _libdir, so the location of the files depends on the architecture it was built on, but it's a noarch package. Plus things reference /usr/lib explicitly while that stuff is in /usr/lib64 on my build. Are you sure the stuff that's in _libdir shouldn't be in _datadir instead? What there is arch-specific? I am having a look. gnumed-server.noarch: W: no-documentation gnumed-server.noarch: W: no-manual-page-for-binary gm-remove_person.sh gnumed-server.noarch: W: no-manual-page-for-binary gm-move_backups_offsite.sh gnumed-server.noarch: W: no-manual-page-for-binary gm-backup_database.sh gnumed-server.noarch: W: no-manual-page-for-binary gm-restore_database.sh gnumed-server.noarch: W: no-manual-page-for-binary gm-adjust_db_settings.sh gnumed-server.noarch: W: no-manual-page-for-binary gm-bootstrap_server gnumed-server.noarch: W: no-manual-page-for-binary gm-backup_data.sh gnumed-server.noarch: W: no-manual-page-for-binary gm-restore_data.sh gnumed-server.noarch: W: no-manual-page-for-binary gm-zip+sign_backups.sh It's nice to have _some_ documentation, and it looks like this package has some. At least there's a license text and some manpages, buried under _libdir. Is there some reason those shouldn't be installed into the proper places? I am having a look. Why are the same scripts installed in two places? How do you configure this package? Nothing is installed under /etc and nothing is marked as %conf. My mistake. Sorry. If this is a server, how do you start and stop it? Doesn't it need an initscript? The way it works is this: One copies it over to relevant directories and then run a bootstrap script which sets up the database and the necessary configurations. Why does this require a local postgres server? Will it not work with a remote one? The upstream wiki mentions that it _requires_ a local server. Thanks for the initial review, I shall get these corrected. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676335] New: Review Request: dmtcp - Checkpoint/Restart functionality for Linux processes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: dmtcp - Checkpoint/Restart functionality for Linux processes https://bugzilla.redhat.com/show_bug.cgi?id=676335 Summary: Review Request: dmtcp - Checkpoint/Restart functionality for Linux processes Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: ka...@ccs.neu.edu QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: www.ccs.neu.edu/home/kapil/fedora_rpms/dmtcp.spec SRPM URL: www.ccs.neu.edu/home/kapil/fedora_rpms/dmtcp-1.2.0+svn886-1.src.rpm Description: DMTCP (Distributed MultiThreaded Checkpointing) is a tool to transparently checkpointing the state of an arbitrary group of programs including multi-threaded and distributed computations. It operates directly on the user binary executable, with no Linux kernel modules or other kernel mods. Among the applications supported by DMTCP are OpenMPI, MATLAB, Python, Perl, and many programming languages and shell scripting languages. DMTCP also supports GNU screen sessions, including vim/cscope and emacs. With the use of TightVNC, it can also checkpoint and restart X-Windows applications, as long as they do not use extensions (e.g.: no OpenGL, no video). NOTE: This is my first package and I need a sponsor. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676335] Review Request: dmtcp - Checkpoint/Restart functionality for Linux processes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=676335 Kapil Arya ka...@ccs.neu.edu changed: What|Removed |Added Platform|All |Unspecified Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 427121] Review Request: grib_api - ECMWF encoding/decoding GRIB software
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=427121 Orion Poplawski or...@cora.nwra.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #21 from Orion Poplawski or...@cora.nwra.com 2011-02-09 10:42:23 EST --- Package Change Request == Package Name: grib_api New Branches: el6 el5 Owners: orion -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 237041] Review Request: perl-JSON - Parse and convert to JSON (JavaScript Object Notation)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=237041 --- Comment #9 from Jon Stanley jonstan...@gmail.com 2011-02-09 11:35:44 EST --- Apologies for not seeing this bugmail before now, if it's in the client channels and not the server, it should be in server optional. I haven't looked, but that'd be my guess. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 675914] Review Request: flush - GTK-based BitTorrent client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=675914 --- Comment #5 from Oksana okurysh...@yahoo.com 2011-02-09 12:10:20 EST --- Fixed everything. There were some problems with new API of new libnotify in rawhide. http://koji.fedoraproject.org/koji/taskinfo?taskID=2823012 Spec URL: http://www.ossportal.ru/sites/default/files/packages/flush.spec SRPM URL: http://www.ossportal.ru/sites/default/files/packages/flush-0.9.9-1.fc14.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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 669010] Review Request: libfap - C port of Ham::APRS::FAP APRS Parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=669010 --- Comment #16 from Fedora Update System upda...@fedoraproject.org 2011-02-09 12:24:58 EST --- libfap-1.0-3.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 669010] Review Request: libfap - C port of Ham::APRS::FAP APRS Parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=669010 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||libfap-1.0-3.el6 Resolution||ERRATA Last Closed||2011-02-09 12:25:05 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676392] Review Request: foghorn - DBus signal to SNMP trap service
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=676392 Ryan O'Hara roh...@redhat.com changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676392] New: Review Request: foghorn - DBus signal to SNMP trap service
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: foghorn - DBus signal to SNMP trap service https://bugzilla.redhat.com/show_bug.cgi?id=676392 Summary: Review Request: foghorn - DBus signal to SNMP trap service Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: roh...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://people.redhat.com/~rohara/foghorn/foghorn.spec SRPM URL: http://people.redhat.com/~rohara/foghorn/foghorn-0.1.2-1.el6.src.rpm Description: The foghorn service listens for specific dbus signals and maps those signals to SNMPv2 traps. This service is currently used by various cluster components (fenced, corosync, rgmanager) as a means to generate SNMPv2 traps for certains events. This is my first Fedora package and I will need a sponsor. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676392] Review Request: foghorn - DBus signal to SNMP trap service
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=676392 Ryan O'Hara roh...@redhat.com changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | Depends on||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 596138] Review Request: nss-gui - A graphical user interface for NSS security databases
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=596138 Kai Engert (kaie) keng...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE Last Closed||2011-02-09 13:24:56 --- Comment #23 from Kai Engert (kaie) keng...@redhat.com 2011-02-09 13:24:56 EST --- I was able to commit and build. Thanks again. Closing. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 646608] Rename review: drupal-service_links - drupal6-service_links
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=646608 --- Comment #9 from Jon Ciesla l...@jcomserv.net 2011-02-09 13:49:51 EST --- See comment on workspace BZ. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673919] Review Request: monkey - Very small, fast and scalable web server for Linux.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673919 --- Comment #2 from Guillermo Gómez guillermo.go...@gmail.com 2011-02-09 13:29:52 EST --- MUST rpmlint output - MISSING Package name - OK Spec file matches base package - OK License must be licensed with a Fedora approved license and meet the Licensing Guidelines - OK License in spec must match actual license - OK License file include in %doc - OK Spec file written in American English - OK Spec file legible - OK Tar ball matches upstream - OK Package successfully builds binary RPMs - OK (koji f13,f14, local f14 x86_64) No duplicate files - OK Macro use must be consistant - OK At the beginning of %install, each package MUST run rm -rf %{buildroot} - OK All filenames in rpm packages must be valid UTF-8 - OK Your package should contain man pages for binaries/scripts. If it doesn't, work with upstream to add them where they make sense - SHOULD /etc/rc.d/init.d/banana sysv initscript provided but not configured. Please follow http://fedoraproject.org/wiki/Packaging:SysVInitScript to include the initscript. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 665168] Review Request: nautilus-sendto-trac - Nautilus context menu for sending files to Trac
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=665168 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||nautilus-sendto-trac-0.3.1- ||2.fc14 Resolution||ERRATA Last Closed||2011-02-09 15:19:32 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 634025] Review Request: PolarSSL - Light-weight cryptographic and SSL/TLS 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=634025 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||polarssl-0.14.0-4.fc14 Resolution||ERRATA Last Closed||2011-02-09 15:20:29 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 665168] Review Request: nautilus-sendto-trac - Nautilus context menu for sending files to Trac
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=665168 --- Comment #13 from Fedora Update System upda...@fedoraproject.org 2011-02-09 15:19:27 EST --- nautilus-sendto-trac-0.3.1-2.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 669010] Review Request: libfap - C port of Ham::APRS::FAP APRS Parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=669010 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|libfap-1.0-3.el6|libfap-1.0-3.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 634025] Review Request: PolarSSL - Light-weight cryptographic and SSL/TLS 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=634025 --- Comment #15 from Fedora Update System upda...@fedoraproject.org 2011-02-09 15:20:23 EST --- polarssl-0.14.0-4.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 669010] Review Request: libfap - C port of Ham::APRS::FAP APRS Parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=669010 --- Comment #17 from Fedora Update System upda...@fedoraproject.org 2011-02-09 15:23:48 EST --- libfap-1.0-3.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676129] Review Request: qconf - Allows you to have a nice configure script for your qmake-based project
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=676129 Erik van Pienbroek erik-fed...@vanpienbroek.nl changed: What|Removed |Added Status|NEW |ASSIGNED CC||erik-fed...@vanpienbroek.nl AssignedTo|nob...@fedoraproject.org|erik-fed...@vanpienbroek.nl Flag||fedora-review? --- Comment #1 from Erik van Pienbroek erik-fed...@vanpienbroek.nl 2011-02-09 17:16:16 EST --- Package looks good. Just two small issues: - The BuildRoot tag can be dropped - Please replace %defattr(-,root,root) with %defattr(-,root,root,-) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 674180] Review Request: knights - A chess board for KDE
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674180 --- Comment #12 from Golo Fuchert packa...@golotop.de 2011-02-09 17:22:36 EST --- Julian, the package is almost ready, only the permissions for one file needs to be corrected. $ rpmlint SPECS/knights.spec SRPMS/knights-2.2.0-3.fc14.src.rpm RPMS/i686/knights-2.2.0-3.fc14.i686.rpm knights.src: W: spelling-error %description -l en_US multiplayer - multilayer, multiplexer, multiplier knights.i686: W: spelling-error %description -l en_US multiplayer - multilayer, multiplexer, multiplier knights.i686: E: script-without-shebang /usr/share/applications/kde4/knights.desktop knights.i686: W: no-manual-page-for-binary knights 2 packages and 1 specfiles checked; 1 errors, 3 warnings. Spelling errors are false positives, the .desktop file seems to have wrong permissions (755). - key: [+] OK [.] OK, not applicable [X] needs work - [+] MUST: The package must be named according to the Package Naming Guidelines. [+] MUST: The spec file name must match the base package %{name}. [+] MUST: The package must meet the Packaging Guidelines. [+] MUST: The package must be licensed with a Fedora approved license. - GPLv2 or GPLv3 (with the possibility for an extension by KDE e.V.) [+] MUST: The License field in the package spec file must match the actual license. [.] MUST: The file containing the text of the license(s) for the package must be included in %doc. [+] MUST: The spec file must be written in American English. [+] MUST: The spec file for the package MUST be legible. [+] MUST: The sources used to build the package must match the upstream source. $ md5sum knights-2.2.0.tar.bz2{,.packaged} 6196e8ef8d2e7c16e38307469708f7cf knights-2.2.0.tar.bz2 6196e8ef8d2e7c16e38307469708f7cf knights-2.2.0.tar.bz2.packaged [+] MUST: The package MUST successfully compile and build into binary rpms on at least one primary architecture. [.] MUST: If the package does not successfully compile, build or work on an architecture, ... [+] MUST: All build dependencies must be listed in BuildRequires. [+] MUST: When compiling C, C++, or Fortran files, %{optflags} must be applied. - implicitly by %cmake_kde4 [+] MUST: The spec file MUST handle locales properly. [+] MUST: If a package installs files below %{_datadir}/icons, the icon cache must be updated. [.] MUST: Packages storing shared library files (not just symlinks) must call ldconfig in %post and %postun. [+] MUST: Packages must NOT bundle copies of system libraries. [.] MUST: If the package is designed to be relocatable, ... [+] MUST: A package must own all directories that it creates. [+] MUST: A Fedora package must not list a file more than once in %files. [X] MUST: Permissions on files must be set properly. - see the rpmlint output above [+] MUST: Packages must not provide RPM dependency information when that information is not global in nature, or are otherwise handled. [.] MUST: When filtering automatically generated RPM dependency information, the filtering system implemented by Fedora must be used. [+] MUST: Each package must consistently use macros. [+] MUST: The package must contain code, or permissable content. [.] MUST: Large documentation files must go in a -doc subpackage. [+] MUST: Files in %doc must not affect the runtime of the application. [.] MUST: Header files must be in a -devel package. [.] MUST: Static libraries must be in a -static package. [.] MUST: If a package contains library files with a suffix (e.g. libfoo.so.1.1), ... [.] MUST: devel packages must require the base package using a fully versioned dependency. [+] MUST: Packages must NOT contain any .la libtool archives. [+] MUST: Packages containing GUI applications must include a %{name}.desktop file [+] MUST: .desktop files must be properly installed with desktop-file-install in the %install section. [+] MUST: Packages must not own files or directories already owned by other packages. [+] MUST: All filenames in rpm packages must be valid UTF-8. [+] SHOULD: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream... - has been/will be done by the packager. [+] SHOULD: Timestamps of files should be preserved. [+] SHOULD: The reviewer should test that the package builds in mock. [+] SHOULD: The reviewer should test that the package functions as described. - Almost, the description didn't warn the reviewer to lose against gnuchess. ;-) [+] SHOULD: If scriptlets are used, those scriptlets must be sane. [.] SHOULD: Usually, subpackages other than devel should require the base package using a fully versioned dependency. [.] SHOULD: pkgconfig(.pc) files should be placed in a -devel pkg. [.] SHOULD: If the package has file dependencies outside of /etc, /bin, /sbin, /usr/bin, or
[Bug 592487] Review Request: ffgtk - A solution for controlling Fritz!Box or compatible routers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=592487 Louis Lagendijk lo...@lagendijk.xs4all.nl changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 675914] Review Request: flush - GTK-based BitTorrent client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=675914 --- Comment #6 from Oksana okurysh...@yahoo.com 2011-02-09 17:35:49 EST --- Sorry, not everything. I forgot about the third point. I will check deps tomorrow, they aren't good now. Don't check a package before this, please.. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676335] Review Request: dmtcp - Checkpoint/Restart functionality for Linux processes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=676335 Neal Becker ndbeck...@gmail.com changed: What|Removed |Added CC||ndbeck...@gmail.com --- Comment #1 from Neal Becker ndbeck...@gmail.com 2011-02-09 19:53:28 EST --- 1. Needs patch. %build sed -i -e 's/enable_option_checking=fatal/enable_option_checking=no/'\ configure.ac autoreconf --force %configure --disable-option-checking 2. A number of 'file listed twice' warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676335] Review Request: dmtcp - Checkpoint/Restart functionality for Linux processes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=676335 --- Comment #2 from Neal Becker ndbeck...@gmail.com 2011-02-09 19:58:43 EST --- %{_libdir}/libdmtcpaware.so.* %{_libdir}/%{name}/libdmtcpaware.so.* That seems odd. Install same lib in both places? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676335] Review Request: dmtcp - Checkpoint/Restart functionality for Linux processes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=676335 --- Comment #3 from Kapil Arya ka...@ccs.neu.edu 2011-02-09 20:08:18 EST --- Thanks for review Neal! I have updated %build section in dmtcp.spec as you suggested. About the file listed twice warnings, I am not sure how to eliminate them. My guess is that this is due to the fact that we have all the binaries and helping files in %{_libdir}/dmtcp/* and the binaries %{_bindir}/dmtcp_* are actually symlinks to the real binaries in %{_libdir}/dmtcp/dmtcp_*. Is there a better way to do that? Similarly %{_libdir}/libdmtcpaware.so.* are symlinks to %{_libdir}/dmtcp/libdmtcpaware.so.* Should I keep them as is, or move these .so files from %{_libdir}/dmtcp/ to %{_libdir}/? Thanks, -Kapil -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673027] Review Request: manchu-fonts - A Manchu OpenType (TrueType-flavored) 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=673027 Peng Wu p...@redhat.com changed: What|Removed |Added CC||j...@redhat.com Flag||fedora-cvs? --- Comment #4 from Peng Wu p...@redhat.com 2011-02-09 21:51:25 EST --- New Package SCM Request === Package Name: manchu-fonts Short Description: A Manchu OpenType (TrueType-flavored) font Owners: pwu Branches: f13 f14 f15 el4 el5 el6 InitialCC: i18n-team petersen jamesni -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673029] Review Request: sil-nuosu-fonts - The Nuosu SIL 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=673029 Peng Wu p...@redhat.com changed: What|Removed |Added CC||j...@redhat.com Flag||fedora-cvs? --- Comment #4 from Peng Wu p...@redhat.com 2011-02-09 21:54:52 EST --- New Package SCM Request === Package Name: sil-nuosu-fonts Short Description: The Nuosu SIL Font Owners: pwu Branches: f13 f14 f15 el4 el5 el6 InitialCC: i18n-team petersen jamesni -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676187] Csync2 is a cluster synchronization 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=676187 --- Comment #1 from Angus Salkeld asalk...@redhat.com 2011-02-09 22:15:07 EST --- builds on ppc: http://ppc.koji.fedoraproject.org/koji/taskinfo?taskID=129364 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 637360] Review Request: ghc-parameterized-data - Parameterized data 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=637360 Lakshmi Narasimhan lakshminaras2...@gmail.com changed: What|Removed |Added Flag||fedora-review+ --- Comment #2 from Lakshmi Narasimhan lakshminaras2...@gmail.com 2011-02-09 22:25:32 EST --- [+]MUST: rpmlint must be run on every package. The output should be posted in the review. rpmlint -i ghc-parameterized-data-*.rpm ../ghc-parameterized-data.spec ghc-parameterized-data.i686: W: spelling-error Summary(en_US) Haskell - Gaskell, Gaitskell, Skellum The value of this tag appears to be misspelled. Please double-check. ghc-parameterized-data.src: W: spelling-error Summary(en_US) Haskell - Gaskell, Gaitskell, Skellum The value of this tag appears to be misspelled. Please double-check. ghc-parameterized-data-devel.i686: W: spelling-error Summary(en_US) Haskell - Gaskell, Gaitskell, Skellum The value of this tag appears to be misspelled. Please double-check. ghc-parameterized-data-prof.i686: E: devel-dependency ghc-parameterized-data-devel Your package has a dependency on a devel package but it's not a devel package itself. ghc-parameterized-data-prof.i686: W: spelling-error Summary(en_US) Haskell - Gaskell, Gaitskell, Skellum The value of this tag appears to be misspelled. Please double-check. ghc-parameterized-data-prof.i686: W: no-documentation The package contains no documentation (README, doc, etc). You have to include documentation files. ghc-parameterized-data-prof.i686: W: devel-file-in-non-devel-package /usr/lib/ghc-6.12.3/parameterized-data-0.1.5/libHSparameterized-data-0.1.5_p.a A development file (usually source code) is located in a non-devel package. If you want to include source code in your package, be sure to create a development package. 4 packages and 1 specfiles checked; 1 errors, 6 warnings. [+]MUST: The package must be named according to the Package Naming Guidelines. [+]MUST: The spec file name must match the base package %{name}, in the format %{name}.spec [+]MUST: The package must meet the Packaging Guidelines. Naming-Yes Version-release - Matches License - OK, BSD 3 clause No prebuilt external bits - OK Spec legibity - OK Package template - OK, please use cabal2spec 0.22.4. Also you can enter ghc deps using ghc_pkg_deps macro (as per the template) Arch support - OK Libexecdir - OK rpmlint - yes changelogs - OK Source url tag - OK, validated. Buildroot is ignored - present anyway. OK %clean is ignored - present anyway. OK Build Requires list - OK, Please use a ghc_pkg_deps macro to enter the deps (to adhere to the template). Summary and description - OK API documentation - OK [+]MUST: The package must be licensed with a Fedora approved license and meet the Licensing Guidelines . BSD 3 clause license [+]MUST: The License field in the package spec file must match the actual license. [+]MUST: If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package must be included in %doc. Yes LICENSE file included. [+]MUST: The spec file must be written in American English. [+]MUST: The spec file for the package MUST be legible. [+]MUST: The sources used to build the package must match the upstream source,as provided in the spec URL. Reviewers should use md5sum for this task. md5sum parameterized-data-0.1.5.tar.gz fe332c77def5865a01fe3b1af9ec1b78 parameterized-data-0.1.5.tar.gz md5sum parameterized-data-0.1.5.tar.gz fe332c77def5865a01fe3b1af9ec1b78 parameterized-data-0.1.5.tar.gz [+]MUST: The package MUST successfully compile and build into binary rpms on at least one primary architecture. Built on i686. [+]MUST: If the package does not successfully compile, build or work on an architecture, then those architectures should be listed in the spec in ExcludeArch. [+]MUST: All build dependencies must be listed in BuildRequires. [NA]MUST: The spec file MUST handle locales properly using the %find_lang macro [NA]MUST: Packages stores shared library files must call ldconfig in %post and %postun. [+]MUST: Packages must NOT bundle copies of system libraries. checked with rpmquery --list [NA]MUST: If the package is designed to be relocatable, the packager must state this fact in the request for review. [+]MUST: A package must own all directories that it creates. checked with rpmquery --whatprovides [+]MUST: A Fedora package must not list a file more than once in the spec file's %files listings. [+]MUST: Permissions on files must be set properly. checked with ls -lR [+]MUST: Each package must consistently use macros. [+]MUST: The package must contain
[Bug 676159] Review Request: crlibm - Correctly Rounded mathematical 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=676159 Dmitrij S. Kryzhevich kr...@land.ru changed: What|Removed |Added CC||kr...@land.ru --- Comment #3 from Dmitrij S. Kryzhevich kr...@land.ru 2011-02-10 00:46:41 EST --- Looking at cvs, last modified 10 month ago. Is it still alive? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 675914] Review Request: flush - GTK-based BitTorrent client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=675914 --- Comment #7 from Oksana okurysh...@yahoo.com 2011-02-10 01:09:12 EST --- New version: http://www.ossportal.ru/sites/default/files/packages/flush.spec I hope deps are ok. http://koji.fedoraproject.org/koji/taskinfo?taskID=2828919 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673026] Review Request: ukij-tuz-fonts - Uyghur Computer Science Association (UKIJ) Unicode 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=673026 --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2011-02-10 02:30:58 EST --- ukij-tuz-fonts-2.0.0-3.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/ukij-tuz-fonts-2.0.0-3.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673026] Review Request: ukij-tuz-fonts - Uyghur Computer Science Association (UKIJ) Unicode 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=673026 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676544] Review Request: Pidgin-logviewer - User friendly and intuitive way of viewing chat logs in Pidgin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=676544 Praveen Kumar kumarpraveen.nit...@gmail.com changed: What|Removed |Added CC||kumarpraveen.nitdgp@gmail.c ||om Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676544] New: Review Request: Pidgin-logviewer - User friendly and intuitive way of viewing chat logs in Pidgin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: Pidgin-logviewer - User friendly and intuitive way of viewing chat logs in Pidgin https://bugzilla.redhat.com/show_bug.cgi?id=676544 Summary: Review Request: Pidgin-logviewer - User friendly and intuitive way of viewing chat logs in Pidgin Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: kumarpraveen.nit...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://kumarpraveen.fedorapeople.org/pidgin-logviewer/pidgin-logviewer.spec SRPM URL: http://kumarpraveen.fedorapeople.org/pidgin-logviewer/pidgin-logviewer-0.2-1.fc14.src.rpm Description: This project aims to present an alternative, user-friendly and intuitive way of viewing chat logs in Pidgin. Currently the project is in an early stage, and is designed as a plugin, but it eventually aims to replace pidgin's default log viewer. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review