[Bug 1086444] Review Request: ghc-optparse-applicative - Utilities and combinators for parsing command line options

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1086444

Ricky Elrod rel...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #6 from Ricky Elrod rel...@redhat.com ---
New Package SCM Request
===
Package Name: ghc-optparse-applicative
Short Description: Utilities and combinators for parsing command line options
Owners: codeblock petersen
Branches: f20 epel7 f19
InitialCC: haskell-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1081782] Review Request: astloch - Astloch fonts

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1081782



--- Comment #5 from Julien Enselme juj...@jujens.eu ---
I modified the fontconfig file according to the template posted on the fonts
mailing list.

The links are the same. The package looks fine for me.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 881590] Review Request: leveldbjni - A Java Native Interface to LevelDB

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=881590



--- Comment #11 from Fedora Update System upda...@fedoraproject.org ---
leveldbjni-1.8-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/leveldbjni-1.8-1.fc20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1087046] Review Request: php-horde-content - Tagging application

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1087046

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #4 from Remi Collet fed...@famillecollet.com ---
Thanks for the review !

New Package SCM Request
===
Package Name: php-horde-content
Short Description: Tagging application 
Owners: remi
Branches: f20 el6 epel7
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1075806] Review Request: fcgiwrap - Simple FastCGI wrapper for CGI scripts

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1075806



--- Comment #7 from Christopher Meng cicku...@gmail.com ---
(In reply to Justin Zhang from comment #6)
 Hi Christopher,
   It's not re-inventing the wheel. It's just trying to contribute to help
 move this issue forward. I think this is beneficial to the fedora project.
 
   Maybe let's contact Sebastian Dyroff to see if he can expedite the fix and
 if he would accept some of ideas of my implementation.
 
   Thanks!

We have many bugs like this, not stalled tickets, but the reporter is busy or
because of something else.

Please don't ask me to do anything here anymore. 

Thanks.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1087734] Review Request: php-horde-imp - A web based webmail system

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1087734



--- Comment #4 from Remi Collet fed...@famillecollet.com ---
Good catch !

https://github.com/remicollet/remirepo/commit/14f2b9bd6182eb35e75497aa22b3b8c7571905eb


Spec:
https://raw.githubusercontent.com/remicollet/remirepo/14f2b9bd6182eb35e75497aa22b3b8c7571905eb/php/horde/php-horde-imp/php-horde-imp.spec
Srpm: http://rpms.famillecollet.com/SRPMS/php-horde-imp-6.1.7-2.remi.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 462446] Review Request: ttf2pt1 - TrueType to Adobe Type 1 font converter

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=462446



--- Comment #29 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 462446] Review Request: ttf2pt1 - TrueType to Adobe Type 1 font converter

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=462446

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 999136] Review Request: python-backports-lzma - Backport of Python 3.3's lzma module

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=999136

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 999136] Review Request: python-backports-lzma - Backport of Python 3.3's lzma module

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=999136



--- Comment #20 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1058198] Review Request: ghc-rfc5051 - Simple unicode collation as per RFC5051

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1058198



--- Comment #5 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1058198] Review Request: ghc-rfc5051 - Simple unicode collation as per RFC5051

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1058198

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069451] Review Request: ghc-fsnotify - Cross platform library for file change notification

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069451



--- Comment #5 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1069451] Review Request: ghc-fsnotify - Cross platform library for file change notification

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069451

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083234] Review Request: mariadb-galera - MariaDB with wsrep patches

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083234

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083234] Review Request: mariadb-galera - MariaDB with wsrep patches

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083234



--- Comment #6 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1086444] Review Request: ghc-optparse-applicative - Utilities and combinators for parsing command line options

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1086444



--- Comment #7 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1086444] Review Request: ghc-optparse-applicative - Utilities and combinators for parsing command line options

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1086444

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1087046] Review Request: php-horde-content - Tagging application

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1087046

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1087046] Review Request: php-horde-content - Tagging application

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1087046



--- Comment #5 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1089559] Review Request: tlp - Advanced power management tool for Linux

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089559

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1089559] Review Request: tlp - Advanced power management tool for Linux

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089559



--- Comment #12 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091361] Review Request: perl-HTML-FormFu-Element-reCAPTCHA - reCAPTCHA component for HTML::FormFu frame work

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091361

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1090802] Review Request: ghc-streaming-commons - Common lower-level functions for streaming data libraries

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1090802

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091361] Review Request: perl-HTML-FormFu-Element-reCAPTCHA - reCAPTCHA component for HTML::FormFu frame work

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091361



--- Comment #4 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1090802] Review Request: ghc-streaming-commons - Common lower-level functions for streaming data libraries

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1090802



