[Bug 753240] Review Request: colorhug-client - Tools for the Hughski Colorimeter

2011-11-27 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=753240

Richard Hughes rhug...@redhat.com changed:

   What|Removed |Added

   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 757484] New: Review Request: perl-Router-Simple - Simple HTTP router

2011-11-27 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: perl-Router-Simple - Simple HTTP router

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

   Summary: Review Request: perl-Router-Simple - Simple HTTP
router
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: emmanuel.sey...@club-internet.fr
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL:
http://people.parinux.org/~seyman/fedora/perl-Router-Simple/perl-Router-Simple.spec
SRPM URL:
http://people.parinux.org/~seyman/fedora/perl-Router-Simple/perl-Router-Simple-0.09-1.fc16.src.rpm
Description:
Router::Simple is a simple router class.

-- 
Configure 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 744952] Review Request: light-theme-gnome - Light Themes (Ambiance and Radiance)

2011-11-27 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=744952

--- Comment #5 from Christoph Wickert cwick...@fedoraproject.org 2011-11-27 
08:59:50 EST ---
Can we please have this packaged modular to allow non-GNOME users to use it?
nimbus is a good example how this can be achieved. The metacity bits for
example shoule be packaged separately and that package is then supposed to
require metacity. I am not sure that the unitiy or UbuntuOne bits should be
packaged at all, they are useless in Fedora (as long as we don't have Unity or
UbuntuOne packaged).

-- 
Configure 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 753583] Review Request: gedit-code-assistance - gedit plugin for code assistance for C, C++ and Objective-C

2011-11-27 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=753583

Elad Alfassa el...@doom.co.il changed:

   What|Removed |Added

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

--- Comment #7 from Elad Alfassa el...@doom.co.il 2011-11-27 09:50:01 EST ---
Seems fine to me, package is APPROVED.



-- 
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 754975] Review Request: pion-net - Pion Network Library

2011-11-27 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=754975

Michael Schwendt mschwe...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mschwe...@gmail.com
   Flag|needinfo?(mschwendt@gmail.c |fedora-review+
   |om) |

--- Comment #7 from Michael Schwendt mschwe...@gmail.com 2011-11-27 10:18:49 
EST ---
[pkg-config file changes]

 I hope it is correct.

Yes, it makes sense to move the external libs into Libs.private. Just that at
Fedora we try to avoid static linking, so it might be that some of the libs in
Libs.private are not available as -static packages.


 - A typo in configure option (--with-pic).

:)  Didn't make a difference, though. The default pic mode also compiled with
-fPIC -DPIC.


 - libicu was hardcoded in autoconf. I removed it and libicu-devel in
 BuildRequires is no longer needed.

Confirmed. BuildRequires now match the list at
http://www.pion.org/projects/pion-network-library/libraries
however zlib and bzip2 dependency is a mystery, too. They seem to be unused but
are linked with.

$ grep zlib * -R|grep '#include'
common/build/pion-config.inc: AC_TRY_LINK([#include zlib.h],[ zlibVersion();
return(0); ],
common/build/pion-config.inc: AC_TRY_LINK([#include bzlib.h],[
BZ2_bzlibVersion(); return(0); ],
configure:#include zlib.h
configure:#include bzlib.h


 - Unversioned .so files were moved into the -devel package.

Confirmed.


* Exlicit -devel Requires could be arch-specific using %{?_isa}.


* Upstream download section offers a shorter .tar.bz2 source archive.


* APPROVED

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

[Bug 753583] Review Request: gedit-code-assistance - gedit plugin for code assistance for C, C++ and Objective-C

2011-11-27 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=753583

Ignacio Casal Quinteiro (nacho) i...@gnome.org changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Ignacio Casal Quinteiro (nacho) i...@gnome.org 2011-11-27 
11:23:05 EST ---
New Package SCM Request
===
Package Name: gedit-code-assistance
Short Description: gedit plugin for code assistance for C, C++ and Objective-C
Owners: nacho
Branches: f16
InitialCC:

-- 
Configure 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 744339] Review Request: dieharder - Random number generator tester and timer

2011-11-27 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=744339

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #34 from Fedora Update System upda...@fedoraproject.org 
2011-11-27 13:37:23 EST ---
dieharder-3.31.1-4.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 744977] Review Request: libhtp - Security-aware parser for the HTTP protocol and the related bits and pieces

2011-11-27 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=744977

