[Bug 238367] Review Request: xu4 - Ultima IV recreated

2007-05-11 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: xu4 - Ultima IV recreated
Alias: xu4

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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 03:49 EST ---
(In reply to comment #4)
 I'm thinking that it might not be such a good idea to ship an old beta instead
 of a CVS pull.  In fact I'm not sure beta3 is even winnable (since you can't
 talk to Hawkwind.)

Okay, this new version is based on CVS now

(In reply to comment #16 and #18)
  MUST FIX 
 - Add Requires: desktop-file-utils
Fixed

  SHOULD FIX 
 please also add a comment in spec why you use %{makeinstall}
Fixed


New version here:
Spec URL: http://people.atrpms.net/~hdegoede/xu4.spec
SRPM URL: http://people.atrpms.net/~hdegoede/xu4-1.1-0.1.20070510.fc7.src.rpm

Notice that this now has version 1.1, as that is how the CVS tree identifies
itself in the games about box.



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239770] New: Review Request: gsm - GSM speech compressor library

2007-05-11 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/bugzilla/show_bug.cgi?id=239770

   Summary: Review Request: gsm - GSM speech compressor library
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://rpm.greysector.net/extras/gsm.spec
SRPM URL: http://rpm.greysector.net/extras/gsm-1.0.12-2.src.rpm
Description:

libgsm, an implementation of the European GSM 06.10 provisional
standard for full-rate speech transcoding, prI-ETS 300 036,
which uses RPE/LTP (residual pulse excitation/long term prediction)
coding at 13 kbit/s.

GSM 06.10 compresses frames of 162 13-bit samples (8 kHz sampling
rate, i.e. a frame rate of 50 Hz) into 260 bits; for compatibility
with typical UNIX applications, our implementation turns frames of 160
16-bit linear samples into 33-byte frames (1650 Bytes/s).
The quality of the algorithm is good enough for reliable speaker
recognition; even music often survives transcoding in recognizable
form (given the bandwidth limitations of 8 kHz sampling rate).

The interfaces offered are a front end modelled after compress(1), and
a library API.  Compression and decompression run faster than realtime
on most SPARCstations.  The implementation has been verified against the
ETSI standard test patterns.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 206871] Review Request: ekg2 - Multi-protocol instant messaging and chat client

2007-05-11 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: ekg2 - Multi-protocol instant messaging and chat client


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn|228186  |239770




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239770] Review Request: gsm - GSM speech compressor library

2007-05-11 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: gsm - GSM speech compressor library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||206871
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 228186] Review Request: gsm - Shared libraries for GSM speech compressor

2007-05-11 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: gsm - Shared libraries for GSM speech compressor


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|206871  |
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239785] New: Review Request: rott - Rise of the Triad

2007-05-11 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/bugzilla/show_bug.cgi?id=239785

   Summary: Review Request: rott - Rise of the Triad
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://people.atrpms.net/~hdegoede/rott.spec
SRPM URL: http://people.atrpms.net/~hdegoede/rott-1.0-1.fc7.src.rpm
Description:
This is the icculus.org Linux port of Apogee's classic 3d shooter Rise of the
Triad, which has been released under the GPL by Apogee. This version is
enhanced with the high resolution rendering from the winrott port.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 225575] Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client

2007-05-11 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: roundcubemail - Round Cube Webmail is a browser-based 
multilingual IMAP client


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 08:00 EST ---
Applied the above.  I bow to your superior script-fu. :)

Spec URL: http://zanoni.jcomserv.net/extras/roundcubemail/roundcubemail.9.spec
SRPM
URL:http://zanoni.jcomserv.net/extras/roundcubemail/roundcubemail-0.1-0.4.beta2.2.src.rpm



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239800] New: php-pear maintainer updates

2007-05-11 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/bugzilla/show_bug.cgi?id=239800

   Summary: php-pear maintainer updates
   Product: Fedora Core
   Version: devel
  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]


Description of problem:


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239800] php-pear maintainer updates

2007-05-11 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: php-pear maintainer updates


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 09:18 EST ---
Package Change Request
==
Package Name: php-pear
Updated Fedora Owners: [EMAIL PROTECTED],[EMAIL PROTECTED]


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 228303] Review Request: python-tag - Python bindings for TagLib to read and write music files tags

2007-05-11 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-tag - Python bindings for TagLib to read and 
write music files tags


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 10:45 EST ---
It seem that the tagpy-LISENCE file is included in srpm file but not installed
from built rpm package.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 228186] Review Request: gsm - Shared libraries for GSM speech compressor

