[Bug 1104476] New: Review Request: python-wsgilog - WSGI logging and event reporting

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

Bug ID: 1104476
   Summary: Review Request: python-wsgilog - WSGI logging and
event reporting
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: rhbgs.10.bigi_g...@xoxy.net
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://id-install-prd.ethz.ch/packages/python-wsgilog.spec
SRPM URL:
http://id-install-prd.ethz.ch/packages/python-wsgilog-0.3-1.fc20.src.rpm

Description: Supports logging events in WSGI applications to STDOUT, time
rotated log files, email, syslog, and web servers. Also supports catching and
sending HTML-formatted exception tracebacks to a web browser for debugging.

Fedora Account System Username: pagiator

Koji build URLs: 
f20: http://koji.fedoraproject.org/koji/taskinfo?taskID=6922976
rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=6922978

-- 
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 1086217] Review Request: nodejs-strip-json-comments - Strip comments from JSON

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



--- Comment #7 from anish apa...@redhat.com ---
Hi  Tom Hughes!

Thank you for your review comments, please find new SRPMS and spec file on 

Spec URL: http://anishpatil.fedorapeople.org/nodejs-strip-json-comments.spec


SRPM URL:
http://anishpatil.fedorapeople.org/nodejs-strip-json-comments-0.1.3-4.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 1086231] Review Request: nodejs-jsonparse - Pure-js JSON streaming parser for node.js

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



--- Comment #7 from anish apa...@redhat.com ---
Thanks for review comments, please find new spec file and SRPM file 


SPEC file:-  http://anishpatil.fedorapeople.org/nodejs-jsonparse.spec

SRPM file :-
http://anishpatil.fedorapeople.org/nodejs-jsonparse-0.0.6-4.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 1092057] Review Request: golang-github-kdar-factorlog - Fast logging infrastructure for Go

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

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

   What|Removed |Added

   Fixed In Version|golang-github-kdar-factorlo |golang-github-kdar-factorlo
   |g-0-0.1.git814d8f7.el6  |g-0-0.1.git814d8f7.fc20



--- Comment #9 from Fedora Update System upda...@fedoraproject.org ---
golang-github-kdar-factorlog-0-0.1.git814d8f7.fc20 has been pushed to the
Fedora 20 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 1060989] Review Request: ghc-io-streams - Simple, composable, easy-to-use stream I/O

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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
ghc-io-streams-1.1.4.5-1.fc20 has been pushed to the Fedora 20 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 908088] Review Request: ascend - ASCEND modelling environment

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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #28 from Fedora Update System upda...@fedoraproject.org ---
Package ascend-0.9.8-6.20140211svn4638.fc20:
* should fix your issue,
* was pushed to the Fedora 20 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing
ascend-0.9.8-6.20140211svn4638.fc20'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2014-7058/ascend-0.9.8-6.20140211svn4638.fc20
then log in and leave karma (feedback).

-- 
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-06-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1092057

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

   What|Removed |Added

   Fixed In Version|golang-github-kdar-factorlo |golang-github-kdar-factorlo
   |g-0-0.1.git814d8f7.fc20 |g-0-0.1.git814d8f7.fc19



--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
golang-github-kdar-factorlog-0-0.1.git814d8f7.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 1069259] Review Request: ndoutils - Store Nagios configuration and event data in a database

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

Simone Caronni negativ...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2014-06-04 03:59:05



-- 
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 1086217] Review Request: nodejs-strip-json-comments - Strip comments from JSON

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

Tom Hughes t...@compton.nu changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #8 from Tom Hughes t...@compton.nu ---
That looks great now. Package is 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 1086231] Review Request: nodejs-jsonparse - Pure-js JSON streaming parser for node.js

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

Tom Hughes t...@compton.nu changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #8 from Tom Hughes t...@compton.nu ---
Looks great now. Package 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 1086217] Review Request: nodejs-strip-json-comments - Strip comments from JSON

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