--- Comment #5 from Martin Gieseking martin.giesek...@uos.de 2011-11-27 
14:07:30 EST ---
(In reply to comment #4)
 - I suggest to build the doxygen API documentation (cd into docs/ and run 
   doxygen doxygen.conf) the devel package.

I meant: I suggest to build the doxygen API documentation, and to add it to the
devel package. ;)

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

[Bug 744977] Review Request: libhtp - Security-aware parser for the HTTP protocol and the related bits and pieces

2011-11-27 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=744977

Martin Gieseking martin.giesek...@uos.de changed:

   What|Removed |Added

 CC||martin.giesek...@uos.de

--- Comment #4 from Martin Gieseking martin.giesek...@uos.de 2011-11-27 
14:04:53 EST ---
There are some things that should be addressed before the package is checked
in:

- the devel package should require the base package this way:
  http://fedoraproject.org/wiki/PackagingGuidelines#Requiring_Base_Package

- Don't add the %doc files several times. Drop AUTHORS, LICENSE, and COPYING 
  from the devel package. Since it requires the base package, these files are 
  installed anyway.

- add README and NOTICE to the base package (with %doc) and doc/QUICK_START to
  the devel package

- I suggest to build the doxygen API documentation (cd into docs/ and run 
  doxygen doxygen.conf) the devel package.

- Either add a Group field to the base package (System Environment/Libraries), 
  or remove it from the devel package. Currently, the Group field is used
  inconsistently.

-- 
Configure 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 666311] Review Request: fritzing - Intuitive EDA platform from prototype to product

2011-11-27 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=666311

Chitlesh GOORAH chitl...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||chitl...@gmail.com
 Blocks|177841(FE-NEEDSPONSOR)  |
 AssignedTo|nob...@fedoraproject.org|chitl...@gmail.com

--- Comment #11 from Chitlesh GOORAH chitl...@gmail.com 2011-11-27 14:11:05 
EST ---
Hello Ed,

can you please update the srpm for version 0.6.3b?

I'll start the review after you update it.

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

[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the gtk+ toolkit

2011-11-27 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=754754

Mario Blättermann mario.blaetterm...@gmail.com changed:

   What|Removed |Added

 CC||mario.blaetterm...@gmail.co
   ||m

--- Comment #2 from Mario Blättermann mario.blaetterm...@gmail.com 2011-11-27 
14:17:21 EST ---
Koji scratch build fails on Rawhide:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3544890

See build.log for more details:
http://koji.fedoraproject.org/koji/getfile?taskID=3544891name=build.log

-- 
Configure 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 754975] Review Request: pion-net - Pion Network Library

2011-11-27 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=754975

--- Comment #8 from Jan Vcelak jvce...@redhat.com 2011-11-27 15:33:35 EST ---
(In reply to comment #7)
 * APPROVED

Thank you.

-- 
Configure 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 754975] Review Request: pion-net - Pion Network Library

2011-11-27 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=754975

Jan Vcelak jvce...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #9 from Jan Vcelak jvce...@redhat.com 2011-11-27 15:37:43 EST ---
New Package SCM Request
===
Package Name: pion-net
Short Description: C++ library for building lightweight HTTP interfaces
Owners: jvcelak
Branches: f15 f16
InitialCC:

-- 
Configure 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 705587] Review Request: android-tools - Android platform tools (adb, fastboot, etc)

2011-11-27 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=705587

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|NEW |MODIFIED

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

[Bug 705587] Review Request: android-tools - Android platform tools (adb, fastboot, etc)

2011-11-27 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=705587

--- Comment #28 from Fedora Update System upda...@fedoraproject.org 
2011-11-27 16:30:36 EST ---
android-tools-2020git4a25390-3.fc15 has been submitted as an update for
Fedora 15.
https://admin.fedoraproject.org/updates/android-tools-2020git4a25390-3.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 705587] Review Request: android-tools - Android platform tools (adb, fastboot, etc)

2011-11-27 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=705587

--- Comment #29 from Fedora Update System upda...@fedoraproject.org 
2011-11-27 16:30:46 EST ---
android-tools-2020git4a25390-3.fc16 has been submitted as an update for
Fedora 16.
https://admin.fedoraproject.org/updates/android-tools-2020git4a25390-3.fc16

-- 
Configure 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] Review Request: gsi-openssh - An implementation of the SSH protocol with GSI authentication

