[Bug 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment

2008-07-01 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: ristretto - Image-viewer for the Xfce desktop 
environment


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 02:57 EST ---
Hehe, you could better pick the diff from that revision and apply it against
0.0.20. Currently, trunk has a few partial modifications to allow saving an
image rotated. But these are not yet fully implemented.

So, when that fix is tested... 0.0.20 + patch is a better solution ;-)

-- 
Configure 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 423821] Review Request: nagios-plugins-rsync - Nagios plugin to monitor remote rsync servers

2008-07-01 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: nagios-plugins-rsync - Nagios plugin to monitor remote 
rsync servers


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 02:55 EST ---
I'll review 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 453016] Review Request: un-fonts-core - Korean TrueType fonts

2008-07-01 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: un-fonts-core - Korean TrueType fonts


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 03:08 EST ---
Thanks, Dennis for the clear update.

 #1 Rebuild and repackaging for rpmlint
 #2 Changed package name to Un series Korean TrueType fonts
 #3 Summary and description of the changes, added to Korean

Thanks

 #4 Added %define archiveversion 080608, because 080608 is snapshot version

OK - I am still a little unclear on the meaning of the snapshot date in 
relation to
the version 1.0.2.  Does it mean prerelease or postrelease or is it just a 
label?

In any case I don't think it should go into the version field but the release:

 #5 I have a problem in korean spacing words for rpmlint. I don't fix it

Okay - maybe we should file a bug for rpmlint for 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 453016] Review Request: un-fonts-core - Korean TrueType fonts

2008-07-01 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: un-fonts-core - Korean TrueType fonts


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   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 453016] Review Request: un-fonts-core - Korean TrueType fonts

2008-07-01 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: un-fonts-core - Korean TrueType fonts


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 03:19 EST ---
Created an attachment (id=310643)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=310643action=view)
un-fonts-core.spec-1.patch

some suggested changes

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 453016] Review Request: un-fonts-core - Korean TrueType fonts

2008-07-01 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: un-fonts-core - Korean TrueType fonts


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 03:18 EST ---
(In reply to comment #2)
 #6 fixed License: GPLv2+

Did you check upstream about the GPL version and ask if they will update the 
license
text in the fonts themselves?

-- 
Configure 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 453016] Review Request: un-fonts-core - Korean TrueType fonts

2008-07-01 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: un-fonts-core - Korean TrueType fonts


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 03:16 EST ---
BTW just a note that if upstream released the fontforge .sfd files
it would be better to build those than shipping the .ttf files but that is not
blocker.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 453016] Review Request: un-fonts-core - Korean TrueType fonts

2008-07-01 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: un-fonts-core - Korean TrueType fonts


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 03:26 EST ---
Also please update the changelog in the spec file next time you update.

-- 
Configure 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 453520] New: Review Request: libUnihan - C library for Unihan character database in 5NF

2008-07-01 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=453520

   Summary: Review Request: libUnihan - C library for Unihan
character database in 5NF
   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://downloads.sourceforge.net/libunihan/libUnihan.spec
SRPM URL: 
http://downloads.sourceforge.net/libunihan/libUnihan-0.3-0.fc9.src.rpm  
Description:

Hi, I just finished packing the libUnihan and libUnihan-data. A review will be 
appreciated.

The project has two packages, one is libUnihan-data, which holds a large 
database (75M), the other is libUnihan, provides C library, header files, and 
so on.

The package to be reviewed is libUnihan. Please tell me what you think.

Regards,
Ding-Yi Chen

-- 
Configure 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 453519] New: Review Request: libUnihan-data - The Unihan character database in 5NF

2008-07-01 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=453519

   Summary: Review Request: libUnihan-data - The Unihan character
database in 5NF
   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://downloads.sourceforge.net/libunihan/libUnihan-data.spec
SRPM URL: 
http://downloads.sourceforge.net/libunihan/libUnihan-data-5.1-0.fc9.src.rpm  
Description:

Hi, I just finished packing the libUnihan and libUnihan-data. A review will be 
appreciated.

The project has two packages, one is libUnihan-data, which holds a large 
database (75M), the other is libUnihan, provides C library, header files, and 
so on.

The package to be reviewed is libUnihan-data. Please tell me what you think.

Regards,
Ding-Yi Chen

-- 
Configure 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 453520] Review Request: libUnihan - C library for Unihan character database in 5NF

2008-07-01 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: libUnihan - C library for Unihan character database in 
5NF


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 03:31 EST ---
Sorry, the SRPM URL should be:
http://downloads.sourceforge.net/libunihan/libUnihan-0.3.0-0.fc9.src.rpm

Regards,
Ding-Yi Chen

-- 
Configure 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 453016] Review Request: un-fonts-core - Korean TrueType fonts

2008-07-01 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: un-fonts-core - Korean TrueType fonts


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 03:32 EST ---
One more thing: since the fonts are big and probably most non-Korean users won't
need all of them installed I would like to subpackage the less used fonts so 
that
we can install just main ones by default and the rest for Korean users.

Default font for Korean should be Batang?

-- 
Configure 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 453016] Review Request: un-fonts-core - Korean TrueType fonts

2008-07-01 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: un-fonts-core - Korean TrueType fonts


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 03:46 EST ---
 Default font for Korean should be Batang?

I tested now - well I guess the default Un Dotum should be ok?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 433253] Review Request: dotconf - Required for speech dispatcher on OLPC XO

2008-07-01 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: dotconf - Required for speech dispatcher on OLPC XO


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 06:05 EST ---
Okay, then would you try to build F-8 and devel branch (not by scratch)?
Also, for F-9/F-8 branches, please visit
https://admin.fedoraproject.org/updates/ and submit a push request to 
repositories.
Then you can close this bug as 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 453016] Review Request: un-fonts-core - Korean TrueType fonts

2008-07-01 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: un-fonts-core - Korean TrueType fonts


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 06:15 EST ---
(In reply to comment #9)
  Default font for Korean should be Batang?
 I tested now - well I guess the default Un Dotum should be ok?

Debin and ubuntu at default korean font is Un Dotum ^^



-- 
Configure 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 453016] Review Request: un-fonts-core - Korean TrueType fonts

2008-07-01 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: un-fonts-core - Korean TrueType fonts


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 06:19 EST ---
(In reply to comment #5)
 (In reply to comment #2)
  #6 fixed License: GPLv2+
 Did you check upstream about the GPL version and ask if they will update the 
license
 text in the fonts themselves?

I asked them to check again. They gave answers to a GPLv2


-- 
Configure 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 452832] Review Request: perl-Math-FFT - Perl extension for Fast Fourier Transforms

2008-07-01 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-Math-FFT - Perl extension for Fast Fourier 
Transforms


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 07:35 EST ---
Successfully built:
http://koji.fedoraproject.org/koji/taskinfo?taskID=689467

-- 
Configure 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 453567] New: Review Request: libfonts - Java TrueType Font Layouting