--- Comment #9 from anish apa...@redhat.com ---
New Package SCM Request
===
Package Name: nodejs-strip-json-comments
Short Description: strip comments from json 
Owners: anishpatil
Branches: f20

-- 
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 1086217] Review Request: nodejs-strip-json-comments - Strip comments from JSON

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

anish apa...@redhat.com changed:

   What|Removed |Added

  Flags||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 1086231] Review Request: nodejs-jsonparse - Pure-js JSON streaming parser for node.js

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



--- Comment #9 from anish apa...@redhat.com ---
New Package SCM Request
===
Package Name: nodejs-jsonparse
Short Description: Pure-js JSON streaming parser for node.js
Owners: anishpatil
Branches: f20

-- 
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 1086231] Review Request: nodejs-jsonparse - Pure-js JSON streaming parser for node.js

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

anish apa...@redhat.com changed:

   What|Removed |Added

  Flags||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 1104579] New: Review Request: ghc-ieee754 - Utilities for dealing with IEEE floating point numbers

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

Bug ID: 1104579
   Summary: Review Request: ghc-ieee754 - Utilities for dealing
with IEEE floating point numbers
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: peter...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org




Spec URL: http://petersen.fedorapeople.org/reviews/ghc-ieee754.spec
SRPM URL:
http://petersen.fedorapeople.org/reviews/ghc-ieee754-0.7.3-1.fc20.src.rpm

Description:
Utilities for dealing with IEEE floating point numbers, ported from the Tango
math library; approximate and exact equality comparisons for general types.

-- 
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 1103554] Review Request: cppcms - a Free High Performance Web Development Framework

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



--- Comment #9 from Michael Schwendt bugs.mich...@gmx.net ---
 Why Bugzilla doesn't accept that??

At  https://bugzilla.redhat.com/userprefs.cgi?tab=account  there are two
(separate) fields for real name and email address. Enter the data and your
password to confirm. That has worked before for other people, who had been
asked to enter the real 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 1104579] Review Request: ghc-ieee754 - Utilities for dealing with IEEE floating point numbers

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



--- Comment #1 from Jens Petersen peter...@redhat.com ---
This package built on koji: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=6923287

-- 
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 1100961] Review Request: antimicro - Graphical program used to map keyboard buttons and mouse controls to a gamepad

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



--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
antimicro-2.4-1.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/antimicro-2.4-1.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 1100961] Review Request: antimicro - Graphical program used to map keyboard buttons and mouse controls to a gamepad

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



--- Comment #14 from Fedora Update System upda...@fedoraproject.org ---
antimicro-2.4-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/antimicro-2.4-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 1086217] Review Request: nodejs-strip-json-comments - Strip comments from JSON

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



--- Comment #10 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 877096] Review Request: perl-Fsdb - A set of commands for manipulating flat-text databases from the shell

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



--- Comment #35 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 1086217] Review Request: nodejs-strip-json-comments - Strip comments from JSON

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

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 877096] Review Request: perl-Fsdb - A set of commands for manipulating flat-text databases from the shell

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

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 1086231] Review Request: nodejs-jsonparse - Pure-js JSON streaming parser for node.js

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

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 1086231] Review Request: nodejs-jsonparse - Pure-js JSON streaming parser for node.js

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



--- Comment #10 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 1103466] Review Request: perl-Net-Statsd - Sends statistics to the stats daemon over UDP

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



--- 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 1103466] Review Request: perl-Net-Statsd - Sends statistics to the stats daemon over UDP

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

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 1104604] New: Review Request: erlang-sd_notify - Erlang interface to systemd notify subsystem

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

Bug ID: 1104604
   Summary: Review Request: erlang-sd_notify - Erlang interface to
systemd notify subsystem
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: lemen...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://peter.fedorapeople.org/erlang-sd_notify.spec
SRPM URL: http://peter.fedorapeople.org/erlang-sd_notify-0.1-1.fc21.src.rpm
Description: Erlang interface to systemd notify subsystem
Fedora Account System Username: peter

Koji scratch build for Rawhide:

* http://koji.fedoraproject.org/koji/taskinfo?taskID=6921361

-- 
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 1104604] Review Request: erlang-sd_notify - Erlang interface to systemd notify subsystem

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

Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

 Blocks||1103524




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1103524
[Bug 1103524] Change rabbitmq-server systemd service to Type=notify
-- 
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 1103466] Review Request: perl-Net-Statsd - Sends statistics to the stats daemon over UDP

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



--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
perl-Net-Statsd-0.08-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/perl-Net-Statsd-0.08-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 1103466] Review Request: perl-Net-Statsd - Sends statistics to the stats daemon over UDP

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

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 1103466] Review Request: perl-Net-Statsd - Sends statistics to the stats daemon over UDP

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



--- Comment #9 from Fedora Update System upda...@fedoraproject.org ---
perl-Net-Statsd-0.08-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/perl-Net-Statsd-0.08-1.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 1079732] Review Request: perl-Catalyst-Plugin-CustomErrorMessage - Catalyst plugin to have more cute error message

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



--- Comment #11 from David Dick dd...@cpan.org ---
Hi Sven,

Just a ping for the git work on 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 1086445] Review Request: dbusmenu-qt5 - A Qt 5 implementation of the DBusMenu protocol

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

Dan Vrátil dvra...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Dan Vrátil dvra...@redhat.com ---
New Package SCM Request
===
Package Name: dbusmenu-qt5
Short Description: A Qt 5 implementation of the DBusMenu protocol 
Upstream URL: https://launchpad.net/libdbusmenu-qt/
Owners: dvratil jgrulich ltinkl kkofler rdieter
Branches: f20
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 1086231] Review Request: nodejs-jsonparse - Pure-js JSON streaming parser for node.js

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



--- Comment #11 from Fedora Update System upda...@fedoraproject.org ---
nodejs-jsonparse-0.0.6-4.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/nodejs-jsonparse-0.0.6-4.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 1086231] Review Request: nodejs-jsonparse - Pure-js JSON streaming parser for node.js

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

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 1086445] Review Request: dbusmenu-qt5 - A Qt 5 implementation of the DBusMenu protocol

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

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 1086445] Review Request: dbusmenu-qt5 - A Qt 5 implementation of the DBusMenu protocol

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



--- 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 1086217] Review Request: nodejs-strip-json-comments - Strip comments from JSON

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



--- Comment #11 from Fedora Update System upda...@fedoraproject.org ---
nodejs-strip-json-comments-0.1.3-4.fc20 has been submitted as an update for
Fedora 20.
https://admin.fedoraproject.org/updates/nodejs-strip-json-comments-0.1.3-4.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 1086217] Review Request: nodejs-strip-json-comments - Strip comments from JSON

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

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 1096704] Review Request: python-bintrees - Binary-, RedBlack- and AVL-Trees in Python and Cython

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

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 1096704] Review Request: python-bintrees - Binary-, RedBlack- and AVL-Trees in Python and Cython

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



--- Comment #7 from Fedora Update System upda...@fedoraproject.org ---
python-bintrees-2.0.1-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/python-bintrees-2.0.1-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 1070406] Review Request: signon - Accounts framework for Linux and POSIX based platforms

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

Dan Vrátil dvra...@redhat.com changed:

   What|Removed |Added

  Flags|needinfo?(dvra...@redhat.co |
   |m)  |



--- Comment #7 from Dan Vrátil dvra...@redhat.com ---
Updated spec with a patch to fix build (submitted upstream)

Updated spec: http://dvratil.fedorapeople.org/spec/signon.spec
Updated SRPML: http://dvratil.fedorapeople.org/spec/signon-8.56-3.fc20.src.rpm
Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6923844

-- 
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 1103917] Review Request: python-cornice - Define Web Services in Pyramid

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

Haïkel Guémar karlthe...@gmail.com changed:

   What|Removed |Added

  Flags||fedora-review+



