[Bug 538057] Review Request: rhnmd - Red Hat Network Monitoring Daemon

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Jan Pazdziora  2011-03-10 02:28:25 
EST ---
(In reply to comment #6)
> 
> On the other hand, because this package is new to Fedora, I question whether
> you need to obsolete these at all -- how many Fedora machines are likely to
> have the arch-specific versions of the packages installed on them?

The package was shipped by Spacewalk project for Fedora in the past. So you may
well have installations of old non-Fedora package and now upgrade with package
that made it to Fedora.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683725] Review Request: marathi-typing-booster - Marathi Typing Booster Table IMEngine for ibus

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Pravin Satpute  changed:

   What|Removed |Added

 CC||i18n-bugs@lists.fedoraproje
   ||ct.org, panem...@gmail.com
   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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683725] New: Review Request: marathi-typing-booster - Marathi Typing Booster Table IMEngine for ibus

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: marathi-typing-booster - Marathi Typing Booster Table 
IMEngine for ibus

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

   Summary: Review Request: marathi-typing-booster - Marathi
Typing Booster Table IMEngine for ibus
   Product: Fedora
   Version: 15
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: psatp...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://pravins.fedorapeople.org/marathi-typing-booster.spec
SRPM URL:
http://pravins.fedorapeople.org/marathi-typing-booster-0.0.1-1.fc14.src.rpm

Description: Marathi Typing Booster Table IMEngine for Ibus

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 538057] Review Request: rhnmd - Red Hat Network Monitoring Daemon

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Ruediger Landmann  changed:

   What|Removed |Added

   Flag||needinfo?(msu...@redhat.com
   ||)

--- Comment #6 from Ruediger Landmann  2011-03-10 
01:46:18 EST ---
Thanks: looks good except for:

rhnmd.spec:19: W: unversioned-explicit-obsoletes rhnmd.i386
rhnmd.spec:20: W: unversioned-explicit-obsoletes rhnmd.x86_64
rhnmd.spec:19: W: mixed-use-of-spaces-and-tabs (spaces: line 7, tab: line 19)

rhnmd.noarch: W: obsolete-not-provided rhnmd.i386
rhnmd.noarch: W: obsolete-not-provided rhnmd.x86_64

If you're going to obsolete the arch-specific versions of the packages, you
should also 

1. specify the last version of the arch-specific packages
2. add corresponding "Provides:" for the arch-specific packages

On the other hand, because this package is new to Fedora, I question whether
you need to obsolete these at all -- how many Fedora machines are likely to
have the arch-specific versions of the packages installed on them?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 682130] Review Request: ibus-indic-table - Indic Table engine for IBus platform

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Naveen Kumar  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
Last Closed||2011-03-10 01:47:26

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 682130] Review Request: ibus-indic-table - Indic Table engine for IBus platform

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Naveen Kumar  2011-03-10 00:53:43 EST 
---
Thanks Jason :)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 678931] Review Request: perl-Math-Random-Secure - Cryptographically-secure, cross-platform replacement for rand()

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
   Fixed In Version||perl-Math-Random-Secure-0.0
   ||6-2.fc15
 Resolution||ERRATA
Last Closed||2011-03-09 22:14:38

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 678931] Review Request: perl-Math-Random-Secure - Cryptographically-secure, cross-platform replacement for rand()

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Fedora Update System  2011-03-09 
22:14:33 EST ---
perl-Math-Random-Secure-0.06-2.fc15 has been pushed to the Fedora 15 stable
repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 680423] Review Request: drupal6-authfas - Drupal 6 module for authentication via a Fedora Account System

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #10 from Fedora Update System  
2011-03-09 20:47:43 EST ---
drupal6-authfas-0.2.1-1.fc15 has been pushed to the Fedora 15 testing
repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683684] New: Review Request: dtrx - Intelligent archive extractor

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: dtrx - Intelligent archive extractor

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

   Summary: Review Request: dtrx - Intelligent archive extractor
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: seb...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://sebelk.fedorapeople.org/dtrx.spec
SRPM URL: http://sebelk.fedorapeople.org/dtrx-7.0-1.fc14.src.rpm
Description:

dtrx is a simple but powerful python script that works as a front-end for a lot
of archivers. It can do recursive extraction, for example extract tarballs from
source rpms.
dtrx extracts archives in a number of different formats; it currently
supports tar, zip (including self-extracting .exe files), cpio, rpm, deb,
gem, 7z, cab, rar (unrar is on a third party repository),
and InstallShield files.  It can also decompress files
compressed with gzip, bzip2, lzma, xz, or compress.

In addition to providing one command to handle many different archive
types, dtrx also aids the user by extracting contents consistently.  By
default, everything will be written to a dedicated directory that's named
after the archive.  dtrx will also change the permissions to ensure that the
owner can read and write all 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683638] Review Request: pyes- Python library for connecting to and managing Elasticsearch

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Tavis Aitken  changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

--- Comment #1 from Tavis Aitken  2011-03-09 17:06:47 EST ---
SRPM URL: should be
http://rpms.tavisto.net/fedora-rpms/pyes-0.14.1-1.fc14.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683638] New: Review Request: pyes- Python library for connecting to and managing Elasticsearch

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: pyes- Python library for connecting to and managing 
Elasticsearch

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

   Summary: Review Request: pyes- Python library for connecting to