--- Comment #5 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091616] Review Request: perl-MetaCPAN-Client - A comprehensive, DWIM-featured client to the MetaCPAN API

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091616

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091616] Review Request: perl-MetaCPAN-Client - A comprehensive, DWIM-featured client to the MetaCPAN API

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091616



--- Comment #4 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083232] Review Request: galera - Galera replication engine

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083232



--- Comment #13 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083232] Review Request: galera - Galera replication engine

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083232

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1038238] Review Request: kcm-touchpad - KDE Systemsettings module for touchpads

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1038238

Kevin Kofler ke...@tigcc.ticalc.org changed:

   What|Removed |Added

 CC||hotmusic...@mail.bg



--- Comment #6 from Kevin Kofler ke...@tigcc.ticalc.org ---
*** Bug 1091760 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091361] Review Request: perl-HTML-FormFu-Element-reCAPTCHA - reCAPTCHA component for HTML::FormFu frame work

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091361

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-HTML-FormFu-Element-re
   ||CAPTCHA-1.00-1.fc21
 Resolution|--- |RAWHIDE
Last Closed||2014-04-28 08:31:01



--- Comment #5 from Petr Pisar ppi...@redhat.com ---
Thank you for the review and the repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1087046] Review Request: php-horde-content - Tagging application

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1087046

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

   What|Removed |Added

 Status|NEW |MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1087046] Review Request: php-horde-content - Tagging application

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1087046



--- Comment #6 from Fedora Update System upda...@fedoraproject.org ---
php-horde-content-2.0.3-2.el6 has been submitted as an update for Fedora EPEL
6.
https://admin.fedoraproject.org/updates/php-horde-content-2.0.3-2.el6

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1087046] Review Request: php-horde-content - Tagging application

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1087046



--- Comment #7 from Fedora Update System upda...@fedoraproject.org ---
php-horde-content-2.0.3-2.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/php-horde-content-2.0.3-2.fc20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987717] Review Request: perl-GraphViz2 - A wrapper for ATT's Graphviz

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987717

Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

 CC||lkund...@v3.sk



--- Comment #5 from Lubomir Rintel lkund...@v3.sk ---
What's the status of this? How is this different from perl-GraphViz (you my
want to clarify that in %description).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 235783] Review Request: perl-GraphViz - Interface to the GraphViz graphing tool

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=235783

Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

 CC||lkund...@v3.sk
  Flags|fedora-cvs+ |fedora-cvs?



--- Comment #6 from Lubomir Rintel lkund...@v3.sk ---
Package Change Request
==
Package Name: perl-GraphViz
New Branches: epel7
Owners: lkundrak
InitialCC: perl-sig

Fedora maintainer does not maintain EPEL packages and el6 maintainer is
unresponsive to e-mail.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083232] Review Request: galera - Galera replication engine

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083232

Mukundan Ragavan nonamed...@gmail.com changed:

   What|Removed |Added

   Assignee|nonamed...@gmail.com|255...@mail.muni.cz



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083232] Review Request: galera - Galera replication engine

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083232



--- Comment #14 from Mukundan Ragavan nonamed...@gmail.com ---
Sorry, I had to be away for a while and couldn't take care of this review. My
apologies. Glad to see everything has been taken care of.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083234] Review Request: mariadb-galera - MariaDB with wsrep patches

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083234

Mukundan Ragavan nonamed...@gmail.com changed:

   What|Removed |Added

   Assignee|nonamed...@gmail.com|255...@mail.muni.cz



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083234] Review Request: mariadb-galera - MariaDB with wsrep patches

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083234



--- Comment #7 from Mukundan Ragavan nonamed...@gmail.com ---
Sorry, I had to be away for a while and couldn't take care of this review. My
apologies. Glad to see everything has been taken care of.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1081782] Review Request: astloch - Astloch fonts

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1081782

Paul Flo Williams p...@frixxon.co.uk changed:

   What|Removed |Added

 CC||p...@frixxon.co.uk



--- Comment #6 from Paul Flo Williams p...@frixxon.co.uk ---
Just a few points:

1. The FontForge sources for these fonts are available from the Google font
directory so, according to our packaging policy, these fonts SHOULD be built
from source.

2. The word fracture in the description should be Fraktur.

3. Our fonts policy now says that fonts must include correct licensing fields
in the metadata. In this case, you'll need to correct the metadata after
building, using the ttname tool.

4. The fontconfig file isn't correct. You need to register both fonts with the
generic name cursive using the basic-font-template.conf supplied with
fontpackages-devel.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1081782] Review Request: astloch - Astloch fonts

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1081782