2007-05-11 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: gsm - Shared libraries for GSM speech compressor


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||239770
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239236] Review Request: conduit - Conduit is a synchronization solution for GNOME

2007-05-11 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: conduit - Conduit is a synchronization solution for 
GNOME


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 11:05 EST ---
it seem that pygoocanvas isn't included yet in branch, to check mock build.
i'll re-try this night or tommorrow.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239812] Review Request: nagi - An interpreter for AGI games

2007-05-11 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: nagi - An interpreter for AGI games


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239811] Review Request: agistudio - AGI integrated development environment

2007-05-11 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: agistudio - AGI integrated development environment


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 10:54 EST ---
nice...
i'll check this out

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239156] Review Request: perl-DBIx-SQLite-Simple - Easy access to SQLite databases using objects

2007-05-11 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-DBIx-SQLite-Simple -  Easy access to SQLite 
databases using objects
Alias: DBIx-SQLite-Simple

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 10:46 EST ---
I botched the last one:

New Package CVS Request
===
Package Name: perl-DBIx-SQLite-Simple
Short Description: Easy Access to SQLite databases using objects
Owners: [EMAIL PROTECTED]
Branches: FC-5 FC-6 EL-4 EL-5
InitialCC: [EMAIL PROTECTED]

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 237373] Review Request: mcpp - Alternative C/C++ preprocessor

2007-05-11 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: mcpp -  Alternative C/C++ preprocessor


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 10:44 EST ---
New Package CVS Request
===
Package Name: mcpp
Short Description: Alternative C/C++ preprocessor
Owners: [EMAIL PROTECTED]
Branches: FC-6
InitialCC: 


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239157] Review Request: perl-Net-Pcap - Interface to pcap LBL packet capture library

2007-05-11 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-Net-Pcap - Interface to pcap LBL packet capture 
library
Alias: perl-Net-Pcap

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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 10:34 EST ---
Updated: 

- Add missing BR
- Chance License to GPL or Artistic
- Update %%files
- Include t/ in %%doc

Spec URL: http://folk.ntnu.no/sindrb/packages/green_nyc/perl-Net-Pcap.spec
SRPM URL:
http://folk.ntnu.no/sindrb/packages/green_nyc/perl-Net-Pcap-0.14-2.fc7.src.rpm


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239811] New: Review Request: agistudio - AGI integrated development environment

2007-05-11 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/bugzilla/show_bug.cgi?id=239811

   Summary: Review Request: agistudio - AGI integrated development
environment
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://zanoni.jcomserv.net/fedora/agistudio/agistudio.spec
SRPM URL: http://zanoni.jcomserv.net/fedora/agistudio/agistudio-1.2.3-1.src.rpm
Description: AGI (Adventure Game Interpreter) is the adventure game engine used 
by
Sierra On-Line(tm) to create some of their early games. QT AGI Studio
is a program which allows you to view, create and edit AGI games.

This can use either NAGI or Sarien as a testing interpreter.  I will be 
submitting NAGI for review also.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239525] Review Request: gammu - Command Line utility and Libraries to work with mobile phones

2007-05-11 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: gammu - Command Line utility and Libraries to work 
with mobile phones


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 10:15 EST ---
From rpmlint 0.80-1 changelog:
- Filter messages about doc file dependencies on /bin/sh.

FWIW, I don't think a dependency on /bin/sh is anything to worry about, doc or 
not.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239812] New: Review Request: nagi - An interpreter for AGI games

2007-05-11 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/bugzilla/show_bug.cgi?id=239812

   Summary: Review Request: nagi - An interpreter for AGI games
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://zanoni.jcomserv.net/fedora/nagi/nagi.spec
SRPM URL: http://zanoni.jcomserv.net/fedora/nagi/nagi-0.1-2002114.src.rpm
Description: NAGI is an interpreter for AGI games, such as the early Space 
Quest,
Leisure Suit Larry and King's Quest games.

This can run content created with agistudio, which is also up for review.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239185] Review Request: nbtscan - Tool to gather NetBIOS info from Windows networks

2007-05-11 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: nbtscan - Tool to gather NetBIOS info from Windows 
networks


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 11:15 EST ---
New Package CVS Request
===
Package Name: nbtscan
Short Description: Tool to gather NetBIOS info from Windows Networks
Owners: [EMAIL PROTECTED]
Branches: FC-5 FC-6 EL-4 EL-5
InitialCC: [EMAIL PROTECTED]

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 228303] Review Request: python-tag - Python bindings for TagLib to read and write music files tags