and managing Elasticsearch
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: tavi...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: https://github.com/tavisto/rpm-specs/blob/master/pyes.spec
SRPM URL: rpms.tavisto.net/fedora-rpms/pyes-0.14.1-1.fc14.src.rpm
Description: pyes is a connector to use elasticsearch from python.

This is my first package that I am trying to get into fedora and will need a
sponsor. I appreciate any feedback anyone is willing to give me.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683610] Review Request: hxtools - A collection of several tools

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Till Maas  changed:

   What|Removed |Added

 CC||opensou...@till.name

--- Comment #1 from Till Maas  2011-03-09 16:11:29 EST ---
Spec URL: http://till.fedorapeople.org/review/hxtools.spec
SRPM URL:
http://till.fedorapeople.org/review/hxtools-20110308-2.tillf14.src.rpm

Changelog:
- Include license 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 680308] Review Request: php-pear-MDB2-Schema - XML based database schema manager

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|php-pear-MDB2-Schema-0.8.5- |php-pear-MDB2-Schema-0.8.5-
   |1.fc13  |1.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 662257] Review Request: php-pear-CodeGen-PECL - Tool to generate PECL extensions from an XML description

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|php-pear-CodeGen-PECL-1.1.3 |php-pear-CodeGen-PECL-1.1.3
   |-3.fc14 |-3.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 662257] Review Request: php-pear-CodeGen-PECL - Tool to generate PECL extensions from an XML description

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #15 from Fedora Update System  
2011-03-09 15:55:50 EST ---
php-pear-CodeGen-PECL-1.1.3-3.fc13 has been pushed to the Fedora 13 stable
repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 680308] Review Request: php-pear-MDB2-Schema - XML based database schema manager

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #12 from Fedora Update System  
2011-03-09 15:55:08 EST ---
php-pear-MDB2-Schema-0.8.5-1.fc14 has been pushed to the Fedora 14 stable
repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 680308] Review Request: php-pear-MDB2-Schema - XML based database schema manager

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|php-pear-MDB2-Schema-0.8.5- |php-pear-MDB2-Schema-0.8.5-
   |1.fc15  |1.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 662257] Review Request: php-pear-CodeGen-PECL - Tool to generate PECL extensions from an XML description

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|php-pear-CodeGen-PECL-1.1.3 |php-pear-CodeGen-PECL-1.1.3
   |-3.fc15 |-3.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683610] New: Review Request: hxtools - A collection of several tools

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: hxtools - A collection of several tools

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

   Summary: Review Request: hxtools - A collection of several
tools
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: opensou...@till.name
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://till.fedorapeople.org/review/hxtools.spec
SRPM URL:
http://till.fedorapeople.org/review/hxtools-20110308-1.tillf14.src.rpm
Description: 
hxtools contains several tools for different tasks written by Jan Engelhardt.

Currently only tools needed for pam_mount are included in this package.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 650667] Review Request: hitori - Hitori game for GNOME

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|hitori-0.2.6-3.fc15 |hitori-0.2.6-3.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 662257] Review Request: php-pear-CodeGen-PECL - Tool to generate PECL extensions from an XML description

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #14 from Fedora Update System  
2011-03-09 15:51:32 EST ---
php-pear-CodeGen-PECL-1.1.3-3.fc14 has been pushed to the Fedora 14 stable
repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 680308] Review Request: php-pear-MDB2-Schema - XML based database schema manager

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Fedora Update System  
2011-03-09 15:50:42 EST ---
php-pear-MDB2-Schema-0.8.5-1.fc13 has been pushed to the Fedora 13 stable
repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 650667] Review Request: hitori - Hitori game for GNOME

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #26 from Fedora Update System  
2011-03-09 15:52:17 EST ---
hitori-0.2.6-3.fc14 has been pushed to the Fedora 14 stable repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 668204] Review Request: buzzbird - Twitter and identi.ca desktop client

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Jan Klepek  2011-03-09 15:16:46 EST 
---
this does not apply to source tarball as far as I know, no bundled/pre-compiled
binary file is present in final rpm.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 682545] Review Request: nutrika - Nutrient content calculator and intake analyzer

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Pavel Zhukov  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|pa...@zhukoff.net

--- Comment #8 from Pavel Zhukov  2011-03-09 14:37:20 EST ---
I'll review it



-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 666572] Review Request: zyGrib - Visualization of meteo data from files in GRIB Format

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Pavel Zhukov  2011-03-09 14:32:36 EST 
---
Fixed all.
http://landgraf.fedorapeople.org/packages/requested/zygrib/zyGrib.spec
http://landgraf.fedorapeople.org/packages/requested/zygrib/zyGrib-3.9.9-3.fc14.src.rpm



-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683591] New: Review Request: tclap - Templatized Command Line Argument Parser

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: tclap - Templatized Command Line Argument Parser

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

   Summary: Review Request: tclap - Templatized Command Line