2008-07-01 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=453567

   Summary: Review Request: libfonts - Java TrueType Font Layouting
   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://people.redhat.com/caolanm/jfreereport/libfonts.spec
SRPM URL: 
http://people.redhat.com/caolanm/jfreereport/libfonts-0.3.4-1.fc9.src.rpm
Description:  Java TrueType Font Layouting

libfonts is a dependency of jfreereport which is a dependency of 
OpenOffice.org3's report generating functionality

-- 
Configure 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 433253] Review Request: dotconf - Required for speech dispatcher on OLPC XO

2008-07-01 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: dotconf - Required for speech dispatcher on OLPC XO


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 08:07 EST ---
I tried building without scratch and this is the error i got:

[EMAIL PROTECTED] ~]$ koji build dist-f8
~/rpmbuild/SRPMS/dotconf-1.0.13-6.fc7.src.rpm
Usage: koji build [options] target URL
(Specify the --help global option for a list of other help options)

koji: error: Destination tag dist-f8 is locked

I generated new keys and tried everything again. Still no change..

Seems like there is some problem in my sshd. When i try to run it in debug mode
it fails to run and gives this error : Bind to port 22 on :: failed: Address
already in use.

Is it cause of this?


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 453569] New: Review Request: libmirage - library to provide access to different image formats

2008-07-01 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=453569

   Summary: Review Request: libmirage - library to provide access to
different image formats
   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://people.fedoraproject.org/~jmoskovc/libmirage.spec
SRPM URL: 
http://people.fedoraproject.org/~jmoskovc/libmirage-1.0.0-2.fc10.src.rpm

Description: 

CD-ROM image access library, and part of the userspace-cdemu suite, a free, GPL 
CD/DVD-ROM device emulator for linux. It is written in C and based on GLib.  
The aim of libMirage is to provide uniform access to the data stored in 
different image formats, by creating a representation of disc stored in image 
file, which is based on GObjects.

rpmlint:

libmirage.i686: E: zero-length /usr/share/doc/libmirage-1.0.0/NEWS
libmirage.i686: E: zero-length /usr/share/doc/libmirage-1.0.0/ChangeLog

These files are really empty, but will be probably used in 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 436330] Review Request: libtrash - Libraries to move files to a trash-folder on delete

2008-07-01 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: libtrash - Libraries to move files to a trash-folder 
on delete


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 08:13 EST ---
First at all, thank you for reviewing :-)

(In reply to comment #2)
 - The spec you mentioned from the URL does not match the spec in the SPRM

Fixed.

 - Make sure that the /sbin/ldconfig lines are as follows:
 
 %post -p /sbin/ldconfig
 
 %postun -p /sbin/ldconfig

Yes, the lines look like those above.

 - Don't use %{name}-latest in the URL, use %{name}-%{version}. Yes, the URL is
 still valid.

Fixed.

 - libtrash.i386: W: devel-file-in-non-devel-package /usr/lib/libtrash.so
   - move %{_libdir}/libtrash.so to a -devel package

This library is the core of libtrash package. So I think, that it doesn't make a
sense to make another package. If so, then the first package would contain only
configuration file and doc files and the second one would contain only the
library. Furthermore, the first one would have to have second one in
requirements, otherwise the whole thing will not work.

 - Other rpmlint errors:
 - libtrash.i386: E: no-ldconfig-symlink /usr/lib/libtrash.so.3.2

Fixed.

Spec URL: http://zprikryl.fedorapeople.org/libtrash.spec
SRPM URL: http://zprikryl.fedorapeople.org/libtrash-3.2-1.fc10.src.rpm


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 453570] New: Review Request: dkms-vhba - a virtual SCSI host bus adapter

2008-07-01 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=453570

   Summary: Review Request: dkms-vhba - a virtual SCSI host bus
adapter
   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://people.fedoraproject.org/~jmoskovc/dkms-vhba.spec
SRPM URL: 
http://people.fedoraproject.org/~jmoskovc/dkms-vhba-1.0.0-3.fc10.src.rpm

Description: 

VHBA kernel module, a virtual SCSI host bus adapter used by CDEmu daemon from 
userspace-cdemu suite.

rpmlint:

dkms-vhba.noarch: W: no-documentation
dkms-vhba.noarch: W: devel-file-in-non-devel-package 
/usr/src/vhba-1.0.0-3.fc10/vhba.c

Upstream doesn't provide any documentation. The package has to contain source 
files because it's dkms module that automatically rebuilds when new kernel is 
booted.

-- 
Configure 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 453570] Review Request: dkms-vhba - a virtual SCSI host bus adapter

2008-07-01 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: dkms-vhba - a virtual SCSI host bus adapter


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 08:47 EST ---
Fedora policy is that kernel modules must be included in the main kernel package
(and preferably in the upstream kernel); separate kernel module packages,
whether binary or dkms-based, are no longer allowed.

http://fedoraproject.org/wiki/Packaging/KernelModules


-- 
Configure 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 453422] Review Request: songbird - Mozilla based multimedia player

2008-07-01 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: songbird - Mozilla based multimedia player


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 08:50 EST ---
I was able to fix the debugedit issue by removing -gstabs+ flags from the build
process. Apparently, the stripper get caught up on those in x86_64, see bug 
453506

The package has now been rebuilt with this fix, and some other small tweaks.
Note, ppc building is not possible at the moment:

http://rpm.rutgers.edu/fedora/songbird-0.6.1-2.fc9.src.rpm
http://rpm.rutgers.edu/fedora/songbird.spec

* Mon Jun 30 2008 David Lee Halik [EMAIL PROTECTED] - 0.6.1-2
- Removed flag -gstabs+ from configure.ac to enable debuginfo, bug 453506
- Removed manual stripping for binaries
- Removed xulrunner source arch macros


-- 
Configure 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 430429] Review Request: python-storm - An object-relational mapper (ORM) for Python

2008-07-01 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-storm - An object-relational mapper (ORM) for 
Python


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 08:56 EST ---
Michel: got a few moments to move this review along ?

-- 
Configure 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 453573] New: Review Request: cdemu-daemon - userspace daemon for use with cdemu suite

2008-07-01 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=453573

   Summary: Review Request: cdemu-daemon - userspace daemon for use
