[Bug 857309] New: Review Request: hash-slinger - Generate various DNS records such as RFC-4255 SSHFP and RFC-698 TLSA

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857309

Bug ID: 857309
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: hash-slinger - Generate various DNS
records such as RFC-4255 SSHFP and RFC-698 TLSA
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: pwout...@redhat.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://people.redhat.com/pwouters/hash-slinger/hash-slinger.spec
SRPM URL:
http://people.redhat.com/pwouters/hash-slinger/hash-slinger-2.0-1.fc17.src.rpm
Description: This package contains various tools to generate special DNS
records:

sshfp   Generate RFC-4255 SSHFP DNS records from known_hosts files
or ssh-keyscan
tlsaGenerate RFC-6698  TLSA DNS records via TLS

It pulls in software from 'sshfp' and 'swede'

Fedora Account System Username: pwouters

-- 
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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=847517

--- Comment #4 from Remi Collet  ---
(In reply to comment #3)
> In %check, I'm assuming "ping -c 1 dns.google.com" is to check whether the
> build host is connected to a public network and will then run all tests? 

Yes.
But it also check that resolver works.
So, as still prefer a name rather that an ip (and a server allowing ping)

> dns.google.com does not resolve on my machine and may not for other building
> machines.  Perhaps instead use one of the Google Public DNS IP addresses
> used in the resolver test file since they respond to pings as well? --
> 8.8.8.8 or 8.8.4.4

In our case, koji won't run this test. This is the designed behavior.

Do you think this is a blocker for the review ?

-- 
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 853463] Review Request: php-redis - Extension for communicating with the Redis key-value store

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853463

Remi Collet  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #11 from Remi Collet  ---
Thanks for the review.


New Package SCM Request
===
Package Name: php-redis
Short Description: Extension for communicating with the Redis key-value store
Owners: remi
Branches: f17 f18 el6
InitialCC:

-- 
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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=847517

Shawn Iwinski  changed:

   What|Removed |Added

 CC||shawn.iwin...@gmail.com
   Assignee|nob...@fedoraproject.org|shawn.iwin...@gmail.com
  Flags||fedora-review?

--- Comment #3 from Shawn Iwinski  ---
In %check, I'm assuming "ping -c 1 dns.google.com" is to check whether the
build host is connected to a public network and will then run all tests? 
dns.google.com does not resolve on my machine and may not for other building
machines.  Perhaps instead use one of the Google Public DNS IP addresses used
in the resolver test file since they respond to pings as well? -- 8.8.8.8 or
8.8.4.4

-- 
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 839098] Review Request: python-flask-silk - Adds silk icons to your Flask application or module, or extension

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=839098

--- Comment #3 from pcpa  ---
Same issue with python-setuptools as #839071

Update:
Spec URL: http://fedorapeople.org/~pcpa/python-flask-silk.spec
SRPM URL: http://fedorapeople.org/~pcpa/python-flask-silk-0.1.1-4.fc19.src.rpm

-- 
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 839097] Review Request: python-flask-autoindex - A mod_autoindex for Flask

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=839097

--- Comment #5 from pcpa  ---
Same issue with python-setuptools as #839071

Update:
Spec URL: http://fedorapeople.org/~pcpa/python-flask-autoindex.spec
SRPM URL:
http://fedorapeople.org/~pcpa/python-flask-autoindex-0.4.1-4.fc19.src.rpm

-- 
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 839071] Review Request: python-flask-babel - Adds i18n/l10n support to Flask applications

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=839071

--- Comment #9 from pcpa  ---
There was indeed an issue with python-setuptools missing. I think it
may have been added to post f18 branch python-devel, but did not
investigate...

New package:
Spec URL: http://fedorapeople.org/~pcpa/python-flask-babel.spec
SRPM URL: http://fedorapeople.org/~pcpa/python-flask-babel-0.8-4.fc19.src.rpm

-- 
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 857193] Review Request: activemq - Open source messaging and Integration Patterns server

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857193

gil cattaneo  changed:

   What|Removed |Added

   Assignee|punto...@libero.it  |nob...@fedoraproject.org
  Flags|fedora-review?  |

-- 
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 857193] Review Request: activemq - Open source messaging and Integration Patterns server

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857193

gil cattaneo  changed:

   What|Removed |Added

 CC||punto...@libero.it
   Assignee|nob...@fedoraproject.org|punto...@libero.it
  Flags||fedora-review?

-- 
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 839071] Review Request: python-flask-babel - Adds i18n/l10n support to Flask applications

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=839071

--- Comment #8 from pcpa  ---
I opened #857266 because it is not passing %check in current rawhide.

-- 
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 843997] Review Request: mlpack - scalable C++ machine learning library

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=843997

José Matos  changed:

   What|Removed |Added

Version|rawhide |18
   Assignee|jama...@fc.up.pt|nob...@fedoraproject.org

--- Comment #5 from José Matos  ---
I will start with small issues as it is easier to follow.

1) Why is not the LICENSE.txt in the main package?
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines

2) Another issue that is not an error but I am curious, why is not the
documentation (doc directory content) packaged?

3) In patch 4 there is a reference to the eminent release of 3.6.
I suspect that you referring to armadillo 3.4. :-)

4) Do you intend to release this package for EPEL5? If not the package can be
simplified in some parts.

-- 
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 853463] Review Request: php-redis - Extension for communicating with the Redis key-value store

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853463

Shawn Iwinski  changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

-- 
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 853463] Review Request: php-redis - Extension for communicating with the Redis key-value store

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853463

Shawn Iwinski  changed:

   What|Removed |Added

  Flags||fedora-review?

-- 
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 853463] Review Request: php-redis - Extension for communicating with the Redis key-value store

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853463

--- Comment #10 from Shawn Iwinski  ---
No blockers.

 APPROVED 

-- 
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 853463] Review Request: php-redis - Extension for communicating with the Redis key-value store

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853463

--- Comment #9 from Shawn Iwinski  ---
Created attachment 612642
  --> https://bugzilla.redhat.com/attachment.cgi?id=612642&action=edit