Argument Parser
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: br...@postle.net
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://bugbear.postle.net/~bruno/apt/SPECS/tclap.spec
SRPM URL:
http://bugbear.postle.net/~bruno/apt/fedora/linux/15/i386/SRPMS.panorama/tclap-1.2.0-1.fc15.src.rpm
Description: This is a simple C++ library that facilitates parsing command line
arguments in a type independent manner.  It doesn't conform exactly
to either the GNU or POSIX standards, although it is close.

Note: I'm submitting this as I'm the maintainer of the hugin package which
currently contains a copy of tclap as a bundled lib. This package will allow
for this to be removed, yay.

Note also that this library consists entirely of C++ templates, so it would be
pointless having a separate tclap-devel, it only makes sense to have tclap and
tclap-doc packages - The bogus warnings from rpmlint are a result 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683587] New: Review Request: gsissh - An implementation of the SSH protocol with GSI authentication

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: gsissh - An implementation of the SSH protocol with 
GSI authentication

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

   Summary: Review Request: gsissh - An implementation of the SSH
protocol with GSI authentication
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mattias.ell...@fysast.uu.se
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://www.grid.tsl.uu.se/review/el5/gsissh.spec
http://www.grid.tsl.uu.se/review/el6/gsissh.spec
http://www.grid.tsl.uu.se/review/fc14/gsissh.spec
http://www.grid.tsl.uu.se/review/fc15/gsissh.spec
http://www.grid.tsl.uu.se/review/fc16/gsissh.spec

SRPM URL:
http://www.grid.tsl.uu.se/review/gsissh-4.3p2-1.el5.src.rpm
http://www.grid.tsl.uu.se/review/gsissh-5.3p1-1.el6.src.rpm
http://www.grid.tsl.uu.se/review/gsissh-5.5p1-1.fc14.src.rpm
http://www.grid.tsl.uu.se/review/gsissh-5.6p1-1.fc15.src.rpm
http://www.grid.tsl.uu.se/review/gsissh-5.8p1-1.fc16.src.rpm

Description:
This is a modified version of openssh that supports GSI authentication. It is
not possible to apply the patch to the normal openssh package without loosing
functionality, since you can not compile openssh with support for GSI and
kerberos at the same time, and the normal openssh package is compiled with
kerberos support. Instead this parallel installable package can be used when
GSI authentication is needed.

The source RPMS are based on the normal openssh packages for each release,
using the same versions and applying the same patches with the addition of the
patch that implements the GSI support. This was done in order to ensure
compatibility and minimize the risk of introducing security problems.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 682905] Review Request: lio-utils - Utilities for Generic Target Infrastructure

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Terje Røsten  2011-03-09 13:41:02 
EST ---
Would be simpler to follow if you add a real changelog.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 682905] Review Request: lio-utils - Utilities for Generic Target Infrastructure

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Andy Grover  2011-03-09 13:37:26 EST ---
Whups, added optflags and made disttag conditional.

http://fedorapeople.org/~grover/lio-utils.spec3
http://fedorapeople.org/~grover/srpms/lio-utils-3.2-3.el6.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 673839] Review Request: boost141 - The free peer-reviewed portable C++ source libraries

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #27 from Fedora Update System  
2011-03-09 13:27:49 EST ---
boost141-1.41.0-2.el5 has been pushed to the Fedora EPEL 5 testing repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 682487] Review Request: drupal6-image - Allows uploading, resizing and viewing of images

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #10 from Fedora Update System  
2011-03-09 13:26:03 EST ---
drupal6-image-6.x.1.1-1.el5 has been pushed to the Fedora EPEL 5 testing
repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 682488] Review Request: drupal6-filefield - Defines a file field type

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #10 from Fedora Update System  
2011-03-09 13:27:15 EST ---
drupal6-filefield-6.x.3.9-1.el5 has been pushed to the Fedora EPEL 5 testing
repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 682486] Review Request: drupal6-video - Allows Creation of CCK Video Fields

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #13 from Fedora Update System  
2011-03-09 13:26:09 EST ---
drupal6-video-6.x.4.2.beta2-2.el5 has been pushed to the Fedora EPEL 5 testing
repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 671884] Review Request: erlang-cluster_info - Cluster info/postmortem inspector for Erlang applications

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #16 from Fedora Update System  
2011-03-09 13:26:48 EST ---
erlang-cluster_info-0.1.0-0.3.20101229gitd077716.el6 has been pushed to the
Fedora EPEL 6 stable repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 501101] Review Request: emacs-color-theme - elisp mode to customize emacs look and feel

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #26 from Fedora Update System  
2011-03-09 13:27:32 EST ---
emacs-color-theme-6.6.0-4.el6 has been pushed to the Fedora EPEL 6 stable
repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 682479] Review Request: drupal6-yubikey - Provides YubiKey based strong two-factor user authentication capabilities

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #10 from Fedora Update System  
2011-03-09 13:26:14 EST ---
drupal6-yubikey-6.x.2.0.beta2-1.el5 has been pushed to the Fedora EPEL 5
testing repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 671884] Review Request: erlang-cluster_info - Cluster info/postmortem inspector for Erlang applications

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|erlang-cluster_info-0.1.0-0 |erlang-cluster_info-0.1.0-0
   |.3.20101229gitd077716.fc14  |.3.20101229gitd077716.el6

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 501101] Review Request: emacs-color-theme - elisp mode to customize emacs look and feel

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|emacs-color-theme-6.6.0-4.e |emacs-color-theme-6.6.0-4.e
   |l5  |l6

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 523034] Review Request: python-setuptools_trial - Setuptools plugin that makes unit tests execute with trial instead of pyunit

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Ruben Kerkhof  2011-03-09 13:14:45 
EST ---
New version here:

SPEC:
http://ruben.fedorapeople.org/python-setuptools_trial.spec
SRPM:
http://ruben.fedorapeople.org/python-setuptools_trial-0.5.12-1.fc16.src.rpm

Darcs isn't used anymore and removed from the tarball.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 682905] Review Request: lio-utils - Utilities for Generic Target Infrastructure

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Andy Grover  2011-03-09 13:08:55 EST ---
(In reply to comment #3)
> Some comments:
> 
>  a) no url tag, where is this project living?
>  b) you might want to add the sha1 8ad98e3 to the tarball name?
>  c) -n %{name}-%{version} in %setup is the default and can be dropped 
>(however, consider b))

I decided to keep the tarball name, since future releases should have proper
tarballs.

>  d) mixing of $RPM_BUILD_ROOT and %{buildroot}, pick one.
>  e) use macros for /etc, /sbin and /etc/init.d/
>  f) target is a very generic name, possible problem

True.. it's what upstream is using, so I'm hesitant to diverge.

>  g) change %defattr(-,root,root) ->  %defattr(-,root,root,-)
>  h) the package seems to be free for docs and with no url tag, the user
> will have a hard time be able to use this utility.

Yeah, I'll be working with upstream to author some manpages.

>  i) Release:  1, please add disttag here.

All the rest are done. Revised versions:

http://fedorapeople.org/~grover/lio-utils.spec(2)
http://fedorapeople.org/~grover/srpms/lio-utils-3.2-1.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683552] New: Review Request: mysql-connector-python - MySQL Connector/Python

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: mysql-connector-python - MySQL Connector/Python

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

   Summary: Review Request: mysql-connector-python - MySQL
Connector/Python
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: fed...@famillecollet.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
https://github.com/remicollet/remirepo/raw/master/mysql-connector-python/mysql-connector-python.spec
SRPM URL:
http://remi.fedorapeople.org/mysql-connector-python-0.3.2-1.remi.src.rpm
Description: 
MySQL Connector/Python is implementing the MySQL Client/Server protocol
completely in Python. No MySQL libraries are needed, and no compilation
is necessary to run this Python DB API v2.0 compliant driver.

-- 

This library is an new dependency of MySQL-Workbench (version >= 5.2.32)