with cdemu suite
   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://people.fedoraproject.org/~jmoskovc/cdemu-daemon.spec
SRPM URL: 
http://people.fedoraproject.org/~jmoskovc/cdemu-daemon-1.0.0-2.fc10.src.rpm

Description:

This is CDEmu daemon, the userspace daemon part of the userspace-cdemu suite, a.
free, GPL CD/DVD-ROM device emulator for linux.
It receives SCSI commands from kernel module and processes them, passing the.
requested data back to the kernel. Daemon implements the actual virtual device;.
one instance per each device registered by kernel module. It uses libMirage, an.
image access library that is part of userspace-cdemu suite, for the image.
access (e.g. sector reading).

rpmlint:

cdemu-daemon.i686: E: zero-length /usr/share/doc/cdemu-daemon-1.0.0/ChangeLog
cdemu-daemon.i686: E: zero-length /usr/share/doc/cdemu-daemon-1.0.0/NEWS

These files are empty, but will be used later.

-- 
Configure 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 453576] New: Review Request: cdemu - client for controlling CDEmu daemon

2008-07-01 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=453576

   Summary: Review Request: cdemu - client for controlling CDEmu
daemon
   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://people.fedoraproject.org/~jmoskovc/cdemu.spec
SRPM URL: http://people.fedoraproject.org/~jmoskovc/cdemu-1.0.0-2.fc10.src.rpm

Description:

This is cdemu-client, a simple command-line client for controlling CDEmu
daemon. It is part of the userspace-cdemu suite, a free, GPL CD/DVD-ROM.
device emulator for linux.
It provides a way to perform the key tasks related to controlling the CDEmu
daemon, such as loading and unloading devices, displaying devices' status and
retrieving/setting devices' debug masks.

rpmlint:
cdemu.noarch: E: zero-length /usr/share/doc/cdemu-1.0.0/NEWS
cdemu.noarch: E: zero-length /usr/share/doc/cdemu-1.0.0/ChangeLog

These two files are empty, but will be used later.

-- 
Configure 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 433253] Review Request: dotconf - Required for speech dispatcher on OLPC XO

2008-07-01 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: dotconf - Required for speech dispatcher on OLPC XO


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 09:18 EST ---
No, as you did on F-9 branch, follow
http://fedoraproject.org/wiki/PackageMaintainers/UpdatingPackageHowTo
or from Import Your Package of
http://fedoraproject.org/wiki/PackageMaintainers/Join

-- 
Configure 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 453584] New: Review Request: gcdemu - GNOME applet for controlling CDEmu daemon

2008-07-01 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=453584

   Summary: Review Request: gcdemu - GNOME applet for controlling
CDEmu daemon
   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://people.fedoraproject.org/~jmoskovc/gcdemu-1.0.0-2.fc10.src.rpm
SRPM URL: http://people.fedoraproject.org/~jmoskovc/gcdemu.spec

Description:

This is gCDEmu, a GNOME applet for controlling CDEmu daemon. It is part of the.
userspace-cdemu suite, a free, GPL CD/DVD-ROM device emulator for linux.
It provides a graphic interface that allows performing the key tasks related to
controlling the CDEmu daemon, such as loading and unloading devices, displaying.
devices' status and retrieving/setting devices' debug masks.
In addition, applet listens to the signals emitted by CDEmu daemon and provides
the notification via libnotify's notifications (provided that python bindings
are installed).

rpmlint:

gcdemu.i686: E: no-binary

It's written in python, but it can't be noarch, because it contains some files 
whose location depends on architecture (lib | lib64)

-- 
Configure 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 453083] Review Request: Samba4 - Samba4 CIFS and AD server and client

2008-07-01 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: Samba4 - Samba4 CIFS and AD server and client


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 09:55 EST ---
The License tag is still not correct (should be GPLv3+ and LGPLv3+ and BSD, 
not GPLv3+, LGPLv3+, BSD).

-- 
Configure 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 450164] Review Request: ace-tao - The ADAPTIVE Communication Environment (ACE) and The ACE ORB (TAO)

2008-07-01 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: ace-tao -  The ADAPTIVE Communication Environment 
(ACE) and The ACE ORB (TAO)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 10:05 EST ---
The license included in the source tarball and what is written on
http://www.cs.wustl.edu/~schmidt/ACE-copying.html seems different. Would you
verify which is the correct one? (the reason I am asking this is that IMO the
license
included in the source tarball is problematic and maybe 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 226115] Merge Review: m4

2008-07-01 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: m4


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 10:12 EST ---
Thanks for your suggestions, all issues are fixed in devel branch.

-- 
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 453119] Review Request: libvirt-java: Java bindings for the libvirt library

2008-07-01 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: libvirt-java: Java bindings for the libvirt library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 10:21 EST ---
Okay, finding the right include paths for JNI stuff is fairly messy,
and well gcj-devel has a bug plugging them by default in gcc include
path (fun see #453572).
I think I have ironed out the problems, regenerated a new 0.1.1 release

SRPMS: ftp://libvirt.org/libvirt/java/libvirt-java-0.1.0-1.fc9.src.rpm
spec: ftp://libvirt.org/libvirt/java/libvirt-java.spec

wei:~/libvirt-java - rpmlint
/u/veillard/rpms/SRPMS/libvirt-java-0.1.1-1.fc9.src.rpm
libvirt-java.src: E: unknown-key GPG#de95bc1f
libvirt-java.src: W: strange-permission libvirt-java.spec 0600
1 packages and 0 specfiles checked; 1 errors, 1 warnings.
wei:~/libvirt-java - 

  I think if one ignore the fact that rpmlint doesn't know my PGP key
this looks decent.

  For the spec file I used the following:

%define javajava
and
Requires:   %{java} = 1.5.0
BuildRequires:  %{java}-devel = 1.5.0

because some users may want to define more precisely which java environment
they want for development. Ideally this should remain that flexible (the src
RPM rebuit even on a RHEL4 with Java SDK in /usr/lib/jvm/java-1.5.0-ibm-1.5.0.7)

I hope it's okay now,

Daniel

-- 
Configure 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 453119] Review Request: libvirt-java: Java bindings for the libvirt library

2008-07-01 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: libvirt-java: Java bindings for the libvirt library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 10:22 EST ---
Forgot to update the new SRPM URL, it is at: 
SRPMS: ftp://libvirt.org/libvirt/java/libvirt-java-0.1.1-1.fc9.src.rpm


Daniel

-- 
Configure 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 401101] Review Request: sleuthkit - The Sleuth Kit (TSK)