Generated by fedora-review 0.2.2 (9f8c0e5) last change: 2012-08-09

-- 
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 824703] Review Request: gnome-shell-theme-selene - The Selene gnome-shell theme

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=824703

--- Comment #2 from Mario Blättermann  ---
Any news...?

-- 
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 823682] Review Request: gnome-shell-theme-adwaita - The Adwaita gnome-shell theme created by half_left

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823682

--- Comment #2 from Mario Blättermann  ---
Any news...?

-- 
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 857193] Review Request: activemq - Open source messaging and Integration Patterns server

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857193

--- Comment #1 from Matt Spaulding  ---
Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=4482211

-- 
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 856027] Review Request: activemq-core - The most popular and powerful open source messaging and Integration Patterns server

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856027

Matt Spaulding  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX
Last Closed||2012-09-13 15:17:05

--- Comment #1 from Matt Spaulding  ---
Decided to consolidate activemq into a single package:
https://bugzilla.redhat.com/show_bug.cgi?id=857193

-- 
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 856513] Review Request: python-termcolor - ANSII Color formatting for output in terminal

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856513

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

-- 
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 856027] Review Request: activemq-core - The most popular and powerful open source messaging and Integration Patterns server

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856027

Bug 856027 depends on bug 856026, which changed state.

Bug 856026 Summary: Review Request: kahadb - A file based persistence database
https://bugzilla.redhat.com/show_bug.cgi?id=856026

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX

-- 
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 856026] Review Request: kahadb - A file based persistence database

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856026

Matt Spaulding  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX
Last Closed||2012-09-13 15:16:26

--- Comment #1 from Matt Spaulding  ---
Decided to consolidate activemq into a single package:
https://bugzilla.redhat.com/show_bug.cgi?id=857193

-- 
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 856189] Review Request: python-colorama - Cross-platform colored terminal text

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856189

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

-- 
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 856025] Review Request: activemq-jaas - ActiveMQ Jaas

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856025

Matt Spaulding  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX
Last Closed||2012-09-13 15:15:51

--- Comment #1 from Matt Spaulding  ---
Decided to consolidate activemq into a single package:
https://bugzilla.redhat.com/show_bug.cgi?id=857193

-- 
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 856027] Review Request: activemq-core - The most popular and powerful open source messaging and Integration Patterns server

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856027

Bug 856027 depends on bug 856025, which changed state.

Bug 856025 Summary: Review Request: activemq-jaas - ActiveMQ Jaas
https://bugzilla.redhat.com/show_bug.cgi?id=856025

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX

-- 
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 856161] Review Request: python-openstack-nose-plugin - openstack run_tests.py style output for nosetests

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856161

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

-- 
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 856024] Review Request: activemq-parent - ActiveMQ Parent POM

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856024

Matt Spaulding  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX
Last Closed||2012-09-13 15:15:00

--- Comment #1 from Matt Spaulding  ---
Decided to consolidate activemq into a single package:
https://bugzilla.redhat.com/show_bug.cgi?id=857193

-- 
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 856025] Review Request: activemq-jaas - ActiveMQ Jaas

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856025

Bug 856025 depends on bug 856024, which changed state.

Bug 856024 Summary: Review Request: activemq-parent - ActiveMQ Parent POM
https://bugzilla.redhat.com/show_bug.cgi?id=856024

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX

-- 
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 856026] Review Request: kahadb - A file based persistence database

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856026

Bug 856026 depends on bug 856024, which changed state.

Bug 856024 Summary: Review Request: activemq-parent - ActiveMQ Parent POM
https://bugzilla.redhat.com/show_bug.cgi?id=856024

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX

-- 
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 856027] Review Request: activemq-core - The most popular and powerful open source messaging and Integration Patterns server

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856027

Bug 856027 depends on bug 856024, which changed state.

Bug 856024 Summary: Review Request: activemq-parent - ActiveMQ Parent POM
https://bugzilla.redhat.com/show_bug.cgi?id=856024

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX

-- 
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 832830] Review Request: php-channel-pirum - Adds pear.pirum-project.org channel to PEAR

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832830

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2012-09-13 15:06:29

--- Comment #10 from Fedora Update System  ---
php-channel-pirum-1.0-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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848353] Review Request: python-pthreading - Re-implement threading.Lock, RLock and Condition with libpthread

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848353

--- Comment #16 from Fedora Update System  ---
python-pthreading-0.1.1-1.el6 has been pushed to the Fedora EPEL 6 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
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 856513] Review Request: python-termcolor - ANSII Color formatting for output in terminal

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856513

Matthias Runge  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #2 from Matthias Runge  ---
Thank you for the review!

New Package SCM Request
===
Package Name: python-termcolor
Short Description: ANSII Color formatting for output in terminal
Owners: mrunge
Branches: f18

-- 
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 856161] Review Request: python-openstack-nose-plugin - openstack run_tests.py style output for nosetests

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856161

Matthias Runge  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #5 from Matthias Runge  ---
Thank you for the review!

New Package SCM Request
===
Package Name: python-openstack-nose-plugin
Short Description: openstack run_tests.py style output for nosetests
Owners: mrunge
Branches: f18

-- 
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 856189] Review Request: python-colorama - Cross-platform colored terminal text

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856189

Matthias Runge  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #2 from Matthias Runge  ---
thank you for the quick review! 

New Package SCM Request
===
Package Name: python-colorama
Short Description: Cross-platform colored terminal text
Owners: mrunge
Branches: f18

-- 
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 857193] New: Review Request: activemq - Open source messaging and Integration Patterns server

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857193

Bug ID: 857193
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: activemq - Open source messaging and
Integration Patterns server
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: mspauldin...@gmail.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://madsa.fedorapeople.org/activemq.spec
SRPM URL: http://madsa.fedorapeople.org/activemq-5.6.0-1.fc18.src.rpm
Description:
The most popular and powerful open source messaging and Integration Patterns
server.
Fedora Account System Username: madsa

I had posted package review requests for activemq modules separately, but after
talking with people decided it better to build a single activemq package and
disable the modules that cannot be built yet.