--- Comment #7 from Paul Flo Williams p...@frixxon.co.uk ---
(In reply to Paul Flo Williams from comment #6)

 3. Our fonts policy now says that fonts must include correct licensing
 fields in the metadata. In this case, you'll need to correct the metadata
 after building, using the ttname tool.

Of course, if you build from source, you have the option of patching the SFD
files before building, but UTF-7 dragons lurk there, so I'd still recommend you
use ttname.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092000] New: Review Request: rubygem-jgrep

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092000

Bug ID: 1092000
   Summary: Review Request: rubygem-jgrep
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: lkund...@v3.sk
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



SPEC: http://v3.sk/~lkundrak/SPECS/rubygem-jgrep.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/rubygem-jgrep-1.3.3-1.el7.src.rpm

Description:

JGrep is  Ruby-based CLI tool and API for parsing and displaying JSON data
using a logical expression syntax. It allows you to search a list of JSON
documents and return specific documents or values based on logical truths.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1048460] Review Request: hawaii-shell - Hawaii shell for desktop, netbook and tablet

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1048460

Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2014-04-28 09:24:46



--- Comment #14 from Lubomir Rintel lkund...@v3.sk ---
Imported and built.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092000] Review Request: rubygem-jgrep

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092000

Lukas Bezdicka 255...@mail.muni.cz changed:

   What|Removed |Added

 CC||255...@mail.muni.cz
   Assignee|nob...@fedoraproject.org|255...@mail.muni.cz
  Flags||fedora-review?



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 235783] Review Request: perl-GraphViz - Interface to the GraphViz graphing tool

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=235783

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 235783] Review Request: perl-GraphViz - Interface to the GraphViz graphing tool

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=235783



--- Comment #7 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091616] Review Request: perl-MetaCPAN-Client - A comprehensive, DWIM-featured client to the MetaCPAN API

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091616

Paul Howarth p...@city-fan.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-MetaCPAN-Client-1.0020
   ||00-2.fc21
 Resolution|--- |RAWHIDE
Last Closed||2014-04-28 09:53:26



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1087734] Review Request: php-horde-imp - A web based webmail system

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1087734

Shawn Iwinski shawn.iwin...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #5 from Shawn Iwinski shawn.iwin...@gmail.com ---
[x]: Package contains no bundled libraries without FPC exception.

 This was a license issue/update, not a bundled library issue/update.

[x]: Requires correct, justified where necessary.


HTTPD config updated.


No blockers.


= APPROVED =

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1087734] Review Request: php-horde-imp - A web based webmail system

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1087734

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #6 from Remi Collet fed...@famillecollet.com ---
Thanks!

New Package SCM Request
===
Package Name: php-horde-imp
Short Description: A web based webmail system
Owners: remi
Branches: f20 el6 epel7
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092018] New: Review Request: rubygem-comp_tree - A simple framework for automatic parallelism

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092018

Bug ID: 1092018
   Summary: Review Request: rubygem-comp_tree - A simple framework
for automatic parallelism
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: lkund...@v3.sk
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



SPEC: http://v3.sk/~lkundrak/SPECS/rubygem-comp_tree.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/rubygem-comp_tree-1.1.3-1.el7.src.rpm

Description:

CompTree is a parallel computation tree structure based upon concepts from
pure functional programming.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092022] New: Review Request: rubygem-drake - A branch of Rake supporting automatic parallelizing of tasks

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092022

Bug ID: 1092022
   Summary: Review Request: rubygem-drake - A branch of Rake
supporting automatic parallelizing of tasks
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: lkund...@v3.sk
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



SPEC: http://v3.sk/~lkundrak/SPECS/rubygem-drake.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/rubygem-drake-0.9.2.0.3.1-1.el7.src.rpm

Description:

Drake is an auto-parallelizing branch of Rake, a Make-like program
implemented in Ruby. Tasks and dependencies are specified in standard
Ruby syntax.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092022] Review Request: rubygem-drake - A branch of Rake supporting automatic parallelizing of tasks

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092022

Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

 Depends On||1092018




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1092018
[Bug 1092018] Review Request: rubygem-comp_tree - A simple framework for
automatic parallelism
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092018] Review Request: rubygem-comp_tree - A simple framework for automatic parallelism

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092018

Lubomir Rintel lkund...@v3.sk changed:

   What|Removed |Added

 Blocks||1092022




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1092022
[Bug 1092022] Review Request: rubygem-drake - A branch of Rake supporting
automatic parallelizing of tasks
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1080583] Review Request: compat-qpid-cpp - Compatibility modules for Qpid

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1080583