2008-07-01 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: sleuthkit - The Sleuth Kit (TSK)


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 10:24 EST ---
Package Review
==

Key:
 - = N/A
 x = Check
 ! = Problem
 ? = Not evaluated

=== REQUIRED ITEMS ===
 [x] Package is named according to the Package Naming Guidelines.
 [x] Spec file name must match the base package %{name}, in the format 
%{name}.spec.
 [x] Package meets the Packaging Guidelines.
 [x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
 Tested on: devel/x86_64
 [x] Rpmlint output:
 source RPM: empty
 binary RPM:
 sleuthkit-libs: W: no-documentation
 sleuthkit and sleuthkit-devel: empty
 [x] Package is not relocatable.
 [x] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
 [x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
 [x] License field in the package spec file matches the actual license.
 License type: CPL and IBM and GPLv2+
 [x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
 [x] Spec file is legible and written in American English.
 [x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
 SHA1SUM of package: e2e8ebc41e4be7aa75376fef38e13cf6f2d6206f 
sleuthkit-2.52.tar.gz
 [x] Package is not known to require ExcludeArch
 [x] All build dependencies are listed in BuildRequires, except for any that are
listed in the exceptions section of Packaging Guidelines.
 [-] The spec file handles locales properly.
 [x] ldconfig called in %post and %postun if required.
 [x] Package must own all directories that it creates.
 [x] Package requires other packages for directories it uses.
 [x] Package does not contain duplicates in %files.
 [x] Permissions on files are set properly.
 [x] Package has a %clean section, which contains rm -rf $RPM_BUILD_ROOT.
 [x] Package consistently uses macros.
 [x] Package contains code, or permissable content.
 [-] Large documentation files are in a -doc subpackage, if required.
 [x] Package uses nothing in %doc for runtime.
 [x] Header files in -devel subpackage, if present.
 [-] Static libraries in -devel subpackage, if present.
 [-] Package requires pkgconfig, if .pc files are present.
 [x] Development .so files in -devel subpackage, if present.
 [x] Fully versioned dependency in subpackages, if present.
 [x] Package does not contain any libtool archives (.la).
 [-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
 [x] Package does not own files or directories owned by other packages.

=== SUGGESTED ITEMS ===
 [x] Latest version is packaged.
 [x] Package does not include license text files separate from upstream.
 [-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
 [x] Reviewer should test that the package builds in mock.
 Tested on: all architectures supported by koji
 [ ] Package should compile and build into binary rpms on all supported
architectures.
 Tested on: all architectures supported by koji
 [x] Package functions as described (tested in rawhide/i386).
 [-] Scriptlets must be sane, if used.
 [-] The placement of pkgconfig(.pc) files is correct.
 [-] File based requires are sane.



*** 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 401101] Review Request: sleuthkit - The Sleuth Kit (TSK)

2008-07-01 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: sleuthkit - The Sleuth Kit (TSK)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 445687] Review Request: portreserve - TCP port reservation utility

2008-07-01 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: portreserve - TCP port reservation utility


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 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 244370] Review Request: olpc-hardware-manager - OLPC hardware manager

2008-07-01 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: olpc-hardware-manager - OLPC hardware manager


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Version|devel   |rawhide

[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 11:24 EST ---
Package Change Request
==
Package Name: olpc-hardware-manager
New Branches: OLPC-3

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, 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 452317] Review Request: heuristica-fonts - Heuristica font

2008-07-01 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: heuristica-fonts - Heuristica font


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 11:31 EST ---
I need to talk with the lawyers about this one, please hold. ;)

-- 
Configure 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 442329] Review Request: gnome-lirc-properties - Infrared Remote Controls setup tool

2008-07-01 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: gnome-lirc-properties - Infrared Remote Controls setup 
tool


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


Bug 442329 depends on bug 440231, which changed state.

Bug 440231 Summary: lircd just exits when device isn't supported
https://bugzilla.redhat.com/show_bug.cgi?id=440231

   What|Old Value   |New Value

 Status|NEW |CLOSED
 Resolution||NOTABUG



-- 
Configure 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 452387] Review Request: netris - A console-based, networked clone of Tetris

2008-07-01 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: netris - A console-based, networked clone of Tetris


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 11:45 EST ---
The clone is not the problem, the trademark is. The name of the game cannot use
the registered Tetris trademark, or anything similar to it. Netris is far
too similar to Tetris, as would be T*tris or anything which rhymes with
Tetris.

As is, this package is not acceptable for Fedora, however you can make it
acceptable by renaming the game so that it does not violate the trademark, and
minimizing the use of the Tetris mark to nominative use (basically, you can
say that $FOO is a puzzle game similar to Tetris (tm)). You cannot call it a
Tetris clone.

You could rename the game to something like NetFallingBlocks, NetBlockPuzzle, or
anything else really, as long as it doesn't infringe upon the Tetris mark. You
would need to be sure that the rename occurs in all user visible places
(anything a user might be able to see while playing the game). You would not
need to rename internal functions or the actual binary executable. The RPM name
would need to change.

If you have questions, please let me know.



-- 
Configure 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 442329] Review Request: gnome-lirc-properties - Infrared Remote Controls setup tool

2008-07-01 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: gnome-lirc-properties - Infrared Remote Controls setup 
tool


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 11:50 EST ---
Updated packages:
http://people.redhat.com/bnocera/gnome-lirc-properties/gnome-lirc-properties-0.2.8-1.fc9.src.rpm
http://people.redhat.com/bnocera/gnome-lirc-properties/gnome-lirc-properties.spec

The only thing missing is a requires on a newer PolicyKit (which I built 
yesterday).

-- 
Configure 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 244370] Review Request: olpc-hardware-manager - OLPC hardware manager

2008-07-01 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: olpc-hardware-manager - OLPC hardware manager


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 12:07 EST ---
OLPC-3 branch is not needed and has not been done.

-- 
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 450164] Review Request: ace-tao - The ADAPTIVE Communication Environment (ACE) and The ACE ORB (TAO)

2008-07-01 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: ace-tao -  The ADAPTIVE Communication Environment 
(ACE) and The ACE ORB (TAO)


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 12:16 EST ---
This has seen some attention.  I believe that the copyright does pass muster at
this point; here is email I've received regarding this issue:


On Tue, 2007-12-18 at 07:55 -0800, Ken Sedgwick wrote:
  Jules Colding wrote:
   Hi Ken,
   
   Did you ever progress with the plan to get your RPMs into Fedora?
  
  Keep encountering license issues.  Recently Tom Callaway (from the
  Fedora Team) was looking into this again ... no success yet.