Currently, only core, jaas, and kahadb are enabled.

-- 
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 857180] New: Review Request: gnome-clocks - Clock application designed for GNOME 3

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857180

Bug ID: 857180
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: gnome-clocks - Clock application
designed for GNOME 3
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: yan...@declera.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://declera.com/~yaneti/gnome-clocks/gnome-clocks.spec
SRPM URL:
http://declera.com/~yaneti/gnome-clocks/gnome-clocks-0.1.1-1.fc19.src.rpm
Description: Clock application designed for GNOME 3
Fedora Account System Username: yaneti

gnome-clocks.noarch: E: explicit-lib-dependency libgweather
gnome-clocks.noarch: W: no-manual-page-for-binary gnome-clocks
2 packages and 0 specfiles checked; 1 errors, 1 warnings.

gnome-clocks is using gnome-introspection and the GWeather-3.0.typelib

-- 
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 856858] Review Request: Jokte - Latam CMS, Joomla Fork

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856858

--- Comment #7 from Juan Botero  ---
I updated the spec file with release tag.

Thanks.

-- 
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 856554] python-pottymouth - Transform unstructured, untrusted text to safe, valid XHTML

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856554

--- Comment #1 from Luis Bazan  ---
These is a informal revision


Package Review
==

Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated



 Generic 
[x]: EXTRA Rpmlint is run on all installed packages.
 Note: There are rpmlint messages (see attachment).
[x]: EXTRA Spec file according to URL is the same as in SRPM.
[ ]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported primary architecture.
[ ]: MUST %build honors applicable compiler flags or justifies otherwise.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[ ]: MUST Package contains no bundled libraries.
[ ]: MUST Changelog in prescribed format.
[ ]: MUST Sources contain only permissible code or content.
[!]: MUST Each %files section contains %defattr if rpm < 4.4
 Note: defattr() present in %files section. This is OK if packaging
 for EPEL5. Otherwise not needed
[ ]: MUST Macros in Summary, %description expandable at SRPM build time.
[ ]: MUST Package contains desktop file if it is a GUI application.
[ ]: MUST Development files must be in a -devel package
[ ]: MUST Package requires other packages for directories it uses.
[ ]: MUST Package uses nothing in %doc for runtime.
[ ]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[ ]: MUST Package complies to the Packaging Guidelines
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf would be needed if support for EPEL5 is required
[ ]: MUST Large documentation files are in a -doc subpackage, if required.
[!]: MUST 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.
[!]: MUST License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. No licenses
 found. Please check the source files for licenses manually.
[ ]: MUST Package consistently uses macro is (instead of hard-coded directory
 names).
[x]: MUST Package is named using only allowed ascii characters.
[ ]: MUST Package is named according to the Package Naming Guidelines.
[ ]: MUST Package does not generate any conflict.
 Note: Package contains no Conflicts: tag(s)
[ ]: MUST Package obeys FHS, except libexecdir and /usr/target.
[ ]: MUST If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[ ]: MUST Package must own all directories that it creates.
[ ]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[ ]: MUST Package is not relocatable.
[ ]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
[ ]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[ ]: MUST Package contains systemd file(s) if in need.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[x]: SHOULD Buildroot is not present
 Note: Unless packager wants to package for EPEL5 this is fine
[x]: SHOULD Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean would be needed if support for EPEL5 is required
[ ]: SHOULD 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]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
 /usr/sbin.
[ ]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[ ]: SHOULD Package functions as described.
[ ]: SHOULD Latest version is packaged.
[ ]: SHOULD Package does not include license text files separate from
 upstream.
[x]: SHOULD SourceX tarball generation or download is documented.
[!]: SHOULD SourceX / PatchY prefixed with %{name}.
 Note: Source0 (PottyMouth-2.2.1.tar.gz)
[x]: SHOULD SourceX is a working URL.
[ ]: SHOULD Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[ ]: SHOULD Package should compile and build into binary rpm

[Bug 856979] Review Request: rubygem-openstack - Ruby Openstack Compute and Object-Store bindings

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856979

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #8 from Fedora Update System  ---
rubygem-openstack-1.0.5-2.fc18 has been pushed to the Fedora 18 testing
repository.

-- 
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 851734] Review Request: perl-Regexp-Grammars - Add grammatical parsing features to perl regular expressions

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851734

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #11 from Fedora Update System  ---
perl-Regexp-Grammars-1.021-4.fc18 has been pushed to the Fedora 18 testing
repository.

-- 
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 857102] Review Request: jackson-databind - General data-binding package for Jackson (2.x)

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857102

gil cattaneo  changed:

   What|Removed |Added

 Blocks||857139

-- 
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 857077] Review Request: jackson-core - Core part of Jackson

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857077

gil cattaneo  changed:

   What|Removed |Added

 Blocks||857139

-- 
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 857139] Review Request: jackson-module-mrbean - An extension that implements support for POJO type materialization

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857139

gil cattaneo  changed:

   What|Removed |Added

 Depends On||857102, 857077

-- 
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 857139] New: Review Request: jackson-module-mrbean - An extension that implements support for POJO type materialization

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857139

Bug ID: 857139
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: jackson-module-mrbean - An extension
that implements support for POJO type materialization
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: punto...@libero.it
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://gil.fedorapeople.org/jackson-module-mrbean.spec
SRPM URL:
http://gil.fedorapeople.org/jackson-module-mrbean-2.0.5-1.fc16.src.rpm
Description: Abstract type materialization module: implement interfaces,
abstract classes on run-time.
Fedora Account System Username: gil

-- 
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 857137] Review Request: jackson-module-jaxb-annotations - JAXB annotations support for Jackson (2.x)

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857137

gil cattaneo  changed:

   What|Removed |Added

 Blocks||857138

-- 
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 825347] Review Request: jersey - JAX-RS (JSR 311) production quality Reference Implementation

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=825347

gil cattaneo  changed:

   What|Removed |Added

 Blocks||857138

-- 
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 857138] Review Request: jackson-jaxrs-json-provider - JAX-RS MessageBodyReader and -Writer implementations for JSON

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857138