--- Comment #2 from Darryl L. Pierce dpie...@redhat.com ---
(In reply to Sandro Mani from comment #1)
 These can be removed:
 
 %defattr(-,root,root,-)
 
 %clean
 rm -rf %{buildroot}
 
 
 The order of the sections in the spec is somehwat untypical, but I don't
 think there is any guideline for that.
 
 
 The package fails to build:
 
 /builddir/build/BUILD/qpid-0.24/cpp/src/qpid/ha/BrokerReplicator.cpp:351:6:
 error: 'void
 qpid::ha::{anonymous}::collectQueueReplicators(boost::shared_ptrqpid::
 broker::Exchange, std::setboost::shared_ptrqpid::ha::QueueReplicator
 )' defined but not used [-Werror=unused-function]
  void collectQueueReplicators(
   ^
 cc1plus: all warnings being treated as errors

Thank you for taking on the review and the feedback.

 * I've removed the defattr lines
 * I've removed the clean stanza
 * I've rebased a few patches from upstream that removed the dead code

Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6790196
Updated spec:  http://mcpierce.fedorapeople.org/rpms/compat-qpid-cpp.spec
Updated SRPM: 
http://mcpierce.fedorapeople.org/rpms/compat-qpid-cpp-0.24-3.fc20.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 987717] Review Request: perl-GraphViz2 - A wrapper for ATT's Graphviz

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=987717

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

 Whiteboard|NotReady|



--- Comment #6 from Christopher Meng cicku...@gmail.com ---
(In reply to Lubomir Rintel from comment #5)
 What's the status of this? How is this different from perl-GraphViz (you my
 want to clarify that in %description).

Good catch.

perl-G and perl-G2 are from the same author. perl-G is deprecated now:

http://search.cpan.org/dist/GraphViz/

You can take a deep look:

Interface to ATT's GraphViz. Deprecated. See GraphViz2

That's why I packaged perl-G2. Last year I was trying to package many
dependencies, then back to G2 itself I couldn't find any free time.

I will clean up my old reviews soon.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1081782] Review Request: astloch-fonts - Astloch fonts

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1081782

Christopher Meng cicku...@gmail.com changed:

   What|Removed |Added

Summary|Review Request: astloch -   |Review Request:
   |Astloch fonts   |astloch-fonts - Astloch
   ||fonts



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092057] New: Review Request: golang-github-kdar-factorlog - Fast logging infrastructure for Go

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092057

Bug ID: 1092057
   Summary: Review Request: golang-github-kdar-factorlog - Fast
logging infrastructure for Go
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: l...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://lsm5.fedorapeople.org/golang-github-kdar-factorlog/golang-github-kdar-factorlog.spec
SRPM URL:
http://lsm5.fedorapeople.org/golang-github-kdar-factorlog/SRPMS/golang-github-kdar-factorlog-0-0.1.git814d8f7.fc21.src.rpm

Description:
FactorLog is a fast logging infrastructure for Go that provides numerous
logging functions for whatever your style may be. It could easily be a
replacement for Go's log in the standard library (though it doesn't support
functions such as SetFlags()).

Fedora Account System Username: lsm5

$ rpmlint golang-github-kdar-factorlog.spec
SRPMS/golang-github-kdar-factorlog-0-0.1.git814d8f7.fc21.src.rpm
RPMS/noarch/golang-github-kdar-factorlog-devel-0-0.1.git814d8f7.fc21.noarch.rpm
golang-github-kdar-factorlog.spec:40: W: setup-not-quiet
golang-github-kdar-factorlog.src:40: W: setup-not-quiet
2 packages and 1 specfiles checked; 0 errors, 2 warnings.

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092057] Review Request: golang-github-kdar-factorlog - Fast logging infrastructure for Go

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092057

Michal Fojtik mfoj...@redhat.com changed:

   What|Removed |Added

 CC||mfoj...@redhat.com
  Flags||fedora-review+



--- Comment #1 from Michal Fojtik mfoj...@redhat.com ---
Review+

Licence is OK, the spec file looks sane. No problems found in version/naming.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092057] Review Request: golang-github-kdar-factorlog - Fast logging infrastructure for Go

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092057

Lokesh Mandvekar l...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092057] Review Request: golang-github-kdar-factorlog - Fast logging infrastructure for Go

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092057

Lokesh Mandvekar l...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Lokesh Mandvekar l...@redhat.com ---
New Package SCM Request
===
Package Name: golang-github-kdar-factorlog
Short Description: Fast logging infrastructure for Go
Owners: lsm5
Branches: f19 f20 el6 epel7
InitialCC: golang-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092057] Review Request: golang-github-kdar-factorlog - Fast logging infrastructure for Go

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092057