OK, guys, all the legal barriers are cleared away. The latest version of
the DOC license (what ACE+TAO uses) is Free and GPL Compatible now.

Ken, you can go ahead and start the process here:
http://fedoraproject.org/wiki/PackageMaintainers/Join

If you have any questions along the way, feel free to contact me.

~spot


I've looked at the URL you posted and compared it to the COPYING file
which we ship in the RPM and they appear the same to me, modulo HTML
vs text presentation.

Ken



-- 
Configure 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 453584] Review Request: gcdemu - GNOME applet for controlling CDEmu daemon

2008-07-01 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: gcdemu - GNOME applet for controlling CDEmu daemon


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 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 453573] Review Request: cdemu-daemon - userspace daemon for use with cdemu suite

2008-07-01 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: cdemu-daemon - userspace daemon for use with cdemu 
suite


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||CANTFIX




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 12:21 EST ---
As the package this depends on is unacceptable for Fedora, I will close this
ticket.  If it turns out that the kernel module makes it into the upstream
kernel or Fedora's kernel then feel free to reopen this ticket.

-- 
Configure 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 453576] Review Request: cdemu - client for controlling CDEmu daemon

2008-07-01 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: cdemu - client for controlling CDEmu daemon


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 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 453570] Review Request: dkms-vhba - a virtual SCSI host bus adapter

2008-07-01 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: dkms-vhba - a virtual SCSI host bus adapter


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||CANTFIX




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 12:19 EST ---
As this is unacceptable for Fedora, I am closing this ticket.  Please either
work to get this into the upstream kernel, or (less prefereably) communicate
with the kernel maintainers to see if they are willing to carry the module in
the Fedora kernel packages.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 453520] Review Request: libUnihan - C library for Unihan character database in 5NF

2008-07-01 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: libUnihan - C library for Unihan character database in 
5NF


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||453519




-- 
Configure 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 453519] Review Request: libUnihan-data - The Unihan character database in 5NF

2008-07-01 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: libUnihan-data - The Unihan character database in 5NF


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||453520
  nThis||




-- 
Configure 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 450164] Review Request: ace-tao - The ADAPTIVE Communication Environment (ACE) and The ACE ORB (TAO)

2008-07-01 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: ace-tao -  The ADAPTIVE Communication Environment 
(ACE) and The ACE ORB (TAO)


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 12:29 EST ---
Actually not the same:

   The [12]ACE, [13]TAO, [14]CIAO, and [15]CoSMIC web sites are
   maintained by the [16]DOC Group at the [17]Institute for Software
   Integrated Systems (ISIS) and the [18]Center for Distributed Object
   Computing of Washington University, St. Louis for the development of
   open-source software as part of the open-source software community. 
==
   By
   submitting comments, suggestions, code, code snippets, techniques
   (including that of usage) and algorithms (collectively
   ``Submissions''), submitters acknowledge that they have the right to
   do so, that any such Submissions are given freely and unreservedly,
   and that they waive any claims to copyright or ownership. In addition,
   submitters acknowledge that any such Submission might become part of
   the copyright maintained on the overall body of code that comprises
   the DOC software. By making a Submission, submitter agree to these
   terms. Moreover, submitters acknowledge that the incorporation or
   modification of such Submissions is entirely at the discretion of the
   moderators of the open-source DOC software projects or their
   designees.

   Submissions are provided by the submitter ``as is'' with no warranties
   whatsoever, including any warranty of merchantability, noninfringement
   of third party intellectual property, or fitness for any particular

The parts bound with === are in the license in the tarball, which IMO
is problematic. 

-- 
Configure 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 441141] Review Request: antlr3 - ANother Tool for Language Recognition

2008-07-01 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: antlr3 - ANother Tool for Language Recognition


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 12:50 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 450164] Review Request: ace-tao - The ADAPTIVE Communication Environment (ACE) and The ACE ORB (TAO)

2008-07-01 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: ace-tao -  The ADAPTIVE Communication Environment 
(ACE) and The ACE ORB (TAO)


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 12:51 EST ---
Thanks for catching that!

I'll pursue this and find the answer ...

Ken

-- 
Configure 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 452559] Review Request: tex-zfuzz - Type-checker and LaTeX style for Z spec language

2008-07-01 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: tex-zfuzz - Type-checker and LaTeX style for Z spec 
language


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 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 453287] Review Request: perl-Term-ReadLine-Gnu - Perl extension for the GNU Readline/History Library

2008-07-01 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-Term-ReadLine-Gnu - Perl extension for the GNU 
Readline/History Library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 12:53 EST ---
So here's my poor attempt for a _temporary_ workaround for the problem.
Please note it totally sucks [1], but should not make any harm (unless someone
runs valgrind on perl). For more background see perl RT ticket [2], as it's a
bug in perl.

[1] http://fedorapeople.org/~lkundrak/perl-Term-ReadLine-Gnu-1.17-malloc.patch
[2] http://rt.perl.org/rt3/Ticket/Display.html?id=56500

http://netbsd.sk/~lkundrak/SPECS/perl-Term-ReadLine-Gnu.spec
http://netbsd.sk/~lkundrak/SRPMS/perl-Term-ReadLine-Gnu-1.17a-3.el5.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 446637] Review Request: gssdp - GSSDP implements resource discovery and announcement over SSDP

2008-07-01 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: gssdp - GSSDP implements resource discovery and 
announcement over SSDP


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 12:54 EST ---
Hi Peter,
I have been working on packaging these libraries for the last few days stupidly
not checking if someone else was working on it. Anyway I wanted to look at your
spec files to compare them with the ones I made and see if there was some useful
feedback I could give you, but trying to download the spec file from your server
seems to just time out.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 446637] Review Request: gssdp - GSSDP implements resource discovery and announcement over SSDP

2008-07-01 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: gssdp - GSSDP implements resource discovery and 
announcement over SSDP


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 13:19 EST ---
Yes. I think it just lost power. Waiting for it to come back up. Unfortunately
its in Australia and I'm in London. It should be back up in a couple of hours.

Peter

-- 
Configure 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 446637] Review Request: gssdp - GSSDP implements resource discovery and announcement over SSDP

2008-07-01 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: gssdp - GSSDP implements resource discovery and 
announcement over SSDP


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 13:22 EST ---
Actually I think I can use my space on fedorapeople.org. When I get home I'll
work out how to upload them there.


-- 
Configure 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 452901] Review Request: ocspd - OpenCA OCSP Daemon