gil cattaneo  changed:

   What|Removed |Added

 Depends On||857137, 825347

-- 
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 857138] New: Review Request: jackson-jaxrs-json-provider - JAX-RS MessageBodyReader and -Writer implementations for JSON

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857138

Bug ID: 857138
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: jackson-jaxrs-json-provider - JAX-RS
MessageBodyReader and -Writer implementations for JSON
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: punto...@libero.it
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://gil.fedorapeople.org/jackson-jaxrs-json-provider.spec
SRPM URL:
http://gil.fedorapeople.org/jackson-jaxrs-json-provider-2.0.5-1.fc16.src.rpm
Description: Functionality to handle JSON input/output for JAX-RS
implementations
(like Jersey and RESTeasy) using standard Jackson data binding.
Fedora Account System Username: gil

-- 
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 856722] Review Request: openstack-cinder - OpenStack Volume service

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856722

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

-- 
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 857137] Review Request: jackson-module-jaxb-annotations - JAXB annotations support for Jackson (2.x)

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857137

gil cattaneo  changed:

   What|Removed |Added

 Depends On||857102, 857077

-- 
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 857077] Review Request: jackson-core - Core part of Jackson

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857077

gil cattaneo  changed:

   What|Removed |Added

 Blocks||857137

-- 
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 857102] Review Request: jackson-databind - General data-binding package for Jackson (2.x)

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857102

gil cattaneo  changed:

   What|Removed |Added

 Blocks||857137

-- 
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 857137] New: Review Request: jackson-module-jaxb-annotations - JAXB annotations support for Jackson (2.x)

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857137

Bug ID: 857137
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: jackson-module-jaxb-annotations - JAXB
annotations support for Jackson (2.x)
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: punto...@libero.it
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://gil.fedorapeople.org/jackson-module-jaxb-annotations.spec
SRPM URL:
http://gil.fedorapeople.org/jackson-module-jaxb-annotations-2.0.5-1.fc16.src.rpm
Description: Support for using JAXB annotations as an alternative to
"native" Jackson annotations, for configuring data binding.
Fedora Account System Username: gil

-- 
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 857102] Review Request: jackson-databind - General data-binding package for Jackson (2.x)

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857102

gil cattaneo  changed:

   What|Removed |Added

 Blocks||857136

-- 
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 857136] Review Request: jackson-dataformat-smile - Jackson extension that adds support for Smile

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857136

gil cattaneo  changed:

   What|Removed |Added

 Depends On||857102

-- 
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 857136] New: Review Request: jackson-dataformat-smile - Jackson extension that adds support for Smile

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857136

Bug ID: 857136
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: jackson-dataformat-smile - Jackson
extension that adds support for Smile
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: punto...@libero.it
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://gil.fedorapeople.org/jackson-dataformat-smile.spec
SRPM URL:
http://gil.fedorapeople.org/jackson-dataformat-smile-2.0.6b-1.fc16.src.rpm
Description: Support for reading and writing Smile ("binary JSON") encoded data
using Jackson abstractions (streaming API, data binding, tree model).
Fedora Account System Username: gil

-- 
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 856722] Review Request: openstack-cinder - OpenStack Volume service

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856722

Pádraig Brady  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #5 from Pádraig Brady  ---
New Package SCM Request
===
Package Name: openstack-cinder
Short Description: OpenStack Volume service
Owners: pbrady eharney apevec
Branches: f18 el6
InitialCC:

-- 
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 856161] Review Request: python-openstack-nose-plugin - openstack run_tests.py style output for nosetests

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856161

Pádraig Brady  changed:

   What|Removed |Added

  Flags||fedora-review+

--- Comment #4 from Pádraig Brady  ---
Package Review
==

Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated



 Generic 
[x]: EXTRA Rpmlint is run on all installed packages.
 Note: No rpmlint messages.
[x]: EXTRA Spec file according to URL is the same as in SRPM.
[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported primary architecture.
[x]: MUST %build honors applicable compiler flags or justifies otherwise.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Each %files section contains %defattr if rpm < 4.4
 Note: Note: defattr macros not found. They would be needed for EPEL5
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[-]: MUST Package contains desktop file if it is a GUI application.
[-]: MUST Development files must be in a -devel package
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Package complies to the Packaging Guidelines
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf would be needed if support for EPEL5 is required
[-]: MUST Large documentation files are in a -doc subpackage, if required.
[-]: MUST 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.
[ ]: MUST License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 "Apache (v2.0)" For detailed output of licensecheck see file:
 /home/padraig/856161-python-openstack-nose-plugin/licensecheck.txt
[x]: MUST Package consistently uses macro is (instead of hard-coded directory
 names).
[x]: MUST Package is named using only allowed ascii characters.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generate any conflict.
 Note: Package contains no Conflicts: tag(s)
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[-]: MUST If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
 Note: Installation errors (see attachment)
[x]: MUST Package is not relocatable.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[-]: MUST Package contains systemd file(s) if in need.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[x]: SHOULD Buildroot is not present
 Note: Unless packager wants to package for EPEL5 this is fine
[x]: SHOULD Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean would be needed if support for EPEL5 is required
[!]: SHOULD 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]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
 /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[x]: SHOULD Package functions as described.
[x]: SHOULD Latest version is packaged.
[x]: SHOULD Package does not include license text files separate from
 upstream.