Koji : http://koji.fedoraproject.org/koji/taskinfo?taskID=2897886
Rpmlint is silent

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683552] Review Request: mysql-connector-python - MySQL Connector/Python

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Remi Collet  changed:

   What|Removed |Added

 Blocks||683047

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: libvirt-php - PHP bindings for libvirt virtualization toolkit

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Michal Novotny  2011-03-09 11:58:26 
EST ---
(In reply to comment #5)
> (In reply to comment #4)
> > (In reply to comment #3)
> > > The tarball at the source URL and in the .src.rpm differs.
> > > 
> > > According to the 
> > > https://fedoraproject.org/wiki/Packaging:PHP#Naming_scheme 
> > > the package should be named php-libvirt instead of libvirt-php.
> > > 
> > 
> > 
> > Well, originally the project was named php-libvirt but it got renamed to 
> > comply
> > with the names at http://libvirt.org/git . This was not my idea however I
> > already got used to the libvirt-php name.
> 
> Don't confuse the upstream name and the Fedora name.  They can
> be different if we need them to be.
> 


Well, I maintain the upstream for libvirt-php. If the names can be different
then I rename the project for Fedora.


> > > Note that you have included the html doc in both main package and the -doc
> > > subpackage. Also the %doc must be on the same line as the filename
> > > specification.
> > 
> > This is because rpmlint was complaining the main package was not having any
> > documentation. Shouldn't be I having it in the main package then?
> 
> You can ignore rpmlint if you think it is getting things wrong,
> although it's often a good idea to add a small comment in the
> spec file.


Oh, ok. I won't be putting docs in the main package then.


> 
> In this case, how about putting the README and license file (eg. COPYING)
> into the main package, and the rest of the documentation in the
> -doc subpackage.
> 
> > > 
> > > The licensing is confusing/wrong - in the README you specify that the 
> > > license
> > > is GPL (if so, there should be COPYING with the correct GPL version). In 
> > > the
> > > .spec file there is License: PHP. The source files do not contain any 
> > > copyright
> > > statements nor license names - these are not required but they are 
> > > recommended.
> > 
> > 
> > Oh, I'll fix it. I guess this was done by multiple people contributing to 
> > this
> > so it made some kind of mess there however for the PHP modules the licence
> > should be a PHP licence, right? Or should be easily be GPL licence as well
> > since it's just about the module/extension?
> 
> You really need to be clear about licensing before anything
> can be incorporated into Fedora.  It's a legal requirement and
> could get people into trouble.  Maybe clarify this with upstream
> on libvir-list?


Well, upstream libvir-list has nothing to do with the binding since I maintain
the PHP bindings. However I should ask about the licence.

Michal

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683463] Review Request: trafficserver - Apache Traffic Server

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Ed Marshall  changed:

   What|Removed |Added

 CC||esm+red...@logic.net

--- Comment #1 from Ed Marshall  2011-03-09 11:54:12 EST 
---
Hi, this is a completely unofficial review, but I'd actually like to see
trafficserver in Fedora. :)

The first thing I noticed: you're packaging the unstable version of
trafficserver (2.1.6) rather than the stable release from September (2.0.1).
Upstream doesn't seem to have confidence that 2.1.6 is ready to be declared
stable yet; should Fedora be overriding them?

rpmlint on the SRPM reports a few errors:

trafficserver.src: W: name-repeated-in-summary C TrafficServer
trafficserver.src: E: description-line-too-long C Apache Traffic Server is
fast, scalable and extensible HTTP/1.1 compliant caching proxy server.
trafficserver.src: E: description-line-too-long C Formerly a commercial
product, Yahoo! donated it to the Apache Foundation, and is now an Apache TLP.
trafficserver.src: W: no-version-in-last-changelog
trafficserver.src: W: invalid-license Apache-2.0
trafficserver.src:11: W: hardcoded-packager-tag Zhao
trafficserver.src:30: W: rpm-buildroot-usage %build echo  $RPM_BUILD_ROOT
trafficserver.src:52: W: macro-in-comment %attr
trafficserver.src:17: W: mixed-use-of-spaces-and-tabs (spaces: line 17, tab:
line 3)

So, some simple changes I'd suggest to make it happier:

- Make the summary more descriptive; Name: tells us what it is, Summary: tells
us what it does (and Description: expands on that). See:
http://fedoraproject.org/wiki/Packaging/Guidelines#Summary_and_description

- Wrap the description field at 80 characters or less. Again, think about what
the package does, rather than it's history.

- Add the applicable package versions to the changelog "*" lines. See:
http://fedoraproject.org/wiki/Packaging/Guidelines#Changelogs

- Change license field to "ASL 2.0". See:
http://fedoraproject.org/wiki/Licensing:Main#Documentation_Licenses

- Remove the Packager: tag. (We get credit in the changelog. :) See:
http://fedoraproject.org/wiki/Packaging/Guidelines#Rpmlint_Errors

- Take out the "echo $RPM_BUILD_ROOT" from the spec; this looks like debugging
leftovers?

- Line 17 of the spec has tabs instead of spaces.


rpmlint on the binary you built brings up a few more issues (I've trimmed out
the repeats):

trafficserver.x86_64: W: no-documentation
trafficserver.x86_64: E: non-standard-dir-perm /usr/lib64/trafficserver/plugins
0644L
trafficserver.x86_64: W: devel-file-in-non-devel-package /usr/include/ts/ts.h
trafficserver.x86_64: E: non-standard-dir-perm /usr/lib64/trafficserver 0644L
trafficserver.x86_64: W: devel-file-in-non-devel-package
/usr/include/ts/experimental.h
trafficserver.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/trafficserver/libtsmgmt.so
trafficserver.x86_64: E: non-standard-dir-perm /usr/include/ts 0644L
trafficserver.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/trafficserver/libtsutil.so
trafficserver.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/trafficserver/libtsutil.a
trafficserver.x86_64: W: hidden-file-or-dir
/etc/trafficserver/body_factory/default/.body_factory_info
trafficserver.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/trafficserver/libtsmgmt.a
trafficserver.x86_64: W: devel-file-in-non-devel-package
/usr/include/ts/mgmtapi.h
trafficserver.x86_64: W: devel-file-in-non-devel-package
/usr/include/ts/remap.h
trafficserver.x86_64: W: log-files-without-logrotate /var/log/trafficserver
trafficserver.x86_64: W: no-manual-page-for-binary traffic_shell
trafficserver.x86_64: W: no-manual-page-for-binary tsxs
trafficserver.x86_64: W: no-manual-page-for-binary traffic_server
trafficserver.x86_64: W: no-manual-page-for-binary traffic_manager
trafficserver.x86_64: W: no-manual-page-for-binary traffic_line
trafficserver.x86_64: W: no-manual-page-for-binary traffic_sac
trafficserver.x86_64: W: no-manual-page-for-binary traffic_cop
trafficserver.x86_64: W: no-manual-page-for-binary traffic_logstats
trafficserver.x86_64: W: no-manual-page-for-binary traffic_logcat
trafficserver.x86_64: E: init-script-without-chkconfig-postin
/etc/init.d/trafficserver
trafficserver.x86_64: E: init-script-without-chkconfig-preun
/etc/init.d/trafficserver
trafficserver.x86_64: W: service-default-enabled /etc/init.d/trafficserver
trafficserver.x86_64: E: no-chkconfig-line /etc/init.d/trafficserver
trafficserver.x86_64: E: subsys-not-used /etc/init.d/trafficserver

There's quite a bit here, but the basics are:

- no man pages or documentation; you'll really want to work out the conflicts
with the man pages, and I'd suggest adding a %doc section for the README and
other documentation files in the distribution.

- file and directory

[Bug 480552] Review Request: poweradmin - A friendly web-based DNS administration tool for Bert Hubert's PowerDNS server

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Ruben Kerkhof  2011-03-09 11:08:34 
EST ---
Yes, thanks a lot Jason!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 646606] Rename review: drupal-workspace -> drupal6-workspace

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #13 from Jon Ciesla  2011-03-09 10:47:59 EST ---
This and service_links are all that block the drupal->drupal6 rename.  Is all
you're objecting to the Epoch?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 680423] Review Request: drupal6-authfas - Drupal 6 module for authentication via a Fedora Account System

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Fedora Update System  2011-03-09 
10:29:47 EST ---
drupal6-authfas-0.2.1-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/drupal6-authfas-0.2.1-1.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 680205] Review Request: allegro5 - Allegro 5 is a game programming library.

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Brandon McCaig  2011-03-09 10:21:03 
EST ---
Spec URL: http://castopulence.org/rpm/allegro5/5.0.0-3/allegro5.spec
SRPM URL:
http://castopulence.org/rpm/allegro5/5.0.0-3/allegro5-5.0.0-3.fc13.src.rpm