2008-07-01 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: ocspd - OpenCA OCSP Daemon


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|182235  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 13:23 EST ---
This license is functionally identical (only changes are trademarks and
copyright holders) to Apache 1.0. Use:

License: ASL 1.0

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 436330] Review Request: libtrash - Libraries to move files to a trash-folder on delete

2008-07-01 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: libtrash - Libraries to move files to a trash-folder 
on delete


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 13:24 EST ---
(In reply to comment #3)
 This library is the core of libtrash package. So I think, that it doesn't 
 make a
 sense to make another package. If so, then the first package would contain 
 only
 configuration file and doc files and the second one would contain only the
 library. Furthermore, the first one would have to have second one in
 requirements, otherwise the whole thing will not work.

You need to keep the libraries with numbered extensions (libtrash.so.3.2 and
libtrash.so.3) in the main package, while moving the non-numbered one
(libtrash.so) to the -devel package.

More things:
- You should have updated the NVR (3.2-2 instead of 3.2-1) for the updated
release of this package.

- E: no-ldconfig-symlink /usr/lib/libtrash.so.3.2 is still not fixed. This
message only comes up when building the binary 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 452872] Review Request: bmake - The NetBSD make(1) tool

2008-07-01 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: bmake - The NetBSD make(1) tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 13:29 EST ---
Some random comments 0.20080515-1:

* Versioning
  - If you think %bmake_date should not be treated as a formal
version, then versioning this package as
0-0.X.%{bmake_date}%{?dist}
(i.e. version 0, release number 0.X.%{bmake_date} with suffix
  %{?dist} where X is incremented every time you modify
  your spec file)
is better.

  - Also please remove Epoch.

* BuildRequires
  - This does not build on dist-f10:
http://koji.fedoraproject.org/koji/taskinfo?taskID=687706
BuildRequires: util-linux is needed (on F-10 util-linux-ng
Provides util-linux)

* %prep usage

%setup -q -n bmake
tar -x -z -f %{SOURCE1}

  - can be replaced by

%setup -q -n %{name} -a 1


* %defattr
  - Now we recommend %defattr(-,root,root,-)

* Directory issue
  - The man directory %{_mandir}/cat1 is nonusual. Please move
the man file to section 1 or explain why you want to create
cat1 directory.

* Permission issue
  - The directory %{_datadir}/mk has 0775 permission, which should usually
be 0755.

  - And bmake.1 has 0444 permission, which should be 0644.

! Timestamps
  - This package installs many .mk files which are not modified during
build and preserving timestamps on those files are preferable.
Consider to add
-
sed -i.timestamp -e 's|^cp_f=-f|cp_f=-pf|' mk/install-mk
-
at %prep to keep timestamps on those files.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 453287] Review Request: perl-Term-ReadLine-Gnu - Perl extension for the GNU Readline/History Library

2008-07-01 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-Term-ReadLine-Gnu - Perl extension for the GNU 
Readline/History Library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 13:30 EST ---
Soo there's probably a better solution to this problem: bug #453646
Hopefully the maintainers (I've mailed them) won't mind if that gets fixed with
https://admin.fedoraproject.org/updates/F9/FEDORA-2008-5875

-- 
Configure 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-07-01 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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|182235  |
  nThis||




-- 
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 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment

2008-07-01 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: ristretto - Image-viewer for the Xfce desktop 
environment


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 13:39 EST ---
(In reply to comment #52)
 Hehe, you could better pick the diff from that revision and apply it against
 0.0.20.

Ok. To be honest I did not know what you meant by cherry-picking, but now I got
it :)

New SRPM:
http://cwickert.fedorapeople.org/review/ristretto-0.0.20-2.fc10.src.rpm
SPEC: http://cwickert.fedorapeople.org/review/ristretto.spec

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 452901] Review Request: ocspd - OpenCA OCSP Daemon

2008-07-01 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: ocspd - OpenCA OCSP Daemon


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 13:40 EST ---
Thanks for the legal advice.
License change applied.

-- 
Configure 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 452470] Review-Request: perl-Test-WWW-Selenium - Perl Client for the Selenium Remote Control test tool

2008-07-01 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-Test-WWW-Selenium - Perl Client for the Selenium 
Remote Control test tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 14:01 EST ---
Imported and built. 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


[Bug 447751] Review Request: diary - A application to allow you to keep a diary of your life