2007-05-11 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-tag - Python bindings for TagLib to read and 
write music files tags


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 10:53 EST ---
D'oh! :-)

Spec URL: http://ftp.es6.freshrpms.net/tmp/extras/python-tag/python-tag.spec
SRPM URL:
http://ftp.es6.freshrpms.net/tmp/extras/python-tag/python-tag-0.91-3.src.rpm

* Fri May 11 2007 Matthias Saou http://freshrpms.net/ 0.91-3
- Actually include LICENSE file.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 236177] Review Request: perl-NetAddr-IP - Manages IPv4 and IPv6 addresses and subnets

2007-05-11 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-NetAddr-IP - Manages IPv4 and IPv6 addresses and 
subnets
Alias: perl-NetAddr-IP

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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 11:34 EST ---
ping?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239157] Review Request: perl-Net-Pcap - Interface to pcap LBL packet capture library

2007-05-11 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-Net-Pcap - Interface to pcap LBL packet capture 
library
Alias: perl-Net-Pcap

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 11:20 EST ---
APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 230549] Review Request: oxine - Lightweight, purely OSD based xine frontend

2007-05-11 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: oxine - Lightweight, purely OSD based xine frontend


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 11:20 EST ---
Spec URL: http://ftp.es6.freshrpms.net/tmp/extras/oxine/oxine.spec
SRPM URL: http://ftp.es6.freshrpms.net/tmp/extras/oxine/oxine-0.6.6-2.src.rpm

* Fri May 11 2007 Matthias Saou http://freshrpms.net/ 0.6.6-2
- Fix black screen problem by requiring xine-lib-extras, where
  xineplug_decode_image.so is available.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 238562] Review Request: machineball - A futuristic ball game with simple rules

2007-05-11 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: machineball - A futuristic ball game with simple rules


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 11:22 EST ---
(In reply to comment #8)
 The fix is easy, the real version is 1.0, the added -1 in the src tarbal (...)

Right. The easiest solutions are the best ones ;)

I like the package now and nothing stands in the way to approve it.
The game looks quite nice, but it's a pity that there's no AI mode ;)

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239525] Review Request: gammu - Command Line utility and Libraries to work with mobile phones

2007-05-11 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: gammu - Command Line utility and Libraries to work 
with mobile phones


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 11:59 EST ---
(In reply to comment #11)
 From rpmlint 0.80-1 changelog:
 - Filter messages about doc file dependencies on /bin/sh.
 
 FWIW, I don't think a dependency on /bin/sh is anything to worry about, doc or
not.

hmm. I was expecting something similar. Today was so busy working day that I
couldn't got time to upgrade rpmlint on FC6 machine.
So new version was already available. Ok

why don't you try to move then executable scripts to /etc then?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 238562] Review Request: machineball - A futuristic ball game with simple rules

2007-05-11 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: machineball - A futuristic ball game with simple rules


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 11:39 EST ---
New Package CVS Request
===
Package Name:  machineball
Short Description: A futuristic ball game with simple rules
Owners:[EMAIL PROTECTED]
Branches:  FC-6 devel
InitialCC: empty




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239157] Review Request: perl-Net-Pcap - Interface to pcap LBL packet capture library

2007-05-11 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-Net-Pcap - Interface to pcap LBL packet capture 
library
Alias: perl-Net-Pcap

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 11:29 EST ---
New Package CVS Request
===
Package Name: perl-Net-Pcap
Short Description: Interface to pcap LBL packet capture library
Owners: [EMAIL PROTECTED]
Branches: FC-5 FC-6 EL-4 EL-5
InitialCC: [EMAIL PROTECTED]

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239236] Review Request: conduit - Conduit is a synchronization solution for GNOME

2007-05-11 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: conduit - Conduit is a synchronization solution for 
GNOME


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 12:12 EST ---
You can grab the buildsys rpms here:

http://buildsys.fedoraproject.org/plague-results/fedora-6-extras/pygoocanvas/0.6.0-2.fc6/
or for F7 here:
http://koji.fedoraproject.org/koji/buildinfo?buildID=6421

I'm leaving town May 12 and will not be able to do any updates until May 21 so
take your time.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 205755] Review Request: elsa - manages group of processes and allows accounting

2007-05-11 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: elsa - manages group of processes and allows accounting


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 13:07 EST ---
So(In reply to comment #31)

 
 All that needs to be done here is to build the package with 'make tag build' 
 in
 package directory and close this bug as Resolved: RAWHIDE
 

Who has that privilage to do that?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 236162] Review Request: kadischi - Fedora based LiveCD/LiveDVD creation utility