[x]: SHOULD SourceX tarball generation or download is documented.
[!]: SHOULD SourceX / PatchY prefixed with %{name}.
 Note: Source0 (openstack.nose

[Bug 855656] Review Request: perl-Safe-Isa - Call isa, can, does and DOES safely on things that may not be objects

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855656

Petr Pisar  changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #1 from Petr Pisar  ---
Source tar ball is original (verified from
,
SHA-256: 74f3fbbc296183a7cacada11304073a67a4beca4fd77aa2eff89af6fbc23591e). Ok.
Source0 and URL are usable. Ok.
Summary verified from lib/Safe/Isa.pm. Ok.
Description verified from lib/Safe/Isa.pm. Ok.
License verified from lib/Safe/Isa.pm. Ok.
No XS code, noarch BuildArch is Ok.
Build-time dependencies are Ok.
All tests pass. Ok.

$ rpmlint perl-Safe-Isa.spec ../SRPMS/perl-Safe-Isa-1.02-1.fc19.src.rpm
../RPMS/noarch/perl-Safe-Isa-1.02-1.fc19.noarch.rpm 
perl-Safe-Isa.src: W: spelling-error %description -l en_US Util -> Til, U til,
Until
perl-Safe-Isa.noarch: W: spelling-error %description -l en_US Util -> Til, U
til, Until
2 packages and 1 specfiles checked; 0 errors, 2 warnings.
rpmlint us Ok.

$ rpm -q -lv -p ../RPMS/noarch/perl-Safe-Isa-1.02-1.fc19.noarch.rpm 
drwxr-xr-x2 rootroot0 Sep 13 17:29
/usr/share/doc/perl-Safe-Isa-1.02
-rw-r--r--1 rootroot  232 Jul 19 21:00
/usr/share/doc/perl-Safe-Isa-1.02/Changes
-rw-r--r--1 rootroot 3713 Jul 19 21:00
/usr/share/doc/perl-Safe-Isa-1.02/README
-rw-r--r--1 rootroot 3232 Sep 13 17:29
/usr/share/man/man3/Safe::Isa.3pm.gz
drwxr-xr-x2 rootroot0 Sep 13 17:29
/usr/share/perl5/vendor_perl/Safe
-rw-r--r--1 rootroot 4003 Jul 19 21:00
/usr/share/perl5/vendor_perl/Safe/Isa.pm
File permissions and layout are Ok.

$ rpm -q --requires -p ../RPMS/noarch/perl-Safe-Isa-1.02-1.fc19.noarch.rpm
| sort | uniq -c
  1 perl(base)
  1 perl(Exporter)
  1 perl(:MODULE_COMPAT_5.16.1)
  1 perl(Scalar::Util)
  1 perl(strict)
  1 perl(warnings)
  1 rpmlib(CompressedFileNames) <= 3.0.4-1
  1 rpmlib(FileDigests) <= 4.6.0-1
  1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  1 rpmlib(PayloadIsXz) <= 5.2-1
Binary requires are Ok.

$ rpm -q --provides  -p ../RPMS/noarch/perl-Safe-Isa-1.02-1.fc19.noarch.rpm
| sort | uniq -c
  1 perl(Safe::Isa) = 1.02
  1 perl-Safe-Isa = 1.02-1.fc19
Binary provides are Ok.

$ resolvedeps rawhide ../RPMS/noarch/perl-Safe-Isa-1.02-1.fc19.noarch.rpm
Binary dependencies resolvable. Ok.

Package builds in F19
(http://koji.fedoraproject.org/koji/taskinfo?taskID=4481430). Ok.

Package is in line with Fedora and Perl packaging guidelines.


Resolution: Package APPROVED.

-- 
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 856513] Review Request: python-termcolor - ANSII Color formatting for output in terminal

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856513

Pádraig Brady  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||p...@draigbrady.com
   Assignee|nob...@fedoraproject.org|p...@draigbrady.com
  Flags||fedora-review+

--- Comment #1 from Pádraig Brady  ---
why does upstream describe as ANSII rather than ANSI ?
There is very little in this package really.


Package Review
==

Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated



 Generic 
[x]: EXTRA Rpmlint is run on all installed packages.
 Note: There are rpmlint messages (see attachment).
[x]: EXTRA Spec file according to URL is the same as in SRPM.
[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported primary architecture.
[-]: MUST %build honors applicable compiler flags or justifies otherwise.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Each %files section contains %defattr if rpm < 4.4
 Note: Note: defattr macros not found. They would be needed for EPEL5
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[-]: MUST Package contains desktop file if it is a GUI application.
[-]: MUST Development files must be in a -devel package
[-]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Package complies to the Packaging Guidelines
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf would be needed if support for EPEL5 is required
[-]: MUST Large documentation files are in a -doc subpackage, if required.
[x]: MUST 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]: MUST 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)" For detailed output of licensecheck see file:
 /home/padraig/856513-python-termcolor/licensecheck.txt
[x]: MUST Package consistently uses macro is (instead of hard-coded directory
 names).
[x]: MUST Package is named using only allowed ascii characters.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generate any conflict.
 Note: Package contains no Conflicts: tag(s)
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[-]: MUST If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Package is not relocatable.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint is run on all rpms the build produces.
 Note: No rpmlint messages.
[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[-]: MUST Package contains systemd file(s) if in need.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[x]: SHOULD Buildroot is not present
 Note: Unless packager wants to package for EPEL5 this is fine
[x]: SHOULD Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean would be needed if support for EPEL5 is required
[-]: SHOULD 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]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
 /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[x]: SHOULD Package functions as described.
[x]: SHOULD Latest version is pack

[Bug 856161] Review Request: python-openstack-nose-plugin - openstack run_tests.py style output for nosetests

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856161

--- Comment #3 from Pádraig Brady  ---
It's strange that it needs both colorama and termcolor, since they do the same
thing. Colorama has more substance to it as it's cross platform. termcolor is
just a few lines of python mapping color names to escape sequences.

-- 
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 855656] Review Request: perl-Safe-Isa - Call isa, can, does and DOES safely on things that may not be objects

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855656

Petr Pisar  changed:

   What|Removed |Added

  Flags||fedora-review?
 Status|NEW |ASSIGNED
 CC||ppi...@redhat.com
   Assignee|nob...@fedoraproject.org|ppi...@redhat.com

-- 
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 835804] Review Request: perl-Module-Install-ReadmeMarkdownFromPod - Create README.mkdn from POD

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=835804

Petr Pisar  changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #3 from Petr Pisar  ---
Spec file changes:
--- perl-Module-Install-ReadmeMarkdownFromPod.spec.old  2012-06-27
09:18:38.0 +0200
+++ perl-Module-Install-ReadmeMarkdownFromPod.spec  2012-09-13
16:52:23.0 +0200
@@ -9,12 +9,20 @@
 BuildArch:  noarch
 BuildRequires:  perl >= 1:5.6.0
 # XXX: We cannot remove ./inc because it build-requires this module