2008-07-01 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: diary - A application to allow you to keep a diary of 
your life


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 13:58 EST ---
What are the problems with having a generically-named package? Before I named
the project (I'm the developer), I checked and I could find no projects which
use the name diary, and it hasn't collided with anything 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 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment

2008-07-01 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: ristretto - Image-viewer for the Xfce desktop 
environment


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 14:00 EST ---
OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name.
OK - Spec has consistant macro usage.
OK - Meets Packaging Guidelines.
OK - License (GPLv2+)
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
12eaaea1aaaea024ad3ce8a114ff9e6d  ristretto-0.0.20.tar.gz
12eaaea1aaaea024ad3ce8a114ff9e6d  ristretto-0.0.20.tar.gz.orig
OK - BuildRequires correct
OK - Spec handles locales/find_lang
OK - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Package has correct buildroot
OK - Package is code or permissible content.
OK - Packages %doc files don't affect runtime.
OK - Package has rm -rf RPM_BUILD_ROOT at top of %install

OK - Package is a GUI app and has a .desktop file

OK - Package compiles and builds on at least one arch.
OK - Package has no duplicate files in %files.
OK - Package doesn't own any directories other packages own.
OK - Package owns all the directories it creates.
See below - No rpmlint output.
OK - final provides and requires are sane.

SHOULD Items:

OK - Should build in mock.
OK - Should build on all supported archs
OK - Should function as described.
OK - Should have sane scriptlets.
OK - Should have dist tag
OK - Should package latest version

Issues:

1. rpmlint says:

ristretto.i386: W: incoherent-version-in-changelog 0.0.20.2 0.0.20-2.fc10
You have a typo there... a . that should be a -

Thats super minor, fix when you import?
This package is APPROVED.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment

2008-07-01 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: ristretto - Image-viewer for the Xfce desktop 
environment


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 14:09 EST ---
(In reply to comment #54)
 ristretto.i386: W: incoherent-version-in-changelog 0.0.20.2 0.0.20-2.fc10
 You have a typo there... a . that should be a -
 
 Thats super minor, fix when you import?

Will do, good catch.


New Package CVS Request
===
Package Name: ristretto
Short Description:  Image-viewer for the Xfce desktop environment
Owners: cwickert
Branches: F-8 F-9
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 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment

2008-07-01 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: ristretto - Image-viewer for the Xfce desktop 
environment


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Attachment #265451|0   |1
is obsolete||




-- 
Configure 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 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment

2008-07-01 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: ristretto - Image-viewer for the Xfce desktop 
environment


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Attachment #295114|0   |1
is obsolete||




-- 
Configure 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 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment

2008-07-01 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: ristretto - Image-viewer for the Xfce desktop 
environment


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Attachment #306585|0   |1
is obsolete||




-- 
Configure 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 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment

2008-07-01 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: ristretto - Image-viewer for the Xfce desktop 
environment


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Attachment #295113|0   |1
is obsolete||




-- 
Configure 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 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment

2008-07-01 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: ristretto - Image-viewer for the Xfce desktop 
environment


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 14:13 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 452901] Review Request: ocspd - OpenCA OCSP Daemon

2008-07-01 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: ocspd - OpenCA OCSP Daemon


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 14:15 EST ---
Would you update the new srpm then? 
(note: please change the release number every time you modify your package 
   when version number does not change, otherwise we are just confused.
   Also, if this uses a release candidate tarball, the current versioning
   does not follow Fedora naming guidelines:
  
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Pre-Release_packages )

Some very quick random remarks (only just watching your spec file)
* %SOURCE must be given with full URL:
   https://fedoraproject.org/wiki/Packaging/SourceURL

* Please do not specify Vendor: item. Fedora builder automatically tags this 
item as
  Fedora Project.

* %{openssl_req} tag is redundant. Even Fedora 1 has 0.9.7a openssl.

* Also, Requires: openssl should be removed. rpmbuild automatically adds
libraries dependency
  to binary rpms, which will correctly pick out needed openssl.

* Please remove redundant [ -n ${RPM_BUILD_ROOT} -a ${RPM_BUILD_ROOT} != '/'
] 
  part.

* The directory %{_sysconfdir}/ocspd/ is not owned by any package.

* Perhaps %{_sysconfdir}/ocspd/ocspd.conf is listed twice.

* Please use %{_initrddir} instead of %{_sysconfdir}/rc.d/init.d

* We now recommend %defattr(-,root,root,-)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment

2008-07-01 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: ristretto - Image-viewer for the Xfce desktop 
environment


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 14:22 EST ---
If you want to make any changes before committing, please press Ctrl-C.
Otherwise press Enter to proceed to commit.

cvs commit...

 Access denied: cwickert is not in ACL for rpms/ristretto/devel
cvs commit: Pre-commit check failed
cvs [commit aborted]: correct above errors first!
cvs commit: saving log message in /tmp/cvsYbu2e5

What's wrong? Where to proceed after this is fixed?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment

2008-07-01 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: ristretto - Image-viewer for the Xfce desktop 
environment


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 14:29 EST ---
acls only sync 2x/hour... wait about another 5minutes and try again?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 452387] Review Request: netris - A console-based, networked clone of Tetris

2008-07-01 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: netris - A console-based, networked clone of Tetris


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 14:32 EST ---
Thank you for reply. I have chosen netblocks as the new name. At the moment
I'm patching documentation. Do I have to include some copyright information
about Tetris if I say similar to Tetris (tm)?

-- 
Configure 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 450189] Review Request: guake - Drop-down terminal for GNOME

2008-07-01 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: guake - Drop-down terminal for GNOME


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 14:42 EST ---
Spec URL: http://lokthare.fedorapeople.org/temp/guake.spec
SRPM URL: http://lokthare.fedorapeople.org/temp/guake-0.2.2-4.fc9.src.rpm
- Add BR for GConf
- Fix schemas file

(In reply to comment #10)
 Note: as you are already sponsored, you can check if your srpm actually 
 builds on
 koji beforehand by:
 $ koji build --scratch build target srpm you want to try building
 where build target can be either dist-f10, dist-f9-updates-candidate or
 dist-f8-updates-candidate.
 When the build is successful, the rebuilt binary rpms and some logs are put 
 under
 http://koji.fedoraproject.org/scratch/your FAS name/task_task id/ .

Thanks you for the tip.
I have build guake with koji without problems.
http://koji.fedoraproject.org/koji/taskinfo?taskID=690316

-- 
Configure 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 452387] Review Request: netris - A console-based, networked clone of Tetris

2008-07-01 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: netris - A console-based, networked clone of Tetris


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 14:41 EST ---
Have you perhaps seen http://savannah.nongnu.org/projects/netblocks ?

-- 
Configure 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 452714] Review Request: Quassel - Distributed IRC Client and Core application

2008-07-01 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: Quassel - Distributed IRC Client and Core application


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


[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 452387] Review Request: netris - A console-based, networked clone of Tetris

2008-07-01 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: netris - A console-based, networked clone of Tetris


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 14:53 EST ---
I saw that, but there weren't any releases. I looked at their SVN and it was
empty. Now, after a longer investigation I have found http://games.court-empty. 
Now, after a longer investigation I have found 
http://games.court-jus.net/netblocks#attachments . It seems like I need to find 
another name.

-- 
Configure 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 452714] Review Request: Quassel - Distributed IRC Client and Core application

2008-07-01 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: Quassel - Distributed IRC Client and Core application


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 14:53 EST ---
1.  MUST: use %{_bindir} instead of /usr/bin in %files

2.  MUST: Refer to qt4, not qt, to avoid ambiguity.  In short, s/qt-/qt4-/

3.  MUST: drop kde4_macros_api business.  This is a qt-only app, not a kde4 
one.

4.  SHOULD:  use %{_qt4_qmake} macro instead of qmake-qt4


rpmlint on latest F-9 koji scratch builds: (x86_64)
$ rpmlint *.rpm
quassel.src: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 7)
quassel.src: E: description-line-too-long Quassel IRC is a modern, distributed 
IRC client, meaning that one (or multiple) client(s) can attach to and detach 
from a central core --
quassel.src: E: description-line-too-long much like the popular combination of 
screen and a text-based IRC client such as WeeChat, but graphical
quassel.src: W: no-version-in-last-changelog
quassel.src: W: strange-permission quassel.spec 0777
quassel.x86_64: E: description-line-too-long Quassel IRC is a modern, 
distributed IRC client, meaning that one (or multiple) client(s) can attach to 
and detach from a central core --
quassel.x86_64: E: description-line-too-long much like the popular combination 
of screen and a text-based IRC client such as WeeChat, but graphical
quassel.x86_64: W: no-version-in-last-changelog
quassel-client.x86_64: W: no-version-in-last-changelog
quassel-core.x86_64: E: description-line-too-long The Quassel IRC Core 
maintains a connection with the server, and allows for multiple clients to 
connect
quassel-core.x86_64: W: no-version-in-last-changelog
4 packages and 0 specfiles checked; 5 errors, 6 warnings.