--- Comment #1 from Haïkel Guémar karlthe...@gmail.com ---
I hereby approve of python-cornice into Fedora Packages Collection.
The rpmlint errors could be safely ignored.

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:
 Unknown or generated, *No copyright* MPL (v2.0). 4 files have unknown
 license. Detailed output of licensecheck in /home/haikel/1103917-python-
 cornice/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package requires other packages for directories it uses.
 Note: No known owner of /usr/lib/python3.4/site-packages,
 /usr/lib/python3.4
[x]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/lib/python3.4/site-packages,
 /usr/lib/python3.4
[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.
[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 8 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 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:
[ ]: Python eggs must not download any dependencies during the build process.
[ ]: 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).
[-]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
 python3-cornice , python3-cornice-sphinx
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text 

[Bug 1086445] Review Request: dbusmenu-qt5 - A Qt 5 implementation of the DBusMenu protocol

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

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 1086445] Review Request: dbusmenu-qt5 - A Qt 5 implementation of the DBusMenu protocol

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



--- Comment #4 from Fedora Update System upda...@fedoraproject.org ---
dbusmenu-qt5-0.9.3-0.1.20140604bzr.fc20 has been submitted as an update for
Fedora 20.
https://admin.fedoraproject.org/updates/dbusmenu-qt5-0.9.3-0.1.20140604bzr.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 1103917] Review Request: python-cornice - Define Web Services in Pyramid

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

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Mathieu Bridon boche...@fedoraproject.org ---
Thanks for the review Haikel!

New Package SCM Request
===
Package Name: python-cornice
Short Description: Define Web Services in Pyramid
Upstream URL: https://pypi.python.org/pypi/cornice
Owners: bochecha lmacken hguemar
Branches: devel el6 el7
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 1104746] New: Review Request: soscleaner - sosreport data obfuscation

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

Bug ID: 1104746
   Summary: Review Request: soscleaner - sosreport data
obfuscation
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jdun...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://people.redhahttp://people.redhat.com/jduncan/soscleaner/soscleaner.spec
SRPM URL: http://people.redhat.com/jduncan/soscleaner/soscleaner-0.1-11.src.rpm
Description: 

SOSCleaner helps users in environments that have data export restrictions clean
up an sosreport so it can be safely uploaded to a support group for analysis.
IT IS NOT ALL that should be done for this process, but it does help with the
most common and repetitive items.

This is going to be incorporated into the redhat-support-tool soon, and there
is strong desire to have it as a standalone tool as well. This will, I hope,
begin the process of creating the upstream for the project.

This is my first package submission to Fedora.

Fedora Account System Username: jduncan

Thanks,

Jamie Duncan

-- 
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 1104746] Review Request: soscleaner - sosreport data obfuscation

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



--- Comment #1 from Jamie Duncan jdun...@redhat.com ---
EL6 Candidate Koji Build:
https://bugzilla.redhat.com/show_bug.cgi?id=1104746

F20 Koji Build:
https://bugzilla.redhat.com/show_bug.cgi?id=1104746

-- 
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 1104746] Review Request: soscleaner - sosreport data obfuscation

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



--- Comment #2 from Jamie Duncan jdun...@redhat.com ---
don't know what happened there...

EL6:
http://koji.fedoraproject.org/koji/taskinfo?taskID=6924175

F20:
http://koji.fedoraproject.org/koji/taskinfo?taskID=6924156

-- 
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 1104476] Review Request: python-wsgilog - WSGI logging and event reporting

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

Orion Poplawski or...@cora.nwra.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||or...@cora.nwra.com
   Assignee|nob...@fedoraproject.org|or...@cora.nwra.com
  Flags||fedora-review+



--- Comment #1 from Orion Poplawski or...@cora.nwra.com ---
Package Review
==

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

= ISSUES =

- The .py files have CRLF line endings, would be good to fix that.
- Would be nice to have a separate license file

Neither of these are blockers though, so APPROVED.

= 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.
[-]: 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]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 BSD (3 clause). Detailed output of licensecheck in
 /export/home/orion/redhat/python-wsgilog-0.3/1104476-python-
 wsgilog/licensecheck.txt