Lokesh Mandvekar l...@redhat.com changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|mfoj...@redhat.com



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1087734] Review Request: php-horde-imp - A web based webmail system

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1087734

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1087734] Review Request: php-horde-imp - A web based webmail system

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1087734



--- Comment #7 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092057] Review Request: golang-github-kdar-factorlog - Fast logging infrastructure for Go

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092057



--- Comment #3 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091375] Review Request: tmate - Instant terminal sharing

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091375



--- Comment #1 from Jakub Hrozek jhro...@redhat.com ---
I reviewed the package on IRC and requested some changes:
* Source0 must point to the upstream tarball
* %makeinstall should not be used
* %make %{?_smp_mflags} should be used instead of -j%jobs magic
* BuildRequires gcc-c++ is not needed

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092057] Review Request: golang-github-kdar-factorlog - Fast logging infrastructure for Go

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092057

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092096] New: Review Request: JOrtho - A spell checker for Java

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092096

Bug ID: 1092096
   Summary: Review Request: JOrtho - A spell checker for Java
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: msima...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://msimacek.fedorapeople.org/JOrtho.spec
SRPM URL: http://msimacek.fedorapeople.org/JOrtho-1.0-0.fc20.src.rpm
Description: JOrtho is a spell checker for Java. The library works with any
JTextComponent from the Swing framework and checks as you type.
The dictionary is based on the free Wiktionary.org, and is
applicable for multiple languages. You can select the
spell checking language via a context menu. The features
of JOrtho are the highlighting of potentially wrongly spelled words,
a context menu with suggestions for correct forms of the word
and a context menu with option to change the checking language.
At the moment there are eight languages for spell checking available:
English, German, French, Spanish, Italian, Russian, Polish, and Arabic.

Fedora Account System Username: msimacek

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092057] Review Request: golang-github-kdar-factorlog - Fast logging infrastructure for Go

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092057



--- Comment #4 from Fedora Update System upda...@fedoraproject.org ---
golang-github-kdar-factorlog-0-0.1.git814d8f7.fc20 has been submitted as an
update for Fedora 20.
https://admin.fedoraproject.org/updates/golang-github-kdar-factorlog-0-0.1.git814d8f7.fc20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092057] Review Request: golang-github-kdar-factorlog - Fast logging infrastructure for Go

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092057

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 923959] Review Request: JOrtho - A spell checker for Java

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=923959

hannes johannes.l...@googlemail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |DUPLICATE
Last Closed||2014-04-28 14:08:38



--- Comment #4 from hannes johannes.l...@googlemail.com ---


*** This bug has been marked as a duplicate of bug 1092096 ***

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092096] Review Request: JOrtho - A spell checker for Java

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092096

hannes johannes.l...@googlemail.com changed:

   What|Removed |Added

 CC||johannes.l...@googlemail.co
   ||m



--- Comment #1 from hannes johannes.l...@googlemail.com ---
*** Bug 923959 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092057] Review Request: golang-github-kdar-factorlog - Fast logging infrastructure for Go

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092057



--- Comment #5 from Fedora Update System upda...@fedoraproject.org ---
golang-github-kdar-factorlog-0-0.1.git814d8f7.el6 has been submitted as an
update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/golang-github-kdar-factorlog-0-0.1.git814d8f7.el6

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092057] Review Request: golang-github-kdar-factorlog - Fast logging infrastructure for Go

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092057



--- Comment #6 from Fedora Update System upda...@fedoraproject.org ---
golang-github-kdar-factorlog-0-0.1.git814d8f7.fc19 has been submitted as an
update for Fedora 19.
https://admin.fedoraproject.org/updates/golang-github-kdar-factorlog-0-0.1.git814d8f7.fc19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 976049] Review Request: irclib - Java implementation of the IRC protocol

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=976049



--- Comment #1 from gil cattaneo punto...@libero.it ---
Spec URL: http://gil.fedorapeople.org/irclib.spec
SRPM URL: http://gil.fedorapeople.org/irclib-1.10-1.fc19.src.rpm

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=6791856

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092149] New: Review Request: dib-utils - Standalone parts of diskimage-builder

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092149

Bug ID: 1092149
   Summary: Review Request: dib-utils - Standalone parts of
diskimage-builder
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: bne...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://fedorapeople.org/~bnemec/dib-utils.spec
SRPM URL: http://fedorapeople.org/~bnemec/dib-utils-0.0.1-1.fc20.src.rpm
Description: Pieces of diskimage-builder that are useful standalone
Fedora Account System Username: bnemec