I'll review a bit more after trying these out to see how well things work.


-- 
Configure 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-07-01 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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 15:07 EST ---
I understand.  I'm in no hurry.  I have to wait for the next findbugs release
anyway, which will include some patches I pushed upstream that fix a handful of
build/deployment problems on Fedora.  Thanks again.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 452714] Review Request: Quassel - Distributed IRC Client and Core application

2008-07-01 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: Quassel - Distributed IRC Client and Core application


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 15:33 EST ---
Thanks for the comments.  Uploaded revised SPEC and SRPMS files to
http://tuxbrewr.fedorapeople.org/quassel/


Build logs:
F9: http://koji.fedoraproject.org/koji/taskinfo?taskID=690433
Rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=690451


-- 
Configure 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 452714] Review Request: Quassel - Distributed IRC Client and Core application

2008-07-01 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: Quassel - Distributed IRC Client and Core application


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 15:44 EST ---
seemed to miss item 2.

5. MUST:  .desktop files
Having tried things out a bit, I noticed the included .desktop files are missing
any Category tags, so they don't appear in the menus anywhere.  I'd suggest 
adding
Categories=Qt;Network;InstantMessaging;

-- 
Configure 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 446637] Review Request: gssdp - GSSDP implements resource discovery and announcement over SSDP

2008-07-01 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: gssdp - GSSDP implements resource discovery and 
announcement over SSDP


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 15:48 EST ---
New location:
SPEC: http://pbrobinson.fedorapeople.org/gssdp.spec
SRPM: http://pbrobinson.fedorapeople.org/gssdp-0.6.1-1.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 452387] Review Request: netris - A console-based, networked clone of Tetris

2008-07-01 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: netris - A console-based, networked clone of Tetris


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 15:55 EST ---
Finally, I think everything in docs and software is renamed.

Spec URL: http://rydzyk.net/fedora/netblox.spec
SRPM URL: http://rydzyk.net/fedora/netblox-0.52-4.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 447456] Review Request: gupnp-tools: a collection of dev tools utilising GUPnP and GTK+

2008-07-01 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: gupnp-tools: a collection of dev tools utilising GUPnP 
and GTK+


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 15:54 EST ---
New location

SPEC: http://pbrobinson.fedorapeople.org/gupnp-tools.spec
SRPM: http://pbrobinson.fedorapeople.org/gupnp-tools-0.6-1.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 447457] Review Request: gupnp-av - UPnP-AV is a collection of helpers for building AV upnp apps

2008-07-01 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: gupnp-av - UPnP-AV is a collection of helpers for 
building AV upnp apps


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 15:53 EST ---
New location

SPEC: http://pbrobinson.fedorapeople.org/gupnp-av.spec
SRPM: http://pbrobinson.fedorapeople.org/gupnp-av-0.2.1-1.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 446639] Review Request: gupnp - GUPnP is an framework for creating UPnP devices control points

2008-07-01 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: gupnp - GUPnP is an framework for creating UPnP 
devices  control points


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 15:52 EST ---
New location:

SPEC: http://pbrobinson.fedorapeople.org/gupnp.spec
SRPM: http://pbrobinson.fedorapeople.org/gupnp-0.12.1-1.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 452387] Review Request: netblox - A console-based, networked falling blocks game

2008-07-01 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: netblox - A console-based, networked falling blocks 
game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: netris - A  |Review Request: netblox - A
   |console-based, networked|console-based, networked
   |clone of Tetris |falling blocks game




-- 
Configure 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 452714] Review Request: Quassel - Distributed IRC Client and Core application

2008-07-01 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: Quassel - Distributed IRC Client and Core application


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 16:03 EST ---
I'll trust you to make those changes before import (or build).  APPROVED.

(and thanks to Charles, comment #1, for the partial review)

It's obviously alpha/beta quality atm, but it shows promise.

-- 
Configure 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 452714] Review Request: Quassel - Distributed IRC Client and Core application

2008-07-01 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: Quassel - Distributed IRC Client and Core application


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 16:06 EST ---
Changes made  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


[Bug 452714] Review Request: Quassel - Distributed IRC Client and Core application

2008-07-01 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: Quassel - Distributed IRC Client and Core application


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 16:11 EST ---
New Package CVS Request
===
Package Name: quassel
Short Description: Distributed IRC Client and Core application
Owners: tuxbrewr
Branches: F-8 F-9
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 450323] Review Request: coq - Coq proof management system

2008-07-01 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: coq - Coq proof management system


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 16:14 EST ---
I made one last change to the SPEC file and source RPM: I was unclear as to the 
nature of the message from rpmlint that only binaries should go in /usr/lib, 
as I had only compiled Coq .vo files in /usr/lib. As it turns out that only 
architecture dependent binary files should go in /usr/lib, I moved all these to 
%{_datadir} (which is usually /usr/share) and the message from rpmlint was then 
fixed. The new SPEC file and source RPM remain at the same location at the 
duke.edu address (and the old ones were moved to .v1 and so forth as before).

-- 
Configure 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 445537] Review Request: tightvnc - VNC software

2008-07-01 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: tightvnc - VNC software


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 16:24 EST ---
rpmlint returns:
tightvnc.src:164: E: files-attr-not-set
tightvnc.src:165: E: files-attr-not-set
tightvnc.src:166: E: files-attr-not-set
tightvnc.src:167: E: files-attr-not-set
tightvnc.src: W: strange-permission vncserver.init 0755
1 packages and 0 specfiles checked; 4 errors, 1 warnings.

You forgot to do:
%defattr(-,root,root,-)
in the main %files section

The strange permission warning is, I think, because all files in your package
have 0664 (note the group-write permission)

-- 
Configure 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-07-01 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-07-01 16:38 EST ---
Thanks spot.
Still I'm missing someone for a review here..

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 421241] Review Request: php-ZendFramework - Leading open-source PHP framework

2008-07-01 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-ZendFramework - Leading open-source PHP framework


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 16:35 EST ---
Hi Jess, any news?

I just did a test run on revision 9862 but getTmpDir() doesn't seem to be in
usage yet with the exception of a few optional tests.

Regards
Alex

-- 
Configure 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 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment

2008-07-01 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: ristretto - Image-viewer for the Xfce desktop 
environment


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-01 16:41 EST ---
ristretto-0.0.20-2.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


  1   2   >