[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.
[-]: Package is not known to require an ExcludeArch tag.
[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]: 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

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.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and 

[Bug 1104476] Review Request: python-wsgilog - WSGI logging and event reporting

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



--- Comment #2 from Orion Poplawski or...@cora.nwra.com ---
Sponsored now.

-- 
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 1104746] Review Request: soscleaner - sosreport data obfuscation

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

Jamie Duncan jdun...@redhat.com changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
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 825409] Review Request: gazebo - 3D multi-robot simulator with dynamics

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



--- Comment #38 from Scott K Logan log...@cottsay.net ---
Okay, I've done some more thorough testing. Lets first talk about the things I
mentioned before:

- %cmake macro - Addressed
- make threading - Addressed
- bundled fonts - Addressed
- new version - Addressed

Now for some new things:

- Gazebo 3.0 added a build dependency on gdal-devel. This needs to be added as
a
  run dependency in gazebo-devel [1].
- The fix for linking against player seems to have disappeared [2].
- We talked offline about this, but there is a pretty big difference in the way
  boost is influencing the behaviour of gzclient when there is no gazebo server
  running. The intended behaviour is that gzclient tries for 30 seconds and
  gives up. You should incorporate a patch to fix this, or Gazebo is
essentially
  impossible to start using roslaunch [3].
- Since Gazebo explicitly needs sdformat 2, you should specify that in the
build
  requirement. It took me a while to figure out why I couldn't build the new
  package...of course, updating sdformat to the newest RPM fixed it.
- I found that when I added the correct PATH and LD_LIBRARY_PATH to the %check
  section, a large amount of the tests started working. Additionally, I
analyzed
  the remaining three and found reasons that they may never work in fedora's
  build environment. I then blacklisted these tests, so all of them were
  succeeding when I built in mock on build slave. It looks like there are more
  tests that fail when built in Koji, so removing the /usr/bin/true from `make
  test` probably isn't a good idea, but it might be worth investigating the
  Koji failures eventually. The changes are at [4] and [5].
- I see that the base package depends on -media, but I'm wondering if it
  wouldn't be a good idea for -media to depend on the base as well. That way,
  there is still a separate noarch packge for the assets, but -media will be
  uninstalled with the base package. There really isn't a reason to have one
  without the other...

I've implemented the changes and tested them. Feel free to use them directly,
modify them, or not use them at all. I feel that this package is very close to
being ready!

[1]
https://github.com/cottsay/gazebo-rpm/commit/7944d7b51eb91c1a64df34cdc7c0392f48a4
[2]
https://github.com/cottsay/gazebo-rpm/commit/324f8e798473ca3d75aaed2883732eb6b5f36917
[3]
https://github.com/cottsay/gazebo-rpm/commit/78c86e05018377f6b7acaa40d7845377fb757b99
[4]
https://github.com/cottsay/gazebo-rpm/commit/19df7f9ad2a7cc7ad80160e6296024fd54bb9c04
[5]
https://github.com/cottsay/gazebo-rpm/commit/68b8d4b6cc1891b291e8f2d5316b91e04f2755cc

-- 
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 1103917] Review Request: python-cornice - Define Web Services in Pyramid

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

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 1103917] Review Request: python-cornice - Define Web Services in Pyramid

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



--- 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 1104746] Review Request: soscleaner - sosreport data obfuscation

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



--- Comment #3 from Michael Schwendt bugs.mich...@gmx.net ---
 Spec URL: 
 http://people.redhahttp://people.redhat.com/jduncan/soscleaner/soscleaner.spec

Spec URL: http://people.redhat.com/jduncan/soscleaner/soscleaner.spec

During review, keep the SRPM URL: and Spec URL: lines up-to-date when you
modify the package, so the fedora-review tool can be used. Consider running
fedora-review -b 1104746 to let that tool perform many helpful checks.

The spec file is full of mistakes and pitfalls.


 %define name soscleaner
 %define version 0.1
 %define release 11

