[Bug 225816] Merge Review: gnome-doc-utils (stalled)

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: gnome-doc-utils (stalled)


https://bugzilla.redhat.com/show_bug.cgi?id=225816


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|MODIFIED|ASSIGNED
Version|9   |rawhide




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446160] New: Review Request: file-browser-applet - File Browser Applet for the GNOME Panel

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




https://bugzilla.redhat.com/show_bug.cgi?id=446160

   Summary: Review Request: file-browser-applet - File Browser
Applet for the GNOME Panel
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: ftp://czar.eas.yorku.ca/pub/fba/file-browser-applet.spec
SRPM URL: ftp://czar.eas.yorku.ca/pub/fba/file-browser-apple
Description: File Browser Applet is an applet for the GNOME Panel. It lets you 
browse
and open files in your home directory from the gnome panel, without having to
open a file manager. It is _not_ intended as replacement to proper file
managers like nautilus. Sometimes you (like me) just want to open a file and
don't want to have to open a file manager, browse to and open the file, then
close the file manager. This Applet just saves you couple of clicks.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446159] New: Review Request: xesam-tools - A toolkit for Xesam compliant services

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




https://bugzilla.redhat.com/show_bug.cgi?id=446159

   Summary: Review Request: xesam-tools - A toolkit for Xesam
compliant services
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: ftp://czar.eas.yorku.ca/pub/xesam/xesam-tools.spec
SRPM URL: ftp://czar.eas.yorku.ca/pub/xesam/xesam-tools-0.6.1-1.fc9.src.rpm
Description: xesam-tools provides Python packages to interface with Xesam
compliant services as well as a command line tool to use directly.

It also features a framework for easily implementing xesam compliant
servers via the xesam.server package. There are two examples bundled
with this distribution. Namely demo/xesam-yahoo-service and
demo/xesam-dummy-service. Both include a small header describing
their usage.

The implementation uses gobject and the Python dbus bindings for the gobject
mainloop. There's no gui, only a command line application (xesam-tool).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 225816] Merge Review: gnome-doc-utils (stalled)

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: gnome-doc-utils (stalled)


https://bugzilla.redhat.com/show_bug.cgi?id=225816


[EMAIL PROTECTED] changed:

   What|Removed |Added

Version|rawhide |9




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446158] New: Review Request: xesam-glib - A GObject library for dealing with Xesam services

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




https://bugzilla.redhat.com/show_bug.cgi?id=446158

   Summary: Review Request: xesam-glib - A GObject library for
dealing with Xesam services
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: ftp://czar.eas.yorku.ca/pub/xesam/xesam-glib.spec
SRPM URL: ftp://czar.eas.yorku.ca/pub/xesam/xesam-glib-0.2.1-1.fc9.src.rpm
Description: Xesam-GLib is a GObject library for dealing with Xesam services.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438543] Review Request: Synopsis - Source-code introspection tool.

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: Synopsis - Source-code introspection tool.


https://bugzilla.redhat.com/show_bug.cgi?id=438543





--- Additional Comments From [EMAIL PROTECTED]  2008-05-13 00:39 EST ---
Here is a new snapshot. The most important changes are code cleanup to remove
old XEROX code, and fix the Python parser license header:

http://synopsis.fresco.org/download/srpm/synopsis-2008-05-13-1.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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445018] Review Request: python-beaker - WSGI middleware for sessions

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: python-beaker -  WSGI middleware for sessions


https://bugzilla.redhat.com/show_bug.cgi?id=445018





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 21:52 EST ---
Thanks, Paul.  Updated again:

Spec URL: http://www.kylev.com/pylons-rpm/python-beaker.spec
SRPM URL: http://www.kylev.com/pylons-rpm/python-beaker-0.9.4-4.fc8.src.rpm

Man, I'm sloppy this time 'round!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 171993] Review Request: mpich2 - An implementation of MPI

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: mpich2 -  An implementation of MPI


https://bugzilla.redhat.com/show_bug.cgi?id=171993





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 20:30 EST ---
Hi Orion and all,

Package now builds in koji 
(http://koji.fedoraproject.org/koji/taskinfo?taskID=605687).
Anything else left to pass review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 442693] Review Request: geoclue - Geoclue is a modular geoinformation service

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: geoclue - Geoclue is a modular geoinformation service


https://bugzilla.redhat.com/show_bug.cgi?id=442693





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 19:22 EST ---
I have recently become a sponsor, so I can now sponsor you.

The canonical information about this is 
http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored

You'll have to set up a Fedora Account first. For that, see 

http://fedoraproject.org/wiki/PackageMaintainers/Join#head-a601c13b0950a89568deafa65f505b4b58ee869b

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 435299] Review Request: ocaml-pa-monad - OCaml syntax extension for monads

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-pa-monad - OCaml syntax extension for monads


https://bugzilla.redhat.com/show_bug.cgi?id=435299





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 18:53 EST ---
I don't really care one way or the other about the naming; I agree that an
all-dash scheme looks cleaner but if other distros choose the other method then
there could be some confusion.  If that happens, a simple solution would be for
this package to provide ocaml-pa_monad.

Does the lack of the linking exception cause any problem for actually using this
package?  I don't know if syntax extensions actually get linked into the object
code.

I'd say this package is fine and I'd approve it, but I'd like to be clear on the
license issue first.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445018] Review Request: python-beaker - WSGI middleware for sessions

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: python-beaker -  WSGI middleware for sessions