This package is intended to reflect an upcoming change in the upstream project
(as seen in https://review.openstack.org/#/c/90281/ ), but because we need a
split version of diskimage-builder for inclusion in the Fedora cloud image we
don't have time to wait for the upstream change to go through.  Adding this as
a separate package now should address the immediate needs and allow for an easy
transition to the new upstream repo when it becomes available.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092149] Review Request: dib-utils - Standalone parts of diskimage-builder

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092149

Ben Nemec bne...@redhat.com changed:

   What|Removed |Added

 Blocks||1086494




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1086494
[Bug 1086494] os-refresh-config calls dib-run-parts, which is not installed
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1087046] Review Request: php-horde-content - Tagging application

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1087046

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

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
php-horde-content-2.0.3-2.el6 has been pushed to the Fedora EPEL 6 testing
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1074969] Review Request: perl-Config-Generator - rpm containing global variables used by the Config::Generator modules

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1074969

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||perl-Config-Generator-0.5-1
   ||.el6
 Resolution|--- |ERRATA
Last Closed||2014-04-28 16:37:09



--- Comment #20 from Fedora Update System upda...@fedoraproject.org ---
perl-Config-Generator-0.5-1.el6 has been pushed to the Fedora EPEL 6 stable
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1074969] Review Request: perl-Config-Generator - rpm containing global variables used by the Config::Generator modules

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1074969

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

   What|Removed |Added

   Fixed In Version|perl-Config-Generator-0.5-1 |perl-Config-Generator-0.5-1
   |.el6|.el5



--- Comment #21 from Fedora Update System upda...@fedoraproject.org ---
perl-Config-Generator-0.5-1.el5 has been pushed to the Fedora EPEL 5 stable
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092149] Review Request: dib-utils - Standalone parts of diskimage-builder

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092149

Steven Hardy sha...@redhat.com changed:

   What|Removed |Added

 CC||sha...@redhat.com
   Assignee|nob...@fedoraproject.org|sha...@redhat.com



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1040517] Review Request: julia - High-level, high-performance dynamic language for technical computing

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1040517



--- Comment #12 from Milan Bouchet-Valat nalimi...@club.fr ---
Are you OK with the new version? (FWIW, ARM support in Julia might well be
ready for the next 0.3 version.)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1089500] Review Request: openlibm - High quality system independent, open source libm

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089500



--- Comment #6 from Milan Bouchet-Valat nalimi...@club.fr ---
Any more comments on the new .spec file?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092176] New: Review Request: adwaita-icon-theme - Adwaita icon theme

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092176

Bug ID: 1092176
   Summary: Review Request: adwaita-icon-theme - Adwaita icon
theme
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: kalevlem...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://kalev.fedorapeople.org/adwaita-icon-theme.spec
SRPM URL:
http://kalev.fedorapeople.org/adwaita-icon-theme-3.13.1-1.fc21.src.rpm
Description:
In upstream, gnome-icon-theme and gnome-icon-theme-symbolic were merged
together into a single icon theme, available under the adwaita-icon-theme name.

Fedora Account System Username: kalev

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092176] Review Request: adwaita-icon-theme - Adwaita icon theme

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092176

Adam Williamson awill...@redhat.com changed:

   What|Removed |Added

 CC||awill...@redhat.com
   Assignee|nob...@fedoraproject.org|awill...@redhat.com
  Flags||fedora-review+



--- Comment #1 from Adam Williamson awill...@redhat.com ---
Accepting for review. All MUST and SHOULD items at
https://fedoraproject.org/wiki/Packaging:ReviewGuidelines are 'good' or 'N/A'.

rpmlint output - only warning is:

adwaita-icon-theme-devel.noarch: W: no-documentation

it depends on the main package, so that's fine.

Package built correctly in Koji for Rawhide (scratch build test) with no
obvious issues in build.log, contents are as expected.

Package is approved.

(note for those playing along at home: though this is essentially intended to
replace gnome-icon-theme{-symbolic}, it's not a transparent replacement, the
theme name was actually changed, so it can't simply immediately
obsolete/provide those two; dependent packages need to be updated to use the
new theme name.)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 772243] Review Request: rds-tools - utilities for testing rds protocol networking

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772243

Doug Ledford dledf...@redhat.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NOTABUG
Last Closed||2014-04-28 18:52:32



--- Comment #11 from Doug Ledford dledf...@redhat.com ---
Considering that upstream rds kernel support is totally broken, I don't really
care much about this package.  I generally recommend that rds support in the
kernel be entirely disabled at this point.  If that changes, I can issue a new
review request against an updated version of this package.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1092176] Review Request: adwaita-icon-theme - Adwaita icon theme

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092176