2007-05-11 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: kadischi - Fedora based LiveCD/LiveDVD creation utility


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 13:21 EST ---
All issues have been corrected.
The new SPEC and SRPM are located here:

http://autopsy.podzone.org/~autopsy/kadischi.spec
http://autopsy.podzone.org/~autopsy/kadischi-3.6-2.20070511cvs.fc6.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239770] Review Request: gsm - GSM speech compressor library

2007-05-11 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: gsm - GSM speech compressor library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 13:29 EST ---
New Package CVS Request
===
Package Name: gsm
Short Description: GSM speech compressor library
Owners: [EMAIL PROTECTED]
Branches: FC-5 FC-6
InitialCC: 


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239770] Review Request: gsm - GSM speech compressor library

2007-05-11 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: gsm - GSM speech compressor library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 13:32 EST ---
Thank you, Michael.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 223557] Review Request: mdbtools - tools for extracting things from Access databases

2007-05-11 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: mdbtools - tools for extracting things from Access 
databases


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NOTABUG
OtherBugsDependingO|163778  |201449
  nThis||
   Flag|fedora-review?  |fedora-review-




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 13:43 EST ---
No response, closing.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 224245] Merge Review: squirrelmail

2007-05-11 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: squirrelmail


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 225522] Review Request: cinepaint - CinePaint is a tool for manipulating images

2007-05-11 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: cinepaint - CinePaint is a tool for manipulating images


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
OtherBugsDependingO|163776  |163778
  nThis||
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 14:50 EST ---
Well, I use rawhide and so I have not checked on
FC-6.

On rawhide, 0.22.0-2 contained several problems.

---
* Compilation
  - Actually
 yes indeed...(was still working on it but was still 
 failling on icc_examin on devel). 
This needs a patch (Patch0)

* Directory ownership issue
  - Some directories were owned properly.

* Undefined non-weak symbols
  - Some libraries in cinepaint contained undefined non-weak
symbols. 
This cannot be accepted for a package which provides
-devel subpackage because leaving these symbols leads to
linkage failure

* Unnecessary/missing (Build)Requires
* Unnecessary %Requires(postun) etc
* Use of binaries for priviledged user
  - Binaries for priviledged user (i.e. binaries under
/sbin /usr/sbin etc..) must be specified by full path.
Otherwise, calling these binaries fails on sudo user,
for example.

* More pkgconfig .pc file fix was needed
* More proper fix for rpath was recommended
  - i.e. don't use chrpath when possible
* Unneeded autoconf call
* Timestamp keeping
* desktop file fix
* Requires:
  - What does Requires: freefont mean?? On my system no package
provides freefont. 
* Build log should be more verbose
---

I hope I fixed the issues above.
http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/extras/development/SPECS/cinepaint.spec
http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/extras/development/SRPMS/cinepaint-0.22.0-3.fc7.src.rpm

mock build on FC-devel i386 is successful.
http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/extras/development/LOGS/MOCK-cinepaint.log

Please check if my spec/srpm is what you expect. If you notice
something, please fix/modify my spec/srpm and upload your spec/srpm

---
NOTE
There are still some items I have to check for this package.
My package may have some problem, so please check my spec/srpm.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 234667] Review Request: dwarves - DWARF Tools

2007-05-11 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: dwarves - DWARF Tools


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 15:13 EST ---
http://oops.ghostprotocols.net:81/acme/dwarves/rpm/SRPMS/ is being kept
frequently  updated with git snapshots. I've addressed most of the above
problems. Now waiting for a new round of review.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 238367] Review Request: xu4 - Ultima IV recreated

2007-05-11 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: xu4 - Ultima IV recreated
Alias: xu4

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 16:29 EST ---
 REVIEW GUIDELINES 
- rpmlint output clean
- package named according to package naming guidelines
- package meets packaging guidelines
- package licensed with open source compatible license
- license matches actual license
- license file included in %doc
- spec written in American english
- spec file legible
- sources match upstream
# diff -ubBr /home/fedora/rpmbuild/SPECS/u4/ /home/chris/cvs/u4/
- package successfully compiles on x86_64 FC6
- all build dependencies in BR
- no locales
- no shared libraries
- package is not relocatable
- package owns all directories it creates
- directories it does not create are brought in through Requires
- no duplicates in %files
- file permissions set properly
- contains proper %clean
- macro usage consistent
- contains code
- no large documentation
- files in %doc do not affect runtime
- no header files
- no static libraries
- no pkgconfig files
- no library with suffix
- no need for devel subpackage
- no libtool archives
- contains proper .desktop file
- does not own files or directories owned by other packages
- contains proper install
- all filenames utf-8

 SHOULD FIX 