2011-11-27 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=683587

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||gsi-openssh-5.6p1-3.fc15
 Resolution||ERRATA
Last Closed||2011-11-27 16:54:20

--- Comment #28 from Fedora Update System upda...@fedoraproject.org 
2011-11-27 16:54:20 EST ---
gsi-openssh-5.6p1-3.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 728256] Review Request: rubygem-activesupport2.3 - rails 2 alongside rails 3

2011-11-27 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=728256

--- Comment #8 from Jon Ciesla l...@jcomserv.net 2011-11-27 16:58:03 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 753583] Review Request: gedit-code-assistance - gedit plugin for code assistance for C, C++ and Objective-C

2011-11-27 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=753583

--- Comment #9 from Jon Ciesla l...@jcomserv.net 2011-11-27 16:59:05 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 683587] Review Request: gsi-openssh - An implementation of the SSH protocol with GSI authentication

2011-11-27 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=683587

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|gsi-openssh-5.6p1-3.fc15|gsi-openssh-5.8p2-2.fc16

--- Comment #29 from Fedora Update System upda...@fedoraproject.org 
2011-11-27 16:56:11 EST ---
gsi-openssh-5.8p2-2.fc16 has been pushed to the Fedora 16 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 754476] Review Request: perl-Class-Load-XS - XS implementation of parts of Class::Load

2011-11-27 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=754476

--- Comment #4 from Jon Ciesla l...@jcomserv.net 2011-11-27 17:02:38 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 754975] Review Request: pion-net - Pion Network Library

2011-11-27 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=754975

--- Comment #10 from Jon Ciesla l...@jcomserv.net 2011-11-27 17:04:33 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 754064] Review Request: python-mimeparse - Python module providing basic functions for parsing mime-type names

2011-11-27 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=754064

--- Comment #6 from Jon Ciesla l...@jcomserv.net 2011-11-27 17:01:32 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 755176] Review Request: ghc-mwc-random - Pseudo random number generation library for Haskell

2011-11-27 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=755176

--- Comment #4 from Jon Ciesla l...@jcomserv.net 2011-11-27 17:07:32 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 757239] Review Request: kactivities - API for using and interacting with Activities

2011-11-27 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=757239

--- Comment #5 from Jon Ciesla l...@jcomserv.net 2011-11-27 17:08:28 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 755139] Review Request: python-nose1.1 - Discovery-based unittest extension for Python

2011-11-27 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=755139

--- Comment #11 from Jon Ciesla l...@jcomserv.net 2011-11-27 17:06:54 EST ---
Git done (by process-git-requests).

Steve, please take ownership of review BZs, thanks!

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

[Bug 186811] Review Request: libnfnetlink - Netfilter netlink userspace library

2011-11-27 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=186811

Paul P Komkoff Jr i...@stingr.net changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #18 from Paul P Komkoff Jr i...@stingr.net 2011-11-27 17:19:43 
EST ---
Package Change Request
==
Package Name: libnfnetlink
New Branches: EL-6
Owners: stingray jrussek

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

[Bug 757290] Review Request: unicode-ucd - Unicode Character Database

2011-11-27 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=757290

Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|panem...@gmail.com

-- 
Configure 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 754476] Review Request: perl-Class-Load-XS - XS implementation of parts of Class::Load

2011-11-27 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=754476

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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

[Bug 754476] Review Request: perl-Class-Load-XS - XS implementation of parts of Class::Load

2011-11-27 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=754476

--- Comment #5 from Fedora Update System upda...@fedoraproject.org 2011-11-27 
19:46:58 EST ---
perl-Class-Load-XS-0.03-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/perl-Class-Load-XS-0.03-1.fc16

-- 
Configure 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 753676] Review Request: gnome-shell-extention-netspeed -an internet speed indicator

2011-11-27 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=753676

--- Comment #9 from Parag AN(पराग) panem...@gmail.com 2011-11-27 22:32:15 EST 
---
Hi Amir,
   Is there any updates by you on reviewing other's 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 755054] Review Request: gnome-font-viewer - Utility for previewing fonts for GNOME

2011-11-27 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=755054

--- Comment #3 from Parag AN(पराग) panem...@gmail.com 2011-11-27 22:32:50 EST 
---
any further updates here?

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

[Bug 757290] Review Request: unicode-ucd - Unicode Character Database

2011-11-27 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=757290

Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #2 from Parag AN(पराग) panem...@gmail.com 2011-11-27 22:31:15 EST 
---
Review:

+ koji scratch(f16) build -
http://koji.fedoraproject.org/koji/taskinfo?taskID=3545268

+ rpmlint on rpms gave
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

+ source verified with upstream as (sha1sum)
d83d2bb7c4d34221cd9eaf53bfa730391ca9d135  UCD.zip
d83d2bb7c4d34221cd9eaf53bfa730391ca9d135  ../SOURCES/UCD.zip

+ follows packaging guidelines.


Suggestions:
1) I see duplication of copyright.html file at places
/usr/share/unicode/ucd/copyright.html
and
/usr/share/doc/unicode-ucd-6.0.0/copyright.html

I think you need to manually remove that file in %install.

2) unzip is already in minimal build environment so no need to BR: it in spec.

APPROVED.

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

[Bug 755054] Review Request: gnome-font-viewer - Utility for previewing fonts for GNOME

2011-11-27 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=755054

--- Comment #4 from AnujMore anujmo...@gmail.com 2011-11-27 22:39:48 EST ---
@Volker: I will upload those changes right away.

@Parag: I am almost there.
Exam season :/

I have an exam tomorrow, which means I should upload the changes by ~2000 IST.

-- 
Configure 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 700352] Review Request: perl-DublinCore-Record - Container for Dublin Core metadata elements

2011-11-27 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=700352

--- Comment #3 from Parag AN(पराग) panem...@gmail.com 2011-11-27 22:54:15 EST 
---
ping any updates here?

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

[Bug 700344] Review Request: perl-Authen-CAS-Client - Interface for authentication via JA-SIG's Central Authentication Service

2011-11-27 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=700344

--- Comment #5 from Parag AN(पराग) panem...@gmail.com 2011-11-27 22:54:07 EST 
---
ping any updates here?

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

[Bug 728823] Review Request: django-tinymce - TinyMCE form field editor for Django applications

2011-11-27 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=728823

Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #10 from Parag AN(पराग) panem...@gmail.com 2011-11-27 23:22:51 
EST ---
Review:

+ koji scratch build
-http://koji.fedoraproject.org/koji/taskinfo?taskID=3545280

+ rpmlint on rpms gave
django-tinymce.src: W: invalid-url Source0:
http://django-tinymce.googlecode.com/files/django-tinymce-1.5.tar.gz HTTP Error
404: Not Found
2 packages and 0 specfiles checked; 0 errors, 1 warnings.
== This looks a false message. 

+ source verified with upstream as (sha1sum)
a181255e8efe7b57a22851ebb184f3b9bf867bac  django-tinymce-1.5.tar.gz
a181255e8efe7b57a22851ebb184f3b9bf867bac  ../SOURCES/django-tinymce-1.5.tar.gz

+ rest follows packaging guidelines.


Suggestions:
1) Remove CFLAGS=$RPM_OPT_FLAGS from %build as this is noarch package

APPROVED.

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

[Bug 757484] Review Request: perl-Router-Simple - Simple HTTP router

2011-11-27 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=757484

Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||panem...@gmail.com
 AssignedTo|nob...@fedoraproject.org|panem...@gmail.com
   Flag||fedora-review+

--- Comment #1 from Parag AN(पराग) panem...@gmail.com 2011-11-27 23:22:10 EST 
---
Review:

+ koji (f16) scratch build
-http://koji.fedoraproject.org/koji/taskinfo?taskID=3545278

+ rpmlint on rpms gave
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

+ source verified with upstream as (sha1sum)
2def3774ad1a280b8cdac9885a92a53e8444decf  Router-Simple-0.09.tar.gz
2def3774ad1a280b8cdac9885a92a53e8444decf  ../SOURCES/Router-Simple-0.09.tar.gz

+ make test gave
All tests successful.
Files=9, Tests=29,  0 wallclock secs ( 0.04 usr  0.01 sys +  0.23 cusr  0.03
csys =  0.31 CPU)

+ Package: perl-Router-Simple-0.09-1.fc16.noarch =
Provides: perl(Router::Simple) = 0.09 perl(Router::Simple::Declare)
perl(Router::Simple::Route) perl(Router::Simple::SubMapper)

Requires: perl = 0:5.00800 perl(Carp) perl(List::Util) perl(Router::Simple)
perl(Router::Simple::Route) perl(Router::Simple::SubMapper) perl(Scalar::Util)
perl(parent) perl(strict) perl(warnings)

+ rest follows packaging guidelines.