Kalev Lember kalevlem...@gmail.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Kalev Lember kalevlem...@gmail.com ---
Thanks for the review, Adam!

New Package SCM Request
===
Package Name: adwaita-icon-theme
Short Description: Adwaita icon theme
Owners: kalev
Branches:
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1071581] Review Request: python-wikitools - Python package for interacting with a MediaWiki wiki

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1071581



--- Comment #4 from Mukundan Ragavan nonamed...@gmail.com ---
Hi Honza, here is a more detailed review. There are couple of issues. Please
see below.


Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed



= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 GPL (v3 or later), Unknown or generated. 2 files have unknown
 license. Detailed output of licensecheck in
 /home/mukundan/personal/pkgs/reviews/1071581-python-
 wikitools/licensecheck.txt

--- No problems.


GPL (v3 or later)
-
wikitools-1.1.1/wikitools/api.py
wikitools-1.1.1/wikitools/category.py
wikitools-1.1.1/wikitools/page.py
wikitools-1.1.1/wikitools/pagelist.py
wikitools-1.1.1/wikitools/user.py
wikitools-1.1.1/wikitools/wiki.py
wikitools-1.1.1/wikitools/wikifile.py

Unknown or generated

wikitools-1.1.1/setup.py
wikitools-1.1.1/wikitools/__init__.py

[!]: Package requires other packages for directories it uses.
 Note: No known owner of /usr/lib/python2.7/site-packages/wikitools
[!]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/lib/python2.7/site-
 packages/wikitools

--- please add the appropriate directory under %files.

[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 51200 bytes in 3 files.
[!]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s)
 for the package is included in %doc.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

Python:
[x]: Python eggs must not download any dependencies during the build process.
[x]: A package which is used by another package via an egg interface should
 provide egg info.
[x]: Package meets the Packaging Guidelines::Python
[x]: Package contains BR: python2-devel or python3-devel
[x]: Binary eggs must be removed in %prep

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: Description 

[Bug 1091479] Review Request: pam-kwallet - PAM module for KWallet

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091479



--- Comment #1 from Rex Dieter rdie...@math.unl.edu ---
Spec URL: http://rdieter.fedorapeople.org/rpms/pam-kwallet/pam-kwallet.spec
SRPM URL:
http://rdieter.fedorapeople.org/rpms/pam-kwallet/pam-kwallet-0-0.2.20140428gitaf786456.fc20.src.rpm

%changelog
* Mon Apr 28 2014 Rex Dieter rdie...@fedoraproject.org
0-0.2.20140428gitaf786456
- 20140428 git snapshot

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083721] Review Request: rendercheck - Tool to verify correct operation of the XRENDER extension

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083721



--- Comment #2 from Mukundan Ragavan nonamed...@gmail.com ---
Sorry for the delayed review. Please see detailed review below.


Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed



= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[!]: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s)
 for the package is included in %doc.

--- Source tarball has the file COPYING which is not included under %doc.
Please fix.