- spec comments should show how to check out code from the cvs on or before the
date of cvsdate.  I think cvs ... -D %{cvsdate} might work.

 APPROVED 


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 238367] Review Request: xu4 - Ultima IV recreated

2007-05-11 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: xu4 - Ultima IV recreated
Alias: xu4

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 16:34 EST ---
New Package CVS Request
===
Package Name:  xu4
Short Description: Ultima IV recreated
Owners:[EMAIL PROTECTED]
Branches:  FC-6 devel
InitialCC: empty


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220862] Review Request: php-channel-phpunit - Add phpunit channel to pear

2007-05-11 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-channel-phpunit - Add phpunit channel to pear
Alias: channel-phpunit

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 196749] Review Request: php-pecl-xdebug - PECL package for debugging PHP scripts

2007-05-11 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-pecl-xdebug - PECL package for debugging PHP 
scripts
Alias: php-pecl-xdebug

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222598] Review Request: php-pecl-radius - Radius client library

2007-05-11 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-pecl-radius - Radius client library
Alias: php-pecl-radius

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 233455] Review Request: autodir - Creates user directories on demand

2007-05-11 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: autodir - Creates user directories on demand


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 236366] Review Request: clutter-gtk - basic GTK clutter widget

2007-05-11 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: clutter-gtk - basic GTK clutter widget


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 237380] Review Request: ruby-gettext-package - Localization Library and Tools for Ruby

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

Summary: Review Request:  ruby-gettext-package - Localization Library and Tools 
for Ruby
Alias: ruby-gettext-package

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 238367] Review Request: xu4 - Ultima IV recreated

2007-05-11 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: xu4 - Ultima IV recreated
Alias: xu4

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 238562] Review Request: machineball - A futuristic ball game with simple rules

2007-05-11 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: machineball - A futuristic ball game with simple rules


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239096] Review Request: perl-libwhisker2 - Perl module geared specificly for HTTP testing

2007-05-11 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-libwhisker2 - Perl module geared specificly for 
HTTP testing


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239156] Review Request: perl-DBIx-SQLite-Simple - Easy access to SQLite databases using objects

2007-05-11 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-DBIx-SQLite-Simple -  Easy access to SQLite 
databases using objects
Alias: DBIx-SQLite-Simple

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239883] New: Review Request: powertop - power usage tool

2007-05-11 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/bugzilla/show_bug.cgi?id=239883

   Summary: Review Request: powertop - power usage tool
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://people.redhat.com/ajackson/powertop.spec
SRPM URL: http://people.redhat.com/ajackson/powertop-1.0-1.src.rpm

Description: powertop is a tool that monitors kernel wakeups and accounts them 
to userspace processes.  You can use this information to find out why your CPU 
isn't able to go to sleep, and therefore why your battery life is so bad.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239157] Review Request: perl-Net-Pcap - Interface to pcap LBL packet capture library

2007-05-11 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-Net-Pcap - Interface to pcap LBL packet capture 
library
Alias: perl-Net-Pcap

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239161] Review Request: perl-Net-Libdnet - Perl Interface to libdnet

2007-05-11 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-Net-Libdnet - Perl Interface to libdnet
Alias: perl-Net-Libdnet

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239185] Review Request: nbtscan - Tool to gather NetBIOS info from Windows networks

2007-05-11 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: nbtscan - Tool to gather NetBIOS info from Windows 
networks


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239770] Review Request: gsm - GSM speech compressor library

2007-05-11 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: gsm - GSM speech compressor library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239800] php-pear maintainer updates

2007-05-11 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: php-pear maintainer updates


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239884] New: Review Request: liberation-fonts - Fonts to replace commonly used Microsoft Windows Fonts

2007-05-11 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/bugzilla/show_bug.cgi?id=239884

   Summary: Review Request: liberation-fonts -  Fonts to replace
commonly used Microsoft Windows Fonts
   Product: Fedora Core
   Version: devel
  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/mclasen/liberation-fonts.spec
SRPM URL: http://people.redhat.com/mclasen/liberation-fonts-0.1-5.fc7.src.rpm
Description: 

The Liberation Fonts are intended to be replacements for the three
most commonly used fonts on Microsoft systems: Times New Roman,
Arial, and Courier New.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239884] Review Request: liberation-fonts - Fonts to replace commonly used Microsoft Windows Fonts