As long as I didn't screw up merging I think I should have all existing
concerns addressed. The rpmdev-diff looks good to me. I'm rather satisfied with
the rpmlint output now too. :) It would be nice if you could do something like
%doc(null) or something to suppress warnings in subpackages. The only thing
that bothers me now is the "W: unused-direct-shlib-dependency" warnings when
tested against the installed 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683487] Review Request: R-ACCLMA - R contrib package for ACC & LMA Graph Plotting

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jason Tibbitts  changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683487] Review Request: R-ACCLMA - R contrib package for ACC & LMA Graph Plotting

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Madhavprasad Pai  changed:

   What|Removed |Added

   Platform|All |noarch

--- Comment #1 from Madhavprasad Pai  2011-03-09 09:51:50 EST 
---
Need sponsor

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683487] New: Review Request: R-ACCLMA - R contrib package for ACC & LMA Graph Plotting

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: R-ACCLMA - R contrib package for ACC & LMA Graph 
Plotting

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

   Summary: Review Request: R-ACCLMA - R contrib package for ACC &
LMA Graph Plotting
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mk...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://pai.fedorapeople.org/R-ACCLMA.spec
SRPM URL: http://pai.fedorapeople.org/R-ACCLMA-1.0-1.fc14.src.rpm
Description: 

This is a contrib package to the R software environment for statistical
computing and graphics. 

The R package summary is

[snip]
The main function is plotLMA(sourcefile,header) that takes a data set and plots
the appropriate LMA and ACC graphs. If no sourcefile (a string) was passed, a
manual data entry window is opened. The header parameter indicates by
TRUE/FALSE (false by default) if the source CSV file has a head row or not. The
data set should contain only one independent variable (X) and one dependent
varialbe (Y) and can contain a weight for each observation
[/snip]

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 682130] Review Request: ibus-indic-table - Indic Table engine for IBus platform

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Jason Tibbitts  2011-03-09 09:28:47 EST 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 464931] Review Request: appliance-os recipe for building the appliance operating system

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jason Tibbitts  changed:

   What|Removed |Added

  Status Whiteboard||NotReady

--- Comment #11 from Jason Tibbitts  2011-03-09 09:06:56 EST 
---
There does not seem to be an actual package to review here.  I've no idea what
to do with this ticket, but I'm going to mark it so that it does not clutter up
the already cluttered package review queue.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: libvirt-php - PHP bindings for libvirt virtualization toolkit

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Tomas Mraz  changed:

   What|Removed |Added

   Flag|needinfo?(tm...@redhat.com) |