APPROVED.

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

[Bug 698051] Review Request: spim - An assembly language MIPS32 simulator

2011-11-27 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=698051

W. Michael Petullo m...@flyn.org changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #11 from W. Michael Petullo m...@flyn.org 2011-11-27 23:30:44 EST 
---
New Package SCM Request
===
Package Name: spim
Short Description: An assembly language MIPS32 simulator
Owners: mikep
Branches: f15 f16
InitialCC:

-- 
Configure 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 700344] Review Request: perl-Authen-CAS-Client - Interface for authentication via JA-SIG's Central Authentication Service

2011-11-27 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=700344

--- Comment #6 from Nicholas van Oudtshoorn vano...@gmail.com 2011-11-28 
00:08:58 EST ---
Thanks for the PING - got busy over here.

Have updated a new SPEC file and SRPM:

Spec URL: http://dl.dropbox.com/u/8648526/packaging/perl-Authen-CAS-Client.spec
SRPM URL:
http://dl.dropbox.com/u/8648526/packaging/perl-Authen-CAS-Client-0.05-2.fc16.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 755139] Review Request: python-nose1.1 - Discovery-based unittest extension for Python

2011-11-27 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=755139

Steve Traylen steve.tray...@cern.ch changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|steve.tray...@cern.ch
  QAContact|steve.tray...@cern.ch   |extras...@fedoraproject.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 700352] Review Request: perl-DublinCore-Record - Container for Dublin Core metadata elements

2011-11-27 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=700352

--- Comment #4 from Nicholas van Oudtshoorn vano...@gmail.com 2011-11-28 
00:17:05 EST ---
Finally - an update! A new spec file and SRPM based on the comments above:

Spec URL: http://dl.dropbox.com/u/8648526/packaging/perl-DublinCore-Record.spec
SRPM URL:
http://dl.dropbox.com/u/8648526/packaging/perl-DublinCore-Record-0.03-2.fc16.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 700353] Review Request: perl-HTTP-OAI - API for the OAI-PMH

2011-11-27 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=700353

--- Comment #3 from Parag AN(पराग) panem...@gmail.com 2011-11-28 00:21:15 EST 
---
ping for update here also :)

-- 
Configure 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 700354] Review Request: perl-Locale-Currency-Format - Perl functions for formatting monetary values

2011-11-27 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=700354

--- Comment #4 from Nicholas van Oudtshoorn vano...@gmail.com 2011-11-28 
00:22:39 EST ---
New spec file and SRPM uploaded:
Spec URL:
http://dl.dropbox.com/u/8648526/packaging/perl-Locale-Currency-Format.spec
SRPM URL:
http://dl.dropbox.com/u/8648526/packaging/perl-Locale-Currency-Format-1.30-1.fc16.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 608574] Review Request: tinymce - Web based Javascript HTML WYSIWYG editor control

2011-11-27 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=608574

--- Comment #21 from Fedora Update System upda...@fedoraproject.org 
2011-11-28 00:33:33 EST ---
tinymce-3.4.3.2-4.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/tinymce-3.4.3.2-4.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 608574] Review Request: tinymce - Web based Javascript HTML WYSIWYG editor control

2011-11-27 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=608574

--- Comment #20 from Fedora Update System upda...@fedoraproject.org 
2011-11-28 00:33:22 EST ---
tinymce-3.4.3.2-4.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/tinymce-3.4.3.2-4.fc16

-- 
Configure 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 608574] Review Request: tinymce - Web based Javascript HTML WYSIWYG editor control

2011-11-27 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=608574

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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

[Bug 608575] Review Request: tinymce-spellchecker - TinyMCE spellchecker plugin

2011-11-27 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=608575

--- Comment #12 from Fedora Update System upda...@fedoraproject.org 
2011-11-28 00:35:50 EST ---
tinymce-spellchecker-2.0.5-5.fc16 has been submitted as an update for Fedora
16.
https://admin.fedoraproject.org/updates/tinymce-spellchecker-2.0.5-5.fc16

-- 
Configure 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 608575] Review Request: tinymce-spellchecker - TinyMCE spellchecker plugin

2011-11-27 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=608575

--- Comment #11 from Fedora Update System upda...@fedoraproject.org 
2011-11-28 00:35:42 EST ---
tinymce-spellchecker-2.0.5-5.fc15 has been submitted as an update for Fedora
15.
https://admin.fedoraproject.org/updates/tinymce-spellchecker-2.0.5-5.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 700353] Review Request: perl-HTTP-OAI - API for the OAI-PMH