[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 MIT/X11 (BSD like), Unknown or generated. 14 files have unknown
 license. Detailed output of licensecheck in
 /home/mukundan/personal/pkgs/reviews/1083721-rendercheck/licensecheck.txt

--- This seems fine to me.

[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[x]: Each %files section contains %defattr if rpm  4.4
 Note: %defattr present but not needed

--- EL 5 support? If not, please consider modernizing the spec.

[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[x]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[?]: Package is not known to require an ExcludeArch tag.

--- s390 and s390x

[!]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 0 bytes in 0 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

Generic:
[?]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: %clean present but not required

--- perhaps modernize the spec?

[-]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 

[Bug 1089110] Review Request: nield - Network Interface Events Logging Daemon

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089110



--- Comment #12 from Tetsumune KISO t2m...@gmail.com ---
Updated so as not to include kernel header files.
Could you review them?

Spec URL:
http://sourceforge.net/projects/nield/files/nield-0.5.1/fedora/nield.spec
SRPM URL:
http://sourceforge.net/projects/nield/files/nield-0.5.1/fedora/nield-0.5.1-1.fc20.src.rpm
Description: The Network Interface Events Logging Daemon is a tool that
receives
notifications from the kernel through the netlink socket, and generates logs
related to link state, neighbor cache (ARP,NDP), IP address (IPv4,IPv6), route,
FIB rules, traffic control.
Fedora Account System Username: t2mune

And I ran licensecheck and rpmlint.
netlink is not a spelling error.

[t2mune@fedora20 ~]$ licensecheck nield-0.5.1/*
nield-0.5.1/src/tcmsg_filter.c: GPL (v2 or later)
nield-0.5.1/src/debug.c: GPL (v2 or later)
nield-0.5.1/src/log.c: GPL (v2 or later)
nield-0.5.1/src/ndmsg.c: GPL (v2 or later)
nield-0.5.1/src/nlmsg.c: GPL (v2 or later)
nield-0.5.1/src/rtmsg.c: GPL (v2 or later)
nield-0.5.1/src/nield.h: GPL (v2 or later)
nield-0.5.1/src/nield.c: GPL (v2 or later)
nield-0.5.1/src/tcmsg_qdisc.c: GPL (v2 or later)
nield-0.5.1/src/frhdr.c: GPL (v2 or later)
nield-0.5.1/src/ifimsg.c: GPL (v2 or later)
nield-0.5.1/src/ifamsg.c: GPL (v2 or later)
nield-0.5.1/src/list.h: GPL (v2 or later)
nield-0.5.1/src/rtnetlink.h: GPL (v2 or later)
nield-0.5.1/src/tcamsg.c: GPL (v2 or later)
[t2mune@fedora20 ~]$ rpmlint rpmbuild/SPECS/nield.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[t2mune@fedora20 ~]$ rpmlint rpmbuild/SRPMS/nield-0.5.1-1.fc20.src.rpm 
nield.src: W: spelling-error %description -l en_US netlink - net link,
net-link, linnet
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083721] Review Request: rendercheck - Tool to verify correct operation of the XRENDER extension

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083721



--- Comment #3 from Mukundan Ragavan nonamed...@gmail.com ---
koji scratch builds -
http://koji.fedoraproject.org/koji/taskinfo?taskID=6792369

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1079967] Review Request: fedwatch - Tool for watching fedmsg messages and running arbitrary scripts

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079967



--- Comment #5 from Mukundan Ragavan nonamed...@gmail.com ---
Sorry for the delayed review.


Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed



= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 LGPL (v2.1 or later), Unknown or generated. 3 files have unknown
 license. Detailed output of licensecheck in
 /home/mukundan/personal/pkgs/reviews/1079967-fedwatch/licensecheck.txt

--- No issues.

[x]: License file installed when any subpackage combination is installed.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[x]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 40960 bytes in 5 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s)
 for the package is included in %doc.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: %config files are marked noreplace or the reason is justified.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: No %config files under /usr.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

Python:
[x]: Python eggs must not download any dependencies during the build process.
[x]: A package which is used by another package via an egg interface should
 provide egg info.
[ ]: Package meets the Packaging Guidelines::Python
[x]: Package contains BR: python2-devel or python3-devel
[x]: Binary eggs must be removed in %prep

= SHOULD items =

Generic:
[x]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[?]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in python-
 fedwatch
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.

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

[-]: %check is present and all tests pass.
[x]: Packages should try to preserve 

[Bug 1089110] Review Request: nield - Network Interface Events Logging Daemon

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089110



--- Comment #13 from Christopher Meng cicku...@gmail.com ---
Read carefully:

https://fedoraproject.org/wiki/Join_the_package_collection_maintainers

Find which step you forgot to follow, then do it.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091479] Review Request: pam-kwallet - PAM module for KWallet

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091479

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

 Blocks||1092216 (F21Target-kde)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1092216
[Bug 1092216] Fedora 21 Target KDE Tracker
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 949038] Review Request: php-horde-horde - Horde Application Framework

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=949038

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2014-04-29 01:12:18



--- Comment #11 from Remi Collet fed...@famillecollet.com ---
Build in rawhide + epel7

Will push all the horde applications at the same time (when all will be
reviewed) in f20 / epel6

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1087734] Review Request: php-horde-imp - A web based webmail system

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1087734

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2014-04-29 01:12:07



--- Comment #8 from Remi Collet fed...@famillecollet.com ---
Build in rawhide + epel7

Will push all the horde applications at the same time (when all will be
reviewed) in f20 / epel6

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1075569] Review Request: ghc-ansi-wl-pprint - The Wadler/Leijen Pretty Printer for colored ANSI terminal output

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1075569

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

   What|Removed |Added

   Fixed In Version|ghc-ansi-wl-pprint-0.6.7.1- |ghc-ansi-wl-pprint-0.6.7.1-
   |1.fc20  |1.fc19



--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
ghc-ansi-wl-pprint-0.6.7.1-1.fc19 has been pushed to the Fedora 19 stable
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1058669] Review Request: ghc-word8 - 8-bit character library

2014-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1058669

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

   What|Removed |Added

   Fixed In Version|ghc-word8-0.0.4-1.fc20  |ghc-word8-0.0.4-1.fc19



--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
ghc-word8-0.0.4-1.fc19 has been pushed to the Fedora 19 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

  1   2   >