2007-05-11 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: liberation-fonts -  Fonts to replace commonly used 
Microsoft Windows Fonts


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 17:48 EST ---
Just FYI, the font config configuration that we are planning for the liberation
fonts is to add them to the lists of metrically equivalent fonts in 
/etc/fonts/conf.d/30-aliases-fedora.conf

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 228303] Review Request: python-tag - Python bindings for TagLib to read and write music files tags

2007-05-11 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-tag - Python bindings for TagLib to read and 
write music files tags


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 17:55 EST ---
^^

Well,
In addition/resume to the full review above:

+
OK -- Lisence is BSD
OK -- Lisence file included in package




 *APPROVED*


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 228303] Review Request: python-tag - Python bindings for TagLib to read and write music files tags

2007-05-11 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-tag - Python bindings for TagLib to read and 
write music files tags


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 17:55 EST ---
^^

Well,
In addition/resume to the full review above:

+
OK -- Lisence is BSD
OK -- Lisence file included in package




 *APPROVED*


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239883] Review Request: powertop - power usage tool

2007-05-11 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: powertop - power usage tool


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 18:01 EST ---
Checked against http://fedoraproject.org/wiki/Packaging/ReviewGuidelines, and OK
for all MUST points.

Detail from rpmlint:

- rpmlint powertop-1.0-1.fc7.i386.rpm:
  W: powertop no-documentation


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239883] Review Request: powertop - power usage tool

2007-05-11 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: powertop - power usage tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
OtherBugsDependingO||163778
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239883] Review Request: powertop - power usage tool

2007-05-11 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: powertop - power usage tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||fedora-review?




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239236] Review Request: conduit - Conduit is a synchronization solution for GNOME

2007-05-11 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: conduit - Conduit is a synchronization solution for 
GNOME


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 18:27 EST ---
okay, i'll check this out. ;-)

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239892] New: Review Request: eclipse-checkstyle - a checkstyle plugin for eclipse

2007-05-11 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/bugzilla/show_bug.cgi?id=239892

   Summary: Review Request: eclipse-checkstyle - a checkstyle plugin
for eclipse
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: 
http://www.stl.gtri.gatech.edu/rmyers/eclipse-checkstyle/eclipse-checkstyle.spec
SRPM URL: 
http://www.stl.gtri.gatech.edu/rmyers/eclipse-checkstyle/eclipse-checkstyle-4.0.1-1.fc7.src.rpm
Description: The Eclipse Checkstyle plug-in integrates the Checkstyle Java code
auditor into the Eclipse IDE. The plug-in provides real-time feedback
to the user about violations of rules that check for coding style and
possible error prone code constructs. 

Note that you will need this script to build the tar: 
http://www.stl.gtri.gatech.edu/rmyers/eclipse-checkstyle/checkout_and_build_tarball.sh

This is my first package and am seeking a sponsor.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239892] Review Request: eclipse-checkstyle - a checkstyle plugin for eclipse

2007-05-11 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: eclipse-checkstyle - a checkstyle plugin for eclipse


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239163] Review Request: perl-Net-IPv6Addr - Perl module to check validity of IPv6 addresses

2007-05-11 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-Net-IPv6Addr - Perl module to check validity of 
IPv6 addresses
Alias: perl-Net-IPv6Addr

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 19:52 EST ---
This package has a prereq on Math::Base85, but Math::Base85 is currently not
available in the fedora collection.  This is needed both for certain features of
this module as well as to fully enable the test suite.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239193] Review Request: perl-Module-ExtractUse - Find out what modules are used

2007-05-11 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-Module-ExtractUse - Find out what modules are used
Alias: Module-ExtractUse

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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 19:56 EST ---
Apologies for taking so long -- it's been a busy week and I wanted to make sure
I put appropriate thought into this.