--- Comment #6 from Tomas Mraz  2011-03-09 08:58:42 EST ---
Basically everything was already answered by Richard. The licensing must be
clear and the Fedora package name can be different from the upstream tarball
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: libvirt-php - PHP bindings for libvirt virtualization toolkit

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Richard W.M. Jones  2011-03-09 08:53:05 
EST ---
(In reply to comment #4)
> (In reply to comment #3)
> > The tarball at the source URL and in the .src.rpm differs.
> > 
> > According to the https://fedoraproject.org/wiki/Packaging:PHP#Naming_scheme 
> > the package should be named php-libvirt instead of libvirt-php.
> > 
> 
> 
> Well, originally the project was named php-libvirt but it got renamed to 
> comply
> with the names at http://libvirt.org/git . This was not my idea however I
> already got used to the libvirt-php name.

Don't confuse the upstream name and the Fedora name.  They can
be different if we need them to be.

> > Note that you have included the html doc in both main package and the -doc
> > subpackage. Also the %doc must be on the same line as the filename
> > specification.
> 
> This is because rpmlint was complaining the main package was not having any
> documentation. Shouldn't be I having it in the main package then?

You can ignore rpmlint if you think it is getting things wrong,
although it's often a good idea to add a small comment in the
spec file.

In this case, how about putting the README and license file (eg. COPYING)
into the main package, and the rest of the documentation in the
-doc subpackage.

> > 
> > The licensing is confusing/wrong - in the README you specify that the 
> > license
> > is GPL (if so, there should be COPYING with the correct GPL version). In the
> > .spec file there is License: PHP. The source files do not contain any 
> > copyright
> > statements nor license names - these are not required but they are 
> > recommended.
> 
> 
> Oh, I'll fix it. I guess this was done by multiple people contributing to this
> so it made some kind of mess there however for the PHP modules the licence
> should be a PHP licence, right? Or should be easily be GPL licence as well
> since it's just about the module/extension?

You really need to be clear about licensing before anything
can be incorporated into Fedora.  It's a legal requirement and
could get people into trouble.  Maybe clarify this with upstream
on libvir-list?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: libvirt-php - PHP bindings for libvirt virtualization toolkit

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Michal Novotny  changed:

   What|Removed |Added

   Flag|needinfo?(minovotn@redhat.c |needinfo?(tm...@redhat.com)
   |om) |

--- Comment #4 from Michal Novotny  2011-03-09 08:48:49 
EST ---
(In reply to comment #3)
> The tarball at the source URL and in the .src.rpm differs.
> 
> According to the https://fedoraproject.org/wiki/Packaging:PHP#Naming_scheme 
> the package should be named php-libvirt instead of libvirt-php.
> 


Well, originally the project was named php-libvirt but it got renamed to comply
with the names at http://libvirt.org/git . This was not my idea however I
already got used to the libvirt-php name.


> Note that you have included the html doc in both main package and the -doc
> subpackage. Also the %doc must be on the same line as the filename
> specification.


This is because rpmlint was complaining the main package was not having any
documentation. Shouldn't be I having it in the main package then?


> 
> The licensing is confusing/wrong - in the README you specify that the license
> is GPL (if so, there should be COPYING with the correct GPL version). In the
> .spec file there is License: PHP. The source files do not contain any 
> copyright
> statements nor license names - these are not required but they are 
> recommended.


Oh, I'll fix it. I guess this was done by multiple people contributing to this
so it made some kind of mess there however for the PHP modules the licence
should be a PHP licence, right? Or should be easily be GPL licence as well
since it's just about the module/extension?

Michal

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 677943] Review Request: liblouis - Braille translation and back-translation library

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Martin Gieseking  2011-03-09 
08:44:11 EST ---
Thank you for the review. Since you're now a member of the packager group, you
are also allowed to approve this package if you can't find any packaging
mistakes.


(In reply to comment #6)
> Of course, the other subpackages has set the correct dependencies. The "utils"
> package could have this dependency optionally, but it's not really required.

Yes, it's not necessary because the -utils package depends on liblouis.so.*
provided by the base package:
$ rpm -qp --requires liblouis-utils-2.2.0-2.fc14.i686.rpm 
...
liblouis.so.2  
...

$ rpm -qp --provides liblouis-2.2.0-2.fc14.i686.rpm
bundled(gnulib) = 2009
liblouis.so.2  
liblouis = 2.2.0-2.fc14
liblouis(x86-32) = 2.2.0-2.fc14

Thus, the base package is automatically pulled in as a dependency.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683463] New: Review Request: trafficserver - Apache Traffic Server

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: trafficserver - Apache Traffic Server

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

   Summary: Review Request: trafficserver - Apache Traffic Server
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: ming@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://zymlinux.net/trafficserver/rpm/trafficserver.spec
SRPM URL: http://zymlinux.net/trafficserver/rpm/trafficserver-2.1.6-1.src.rpm
Description: Apache Traffic Server is fast, scalable and extensible HTTP/1.1
compliant caching proxy server.
Formerly a commercial product, Yahoo! donated it to the Apache Foundation, and
is now an Apache TLP.
site: http://trafficserver.apache.org/
version 2.1.6 is the most stable+useable release.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683463] Review Request: trafficserver - Apache Traffic Server

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Zhao Yongming  changed:

   What|Removed |Added

URL||http://trafficserver.apache
   ||.org/

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: libvirt-php - PHP bindings for libvirt virtualization toolkit

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Tomas Mraz  changed:

   What|Removed |Added

   Flag||needinfo?(minovotn@redhat.c
   ||om)

--- Comment #3 from Tomas Mraz  2011-03-09 08:33:25 EST ---
The tarball at the source URL and in the .src.rpm differs.

According to the https://fedoraproject.org/wiki/Packaging:PHP#Naming_scheme 
the package should be named php-libvirt instead of libvirt-php.

Note that you have included the html doc in both main package and the -doc
subpackage. Also the %doc must be on the same line as the filename
specification.

The licensing is confusing/wrong - in the README you specify that the license
is GPL (if so, there should be COPYING with the correct GPL version). In the
.spec file there is License: PHP. The source files do not contain any copyright
statements nor license names - these are not required but they are recommended.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683257] Review Request: perl-Test-Command - Test routines for external commands

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Emmanuel Seyman  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||emmanuel.seyman@club-intern
   ||et.fr
 AssignedTo|nob...@fedoraproject.org|emmanuel.seyman@club-intern
   ||et.fr
   Flag||fedora-review?