-BuildRequires:  perl(ExtUtils::MakeMaker)
 BuildRequires:  perl(base)
-BuildRequires:  perl(Module::Install::Base)
-BuildRequires:  perl(Module::Install::ReadmeFromPod)
-BuildRequires:  perl(Pod::Markdown)
-BuildRequires:  perl(Test::More) >= 0.70
+BuildRequires:  perl(Carp)
+BuildRequires:  perl(CPAN) >= 1.89
+BuildRequires:  perl(Cwd)
+BuildRequires:  perl(ExtUtils::Command)
+BuildRequires:  perl(ExtUtils::MakeMaker)
+BuildRequires:  perl(ExtUtils::Manifest)
+BuildRequires:  perl(File::Path)
+BuildRequires:  perl(File::Spec)
+BuildRequires:  perl(Pod::Parser)
+BuildRequires:  perl(Pod::Text)
+BuildRequires:  perl(Test::Builder::Module)
+BuildRequires:  perl(Test::Harness)
+BuildRequires:  perl(URI::Escape)
 Requires:   perl(Module::Install)
 Requires:   perl(Module::Install::ReadmeFromPod)
 Requires:   perl(Pod::Markdown)
@@ -29,16 +37,17 @@
 %prep
 %setup -q -n Module-Install-ReadmeMarkdownFromPod-%{version}

+# README is ISO-8859-1 encoded
+iconv -f iso-8859-1 -t utf8 < README > README.utf8
+mv README.utf8 README
+
 %build
 %{__perl} Makefile.PL INSTALLDIRS=vendor
 make %{?_smp_mflags}

 %install
 make pure_install PERL_INSTALL_ROOT=$RPM_BUILD_ROOT
-
 find $RPM_BUILD_ROOT -type f -name .packlist -exec rm -f {} \;