https://bugzilla.redhat.com/show_bug.cgi?id=445018





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 18:43 EST ---
You need %{python_sitelib}/beaker/ rather than %{python_sitelib}/beaker/* in the
%files list so that the directory %{python_sitelib}/beaker/ (as well as its
contents) is owned by 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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 435287] Review Request: ocaml-json-static - OCaml JSON validator and converter (syntax extension)

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-json-static - OCaml JSON validator and converter 
(syntax extension)


https://bugzilla.redhat.com/show_bug.cgi?id=435287


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 18:35 EST ---
OK, this looks fine.  Perhaps we can tweak the guidelines to talk a bit about
detecting syntax extensions.

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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 427824] Review Request: tmispell-voikko - An Ispell compatible front-end for spell-checking modules

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tmispell-voikko - An Ispell compatible front-end for 
spell-checking modules


https://bugzilla.redhat.com/show_bug.cgi?id=427824





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 18:30 EST ---
Static libraries in -devel are deprecated, they should be either dropped or 
shipped in a -static package according to the current guidelines.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 444625] Review Request: GMT-doc - Documentation for Generic Mapping Tools

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: GMT-doc - Documentation for Generic Mapping Tools


https://bugzilla.redhat.com/show_bug.cgi?id=444625


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 18:24 EST ---
New Package CVS Request
===
Package Name: GMT-doc
Short Description: Documentation for Generic Mapping Tools
Owners: orion
Branches: F-9 F-8 F-7 EL-5 EL-4
InitialCC: 
Cvsextras Commits: yes


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445018] Review Request: python-beaker - WSGI middleware for sessions

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: python-beaker -  WSGI middleware for sessions


https://bugzilla.redhat.com/show_bug.cgi?id=445018





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 18:21 EST ---
Upstream responded, it's going all BSD:
  http://beaker.groovie.org/beaker/changeset/141%3Abd2a1de25888/setup.py

Spec file updated to reflect that, with patch to make it look correct (he forgot
to update the classifiers, but he will):

Spec URL: http://www.kylev.com/pylons-rpm/python-beaker.spec
SRPM URL: http://www.kylev.com/pylons-rpm/python-beaker-0.9.4-3.fc8.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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446134] Review Request: jsr-305 - reference implementation of JSR-305

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: jsr-305 - reference implementation of JSR-305


https://bugzilla.redhat.com/show_bug.cgi?id=446134


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: jsr-305 -   |Review Request: jsr-305 -
   |reference implementation of |reference implementation of
   |JSR-305 |JSR-305




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 18:22 EST ---
I forgot to note that I asked upstream to include a license file:
http://groups.google.com/group/jsr-305/msg/959fee95941a8743.  I made this
request on 18 Apr 2008, with no response so far.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445604] Review Request: Tennix! - A funny 2D tennis game

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: Tennix! - A funny 2D tennis game


https://bugzilla.redhat.com/show_bug.cgi?id=445604





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 18:21 EST ---
Hi Paul,

- substituted "install -s" with "install" in the makefile (I have really a lot
to learn about makefiles)
- used 'CFLAGS="%{optflags}" make' to build the binaries
- added desktop-file-utils as build requirement (oops! forgot it!)

New SPEC file and SRPM are available at
Spec URL: http://www.claudiotomasoni.it/files/RPMS/tennix.spec
SRPM URL: http://www.claudiotomasoni.it/files/RPMS/tennix-0.6.1-2.fc8.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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 439100] Review Request: octaviz - 3D visualization system for Octave

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: octaviz - 3D visualization system for Octave


https://bugzilla.redhat.com/show_bug.cgi?id=439100





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 18:25 EST ---
Many thanks for the review and for your helpfulness!
I will proceed as soon as I'm sponsored (I hope soon...)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446134] New: Review Request: jsr-305 - reference implementation of JSR-305

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




https://bugzilla.redhat.com/show_bug.cgi?id=446134

   Summary: Review Request: jsr-305 - reference implementation of
JSR-305
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://jjames.fedorapeople.org/jsr-305/jsr-305.spec
SRPM URL: 
http://jjames.fedorapeople.org/jsr-305/jsr-305-0-0.1.20071105svn.src.rpm
Description: This package contains reference implementations, test cases, and 
other documents for Java Specification Request 305: Annotations for Software 
Defect Detection.

This is a prerequisite for findbugs.  There has been no official release, so it 
is a snapshot of the Subversion repository, whose last checkin occurred on 5 
Nov 2007.  The license terms do not appear in any file in the repository, so I 
have included a downloaded copy of http://code.google.com/p/jsr-305/index.html, 
which claims that the project is available under the "New BSD License", which 
is the "BSD" license tag for Fedora.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 444625] Review Request: GMT-doc - Documentation for Generic Mapping Tools

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: GMT-doc - Documentation for Generic Mapping Tools


https://bugzilla.redhat.com/show_bug.cgi?id=444625


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 18:14 EST ---
* rpmlint not problematic
GMT-doc.noarch: E: zero-length /usr/share/doc/GMT-4.3.0/html/GMT_Docs/img246.png
* free software, license included
* follow guidelines
* match upstream
e068d5be0859c1ea5a8a0b15d4ad5810  GMT4.3.0_pdf.tar.bz2
633f4e648c80d920c4c2080b58fc1fe8  GMT4.3.0_tut.tar.bz2
3fcef23a62b61d4d6abcb8b5b1a4a701  GMT4.3.0_web.tar.bz2
* %files section right

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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438750] Review Request: qtoctave - fronted for octave written using qt4 widgets

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: qtoctave - fronted for octave written using qt4 widgets


https://bugzilla.redhat.com/show_bug.cgi?id=438750





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 18:12 EST ---
Hi Mamoru,

about last issues:
- removed .fc8 from Changelog
- I took the icon from an icon set downloaded from internet a lot of time ago. I
guess it was freely usable, but since I can't find the original (and thus read
the license), I'm not 100% sure. So, I have drawn a little new icon (a Gauss
curve, made in qtoctave, of course) which is released under GPL and included in
the new revision.

Spec URL: http://www.claudiotomasoni.it/files/RPMS/qtoctave.spec
SRPM URL: http://www.claudiotomasoni.it/files/RPMS/qtoctave-0.7.4-2.fc8.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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217259] Review Request: alsa-firmware - Firmware for several ALSA-supported sound card

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: alsa-firmware - Firmware for several ALSA-supported 
sound card


https://bugzilla.redhat.com/show_bug.cgi?id=217259





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 18:01 EST ---
OK, in what must rate as one of the most unrewarding bits of work on packaging
ever, I have been through the package licensing in exhaustive detail at a source
level (the licenses even vary between firmware from the same manufacturer for
minor hardware revisions!) and offer the following rather clumsy but unambiguous
revision:

Spec URL: http://timj.fedorapeople.org/SPECS/alsa-firmware.spec
SRPM URL: http://timj.fedorapeople.org/SRPMS/alsa-firmware-1.0.16-1.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445980] Review Request: odpdom - Oversized Document Parser

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: odpdom - Oversized Document Parser


https://bugzilla.redhat.com/show_bug.cgi?id=445980





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 17:53 EST ---
Well, it certainly doesn't hurt to discuss things with the upstream developers.
 Rathann, do you know their opinions on the topic?

However, please keep in mind that we never know what upstream may do.  Any
upstream developer of any library in the distro could do what Patrice describes
in his wiki page at any time anyway, so we have to be prepared to deal with that
with rebuilds if necessary.  We can cover ourselves as well as possible by
picking the lowest possible version.

Besides, Patrice's argument has been used to prevent introduction of shared
libraries for packages which really could use them (and give us security
exposure because of it; see libnet, for example).  I stand firmly against it. 
We have to be able to make a reasonable distro even in the face of upstreams
which do dumb 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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438892] Review Request: pyPdf - PDF toolkit

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: pyPdf - PDF toolkit


https://bugzilla.redhat.com/show_bug.cgi?id=438892


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 17:30 EST ---
Package Change Request
==
Package Name: pyPdf
New Branches: EL-4 EL-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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438105] Review Request: libconcord - L ibrary to talk to Logitech® Harmony® univ ersal remote controls

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libconcord - Library to talk to Logitech® Harmony® 
universal remote controls


https://bugzilla.redhat.com/show_bug.cgi?id=438105





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 17:24 EST ---
concordance-0.20-4.fc8,libconcord-0.20-5.fc8 has been submitted as an update 
for Fedora 8

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438105] Review Request: libconcord - L ibrary to talk to Logitech® Harmony® univ ersal remote controls

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libconcord - Library to talk to Logitech® Harmony® 
universal remote controls


https://bugzilla.redhat.com/show_bug.cgi?id=438105





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 17:25 EST ---
concordance-0.20-4.fc9,libconcord-0.20-5.fc9 has been submitted as an update 
for Fedora 9

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 328161] Review Request: concordance - Software to program the Logitech Harmony remote control

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: concordance - Software to program the Logitech Harmony 
remote control


https://bugzilla.redhat.com/show_bug.cgi?id=328161





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 17:24 EST ---
concordance-0.20-4.fc8,libconcord-0.20-5.fc8 has been submitted as an update 
for Fedora 8

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 328161] Review Request: concordance - Software to program the Logitech Harmony remote control

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: concordance - Software to program the Logitech Harmony 
remote control


https://bugzilla.redhat.com/show_bug.cgi?id=328161





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 17:25 EST ---
concordance-0.20-4.fc9,libconcord-0.20-5.fc9 has been submitted as an update 
for Fedora 9

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438043] Review Request: GMT - Generic Mapping Tools

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: GMT - Generic Mapping Tools


https://bugzilla.redhat.com/show_bug.cgi?id=438043


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 17:15 EST ---
New Package CVS Request
===
Package Name: GMT
Short Description: Generic Mapping Tools
Owners: orion
Branches: F-9 F-8 F-7 EL-5 EL-4
InitialCC: 
Cvsextras Commits: yes


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 441086] Review Request: ezstream - Command line source client for Icecast media streaming servers

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ezstream - Command line source client for Icecast 
media streaming servers


https://bugzilla.redhat.com/show_bug.cgi?id=441086


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 444733] Review Request: perl-HTTP-Cache-Transparent - Cache the result of http get-requests persistently

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: perl-HTTP-Cache-Transparent - Cache the result of http 
get-requests persistently


https://bugzilla.redhat.com/show_bug.cgi?id=444733


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 17:02 EST ---
New Package CVS Request
===
Package Name: perl-HTTP-Cache-Transparent
Short Description: Cache the result of http get-requests persistently
Owners: kwizart
Branches: F-8 F-9 EL-4 EL-5
InitialCC: perl-sig
Cvsextras Commits: yes

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 433422] Review Request: perl-IP-Country - Fast lookup of country codes from IP addresses

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: perl-IP-Country - Fast lookup of country codes from IP 
addresses


https://bugzilla.redhat.com/show_bug.cgi?id=433422





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 16:59 EST ---
Red Hat Legal agrees with me that the database licenses are non-free. I think
this one is dead in the water, sorry.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438043] Review Request: GMT - Generic Mapping Tools

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: GMT - Generic Mapping Tools


https://bugzilla.redhat.com/show_bug.cgi?id=438043


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 16:51 EST ---
* free software, license included
* follow guidelines
* rpmlint warnings are not problematic:
GMT.i386: W: hidden-file-or-dir /etc/GMT/conf/.gmtdefaults_US
GMT.i386: W: hidden-file-or-dir /usr/share/GMT/conf/.gmtdefaults_US
GMT.i386: W: hidden-file-or-dir /usr/share/GMT/conf/.gmtdefaults_SI
GMT.i386: W: hidden-file-or-dir /etc/GMT/conf/.gmtdefaults_SI
GMT-octave.i386: W: no-documentation
GMT-static.i386: W: no-documentation
* match upstream
41a3c25bf74a151e423f6821dcc2fa25  GMT4.3.0_scripts.tar.bz2
a7ce8c0fa2bd85131dfc8b67d8fe8235  GMT4.3.0_share.tar.bz2
2890ee76b44cc32ee41d54eda425b212  GMT4.3.0_src.tar.bz2
362438041a80570db1842524370c76e1  GMT4.3.0_suppl.tar.bz2
* %files section right

APPROVED

The fact that upstream used a soname with the same number than the
release is a bit frightening, and I think you'll have to be very
careful to check what upstream does.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 266841] Review Request: php-pear-Propel - An Object Relational Mapping (ORM) framework for PHP5

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: php-pear-Propel - An Object Relational Mapping (ORM) 
framework for PHP5


https://bugzilla.redhat.com/show_bug.cgi?id=266841





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 16:46 EST ---
We've updated our license to LGPLv3.  This is currently reflected on our website
and will be reflected in the next release.  To clarify for the sake of the
thread, while FSF doesn't believe the Apache license is compatible w/ LGPLv2,
this is not the belief that has been held by Apache.  It is true that everyone
agrees that version 3 of the LGPL is fine.  We've never explicitly specified the
version of the LGPL that covered the project; however, we did include 2.x, as
that was the current version at the time (and then we got lazy about keeping 
up). 

The Propel developers decided that the simplest way to handle this solution was
to update the license we were distributing and referencing to the LGPLv3.  That
way everyone is happy.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445018] Review Request: python-beaker - WSGI middleware for sessions

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: python-beaker -  WSGI middleware for sessions


https://bugzilla.redhat.com/show_bug.cgi?id=445018





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 16:34 EST ---
We can change the files section to do this:

%{python_sitelib}/beaker/*
%{python_sitelib}/Beaker*

You have to do this to get the egg info for this type of schizophrenic "I'm
'Beaker' and provide 'beaker'" mixed case packages.  Updated:

Spec URL: http://www.kylev.com/pylons-rpm/python-beaker.spec
SRPM URL: http://www.kylev.com/pylons-rpm/python-beaker-0.9.4-2.fc8.src.rpm

I don't think the license issue is a blocker (the variants are all Fedora-safe).
 I just used the one declared in setup.py.  I've written to upstream to see if
they can clarify it in the future.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 441086] Review Request: ezstream - Command line source client for Icecast media streaming servers

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ezstream - Command line source client for Icecast 
media streaming servers


https://bugzilla.redhat.com/show_bug.cgi?id=441086





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 16:21 EST ---
Can we close this now?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 441343] Review Request: guilt - Scripts to manage quilt-like patches on top of git

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: guilt - Scripts to manage quilt-like patches on top of 
git


https://bugzilla.redhat.com/show_bug.cgi?id=441343





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 16:20 EST ---
Can we close this now?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 266841] Review Request: php-pear-Propel - An Object Relational Mapping (ORM) framework for PHP5

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: php-pear-Propel - An Object Relational Mapping (ORM) 
framework for PHP5


https://bugzilla.redhat.com/show_bug.cgi?id=266841





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 16:16 EST ---
(In reply to comment #17)

> We would like to resolve this problem (and respect the license of source 
> work!);
> I am certain that getting the contributors to agree to a license change to a
> Apache-based license (e.g. PHP license?) will not be a problem.

Indeed, this seems like the best resolution to this issue.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438452] Review Request: java-gnome: Java GNOME bindings

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: java-gnome: Java GNOME bindings


https://bugzilla.redhat.com/show_bug.cgi?id=438452


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 16:16 EST ---
Doesn't frysk need the 2.x (API-incompatible AFAIK) bindings?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 439630] Review Request: jogl - Java bindings for OpenGL

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: jogl - Java bindings for OpenGL


https://bugzilla.redhat.com/show_bug.cgi?id=439630





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 16:11 EST ---
lwjgl is no-good for Fedora, native/common/extgl.h is SGI Free B.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445074] Review Request: ocaml-core - Many and various enhancements to the OCaml standard library

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-core - Many and various enhancements to the 
OCaml standard library


https://bugzilla.redhat.com/show_bug.cgi?id=445074


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|NEW
   Flag|needinfo?([EMAIL PROTECTED])|




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 16:10 EST ---
Spec URL: http://www.annexia.org/tmp/ocaml/ocaml-janest-core.spec
SRPM URL: http://www.annexia.org/tmp/ocaml/ocaml-janest-core-0.5.0-2.fc9.src.rpm

* Mon May  5 2008 Richard W.M. Jones <[EMAIL PROTECTED]> - 0.5.0-3
- Rename the package to ocaml-janest-core as per agreement with Debian.



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 434614] Review Request: cdcat - Hyper's CdCatalog

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: cdcat - Hyper's CdCatalog


https://bugzilla.redhat.com/show_bug.cgi?id=434614


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|182235  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 16:07 EST ---
This is fine, there is no encode/decode or playback here, just parsing the
format and id3 tag. Lifting FE-Legal.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 439630] Review Request: jogl - Java bindings for OpenGL

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: jogl - Java bindings for OpenGL


https://bugzilla.redhat.com/show_bug.cgi?id=439630





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 16:03 EST ---
As a possible alternative, someone should investigate:
http://lwjgl.org/



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445018] Review Request: python-beaker - WSGI middleware for sessions

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: python-beaker -  WSGI middleware for sessions


https://bugzilla.redhat.com/show_bug.cgi?id=445018





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 16:04 EST ---
Review:

MUST items
- no rpmlint output for srpm
- package naming ok for python module
- spec file name matches package name
- packaging guidelines are met
- license tag in spec file is 'MIT', upstream names it 'MIT' in setup.py but
  actual license text included in the package is 3 clause BSD (without 
  advertising).

=> I'm not quite what we should do here. At least the license tag in the spec
   file must be updated (3 clause BSD is acceptable for Fedora) but to be sure
   we should ask upstream under what license the code is placed.

Furthermore, there are some file in the package which do use a form of MIT 
license:
* beaker/converters.py uses MIT / "Modern Style with sublicense", no license 
text
  only referenced URL
* beaker/crypto/pbkdf2.py uses MIT / "Old Style with legal disclaimer 3"
  (variant names taken from http://fedoraproject.org/wiki/Licensing/MIT)

What should we do with them? The most part of the code is BSD(?, see above). I
think RPM does only accept a single license tag.

- license text (BSD) is separate file named LICENSE, included in %doc
- spec file written in English and is legible
- sources match upstream (md5sum 9e474576e948d7f80ce238d31c80ade3)
- package builds in mock on x86_64 for Fedora 8
- buildreqs OK
- no shared libraries, static libraries, header files, pkgconfig files, or
  locale files
- package doesn't claim to be relocatable
- directory ownership OK
- no duplicate files
- %defattr(...) present and correct in %files section
- %clean section present and correct
- %install section properly cleans buildroot first
- macro usage is ok
- package does only contain code
- runtime does not depend on docs
- no GUI, therefore no .desktop file
- filenames are all ascii

SHOULD items:
- builds in mock
- package is a library but basic functional test passed
- no file dependencies


Paul Howarth suggest for my own python package that I use a more specific file
list ("%{python_sitelib}/beaker*", ...) which seems to be a good idea for your 
package too:
"this helps catch future changes that create extra files in the package, which
you might want to document further in the changelog etc."



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
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

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-bin-prot - Read and write OCaml values in a 
type-safe binary protocol


https://bugzilla.redhat.com/show_bug.cgi?id=445068





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 16:01 EST ---
My phony explanation for 'W: devel-file-in-non-devel-package' in the
previous comment is completely wrong!  This file _is_ allowed, but is in
the wrong subpackage.

The one is rpmlint clean.

Spec URL: http://www.annexia.org/tmp/ocaml/ocaml-bin-prot.spec
SRPM URL: http://www.annexia.org/tmp/ocaml/ocaml-bin-prot-1.0.5-2.fc9.src.rpm

* Mon May 12 2008 Richard W.M. Jones <[EMAIL PROTECTED]> - 1.0.5-2
- Remove ExclusiveArch, but add a Fedora README file warning about
  shortcomings on non-x86 architectures.
- Added missing dependency ocaml-type-conv.
- Move *.ml file to devel 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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438043] Review Request: GMT - Generic Mapping Tools

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: GMT - Generic Mapping Tools


https://bugzilla.redhat.com/show_bug.cgi?id=438043





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:55 EST ---
(In reply to comment #10)
> ldconfig call should be in %post, not %pre.

Gah, fixed.

> There is a \ missing on the find $RPM_BUILD_ROOT/%{gmthome}/examples -name
> \*.\*sh | line

Not needed - the | works.

> Also you could consider removing the .bat files in examples, rpmlint
> says:
> GMT-examples.i386: W: wrong-file-end-of-line-encoding
> /usr/share/GMT/examples/ex03/job03.bat

Removed.

> There are .in files remaining in 
> /usr/share/GMT/conf/.gmtdefaults_SI.in
> /usr/share/GMT/conf/.gmtdefaults_US.in
> /usr/share/GMT/conf/gmt.conf.in
> I think they should be removed. You could also consider using the 
> timestamps of the .in files to have consistent timestamps across arches
> for the files in /etc/GMT/

Removed, but I really don't see the need to fix timestamps.

> Also maybe the .gmtdefaults_SI and .gmtdefaults_US could be considered 
> as documentation and linked like other config files in /etc/GMT and as
> %config.

Done.

> Another issue revealed by rpmlint is 
> GMT-examples.i386: W: doc-file-dependency 
> /usr/share/GMT/examples/ex26/job26.csh
> /bin/csh
> Maybe the .csh files could be made non executable, and only the 
> sh files would be executable?

Done.

> There is also
> GMT-octave.i386: E: script-without-shebang
> /usr/share/octave/site/api-v32/m/grdinfo.m
> GMT-octave.i386: E: script-without-shebang
> /usr/share/octave/site/api-v32/m/grdwrite.m
> GMT-octave.i386: E: script-without-shebang
> /usr/share/octave/site/api-v32/m/grdread.m
> because these files have the execute bit set. You should correct it
> if it is wrong.

Fixed.

> A last remaining issue is the soname issue. Any comment on that?

Upstream now ships with sonames.

* Mon May 12 2008 Orion Poplawski <[EMAIL PROTECTED]> 4.3.0-2
- Add patch to link libraries properly
- Run ldconfig in %%post, dummy
- Don't ship .bat file
- Don't ship .in files
- Don't make .csh examples executable
- Drop execute bit on .m files

Spec URL: http://www.cora.nwra.com/~orion/fedora/GMT.spec
SRPM URL: http://www.cora.nwra.com/~orion/fedora/GMT-4.3.0-2.fc9.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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445074] Review Request: ocaml-core - Many and various enhancements to the OCaml standard library

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-core - Many and various enhancements to the 
OCaml standard library


https://bugzilla.redhat.com/show_bug.cgi?id=445074


Bug 445074 depends on bug 445070, which changed state.

Bug 445070 Summary: Review Request: ocaml-res - OCaml library for resizing 
arrays and strings
https://bugzilla.redhat.com/show_bug.cgi?id=445070

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445070] Review Request: ocaml-res - OCaml library for resizing arrays and strings

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-res - OCaml library for resizing arrays and 
strings


https://bugzilla.redhat.com/show_bug.cgi?id=445070


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:49 EST ---
Built in F-8/F-9/devel.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 439627] Review Request: gluegen - Java/JNI glue code generator to call out to ANSI C

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gluegen - Java/JNI glue code generator to call out to 
ANSI C


https://bugzilla.redhat.com/show_bug.cgi?id=439627





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:50 EST ---
Ahh, okay. 

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 434707] Review Request: ocaml-sexplib - OCaml library for converting OCaml values to S-expressions

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-sexplib - OCaml library for converting OCaml 
values to S-expressions


https://bugzilla.redhat.com/show_bug.cgi?id=434707


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:46 EST ---
Build fails on F-8/9 because we're still waiting for ocaml-type-conv >= 1.5.0 to
migrate into stable.

Built in devel.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445074] Review Request: ocaml-core - Many and various enhancements to the OCaml standard library

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-core - Many and various enhancements to the 
OCaml standard library


https://bugzilla.redhat.com/show_bug.cgi?id=445074


Bug 445074 depends on bug 434707, which changed state.

Bug 434707 Summary: Review Request: ocaml-sexplib - OCaml library for 
converting OCaml values to S-expressions
https://bugzilla.redhat.com/show_bug.cgi?id=434707

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
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

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-bin-prot - Read and write OCaml values in a 
type-safe binary protocol


https://bugzilla.redhat.com/show_bug.cgi?id=445068


Bug 445068 depends on bug 445067, which changed state.

Bug 445067 Summary: Review Request: ocaml-ounit - Unit test framework for OCaml
https://bugzilla.redhat.com/show_bug.cgi?id=445067

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 228544] Review Request: keepalived - HA monitor built upon LVS, VRRP and service pollers

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: keepalived - HA monitor built upon LVS, VRRP and 
service pollers


https://bugzilla.redhat.com/show_bug.cgi?id=228544


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Priority|normal  |medium
Product|Fedora Extras   |Fedora
Version|devel   |rawhide




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:41 EST ---
I see an EL5 branch was requested, but I don't see packages built... From what I
understand, there may be some kernel-level stuff required. Can anyone shed some
light on what we need to do to get this built for EL5?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445074] Review Request: ocaml-core - Many and various enhancements to the OCaml standard library

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-core - Many and various enhancements to the 
OCaml standard library


https://bugzilla.redhat.com/show_bug.cgi?id=445074


Bug 445074 depends on bug 445067, which changed state.

Bug 445067 Summary: Review Request: ocaml-ounit - Unit test framework for OCaml
https://bugzilla.redhat.com/show_bug.cgi?id=445067

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445067] Review Request: ocaml-ounit - Unit test framework for OCaml

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-ounit - Unit test framework for OCaml


https://bugzilla.redhat.com/show_bug.cgi?id=445067


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:45 EST ---
Built for F-8/F-9/devel.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 275491] Review Request: ocaml-pgocaml - OCaml library for type-safe access to PostgreSQL databases

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-pgocaml - OCaml library for type-safe access to 
PostgreSQL databases


https://bugzilla.redhat.com/show_bug.cgi?id=275491


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:41 EST ---
Built for F-8, F-9, devel.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 439627] Review Request: gluegen - Java/JNI glue code generator to call out to ANSI C

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gluegen - Java/JNI glue code generator to call out to 
ANSI C


https://bugzilla.redhat.com/show_bug.cgi?id=439627


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
   Keywords||Reopened
 Resolution|CANTFIX |




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:37 EST ---
I believe it's the other way around - jogl depends on gluegen.  So this package
should potentially be able to go in.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445105] Review Request: milkytracker - SDL clone of the Fasttracker II module tracker

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: milkytracker - SDL clone of the Fasttracker II module 
tracker


https://bugzilla.redhat.com/show_bug.cgi?id=445105





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:36 EST ---
I've also been notified that the source tarball includes local copies of zlib
and zziplib in the src/compression directory. The author of the software tells
he has modified the libraries to compile cleanly with C++ compilers.

The modifications done to the local zlib aren't very significant, and I could
link against the Fedora-supplied zlib by modifying two Makefile.am files
slightly. The zziplib copy seems more complicated.

I'm now taking care of the issues pointed by Mamoru Tasaka, and also trying to
figure out what to do to the zziplib copy.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 439630] Review Request: jogl - Java bindings for OpenGL

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: jogl - Java bindings for OpenGL


https://bugzilla.redhat.com/show_bug.cgi?id=439630


Bug 439630 depends on bug 439627, which changed state.

Bug 439627 Summary: Review Request: gluegen - Java/JNI glue code generator to 
call out to ANSI C
https://bugzilla.redhat.com/show_bug.cgi?id=439627

   What|Old Value   |New Value

 Status|CLOSED  |ASSIGNED
 Resolution|CANTFIX |



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 439627] Review Request: gluegen - Java/JNI glue code generator to call out to ANSI C

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gluegen - Java/JNI glue code generator to call out to 
ANSI C


https://bugzilla.redhat.com/show_bug.cgi?id=439627


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||CANTFIX




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:32 EST ---
Since this depends on jogl, which is blocked by FE-Legal, I'm closing this
CANTFIX. You may want to take both of them over to Livna/RPMFusion non-free.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 439630] Review Request: jogl - Java bindings for OpenGL

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: jogl - Java bindings for OpenGL


https://bugzilla.redhat.com/show_bug.cgi?id=439630


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||CANTFIX
   Flag||fedora-review-




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:30 EST ---
Closing as CANTFIX. Sorry guys.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 439630] Review Request: jogl - Java bindings for OpenGL

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: jogl - Java bindings for OpenGL


https://bugzilla.redhat.com/show_bug.cgi?id=439630


Bug 439630 depends on bug 439627, which changed state.

Bug 439627 Summary: Review Request: gluegen - Java/JNI glue code generator to 
call out to ANSI C
https://bugzilla.redhat.com/show_bug.cgi?id=439627

   What|Old Value   |New Value

 Status|NEW |CLOSED
 Resolution||CANTFIX



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 426867] Review Request: scala - Hybrid functional/object-oriented language

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: scala - Hybrid functional/object-oriented language


https://bugzilla.redhat.com/show_bug.cgi?id=426867


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|182235  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:28 EST ---
Bootstrapping using prebuilt binaries is ok for the first pass, but only the
first pass.

Go ahead and change the license tag, the intent is clear from upstream. You may
want to leave a comment in there explaining the issue and linking to that URL
until they do a new release.

Lifting FE-Legal.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217259] Review Request: alsa-firmware - Firmware for several ALSA-supported sound card

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: alsa-firmware - Firmware for several ALSA-supported 
sound card


https://bugzilla.redhat.com/show_bug.cgi?id=217259





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:23 EST ---
(In reply to comment #23)
> With some clarifications made in recent releases, I think if we exclude the
> emi_26_62 firmware we will probably be OK. See also
> http://mailman.alsa-project.org/pipermail/alsa-devel/2008-January/005702.html

I think so too, but I'm waiting to see a new package to do the final sign-off.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 427824] Review Request: tmispell-voikko - An Ispell compatible front-end for spell-checking modules

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tmispell-voikko - An Ispell compatible front-end for 
spell-checking modules


https://bugzilla.redhat.com/show_bug.cgi?id=427824





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:17 EST ---
I've been under the impression that *.a files go into the -devel packages and
*.la files should not be packaged at all. Also, the guidelines explicitly say
nothing about *.a files, only about *.la files.

In my F8 system, the command "locate *.a|grep ^/usr/lib|wc -l" shows that I have
at least 109 *.a files in the libdir. Piping the grep result to "xargs rpm -qf"
shows, that most of these files (at least 85) indeed come from -devel packages.

As far as I understand, the *.a files can be used to statically link the library
when compiling something under Fedora. I'm not sure if it's actually that useful
in this case, but I added it for consistency, as the other Enchant providers
also have the *.a files shipped in a -devel package.

This is definitely not a blocker for me, if you want me to remove the *.a file,
I will.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 188542] Review Request: hylafax

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: hylafax


https://bugzilla.redhat.com/show_bug.cgi?id=188542





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:12 EST ---
(In reply to comment #81)

> I don't believe we should package /etc/hylafax/faxcover_example_sgi.ps.  It
contains the old SGI name and logo and I'm pretty certain we shouldn't be
sticking it in /etc whenever someone installs this software.  I'm not even sure
we have the legal right to distribute it, which is reason one for blocking 
FE-Legal.

Indeed, this falls a little outside the realm of fair use. Please get rid of it.

> I'm going to have to get expert assistance with the License: tag; the license
given in the COPYRIGHT file is actually identical to the libtiff license, which
gets its own "libtiff" license tag, but the regex code is clearly the bad
"BSD+Advertising" clause which is GPL-incompatible and thus causes issues. 
Reason two that I'm blocking FE-Legal for guidance.

There's no incompatibility there, but please use: License: libtiff and BSD with
advertising.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 431161] Review Request: mathmap - A gimp plugin and commandline tool

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: mathmap - A gimp plugin and commandline tool


https://bugzilla.redhat.com/show_bug.cgi?id=431161





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:11 EST ---
Ok, I think this new spec and SRPM fix both the optflags problem and the locale
directory problem:

http://people.redhat.com/rnorwood/rpms/mathmap.spec
http://people.redhat.com/rnorwood/rpms/mathmap-1.2.4-3.fc9.src.rpm

Koji build against dist-f10:

http://koji.fedoraproject.org/koji/taskinfo?taskID=605186

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 225617] Merge Review: bitmap-fonts

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: bitmap-fonts


https://bugzilla.redhat.com/show_bug.cgi?id=225617


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|182235  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:04 EST ---
Lucida fonts are acceptable. If we modify the fonts in any way, we'll need to
stop using the Lucida trademarks (similar to Bitstream Vera).

The fixfonts appear to have come from KDE3's "konsole" application, and are
licensed as GPLv2 (http://websvn.kde.org/tags/KDE/3.5.9/kdebase/konsole/).

Use License: GPLv2 and Lucida

Lifting FE-Legal.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446102] Review Request: xdialog - X11 drop in replacement for cdialog

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xdialog -  X11 drop in replacement for cdialog


https://bugzilla.redhat.com/show_bug.cgi?id=446102


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: xdialog -   |Review Request: xdialog -
   |X11 drop in replacement for |X11 drop in replacement for
   |cdialog |cdialog




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 14:59 EST ---
I have set the gtk2 Xdialog to be the default one, even though 
the gtk1 is said to be more stable because the bug reported 
is with non UTF8 locales which should be rare in fedora.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445980] Review Request: odpdom - Oversized Document Parser

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: odpdom - Oversized Document Parser


https://bugzilla.redhat.com/show_bug.cgi?id=445980





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 15:00 EST ---
What is the problem is as written in Patrice's wiki page:
- This package provides libODP.so.0 now
- Upstream comes to provide libODP.so.0 (the same name) in the future 
  with no ABI compatibility with what this (Fedora specific) package
  provides "now".

So IMO we must discuss with upstream developer first if we want
to provide shared library for Fedora.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446102] New: Review Request: xdialog - X11 drop in replacement for cdialog

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




https://bugzilla.redhat.com/show_bug.cgi?id=446102

   Summary: Review Request: xdialog -  X11 drop in replacement for
cdialog
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.environnement.ens.fr/perso/dumas/fc-srpms/xdialog.spec
SRPM URL: 
http://www.environnement.ens.fr/perso/dumas/fc-srpms/xdialog-2.3.1-1.fc9.src.rpm
Description: 

Xdialog is designed to be a drop in replacement for the cdialog program.
It converts any terminal based program into a program with an X-windows
interface. The dialogs are easier to see and use and Xdialog adds even
more functionalities (help button+box, treeview, editbox, file selector,
range box, and much more).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 154392] HelixPlayer should be removed

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: HelixPlayer should be removed


https://bugzilla.redhat.com/show_bug.cgi?id=154392


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Core |Fedora
Version|devel   |rawhide




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445980] Review Request: odpdom - Oversized Document Parser

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: odpdom - Oversized Document Parser


https://bugzilla.redhat.com/show_bug.cgi?id=445980





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 14:48 EST ---
I actually happen to disagree with that; I think it should be our goal to
provide a shared library whenever possible, and see nothing wrong with what's
being done in 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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445980] Review Request: odpdom - Oversized Document Parser

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: odpdom - Oversized Document Parser


https://bugzilla.redhat.com/show_bug.cgi?id=445980


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 14:45 EST ---
Well, as far as I checked this package
- the original tarball does not provide any shared libraries, only
  static archives are provided
- odpdom-p4v.patch is applied to provide a shared library for Fedora

Agreeing with Patrice, I think this situation must be 
avoided (or treated with much care)
because we cannot expect with what soversion 
the upstream developer will provide a shared library
in the future:
http://fedoraproject.org/wiki/PatriceDumas
see also
https://bugzilla.redhat.com/show_bug.cgi?id=438043

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226380] Merge Review: rsync

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: rsync


https://bugzilla.redhat.com/show_bug.cgi?id=226380


[EMAIL PROTECTED] changed:

   What|Removed |Added

Version|devel   |rawhide




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 14:42 EST ---
Created an attachment (id=305156)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=305156&action=view)
Patch to reduce rpmlint warnings, add URL.

I think this review is out of date and could be closed.  It appears to be Re:
rsync 2.6.9, which is out of date (F9 has 3.0.2).

I've attached a patch that could still go in to fix some rpmlint warnings and
implement other suggestions in this review.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 247984] Review Request: Etoys - Squeak-based learning environment for OLPC

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: Etoys - Squeak-based learning environment for OLPC


https://bugzilla.redhat.com/show_bug.cgi?id=247984


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||CANTFIX
   Flag||fedora-review-




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 14:36 EST ---
I don't think this is possible to move forward, given the fact that squeak-vm is
not acceptable for Fedora in its current state. Closing as CANTFIX.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 446097] New: Review Request: pytraffc - Computer version of the board game Rush Hou

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




https://bugzilla.redhat.com/show_bug.cgi?id=446097

   Summary: Review Request: pytraffc - Computer version of the board
game Rush Hou
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://sindrepb.fedorapeople.org/packages/pytraffic.spec
SRPM URL: 
http://sindrepb.fedorapeople.org/packages/pytraffic-2.5.4-1.fc9.src.rpm

Description: 

PyTraffic is a Python version of the board game Rush Hour
created by Binary Arts Coporation. The goal is to remove the red
car out of the grid through the slot on the right. To do this you have to
slide the other cars out of the way.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 247983] Review Request: squeak-vm - Squeak virtual machine for OLPC

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: squeak-vm - Squeak virtual machine for OLPC


https://bugzilla.redhat.com/show_bug.cgi?id=247983


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||CANTFIX
   Flag|fedora-review?  |fedora-review-




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 14:34 EST ---
Yeaaah. I'm closing this one as CANTFIX. It will need a LOT of work to be made
clean for Fedora, not to mention the MP3 problems and the Squeak license 
(non-free).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 200051] Review Request: libutempter

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libutempter


https://bugzilla.redhat.com/show_bug.cgi?id=200051


[EMAIL PROTECTED] changed:

   What|Removed |Added

Version|devel   |rawhide




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 247984] Review Request: Etoys - Squeak-based learning environment for OLPC

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: Etoys - Squeak-based learning environment for OLPC


https://bugzilla.redhat.com/show_bug.cgi?id=247984


Bug 247984 depends on bug 247983, which changed state.

Bug 247983 Summary: Review Request: squeak-vm - Squeak virtual machine for OLPC
https://bugzilla.redhat.com/show_bug.cgi?id=247983

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||CANTFIX



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438043] Review Request: GMT - Generic Mapping Tools

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: GMT - Generic Mapping Tools


https://bugzilla.redhat.com/show_bug.cgi?id=438043





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 14:11 EST ---
ldconfig call should be in %post, not %pre.

There is a \ missing on the find $RPM_BUILD_ROOT/%{gmthome}/examples -name
\*.\*sh | line

Also you could consider removing the .bat files in examples, rpmlint
says:
GMT-examples.i386: W: wrong-file-end-of-line-encoding
/usr/share/GMT/examples/ex03/job03.bat

There are .in files remaining in 
/usr/share/GMT/conf/.gmtdefaults_SI.in
/usr/share/GMT/conf/.gmtdefaults_US.in
/usr/share/GMT/conf/gmt.conf.in
I think they should be removed. You could also consider using the 
timestamps of the .in files to have consistent timestamps across arches
for the files in /etc/GMT/

Also maybe the .gmtdefaults_SI and .gmtdefaults_US could be considered 
as documentation and linked like other config files in /etc/GMT and as
%config.

Another issue revealed by rpmlint is 
GMT-examples.i386: W: doc-file-dependency /usr/share/GMT/examples/ex26/job26.csh
/bin/csh
Maybe the .csh files could be made non executable, and only the 
sh files would be executable?

There is also
GMT-octave.i386: E: script-without-shebang
/usr/share/octave/site/api-v32/m/grdinfo.m
GMT-octave.i386: E: script-without-shebang
/usr/share/octave/site/api-v32/m/grdwrite.m
GMT-octave.i386: E: script-without-shebang
/usr/share/octave/site/api-v32/m/grdread.m
because these files have the execute bit set. You should correct it
if it is wrong.

A last remaining issue is the soname issue. Any comment on 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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445105] Review Request: milkytracker - SDL clone of the Fasttracker II module tracker

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: milkytracker - SDL clone of the Fasttracker II module 
tracker


https://bugzilla.redhat.com/show_bug.cgi?id=445105





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 13:53 EST ---
For 0.90.80-1:

* SourceURL
  - I recommend to use %{name} and %{version}. With this, you won't
have to fix the SourceURL when a new version is released.

* Configure option
  - Would you explain why you want to add --without-jack?
(jack-audio-connection-kit-devel is in Fedora)

* Timestamps
  - When using "install" or "cp" commands, add "-p" option to
keep timestamps on installed files.

* Macros
  - Use macros properly. /usr/share should be %{_datadir}.

* rpmlint issue

milkytracker-debuginfo.i386: W: spurious-executable-perm
/usr/src/debug/milkytracker-0.90.80/src/tracker/TitlePageManager.h
milkytracker-debuginfo.i386: W: spurious-executable-perm
/usr/src/debug/milkytracker-0.90.80/src/ppui/DialogFileSelector.h
(and many lines)

  Summary
  - The permissions of the source code files are incorrect, which
creates lots of debuginfo related rpmlints. Fix this by

%prep
%setup -q
find . -name \*.cpp -or -name \*.h | xargs chmod 0644

for example.

Then as this is a NEEDSPONSOR ticket:
-
NOTE: Before being sponsored:

This package will be accepted with another few work. 
But before I accept this package, someone (I am a candidate) 
must sponsor you.

Once you are sponsored, you have the right to review other 
submitters' review requests and approve the packages formally. 
For this reason, the person who want to be sponsored (like you) 
are required to "show that you have an understanding 
of the process and of the packaging guidelines" as is described
on :
http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored

Usually there are two ways to show this.
A. submit other review requests with enough quality.
B. Do a "pre-review" of other person's review request
   (at the time you are not sponsored, you cannot do
   a formal review)

When you have submitted a new review request or have pre-reviewed other 
person's review request, please write the bug number on this bug report 
so that I can check your comments or review request.

Fedora package collection review requests which are waiting for someone to
review can be checked on:
http://fedoraproject.org/PackageReviewStatus/NEW.html
(NOTE: please don't choose "Merge Review")


Review guidelines are described mainly on:
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines
http://fedoraproject.org/wiki/Packaging/Guidelines
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 431161] Review Request: mathmap - A gimp plugin and commandline tool

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: mathmap - A gimp plugin and commandline tool


https://bugzilla.redhat.com/show_bug.cgi?id=431161


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|ASSIGNED
   Flag|needinfo?([EMAIL PROTECTED]|
   |m)  |




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 13:50 EST ---
Sorry about the delay, I've been knee deep in other stuff for awhile.  I'll take
another look at this today.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445105] Review Request: milkytracker - SDL clone of the Fasttracker II module tracker

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: milkytracker - SDL clone of the Fasttracker II module 
tracker


https://bugzilla.redhat.com/show_bug.cgi?id=445105


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 427824] Review Request: tmispell-voikko - An Ispell compatible front-end for spell-checking modules

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tmispell-voikko - An Ispell compatible front-end for 
spell-checking modules


https://bugzilla.redhat.com/show_bug.cgi?id=427824





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 13:26 EST ---
Which applications use/require the static *.a file, and how?  Static libraries
are frowned upon in Fedora and shouldn't be shipped without a good reason.  If
the *.a here needs to be shipped, some work is needed to meet the packaging
guidelines.
http://fedoraproject.org/wiki/Packaging/Guidelines#head-2302ec1e1f44202c9cc4bcce24cb711266557ad7

Something to forward upstream (if it works): if the versioned *.so.* are not
needed, I think the proper way to get rid of them would be to change
libenchant_voikko_la_LDFLAGS to something like "-lvoikko -module -avoid-version
-no-undefined" in enchant/Makefile.am.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 443797] Review Request: comedilib - Data Acquisition library for the Comedi driver

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: comedilib - Data Acquisition library for the Comedi 
driver
Alias: comedilib-review

https://bugzilla.redhat.com/show_bug.cgi?id=443797


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 13:23 EST ---
thank you both, all 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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445635] Review Request: R-zoo - Z's ordered observations for irregular time series

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: R-zoo - Z's ordered observations for irregular time 
series


https://bugzilla.redhat.com/show_bug.cgi?id=445635


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: R-zoo - Z's |Review Request: R-zoo - Z's
   |ordered observations|ordered observations for
   ||irregular time series




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 13:15 EST ---
* Mon May 12 2008 Orion Poplawski <[EMAIL PROTECTED]> 1.5-3
- Include time series in summary
- Fix up build requires for older versions

Spec URL: http://www.cora.nwra.com/~orion/fedora/R-zoo.spec
SRPM URL: http://www.cora.nwra.com/~orion/fedora/R-zoo-1.5-3.fc9.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, or are watching someone who is.

___
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

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: erlsom - Support for XML Schema in Erlang


https://bugzilla.redhat.com/show_bug.cgi?id=445797





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 13:01 EST ---
I wonder if it wouldn't be better to name this erlang-erlsom.  The only erlang
package we have currently is erlang-esdl, so there's not much precedent, but
it's worth considering.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445143] Review Request: libcaptury - A library for X11/OpenGL video capturing framework

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libcaptury - A library for X11/OpenGL video capturing 
framework


https://bugzilla.redhat.com/show_bug.cgi?id=445143


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 427824] Review Request: tmispell-voikko - An Ispell compatible front-end for spell-checking modules

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tmispell-voikko - An Ispell compatible front-end for 
spell-checking modules


https://bugzilla.redhat.com/show_bug.cgi?id=427824





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 12:56 EST ---
I've gotten a reply from the Voikko main developer, it should be ok to move
libenchant_voikko.so.1.0.0 to libenchant_voikko.so and it does work, so that's
what I've done in this version.

New Spec: http://vpv.fedorapeople.org/packages/tmispell-voikko.spec
New SRPM: 
http://vpv.fedorapeople.org/packages/tmispell-voikko-0.7-0.7.fc8.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, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 433778] Review Request: fsvs - full system versioning using subversion repositories

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: fsvs - full system versioning using subversion 
repositories


https://bugzilla.redhat.com/show_bug.cgi?id=433778


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 12:51 EST ---
fsvs already has a F-9 branch, as all packages were mass branched for F-9 a
while ago. Make sure you are using 'cvs update -d' to get the new directory. 

cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 434707] Review Request: ocaml-sexplib - OCaml library for converting OCaml values to S-expressions

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-sexplib - OCaml library for converting OCaml 
values to S-expressions


https://bugzilla.redhat.com/show_bug.cgi?id=434707


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 12:52 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445070] Review Request: ocaml-res - OCaml library for resizing arrays and strings

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-res - OCaml library for resizing arrays and 
strings


https://bugzilla.redhat.com/show_bug.cgi?id=445070


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 12:55 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445067] Review Request: ocaml-ounit - Unit test framework for OCaml

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-ounit - Unit test framework for OCaml


https://bugzilla.redhat.com/show_bug.cgi?id=445067


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 12:53 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 275491] Review Request: ocaml-pgocaml - OCaml library for type-safe access to PostgreSQL databases

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-pgocaml - OCaml library for type-safe access to 
PostgreSQL databases


https://bugzilla.redhat.com/show_bug.cgi?id=275491


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 12:48 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445601] Review Request: ruby-augeas - Ruby bindings for Augeas

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ruby-augeas - Ruby bindings for Augeas


https://bugzilla.redhat.com/show_bug.cgi?id=445601


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 12:35 EST ---
Imported and built as http://koji.fedoraproject.org/koji/buildinfo?buildID=48599


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438897] Review Request: framewave - Image and signal processing routines

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: framewave - Image and signal processing routines


https://bugzilla.redhat.com/show_bug.cgi?id=438897





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 12:18 EST ---
http://www.cora.nwra.com/~orion/fedora/framewave.spec
http://www.cora.nwra.com/~orion/fedora/framewave-1.1-0.20080505.1.fc9.src.rpm

* Thu May 8 2008 Orion Poplawski <[EMAIL PROTECTED]> - 1.1-0.20080505.1
- 1.1 05MAY08 beta
- Add RPM_OPT_FLAGS to build
- Run UnitTest in %%check

rpmlint still has:

framewave.i386: W: unused-direct-shlib-dependency /usr/lib/libfwBase.so.1.1.0
/lib/libm.so.6

No idea where the above comes from.  fwBase is linked:

g++ -o build/tmp/fwBase_release_shared_32/libfwBase.so -m32 -lboost_thread-mt
-Wl,-soname,libfwBase.so.1 -shared
build/tmp/fwBase_release_shared_32/obj/fwBase.obj
build/tmp/fwBase_release_shared_32/obj/ThreadPool.obj
build/tmp/fwBase_release_shared_32/obj/system.obj
build/tmp/fwBase_release_shared_32/common/obj/Constants.obj

framewave.i386: W: unused-direct-shlib-dependency /usr/lib/libfwImage.so.1.1.0
/usr/lib/libboost_thread-mt.so.3
framewave.i386: W: unused-direct-shlib-dependency /usr/lib/libfwVideo.so.1.1.0
/usr/lib/libboost_thread-mt.so.3
framewave.i386: W: unused-direct-shlib-dependency /usr/lib/libfwJPEG.so.1.1.0
/usr/lib/libboost_thread-mt.so.3
framewave.i386: W: unused-direct-shlib-dependency
/usr/lib/libfwSignal.so.1.1.0/usr/lib/libboost_thread-mt.so.3

I think it's going to be pretty hard to fix these, as currently there is only a
single set of loadflags for all libraries.  I'll poke upstream about it.



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445067] Review Request: ocaml-ounit - Unit test framework for OCaml

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-ounit - Unit test framework for OCaml


https://bugzilla.redhat.com/show_bug.cgi?id=445067


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 12:18 EST ---
New Package CVS Request
===
Package Name: ocaml-ounit
Short Description: Unit test framework for OCaml
Owners: rjones
Branches: F-8 F-9
InitialCC: rjones
Cvsextras Commits: yes

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445067] Review Request: ocaml-ounit - Unit test framework for OCaml

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-ounit - Unit test framework for OCaml


https://bugzilla.redhat.com/show_bug.cgi?id=445067





--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 12:18 EST ---
Oh dear, possible that I added it and didn't bump the release number.  Anyway,
it's all good now.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 444625] Review Request: GMT-doc - Documentation for Generic Mapping Tools

2008-05-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: GMT-doc - Documentation for Generic Mapping Tools


https://bugzilla.redhat.com/show_bug.cgi?id=444625


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: GMT-docs -  |Review Request: GMT-doc -
   |Documentation for Generic   |Documentation for Generic
   |Mapping Tools   |Mapping Tools




--- Additional Comments From [EMAIL PROTECTED]  2008-05-12 12:00 EST ---
* Mon Apr 12 2008 Orion Poplawski <[EMAIL PROTECTED]> 4.3.0-2
- Rename to GMT-doc
- Require only GMT-%%{version}, drop release

Spec URL: http://www.cora.nwra.com/~orion/fedora/GMT-doc.spec
SRPM URL: http://www.cora.nwra.com/~orion/fedora/GMT-doc-4.3.0-2.src.rpm

Thanks!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


  1   2   >