--- Comment #1 from Emmanuel Seyman  
2011-03-09 08:16:27 EST ---
Taking.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: libvirt-php - PHP bindings for libvirt virtualization toolkit

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Tomas Mraz  2011-03-09 07:52:21 EST ---
(In reply to comment #1)
> Here's the Koji build:
> There was no arch-override and it compiled for i386 and x86_64 systems fine 
> so I guess the
> other platforms don't have PHP support.

Not really - Fedora primary architectures are i386 (i686) and x86_64 only. The
other architectures are secondary and handled by a different koji instance and
builds are initiated separately by the secondary arch builders.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 683071] Review Request: libvirt-php - PHP bindings for libvirt virtualization toolkit

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Tomas Mraz  changed:

   What|Removed |Added

 CC||tm...@redhat.com
 AssignedTo|nob...@fedoraproject.org|tm...@redhat.com
   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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 681558] Review Request: libvirt-snmp - SNMP support for libvirt

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Tomas Mraz  2011-03-09 06:14:24 EST ---
The libvirt tarball from the .src.rpm is different from the one at the URL.
Please correct that.

If you update the .src.rpm and spec, please always bump the Release and add
Changelog entry.

I noted that the license is GPLv3 - only - is that intentional? The source
files do not contain any copyright comment - it is not required however it is
recommended that they contain it (at least the non-autogenerated ones). There
is the COPYING file that contains GPLv3 (which by itself allows the code to be
licensed under the later future GPL licenses). However README states that the
license is GPLv3 which limits it.

Why is the LIBVIRT-MIB.txt both in %doc and in the snmp/mibs data dir? I do not
think it should be %doc - it is not in the net-snmp-libs package which contains
most of the mib 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 681558] Review Request: libvirt-snmp - SNMP support for libvirt

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Tomas Mraz  changed:

   What|Removed |Added

   Flag||needinfo?(mprivozn@redhat.c
   ||om)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 682130] Review Request: ibus-indic-table - Indic Table engine for IBus platform

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Naveen Kumar  changed:

   What|Removed |Added

   Flag||fedora-cvs?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 666572] Review Request: zyGrib - Visualization of meteo data from files in GRIB Format

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Peter Lemenkov  2011-03-09 05:45:25 
EST ---
Koji scratchbuild for F-15:

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

Needs love:

* Bundled libraries - bzip2, zlib, proj. I'm pretty sure that the main package
is linked against system-wide copies, but I prefer bundled ones to be removed
at the %prep stage.

* Please, replace macros for sed, cp and rm invocation by real names of these
utilities.

* The package contains bundled font (liberation). I suspect that appropriate
"Requires" should be added. Could you, please, investigate this?

* The License tag should be GPLv3+.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675914] Review Request: flush - GTK-based BitTorrent client

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Oksana  2011-03-09 04:58:05 EST ---
Thanks, will try..

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 682130] Review Request: ibus-indic-table - Indic Table engine for IBus platform

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Naveen Kumar  2011-03-09 04:24:37 EST ---
New Package SCM Request
===
Package Name: ibus-indic-table
Short Description: Indic Table engine for IBus platform
Owners: nkumar
Branches: f14 f15
InitialCC: i18n-team

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 666572] Review Request: zyGrib - Visualization of meteo data from files in GRIB Format

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Pavel Zhukov  2011-03-09 04:24:01 EST ---
moved to fedorapeople
SRPM:
http://landgraf.fedorapeople.org/packages/requested/zygrib/zyGrib-3.9.9-2.fc14.src.rpm

SPEC: http://landgraf.fedorapeople.org/packages/requested/zygrib/zyGrib.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675914] Review Request: flush - GTK-based BitTorrent client

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Peter Lemenkov  2011-03-09 04:10:02 
EST ---
FYI version 0.9.10 has been released.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 682130] Review Request: ibus-indic-table - Indic Table engine for IBus platform

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #8 from Naveen Kumar  2011-03-09 03:58:47 EST ---
(In reply to comment #6)
> Suggestions:
> 1) --disable-static is not needed to %configure
> 2) preserve the timestamps of installed files.
> make DESTDIR=${RPM_BUILD_ROOT} NO_INDEX=true \
> install pkgconfigdir=%{_datadir}/pkgconfig \
>  INSTALL="install -p"
> 
> APPROVED.

Thanks Parag :)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 538057] Review Request: rhnmd - Red Hat Network Monitoring Daemon

2011-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Miroslav Suchý  changed:

   What|Removed |Added

   Flag|needinfo?(msu...@redhat.com |
   |)   |

--- Comment #5 from Miroslav Suchý  2011-03-09 03:05:06 EST 
---
There was only small change:
Spec:
http://miroslav.suchy.cz/fedora/rhnmd/rhnmd.spec

src.rpm:
http://miroslav.suchy.cz/fedora/rhnmd/rhnmd-5.3.7-1.fc14.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review