2011-11-27 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=700353

--- Comment #4 from Nicholas van Oudtshoorn vano...@gmail.com 2011-11-28 
00:35:31 EST ---
Pung! (And updated files):

Spec URL: http://dl.dropbox.com/u/8648526/packaging/perl-HTTP-OAI.spec
SRPM URL:
http://dl.dropbox.com/u/8648526/packaging/perl-HTTP-OAI-3.27-1.fc16.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 608575] Review Request: tinymce-spellchecker - TinyMCE spellchecker plugin

2011-11-27 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=608575

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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

[Bug 700344] Review Request: perl-Authen-CAS-Client - Interface for authentication via JA-SIG's Central Authentication Service

2011-11-27 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=700344

Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #7 from Parag AN(पराग) panem...@gmail.com 2011-11-28 02:22:59 EST 
---
Review:-

+ koji scratch build -
http://koji.fedoraproject.org/koji/taskinfo?taskID=3545434

+ rpmlint on rpms gave
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

+ source verified with upstream as  (sha1sum)
fb13f4789a49df20769a27474b2e9174237b4d15  Authen-CAS-Client-0.05.tar.gz
fb13f4789a49df20769a27474b2e9174237b4d15 
../SOURCES/Authen-CAS-Client-0.05.tar.gz

+ make test gave
All tests successful.
Files=6, Tests=204,  1 wallclock secs ( 0.05 usr  0.01 sys +  0.47 cusr  0.05
csys =  0.58 CPU)

+ Package: perl-Authen-CAS-Client-0.05-2.fc17.noarch
Provides: perl(Authen::CAS::Client) = 0.05 perl(Authen::CAS::Client::Response)
= 0.03 perl(Authen::CAS::Client::Response::AuthFailure)
perl(Authen::CAS::Client::Response::AuthSuccess)
perl(Authen::CAS::Client::Response::Error)
perl(Authen::CAS::Client::Response::Failure)
perl(Authen::CAS::Client::Response::ProxyFailure)
perl(Authen::CAS::Client::Response::ProxySuccess)
perl(Authen::CAS::Client::Response::Success)

Requires: perl = 0:5.006_001 perl(Authen::CAS::Client::Response)
perl(Authen::CAS::Client::Response::Failure)
perl(Authen::CAS::Client::Response::Success) perl(LWP::UserAgent) perl(URI)
perl(URI::QueryParam) perl(XML::LibXML) perl(base) perl(strict) perl(warnings)

+ rest follows packaging guidelines

Suggestions:
1) you don't need following in spec
BuildRequires:  perl = 1:5.6.1

2) Remove =0.00 from all the BuildRequires: lines. you don't need them.


APPROVED.

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

[Bug 700352] Review Request: perl-DublinCore-Record - Container for Dublin Core metadata elements

2011-11-27 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=700352

Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #5 from Parag AN(पराग) panem...@gmail.com 2011-11-28 02:22:55 EST 
---
Review:-

+ koji scratch build -
http://koji.fedoraproject.org/koji/taskinfo?taskID=3545436

+ rpmlint on rpms gave
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

+ source verified with upstream as  (sha1sum)
5fab8fcb9307b787de3831ff9e875c2a5f427bd5  DublinCore-Record-0.03.tar.gz
5fab8fcb9307b787de3831ff9e875c2a5f427bd5 
../SOURCES/DublinCore-Record-0.03.tar.gz

+ ./Build test gave
All tests successful.
Files=6, Tests=135,  0 wallclock secs ( 0.04 usr  0.01 sys +  0.15 cusr  0.02
csys =  0.22 CPU)

+ Package: perl-DublinCore-Record-0.03-2.fc17.noarch
Provides: perl(DublinCore::Element) = 0.03 perl(DublinCore::Record) = 0.03
Requires: perl(Carp) perl(Class::Accessor) perl(DublinCore::Element) perl(base)
perl(strict) perl(warnings)

+ rest follows packaging guidelines


APPROVED.

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

[Bug 757290] Review Request: unicode-ucd - Unicode Character Database

2011-11-27 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=757290

--- Comment #3 from Jens Petersen peter...@redhat.com 2011-11-28 02:58:49 EST 
---
Thanks for the timely review. :-)

Thanks for the suggestions will fix those after importing.

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