(In reply to comment #4)
 TODO is still included in -2.

This is something I'd prefer to leave in, so I don't inadvertently leave it out
in a future release.  I don't have a particularly strong feeling about it,
however, so I'll drop it.

  * tests can make good documentation (sometimes better than the actual docs)
 
 I don't think that's the case in this package.

I actually found t/80_failing.t quite useful in understanding that the parser
will not correctly pick up either 'use constant' or 'use base' constructs.  This
isn't in the documentation, and saved me some pain.
 
  * people might actually want to test the package post-installation
 
 I'm pretty sure that people who install software from rpms don't expect to be
 able to do that, at least when not installing a *-test subpackage.  I'm not
 saying it's necessarily a doomed idea, but should be discussed in distro wide
 context before starting to apply in packages here and there.

Well, it's processes like this that allow us to have the experience to be able
to comment on such distro-wide proposals.  I know I'm varying from the beaten
path here, but it seems like an avenue worth exploring, and am willing to be
flexible as we go along.
 
  * it doesn't hurt anything :)
 
 * It does add files to the package payload that the overwhelming vast majority
   does not have use for.

Technically speaking, 99% of what's in %doc right now the vast majority has no
use for.  Heck, even the minority of fedora packagers probably don't have much
use for it.

 * Test suites are also often quick and dirty code which is not meant to be
   used as an example of anything.

Sometimes.  But some of them are quite excellent -- see, e.g., Moose,
Class::MOP, POE, WWW::Myspace, etc -- and even the quick and dirty ones serve
their purpose.

 * Test suite code is not restricted to using the public API of the software;
   some features can be better tested using knowledge of module internals.  We
   don't want anyone to use such code as an example how to use the packaged 
   software.

I'm not making any judgements as to how people should or shouldn't use software.
 If someone wants to go off and do things the Wrong Way, then a) it doesn't
matter if the test suite is installed or not and b) they're going to do it
anyways.  Enabling people to test their installed s/w isn't going to impact 
that.

 * Test suites have dependencies that the users need to take care of manually,
   or the package will be dependency bloated.  Both are bad, and easily solved
   by not including the test suite.

Hm.  To date, I've been filtering any additional dependencies the suites
introduce  I'm not entirely convinced that this is a bad thing, but it
sounds like an argument for a -test subpackage.

 To summarize, I think including test suite code is acceptable if there's an
 upstream recommendation to use it as an example, upstream installs it too,
 and/or packaging test suites becomes a standard packaging practice (or 
 there's a
 decision/consensus that things should move towards that).

Upstream actually has recommended it in several cases I know (Moose being one),
and others it just makes sense.  I also believe, given the small amount of
additional work needed to enable it, enabling people to test installed software
with the _same_ test suite we used during build is a Good Thing.

Also, as spot pointed out in an email in fedora-perl-devel recently, Blanket
policies help me sleep easier at night.  I'd rather package test suites up
consistently rather than make a subjective judgment call as to their worth.

I know this is a relatively new idea, but I think it's worth exploring.  It's
flexible processes (like the review process) that allow us to gain experience
with various approaches along these lines, rather than waiting/expecting a
directive to be handed down from upon high.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239883] Review Request: powertop - power usage tool

2007-05-11 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: powertop - power usage tool


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 19:59 EST ---
Yeah, there's no documentation yet.  Once there is I'll be happy to add it, but
for now it's just run me and find what's broken.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222960] Review Request: XenMan - Graphical management tool for Xen

2007-05-11 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: XenMan - Graphical management tool for Xen


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|POST




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 20:04 EST ---
Still need to be reviewed (just a lil' up)

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239883] Review Request: powertop - power usage tool

2007-05-11 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: powertop - power usage tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 20:16 EST ---
MUST items:
 - 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 - OK
 - License field in spec matches - OK
 - License file included in package - ***
  *** should include COPYING in doc

 - Spec in American English - OK
 - Spec is legible. - OK
 - Sources match upstream md5sum: - OK

 - Package needs ExcludeArch - N/A
 - BuildRequires correct - OK
 - Spec handles locales/find_lang - N/A
 - Package is relocatable and has a reason to be. - N/A
 - 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
 - Doc subpackage needed/used. - N/A
 - Packages %doc files don't affect runtime. - N/A

 - Headers/static libs in -devel subpackage. - N/A
 - Spec has needed ldconfig in post and postun  - N/A
 - .pc files in -devel subpackage/requires pkgconfig - N/A
 - .so files in -devel subpackage. - N/A
 - -devel package Requires: %{name} = %{version}-%{release}  - N/A
 - .la files are removed. - N/A

 - Package is a GUI app and has a .desktop file - N/A

 - 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. - OK
 - No rpmlint output. - see above
 - final provides and requires are sane: - OK

SHOULD Items:

 - Should build in mock. - didn't test
 - Should build on all supported archs - tested on i386
 - Should function as described. - OK
 - Should have sane scriptlets. - N/A
 - Should have subpackages require base package with fully versioned depend. - 
N/A
 - Should have dist tag - OK
 - Should package latest version - OK

Other things noted - could use $RPM_OPT_FLAGS.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220706] Review Request: linuxwacom-0.7.6_3-3.1.i386.rpm - with wacomcpl tool, man page