%{name}, %{version} and %{release} are implicitly defined by the Name:,
Version: and Release: tags, so it's very poor form to first define these
macros only to have the tags redefine the macros afterwards.

If you want these values at the top of the spec file, move the tags at the top.
Easy as that.


 %define unmangled_version 0.1

https://fedoraproject.org/wiki/Packaging:Guidelines#.25global_preferred_over_.25define



 Source0: %{name}-%{unmangled_version}.tar.gz

That's not an URL:
https://fedoraproject.org/wiki/Packaging:SourceURL


 Group: Applications

Not a group listed in /usr/share/doc/rpm/GROUPS and the Group tag is optional
nowadays. In case of doubt, remove the tag from the spec file:
https://fedoraproject.org/wiki/Packaging:Guidelines#Group_tag


 BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-buildroot

https://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag


 Prefix: %{_prefix}

https://fedoraproject.org/wiki/Packaging:Guidelines#Relocatable_packages


 Vendor: Jamie Duncan jdun...@redhat.com
 Packager: Jamie Duncan jdun...@redhat.com

https://fedoraproject.org/wiki/Packaging:Guidelines#Tags


 %clean
 rm -rf $RPM_BUILD_ROOT

https://fedoraproject.org/wiki/Packaging:Guidelines#.25clean


 %files -f INSTALLED_FILES

As smart as you may find this technique, it hides too much under the carpet.
What about directory ownership, for example? Prefer listing files and
directories in the %files section directly. Use wildcards where helpful.


 %defattr(-,root,root)

https://fedoraproject.org/wiki/Packaging:Guidelines#File_Permissions

-- 
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 1104746] Review Request: soscleaner - sosreport data obfuscation

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



--- Comment #4 from Jamie Duncan jdun...@redhat.com ---
Michael,

Thanks for the feedback. The spec file is *almost* 100% how it came from
distutils.  

I will make the above changes and also take advantage of the tools you mention.

Thanks again,

Jamie Duncan

-- 
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 1081434] Review Request: ip2location - IP2location library

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



--- Comment #20 from Patrick Uiterwijk puiterw...@redhat.com ---
Thanks for the fixes, but i686 doesn't even build, which is one of the primary
architectures: http://koji.fedoraproject.org/koji/taskinfo?taskID=6925071

Please try a scratch build to see if at least the primary architectures build
successfully.

-- 
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 1061985] Review Request: coin-or-lemon - A C++ template library providing many common graph algorithms

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

Andy Lutomirski l...@mit.edu changed:

   What|Removed |Added

  Flags|needinfo?(l...@mit.edu) |



--- Comment #4 from Andy Lutomirski l...@mit.edu ---
Spec URL:
http://web.mit.edu/luto/www/fedora/coin-or-lemon_v2/coin-or-lemon.spec
SRPM URL:
http://web.mit.edu/luto/www/fedora/coin-or-lemon_v2/coin-or-lemon-1.3-2.fc20.src.rpm
Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6925124
Description: LEMON stands for Library for Efficient Modeling and Optimization
in
Networks. It is a C++ template library providing efficient
implementations of common data structures and algorithms with focus on
combinatorial optimization tasks connected mainly with graphs and
networks.
Fedora Account System Username: amluto

This version should be a lot better.

-- 
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 1104746] Review Request: soscleaner - sosreport data obfuscation

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



--- Comment #5 from Jamie Duncan jdun...@redhat.com ---
Michael and everyone else.

It's amazing how much more sense the Packaging Guide makes when you have a
little push in the right direction and a goal.

I've refactored the spec file to be (I think) in line with the guidelines. 
I've also confirmed it builds of course both on my local system and in Koji.

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

I bumped up the release number to 12, with artifacts available at:

SRPM URL: http://people.redhat.com/jduncan/soscleaner/soscleaner-0.1-12.src.rpm
Spec URL: http://people.redhat.com/jduncan/soscleaner/soscleaner.spec

Thanks again for helping to straighten me out.