-find $RPM_BUILD_ROOT -depth -type d -exec rmdir {} 2>/dev/null \;
-
 %{_fixperms} $RPM_BUILD_ROOT/*

 %check


> TODO: You can remove pruning empty directories from %install section.
 %install
 make pure_install PERL_INSTALL_ROOT=$RPM_BUILD_ROOT
-
 find $RPM_BUILD_ROOT -type f -name .packlist -exec rm -f {} \;
-find $RPM_BUILD_ROOT -depth -type d -exec rmdir {} 2>/dev/null \;
-
 %{_fixperms} $RPM_BUILD_ROOT/*
Ok.

> TODO: Do not build-require modules bundled in ./inc or remove them from the
> directory.
> FIX: Specify ./inc dependencies or unbundle the modules (e.g. `perl(Cwd)' at 
> inc/Module/Install.pm:123:use).
-BuildRequires:  perl(ExtUtils::MakeMaker)
 BuildRequires:  perl(base)
-BuildRequires:  perl(Module::Install::Base)
-BuildRequires:  perl(Module::Install::ReadmeFromPod)
-BuildRequires:  perl(Pod::Markdown)
-BuildRequires:  perl(Test::More) >= 0.70
+BuildRequires:  perl(Carp)
+BuildRequires:  perl(CPAN) >= 1.89
+BuildRequires:  perl(Cwd)
+BuildRequires:  perl(ExtUtils::Command)
+BuildRequires:  perl(ExtUtils::MakeMaker)
+BuildRequires:  perl(ExtUtils::Manifest)
+BuildRequires:  perl(File::Path)
+BuildRequires:  perl(File::Spec)
+BuildRequires:  perl(Pod::Parser)
+BuildRequires:  perl(Pod::Text)
+BuildRequires:  perl(Test::Builder::Module)
+BuildRequires:  perl(Test::Harness)
+BuildRequires:  perl(URI::Escape)
Ok.

> FIX: Recode README to UTF-8.
$ rpmlint perl-Module-Install-ReadmeMarkdownFromPod.spec
../SRPMS/perl-Module-Install-ReadmeMarkdownFromPod-0.03-1.fc19.src.rpm
../RPMS/noarch/perl-Module-Install-ReadmeMarkdownFromPod-0.03-1.fc19.noarch.rpm 
2 packages and 1 specfiles checked; 0 errors, 0 warnings.
Ok.

Package builds in F19
(http://koji.fedoraproject.org/koji/taskinfo?taskID=4481355). Ok.

Package is in line with Fedora and Perl packaging guidelines.

Resolution: Package APPROVED.

-- 
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 856189] Review Request: python-colorama - Cross-platform colored terminal text

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856189

Pádraig Brady  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||p...@draigbrady.com
   Assignee|nob...@fedoraproject.org|p...@draigbrady.com
  Flags||fedora-review+

--- Comment #1 from Pádraig Brady  ---
Package Review
==

Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated



 Generic 
[x]: EXTRA Rpmlint is run on all installed packages.
 Note: There are rpmlint messages (see attachment).
[x]: EXTRA Spec file according to URL is the same as in SRPM.
[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported primary architecture.
[-]: MUST %build honors applicable compiler flags or justifies otherwise.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Each %files section contains %defattr if rpm < 4.4
 Note: Note: defattr macros not found. They would be needed for EPEL5
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[-]: MUST Package contains desktop file if it is a GUI application.
[-]: MUST Development files must be in a -devel package
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Package complies to the Packaging Guidelines
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf would be needed if support for EPEL5 is required
[-]: MUST Large documentation files are in a -doc subpackage, if required.
[x]: MUST 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]: MUST License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. No licenses
 found. Please check the source files for licenses manually.
[x]: MUST Package consistently uses macro is (instead of hard-coded directory
 names).
[x]: MUST Package is named using only allowed ascii characters.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generate any conflict.
 Note: Package contains no Conflicts: tag(s)
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[-]: MUST If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: MUST Package must own all directories that it creates.
[-]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[-]: MUST Package is not relocatable.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint is run on all rpms the build produces.
 Note: No rpmlint messages.
[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[-]: MUST Package contains systemd file(s) if in need.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[x]: SHOULD Buildroot is not present
 Note: Unless packager wants to package for EPEL5 this is fine
[x]: SHOULD Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean would be needed if support for EPEL5 is required
[-]: SHOULD 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]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
 /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[x]: SHOULD Package functions as described.
[x]: SHOULD Latest version is packaged.
[x]: SHOULD Package does not include license text files separate from
 upstream.
[x]: SHOULD SourceX tarball generation or download is documented.
[!]: SHOULD Sour

[Bug 856722] Review Request: openstack-cinder - OpenStack Volume service

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856722

Matthias Runge  changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #4 from Matthias Runge  ---
alright, 

Package APPROVED

-- 
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 857102] Review Request: jackson-databind - General data-binding package for Jackson (2.x)

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857102

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)

-- 
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 856161] Review Request: python-openstack-nose-plugin - openstack run_tests.py style output for nosetests

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856161

Matthias Runge  changed:

   What|Removed |Added

 Blocks||856513

--- Comment #2 from Matthias Runge  ---
oops, forgot to mark the corresponding review request, here it is:
https://bugzilla.redhat.com/show_bug.cgi?id=856513

-- 
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 856513] Review Request: python-termcolor - ANSII Color formatting for output in terminal

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856513

Matthias Runge  changed:

   What|Removed |Added

 Depends On||856161

-- 
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 857102] Review Request: jackson-databind - General data-binding package for Jackson (2.x)

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857102

gil cattaneo  changed:

   What|Removed |Added

 Depends On||857080

-- 
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 857080] Review Request: jackson-annotations - Core annotations for Jackson data processor

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857080

gil cattaneo  changed:

   What|Removed |Added

 Blocks||857102

-- 
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 857102] New: Review Request: jackson-databind - General data-binding package for Jackson (2.x)

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857102

Bug ID: 857102
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: jackson-databind - General
data-binding package for Jackson (2.x)
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: punto...@libero.it
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://gil.fedorapeople.org/jackson-databind.spec
SRPM URL: http://gil.fedorapeople.org/jackson-databind-2.0.6-1.fc16.src.rpm
Description: General data-binding functionality for Jackson:
works on core streaming API.
Fedora Account System Username: gil

-- 
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 835804] Review Request: perl-Module-Install-ReadmeMarkdownFromPod - Create README.mkdn from POD

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=835804

--- Comment #2 from Jitka Plesnikova  ---
Updated.

Spec URL:
http://jplesnik.fedorapeople.org/perl-Module-Install-ReadmeMarkdownFromPod/perl-Module-Install-ReadmeMarkdownFromPod.spec
SRPM URL:
http://jplesnik.fedorapeople.org/perl-Module-Install-ReadmeMarkdownFromPod/perl-Module-Install-ReadmeMarkdownFromPod-0.03-1.fc19.src.rpm

-- 
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 851734] Review Request: perl-Regexp-Grammars - Add grammatical parsing features to perl regular expressions

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851734

--- Comment #10 from Fedora Update System  ---
perl-Regexp-Grammars-1.021-4.el6 has been submitted as an update for Fedora
EPEL 6.
https://admin.fedoraproject.org/updates/perl-Regexp-Grammars-1.021-4.el6

-- 
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 857077] Review Request: jackson-core - Core part of Jackson

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857077

gil cattaneo  changed:

   What|Removed |Added

 Blocks||857080

-- 
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 857080] Review Request: jackson-annotations - Core annotations for Jackson data processor

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857080

gil cattaneo  changed:

   What|Removed |Added

 Depends On||857077, 857079

-- 
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 857079] Review Request: fasterxml-oss-parent - FasterXML parent pom

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857079

gil cattaneo  changed:

   What|Removed |Added

 Blocks||857080

-- 
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 857080] New: Review Request: jackson-annotations - Core annotations for Jackson data processor

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857080

Bug ID: 857080
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: jackson-annotations - Core annotations
for Jackson data processor
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: punto...@libero.it
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://gil.fedorapeople.org/jackson-annotations.spec
SRPM URL: http://gil.fedorapeople.org/jackson-annotations-2.0.6-1.fc16.src.rpm
Description: Core annotations used for value types,
used by Jackson data-binding package.
Fedora Account System Username: gil

-- 
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 857079] New: Review Request: fasterxml-oss-parent - FasterXML parent pom

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857079

Bug ID: 857079
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: fasterxml-oss-parent - FasterXML
parent pom
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: punto...@libero.it
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://gil.fedorapeople.org/fasterxml-oss-parent.spec
SRPM URL: http://gil.fedorapeople.org/fasterxml-oss-parent-3-1.fc16.src.rpm
Description: FasterXML is the business behind the Woodstox streaming XML
parser,
Jackson streaming JSON parser, the Aalto non-blocking XML parser, and
a growing family of utility libraries and extensions.

FasterXML offers consulting services for adoption, performance tuning,
and extension.

This package contains the parent pom file for FasterXML.com projects.
Fedora Account System Username: gil

-- 
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 851734] Review Request: perl-Regexp-Grammars - Add grammatical parsing features to perl regular expressions

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851734

--- Comment #9 from Fedora Update System  ---
perl-Regexp-Grammars-1.021-4.fc17 has been submitted as an update for Fedora
17.
https://admin.fedoraproject.org/updates/perl-Regexp-Grammars-1.021-4.fc17

-- 
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 856858] Review Request: Jokte - Latam CMS, Joomla Fork

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856858

Luis Bazan  changed:

   What|Removed |Added

 CC||bazanlui...@gmail.com

--- Comment #6 from Luis Bazan  ---
Hi Juan 

these is a example:

Name:   jokte-cms
Version:1.1
Release:1%{?dist}
Summary:Latin American CMS, fork of Joomla CMS
License:GPL
Group:  Applications/Internet
URL:https://github.com/JokteLatinoamerica/
Source0:   
https://github.com/downloads/JokteLatinoamerica/jokte-cms/Jokte-cms-v1.1.tar.gz
BuildArch:  noarch

Requires:   mysql >= 5.0
BuildRequires:  



Review the
https://fedoraproject.org/wiki/How_to_create_an_RPM_package/es#Creando_un_archivo_spec

remember check use rpmlint
and try test in koji 

Regards!

-- 
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 857077] New: Review Request: jackson-core - Core part of Jackson

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857077

Bug ID: 857077
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: jackson-core - Core part of Jackson
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: punto...@libero.it
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://gil.fedorapeople.org/jackson-core.spec
SRPM URL: http://gil.fedorapeople.org/jackson-core-2.0.6-1.fc16.src.rpm
Description: Core part of Jackson that defines Streaming API as well
as basic shared abstractions.
Fedora Account System Username: gil

-- 
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 856238] Review Request: scratch - Programming language learning environment for stories, games, music and art

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856238

--- Comment #4 from Matthew Miller  ---
 > Seems to me both part from the same Kevin Somervill sources. I think you can
> pick some minor things from my spec too.

Definitely. Anything in particular? I notice you require squeak-vm-nonXOplugins
— are there things in that set of plugins which enhance Scratch?

> Is there any technical reason for split scratch in so many subpackages? I
> mean: is there any external dependency or strong convenience for reuse some
> of it? Maybe some Sugar restriction...  if not, I feel it's better to keep
> it simpler as possible.

I think comment #2 covers this. :)

-- 
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 851734] Review Request: perl-Regexp-Grammars - Add grammatical parsing features to perl regular expressions

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851734

--- Comment #8 from Fedora Update System  ---
perl-Regexp-Grammars-1.021-4.fc18 has been submitted as an update for Fedora
18.
https://admin.fedoraproject.org/updates/perl-Regexp-Grammars-1.021-4.fc18

-- 
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 851734] Review Request: perl-Regexp-Grammars - Add grammatical parsing features to perl regular expressions

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851734

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
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 856161] Review Request: python-openstack-nose-plugin - openstack run_tests.py style output for nosetests

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856161

Pádraig Brady  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||p...@draigbrady.com
   Assignee|nob...@fedoraproject.org|p...@draigbrady.com

--- Comment #1 from Pádraig Brady  ---
This requires python-termcolor which I can't find
Should it be requiring python-xtermcolor?

-- 
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 845934] Review Request: wt - C++ library for developing web applications

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=845934

Michal Minar  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2012-09-13 09:39:02

-- 
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 845934] Review Request: wt - C++ library for developing web applications

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=845934

--- Comment #40 from Fedora Update System  ---
wt-3.2.2-6.p1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/wt-3.2.2-6.p1.fc18

-- 
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 845934] Review Request: wt - C++ library for developing web applications

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=845934

--- Comment #39 from Fedora Update System  ---
wt-3.2.2-6.p1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/wt-3.2.2-6.p1.fc17

-- 
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 845934] Review Request: wt - C++ library for developing web applications

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=845934

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
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 845934] Review Request: wt - C++ library for developing web applications

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=845934

--- Comment #38 from Fedora Update System  ---
wt-3.2.2-6.p1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/wt-3.2.2-6.p1.fc16

-- 
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 856719] Review Request: python-cinderclient - Python API and CLI for OpenStack cinder

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856719

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

-- 
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 851734] Review Request: perl-Regexp-Grammars - Add grammatical parsing features to perl regular expressions

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851734

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

-- 
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 856722] Review Request: openstack-cinder - OpenStack Volume service

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856722

--- Comment #3 from Pádraig Brady  ---
I changed as per all your suggestions above.
However I didn't add the reverse dep between python-cinder and the main
package,
as we don't do that elsewhere in the openstack packages.
I'll consider updating all packages like that in future.
Also related to this, is the doc package should probably be installable
independently of the main package. Again all openstack packages
would need to be updated.

thanks!

Spec URL: http://fedorapeople.org/~pbrady/cinder/openstack-cinder.spec
SRPM URL:
http://fedorapeople.org/~pbrady/cinder/openstack-cinder-2012.2-0.2.f3.fc18.src.rpm
Description: OpenStack Volume (codename Cinder) provides services to manage and
access block storage volumes for use by Virtual Machine instances.
Fedora Account System Username: pbrady

-- 
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 856979] Review Request: rubygem-openstack - Ruby Openstack Compute and Object-Store bindings

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856979

--- Comment #7 from Fedora Update System  ---
rubygem-openstack-1.0.5-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/rubygem-openstack-1.0.5-2.fc17

-- 
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 851734] Review Request: perl-Regexp-Grammars - Add grammatical parsing features to perl regular expressions

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851734

Bill Pemberton  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #6 from Bill Pemberton  ---
New Package SCM Request
===
Package Name: perl-Regexp-Grammars
Short Description: Add grammatical parsing features to perl regular expressions
Owners: wfp
Branches: f17 f18 el6
InitialCC: perl-sig

-- 
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 839071] Review Request: python-flask-babel - Adds i18n/l10n support to Flask applications

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=839071

--- Comment #7 from pcpa  ---
Weird. I will try again tonight (when I "contribute" to fedora :-) to build in
mock (I did confirm it was working for the last package to review). Did you use
mock with a rawhide chroot? I need these packages for rawhide or newer (I was
hoping to have sagemath for fedora 18, but gave up...)

-- 
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 856979] Review Request: rubygem-openstack - Ruby Openstack Compute and Object-Store bindings

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856979

--- Comment #6 from Fedora Update System  ---
rubygem-openstack-1.0.5-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/rubygem-openstack-1.0.5-2.fc18

-- 
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 856979] Review Request: rubygem-openstack - Ruby Openstack Compute and Object-Store bindings

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856979

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
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 856719] Review Request: python-cinderclient - Python API and CLI for OpenStack cinder

2012-09-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856719

Pádraig Brady  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #3 from Pádraig Brady  ---
New Package SCM Request
===
Package Name: python-cinderclient
Short Description: Python API and CLI for OpenStack cinder
Owners: pbrady eharney apevec
Branches: f18 el6
InitialCC:

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

  1   2   >