2007-05-11 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: linuxwacom-0.7.6_3-3.1.i386.rpm - with wacomcpl tool, 
man page


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 20:18 EST ---
Why is this a review request?  linuxwacom has been in core since forever.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239895] New: Review Request: perl-Math-Base85 - perl extension for base 85 numbers,

2007-05-11 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/bugzilla/show_bug.cgi?id=239895

   Summary: Review Request: perl-Math-Base85 - perl extension for
base 85 numbers,
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://folk.ntnu.no/sindrb/packages/green_nyc/perl-Math-Base85.spec
SRPM URL: 
http://folk.ntnu.no/sindrb/packages/green_nyc/perl-Math-Base85-0.2-1.fc7.src.rpm

Description:

RFC 1924 describes a compact, fixed-size representation of IPv6 addresses
which uses a base 85 number system. This module handles some of the uglier
details of it.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239883] Review Request: powertop - power usage tool

2007-05-11 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: powertop - power usage tool


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 20:31 EST ---
Builds in mock for all rhel arches, btw.  Though I doubt it'll do anything
useful on s390x...

Updated spec and srpm in same place as before.  Added COPYING to %doc, patched
CFLAGS to be overridable by the build system and set it to equal RPM_OPT_FLAGS.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239163] Review Request: perl-Net-IPv6Addr - Perl module to check validity of IPv6 addresses

2007-05-11 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-Net-IPv6Addr - Perl module to check validity of 
IPv6 addresses
Alias: perl-Net-IPv6Addr

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||239895




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239895] Review Request: perl-Math-Base85 - perl extension for base 85 numbers,

2007-05-11 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-Base85 - perl extension for base 85 numbers,
Alias: perl-Math-Base85

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Alias||perl-Math-Base85




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239895] Review Request: perl-Math-Base85 - perl extension for base 85 numbers,

2007-05-11 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-Base85 - perl extension for base 85 numbers,
Alias: perl-Math-Base85

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Alias||perl-Math-Base85
OtherBugsDependingO||239163
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239883] Review Request: powertop - power usage tool

2007-05-11 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: powertop - power usage tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 20:35 EST ---
Seems OK to me. Approved.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 234667] Review Request: dwarves - DWARF Tools

2007-05-11 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: dwarves - DWARF Tools


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 20:49 EST ---
Taking for review


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239883] Review Request: powertop - power usage tool

2007-05-11 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: powertop - power usage tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 22:37 EST ---
New Package CVS Request
===
Package Name: powertop
Short Description: Power consumption monitor
Owners: [EMAIL PROTECTED]
Branches: 
InitialCC: 

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 234667] Review Request: dwarves - DWARF Tools

2007-05-11 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: dwarves - DWARF Tools


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 22:41 EST ---
As of 0-24:

% rpmlint i386/*.rpm  
W: dwarves no-version-in-last-changelog
E: dwarves only-non-binary-in-usr-lib
W: dwarves devel-file-in-non-devel-package /usr/lib/ctracer/ctracer_relay.c
W: dwarves devel-file-in-non-devel-package /usr/lib/ctracer/ctracer_relay.h
W: dwarves devel-file-in-non-devel-package /usr/lib/ctracer/ctracer_jprobe.c
W: dwarves-debuginfo no-version-in-last-changelog
W: libdwarves1 no-version-in-last-changelog
W: libdwarves1 no-documentation
W: libdwarves1-devel no-version-in-last-changelog

The one error is unpleasant.  Any reason why that stuff's not in /usr/share? 
systemtap has similar code in its directories in /usr/share, so even though it
technically violates one of the MUSTS in the review guidelines I think there's a
good case for it.

Also, if the SRPM is intended to be the preferred distribution format, then you
need to note that in a comment above Source0.

Otherwise passes all MUSTS as far as I can see.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 237382] Review Request: alexandria - Book collection manager

2007-05-11 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: alexandria - Book collection manager
Alias: alexandria

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


Bug 237382 depends on bug 237380, which changed state.

Bug 237380 Summary: Review Request:  ruby-gettext-package - Localization 
Library and Tools for Ruby
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=237380

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 237380] Review Request: ruby-gettext-package - Localization Library and Tools for Ruby

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

Summary: Review Request:  ruby-gettext-package - Localization Library and Tools 
for Ruby
Alias: ruby-gettext-package

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-05-11 23:32 EST ---
Rebuild done on all branches, closing

Thank you for review!!

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 237373] Review Request: mcpp - Alternative C/C++ preprocessor

2007-05-11 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: mcpp -  Alternative C/C++ preprocessor


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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