Jamie Duncan

-- 
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 1104746] Review Request: soscleaner - sosreport data obfuscation

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



--- Comment #6 from Jamie Duncan jdun...@redhat.com ---
Created attachment 902362
  -- https://bugzilla.redhat.com/attachment.cgi?id=902362action=edit
f-e review.txt

-- 
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 1026252] Review Request: kluppe - a live looping instrument

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



--- Comment #5 from Volker Fröhlich volke...@gmx.at ---
Brendan?

-- 
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 957715] Review Request: gamgi - Build, View and Analyse Atomic Structures

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



--- Comment #4 from Volker Fröhlich volke...@gmx.at ---
Shall we close this 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 1060924] Review Request: rubygem-unicode - Unicode normalization library for Ruby

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



--- Comment #3 from Dan Callaghan dcall...@redhat.com ---
(In reply to Ken Dreyer from comment #1)
 Mind conditionalizing the Requires and Provides to follow the latest Ruby
 Packaging Guidelines? For example:
 
   %if 0%{?fc19} || 0%{?fc20} || 0%{?el7}
   Requires:   ruby
   Requires:   ruby(rubygems)
   %endif
 
   %if 0%{?fc19} || 0%{?fc20} || 0%{?el7}
   Provides: rubygem(%{gem_name}) = %{version}
   %endif

I conditionalized the rubygem Requires/Provides. The requirement on ruby is
still needed though, since this is a C extension so it's only compatible with
MRI.

 Also, I'm getting a build failure on Rawhide:
 http://koji.fedoraproject.org/koji/taskinfo?taskID=6900143
 
 mv: cannot stat
 '/builddir/build/BUILDROOT/rubygem-unicode-0.4.4-1.fc21.x86_64/usr/share/
 gems/gems/unicode-0.4.4/lib/unicode': No such file or directory

Looks like on F21+ rubygems has been made smarter about installing C extensions
to %{_libdir} so I have conditionalized the paths here as well.

I also updated to 0.4.4.1 of the gem, which is just a change to add license
metadata to the gemspec.

http://fedorapeople.org/~dcallagh/rubygem-unicode/rubygem-unicode.spec
http://fedorapeople.org/~dcallagh/rubygem-unicode/rubygem-unicode-0.4.4.1-1.fc21.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 1093406] Review Request: python-junitxml - pyunit extension to output JUnit compatible XML

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

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

   What|Removed |Added

   Fixed In Version|python-junitxml-0.7-1.fc20  |python-junitxml-0.7-1.el6



--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
python-junitxml-0.7-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 1103466] Review Request: perl-Net-Statsd - Sends statistics to the stats daemon over UDP

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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
perl-Net-Statsd-0.08-1.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 1104746] Review Request: soscleaner - sosreport data obfuscation

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



--- Comment #7 from Jamie Duncan jdun...@redhat.com ---
SRPM URL:
http://people.redhat.com/jduncan/soscleaner/soscleaner-0.1-12.fc20.src.rpm
Spec URL: http://people.redhat.com/jduncan/soscleaner/soscleaner.spec

-- 
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 1104746] Review Request: soscleaner - sosreport data obfuscation

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



--- Comment #8 from Jamie Duncan jdun...@redhat.com ---
have gone through fedora-review a few times and it looks solid to me at this
point. Definitely a learning experience.

-- 
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 1104907] New: Free and simple TrueCrypt Implementation based on dm-crypt

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

Bug ID: 1104907
   Summary: Free and simple TrueCrypt Implementation based on
dm-crypt
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: villadalm...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



tcplay is a free (BSD-licensed), pretty much
fully featured (including multiple keyfiles,
cipher cascades, etc) and stable TrueCrypt  
implementation

http://villadalmine.fedorapeople.org/tcplay.spec
http://villadalmine.fedorapeople.org/tc-play-1.0.0rc-1.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 1103917] Review Request: python-cornice - Define Web Services in Pyramid

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

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2014-06-04 22:57:48



--- Comment #4 from Mathieu Bridon boche...@fedoraproject.org ---
Thanks for the Git Jon!

Package built, closing.

-- 
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 1104907] Free and simple TrueCrypt Implementation based on dm-crypt

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



--- Comment #1 from Rino Rondan villadalm...@gmail.com ---
please close.. package duplicate

-- 
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 1092828] Review Request: turses - A Twitter client for the console

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



--- Comment #6 from Eduardo Echeverria echevemas...@gmail.com ---
Ok tweepy has been updated by me in rawhide and f20, please go ahead.

-- 
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-06-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1087734

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

   What|Removed |Added

   Fixed In Version||php-horde-imp-6.1.7-2.fc20
 Resolution|NEXTRELEASE |ERRATA



--- Comment #11 from Fedora Update System upda...@fedoraproject.org ---
php-horde-imp-6.1.7-2.fc20 has been pushed to the Fedora 20 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 949038] Review Request: php-horde-horde - Horde Application Framework

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

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

   What|Removed |Added

   Fixed In Version||php-horde-horde-5.1.6-3.fc2
   ||0
 Resolution|NEXTRELEASE |ERRATA



--- Comment #14 from Fedora Update System upda...@fedoraproject.org ---
php-horde-horde-5.1.6-3.fc20 has been pushed to the Fedora 20 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 1087772] Review Request: php-horde-kronolith - A web based calendar

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

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

   What|Removed |Added

   Fixed In Version||php-horde-kronolith-4.1.5-2
   ||.fc20
 Resolution|RAWHIDE |ERRATA



--- Comment #12 from Fedora Update System upda...@fedoraproject.org ---
php-horde-kronolith-4.1.5-2.fc20 has been pushed to the Fedora 20 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 1086217] Review Request: nodejs-strip-json-comments - Strip comments from JSON

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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #12 from Fedora Update System upda...@fedoraproject.org ---
nodejs-strip-json-comments-0.1.3-4.fc20 has been pushed to the Fedora 20
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 1087742] Review Request: php-horde-turba - A web based address book

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

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

   What|Removed |Added

   Fixed In Version||php-horde-turba-4.1.4-2.fc2
   ||0
 Resolution|RAWHIDE |ERRATA



--- Comment #12 from Fedora Update System upda...@fedoraproject.org ---
php-horde-turba-4.1.4-2.fc20 has been pushed to the Fedora 20 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 1086445] Review Request: dbusmenu-qt5 - A Qt 5 implementation of the DBusMenu protocol

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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #5 from Fedora Update System upda...@fedoraproject.org ---
dbusmenu-qt5-0.9.3-0.1.20140604bzr.fc20 has been pushed to the Fedora 20
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 1087740] Review Request: php-horde-nag - A web based task list manager

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

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

   What|Removed |Added

   Fixed In Version||php-horde-nag-4.1.4-3.fc20
 Resolution|RAWHIDE |ERRATA



--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
php-horde-nag-4.1.4-3.fc20 has been pushed to the Fedora 20 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 1096704] Review Request: python-bintrees - Binary-, RedBlack- and AVL-Trees in Python and Cython

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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
python-bintrees-2.0.1-1.fc20 has been pushed to the Fedora 20 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 1087737] Review Request: php-horde-ingo - An email filter rules manager

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

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

   What|Removed |Added

   Fixed In Version||php-horde-ingo-3.1.4-2.fc20
 Resolution|RAWHIDE |ERRATA



--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
php-horde-ingo-3.1.4-2.fc20 has been pushed to the Fedora 20 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 1086231] Review Request: nodejs-jsonparse - Pure-js JSON streaming parser for node.js

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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #12 from Fedora Update System upda...@fedoraproject.org ---
Package nodejs-jsonparse-0.0.6-4.fc20:
* should fix your issue,
* was pushed to the Fedora 20 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing nodejs-jsonparse-0.0.6-4.fc20'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2014-7097/nodejs-jsonparse-0.0.6-4.fc20
then log in and leave karma (feedback).

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