[Bug 1259005] Review Request: docker-distribution - The Docker toolset to pack, ship, store, and deliver content

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1259005

Lokesh Mandvekar  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|jchal...@redhat.com



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

[Bug 1258878] Review Request: golang-github-coreos-go-iptables - Go wrapper around iptables utility

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258878



--- Comment #5 from Fedora Update System  ---
golang-github-coreos-go-iptables-0-0.1.git74b0926.fc21 has been submitted as an
update to Fedora 21. https://bodhi.fedoraproject.org/updates/FEDORA-2015-14793

-- 
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 1258878] Review Request: golang-github-coreos-go-iptables - Go wrapper around iptables utility

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258878



--- Comment #7 from Fedora Update System  ---
golang-github-coreos-go-iptables-0-0.1.git74b0926.fc23 has been submitted as an
update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-14795

-- 
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 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1254828

Haïkel Guémar  changed:

   What|Removed |Added

  Flags|needinfo?(karlthered@gmail. |
   |com)|



--- Comment #28 from Haïkel Guémar  ---
We're almost done, please apply the following changes:
* fix the following rpmlint errors/warnings
python-networking-bigswitch.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/bsnstacklib/plugins/bigswitch/tests/test_server.py
644 /usr/bin/env
python-networking-bigswitch.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/bsnstacklib/plugins/bigswitch/version.py 644
/usr/bin/env  
python-networking-bigswitch.src:21: W: mixed-use-of-spaces-and-tabs (spaces:
line 6, tab: line 21)

Use sed to drop the shebang, here's a snippet that could help you.
https://fedoraproject.org/wiki/Packaging_tricks#Remove_shebang_from_Python_libraries


As your also an upstream maintainer, please consider updating requirements.txt
as downstream use it to check runtime requirements (rdopkg reqcheck serves that
particular goal).

When you're done, needinfo me and I'll add it in delorean.

-- 
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 1258159] Review Request: nodejs-lolex - Fake JavaScript timers

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258159



--- Comment #10 from Fedora Update System  ---
nodejs-lolex-1.3.1-2.fc22 has been submitted as an update to Fedora 22.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14802

-- 
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 1256435] Review Request: metapixel - photomosaic generator

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256435

Jon Ciesla  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 1258157] Review Request: nodejs-formatio - Human-readable object formatting

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258157



--- Comment #3 from Piotr Popieluch  ---
No, they don't seem to be needed for functioning of the module. Removed them,
no version bump:

Spec URL: https://piotrp.fedorapeople.org/nodejs-formatio.spec
SRPM URL: https://piotrp.fedorapeople.org/nodejs-formatio-1.1.2-1.fc22.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 1259005] New: Review Request: docker-distribution - The Docker toolset to pack, ship, store, and deliver content

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1259005

Bug ID: 1259005
   Summary: Review Request: docker-distribution - The Docker
toolset to pack, ship, store, and deliver content
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: l...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: upcoming
SRPM URL: upcoming
Description: upcoming

Fedora Account System Username: lsm5

-- 
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 1258156] Review Request: nodejs-samsam - Value identification and comparison functions

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258156



--- Comment #8 from Fedora Update System  ---
nodejs-samsam-1.1.2-1.fc21 has been submitted as an update to Fedora 21.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14800

-- 
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 1258159] Review Request: nodejs-lolex - Fake JavaScript timers

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258159



--- Comment #11 from Fedora Update System  ---
nodejs-lolex-1.3.1-2.fc21 has been submitted as an update to Fedora 21.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14803

-- 
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 1259005] Review Request: docker-distribution - The Docker toolset to pack, ship, store, and deliver content

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1259005

Lokesh Mandvekar  changed:

   What|Removed |Added

 Whiteboard||NotReady
  Flags||fedora-review?



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

[Bug 1259005] Review Request: docker-distribution - The Docker toolset to pack, ship, store, and deliver content

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1259005



--- Comment #1 from Lokesh Mandvekar  ---
Upstream: https://github.com/docker/distribution

-- 
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 1258159] Review Request: nodejs-lolex - Fake JavaScript timers

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258159

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.
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 1258159] Review Request: nodejs-lolex - Fake JavaScript timers

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258159



--- Comment #9 from Fedora Update System  ---
nodejs-lolex-1.3.1-2.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14801

-- 
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 1258217] Review Request: nodejs-mkfiletree - Nodejs module to make a tree of files and directories

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258217

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.
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 1258217] Review Request: nodejs-mkfiletree - Nodejs module to make a tree of files and directories

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258217



--- Comment #8 from Fedora Update System  ---
nodejs-mkfiletree-1.0.1-1.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14806

-- 
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 1256338] Review Request: kf5-akonadi - The Akonadi client libraries

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256338

Jon Ciesla  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 1256338] Review Request: kf5-akonadi - The Akonadi client libraries

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256338



--- Comment #5 from Jon Ciesla  ---
Please put group::kde-sig in InitialCC.

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 1258217] Review Request: nodejs-mkfiletree - Nodejs module to make a tree of files and directories

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258217



--- Comment #9 from Fedora Update System  ---
nodejs-mkfiletree-1.0.1-1.fc22 has been submitted as an update to Fedora 22.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14807

-- 
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 1256435] Review Request: metapixel - photomosaic generator

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256435



--- Comment #19 from Jon Ciesla  ---
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 1258878] Review Request: golang-github-coreos-go-iptables - Go wrapper around iptables utility

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258878



--- Comment #6 from Fedora Update System  ---
golang-github-coreos-go-iptables-0-0.1.git74b0926.el6 has been submitted as an
update to Fedora EPEL 6.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-7890

-- 
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 1258156] Review Request: nodejs-samsam - Value identification and comparison functions

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258156



--- Comment #7 from Fedora Update System  ---
nodejs-samsam-1.1.2-1.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14798

-- 
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 1258156] Review Request: nodejs-samsam - Value identification and comparison functions

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258156

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.
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 1258878] Review Request: golang-github-coreos-go-iptables - Go wrapper around iptables utility

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258878

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System  ---
golang-github-coreos-go-iptables-0-0.1.git74b0926.fc21 has been pushed to the
Fedora 21 testing repository. If problems still persist, please make note of it
in this bug report.\nIf you want to test the update, you can install it with \n
su -c 'yum --enablerepo=updates-testing update
golang-github-coreos-go-iptables'. You can provide feedback for this update
here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14793

-- 
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 1151818] Review Request: nodejs-typeahead.js - Autocomplete library for JavaScript

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1151818

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System  ---
nodejs-typeahead.js-0.10.5-3.fc21 has been pushed to the Fedora 21 testing
repository. If problems still persist, please make note of it in this bug
report.\nIf you want to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update nodejs-typeahead.js'. You can provide
feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14729

-- 
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 1252831] Review Request: percona-xtrabackup - online (non-blockable) backup solution for InnoDB and XtraDB engines

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1252831



--- Comment #4 from Pete MacKinnon  ---
Spec URL:
https://pmackinn.fedorapeople.org/percona/xtrabackup/2.2.9/percona-xtrabackup.spec
SRPM URL:
https://pmackinn.fedorapeople.org/percona/xtrabackup/2.2.9/percona-xtrabackup-2.2.9-2.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 1259002] New: Review Request: rudiments - C++ class library for developing systems and applications

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1259002

Bug ID: 1259002
   Summary: Review Request: rudiments - C++ class library for
developing systems and applications
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: david.m...@firstworks.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://rudiments.sourceforge.net/rpm/rudiments.spec
SRPM URL: http://rudiments.sourceforge.net/rpm/rudiments-0.53-1.fc22.src.rpm
Description: A C++ class library for developing systems and applications.
Fedora Account System Username: davidleemuse

Rudiments is the base library for SQL Relay and a few other projects of mine.

This is my first package, and I need a sponsor.

Here's a link to a successful Koji build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=10915810

-- 
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 1258878] Review Request: golang-github-coreos-go-iptables - Go wrapper around iptables utility

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258878

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.
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 1258878] Review Request: golang-github-coreos-go-iptables - Go wrapper around iptables utility

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258878



--- Comment #4 from Fedora Update System  ---
golang-github-coreos-go-iptables-0-0.1.git74b0926.fc22 has been submitted as an
update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-14792

-- 
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 1242086] Review Request: parcimonie.sh - Refresh your GnuPG keyring over Tor

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1242086

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||0-0.4.20150804gitc009937.fc
   ||23
 Resolution|--- |NEXTRELEASE
Last Closed||2015-09-01 14:32: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 1242086] Review Request: parcimonie.sh - Refresh your GnuPG keyring over Tor

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1242086



--- Comment #19 from Fedora Update System  ---
parcimonie.sh-0-0.4.20150804gitc009937.fc23 has been pushed to the Fedora 23
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.
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 1256351] Review Request: kf5-akonadi-server - PIM Storage Service

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256351

Jon Ciesla  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 1256351] Review Request: kf5-akonadi-server - PIM Storage Service

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256351



--- 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.
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 1256341] Review Request: kf5-akonadi-calendar - The Akonadi Calendar client library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256341



--- Comment #5 from Jon Ciesla  ---
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 1256341] Review Request: kf5-akonadi-calendar - The Akonadi Calendar client library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256341

Jon Ciesla  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 1258159] Review Request: nodejs-lolex - Fake JavaScript timers

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258159



--- Comment #5 from Piotr Popieluch  ---
I only need the nodejs part at this moment. Browserify is not in Fedora and I
expect that it has many missing dependencies. I will look into browserify some
other time.


* Tue Sep 01 2015 Piotr Popieluch  - 1.3.1-2
- remove toplevel lolex.js
- remove tests from rpm
- update test for only testing node module



Spec URL: https://piotrp.fedorapeople.org/nodejs-lolex.spec
SRPM URL: https://piotrp.fedorapeople.org/nodejs-lolex-1.3.1-2.fc22.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 1256369] Review Request: kf5-kimap - The KIMAP Library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256369

Jan Grulich  changed:

   What|Removed |Added

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



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

[Bug 1256369] Review Request: kf5-kimap - The KIMAP Library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256369

Jan Grulich  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Jan Grulich  ---
1) License should be GPLv2+ and LGPLv2+

Only minor issue which can be fixed later or during import, 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 1231564] Review Request: elasticdump - Import and export tools for elasticsearch

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1231564



--- Comment #17 from Fedora Update System  ---
elasticdump-0.14.3-1.fc21 has been pushed to the Fedora 21 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.
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 1118980] Review Request: subberthehut - OpenSubtitles.org subtitles downloader written in C

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1118980

Christopher Meng  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |NOTABUG
Last Closed||2015-09-01 03:24:50



--- Comment #2 from Christopher Meng  ---
Lost interest.

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 1231564] Review Request: elasticdump - Import and export tools for elasticsearch

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1231564

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|0.14.3-1.fc23   |0.14.3-1.fc21



-- 
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 1258159] Review Request: nodejs-lolex - Fake JavaScript timers

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258159

Tom Hughes  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #6 from Tom Hughes  ---
That looks good 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 1249211] Review Request: cndoom - Conservative Doom engine for speedrunning competition

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1249211



--- Comment #4 from Rahul Sundaram  ---
I will post more notes later:

cndoom.x86_64: W: wrong-file-end-of-line-encoding
/usr/share/doc/cndoom/ChangeLog
cndoom.x86_64: E: invalid-desktopfile /usr/share/applications/cnsetup.desktop
value "Settings" for string list key "Categories" in group "Desktop Entry" does
not have a semicolon (';') as trailing character
cndoom.x86_64: W: desktopfile-without-binary
/usr/share/applications/cnsetup.desktop cnsetup
cndoom.src:23: W: unversioned-explicit-provides bundled(sha1-gnupg)

I would welcome the common noarch package and can take patches to spec file if
necessary and/or feel free to become a co-maintainer.

-- 
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 1242086] Review Request: parcimonie.sh - Refresh your GnuPG keyring over Tor

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1242086



--- Comment #21 from Fedora Update System  ---
parcimonie.sh-0-0.4.20150804gitc009937.fc21 has been pushed to the Fedora 21
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.
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 1242086] Review Request: parcimonie.sh - Refresh your GnuPG keyring over Tor

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1242086

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|0-0.4.20150804gitc009937.fc |0-0.4.20150804gitc009937.fc
   |22  |21



-- 
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 1258572] Review Request: libipt - Intel Processor Trace Decoder Library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258572

Sergio Durigan Junior  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #4 from Sergio Durigan Junior  ---
This is the package review.  My only comment, based on the rpmlint warnings, is
to use %defattr(0644,root,root,-) in the %files section of the main RPM, just
to make sure that the shlib file gets installed without +x.

Other than that, the package is good and I am setting the fedora-review+ flag.


Package Review
==

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


Issues:
===
- 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 %license.
  Note: License file LICENSE is marked as %doc instead of %license
  See:
  http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text


= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Header files in -devel subpackage, if present.
[x]: ldconfig called in %post and %postun if required.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.
[x]: Development (unversioned) .so files in -devel subpackage, if present.

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: "BSD (3 clause)". Detailed output of licensecheck in /tmp
 /libipt-review/libipt/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[x]: Each %files section contains %defattr if rpm < 4.4
 Note: %defattr present but not needed
[-]: Package contains desktop file if it is a GUI application.
[x]: Development files must be in a -devel package
[-]: 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.
[-]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[-]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 51200 bytes in 3 files.
[-]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: 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]: Dist tag is present.
[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 does not use a name that already exists.
[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

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate
 file from 

[Bug 1155826] Review Request: libu2f-host - Yubico Universal 2nd Factor (U2F) Host C Library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1155826



--- Comment #18 from Seth Jennings  ---
Now that my first package, yubioath-desktop, is fully complete and know the
process and I'm coming back to this.

Based on Andy's previous submissions + changes to address Till's feedback in
comment 15:
https://www.variantweb.net/pub/review/libu2f-host.spec
https://www.variantweb.net/pub/review/libu2f-host-1.0.0-1.fc22.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=10917441

Feedback welcome!

-- 
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 1202805] Review Request: owncloud-client - The ownCloud Client

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1202805

Kevin Kofler  changed:

   What|Removed |Added

 CC||ke...@tigcc.ticalc.org
 Blocks||928937 (qt-reviews)
  Alias||owncloud-client




Referenced Bugs:

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

[Bug 1242086] Review Request: parcimonie.sh - Refresh your GnuPG keyring over Tor

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1242086



--- Comment #20 from Fedora Update System  ---
parcimonie.sh-0-0.4.20150804gitc009937.fc22 has been pushed to the Fedora 22
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.
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 1242086] Review Request: parcimonie.sh - Refresh your GnuPG keyring over Tor

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1242086

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|0-0.4.20150804gitc009937.fc |0-0.4.20150804gitc009937.fc
   |23  |22



-- 
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 1252831] Review Request: percona-xtrabackup - online (non-blockable) backup solution for InnoDB and XtraDB engines

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1252831

Pete MacKinnon  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #5 from Pete MacKinnon  ---
New Package SCM Request
===
Package Name: percona-xtrabackup
Short Description: Online InnoDB and XtraDB backup tool 
Owners: pmackinn
Branches: 21,22,23
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 1258157] Review Request: nodejs-formatio - Human-readable object formatting

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258157

Tom Hughes  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #4 from Tom Hughes  ---
Great. Package approved then.

-- 
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 1162234] Review Request: nss-securepass - NSS library for SecurePass.

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1162234



--- Comment #26 from Giuseppe Paterno'  ---
Hi Parag,
in the meanwhile please accept another RPM packaging contribution on: 
https://bugzilla.redhat.com/show_bug.cgi?id=1259061

I will try making some more reviews.
Best,
  Giuseppe

-- 
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 1259075] New: Review Request: python-jsonpath-rw-ext - Extensions for JSONPath RW

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1259075

Bug ID: 1259075
   Summary: Review Request: python-jsonpath-rw-ext - Extensions
for JSONPath RW
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: karlthe...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://hguemar.fedorapeople.org/reviews/python-jsonpath-rw-ext.spec
SRPM URL:
https://hguemar.fedorapeople.org/reviews/python-jsonpath-rw-ext-0.1.7-1.fc22.src.rpm
Description: Extensions for JSONPath RW
Fedora Account System Username: hguemar

-- 
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 1257329] Review Request: openstack-ironic-python-agent - A python agent for provisioning and deprovisioning Bare Metal servers.

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1257329

John Trowbridge  changed:

   What|Removed |Added

  Flags||needinfo?(chkumar246@gmail.
   ||com)



--- Comment #6 from John Trowbridge  ---
I am unable to build the docs because we are missing
python-sphinxcontrib-pecanwsme. However, I added a note to the spec file to
enable building them when we have that package available, or the requirement is
removed upstream.

I instead just included the raw restructured text in the docs package for 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 1255505] Review Request: dnf-plugin-system-upgrade - DNF plugin to handle system upgrades

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1255505

Will Woods  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #9 from Will Woods  ---
New Package SCM Request
===
Package Name: dnf-plugin-system-upgrade
Short Description: System Upgrade plugin for DNF
Upstream URL:
https://github.com/rpm-software-management/dnf-plugin-system-upgrade
Owners: wwoods packaging-team
Branches: f21 f22 f23
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 1194923] Review Request: nodejs-ava - Simple test runner

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194923



--- Comment #3 from Piotr Popieluch  ---
There is a new release but it introduces many new dependencies which are not in
Fedora yet.

-- 
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 1258878] Review Request: golang-github-coreos-go-iptables - Go wrapper around iptables utility

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258878



--- Comment #9 from Fedora Update System  ---
golang-github-coreos-go-iptables-0-0.1.git74b0926.fc23 has been pushed to the
Fedora 23 testing repository. If problems still persist, please make note of it
in this bug report.\nIf you want to test the update, you can install it with \n
su -c 'yum --enablerepo=updates-testing update
golang-github-coreos-go-iptables'. You can provide feedback for this update
here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14795

-- 
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 1251238] Review Request: yubioath-desktop - Yubikey tool for generating OATH tokens

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1251238



--- Comment #14 from Fedora Update System  ---
yubioath-desktop-2.2.1-1.fc23 has been pushed to the Fedora 23 testing
repository. If problems still persist, please make note of it in this bug
report.\nIf you want to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update yubioath-desktop'. You can provide feedback
for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14790

-- 
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 1259075] Review Request: python-jsonpath-rw-ext - Extensions for JSONPath RW

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1259075

Haïkel Guémar  changed:

   What|Removed |Added

 Blocks||1243533
   ||(RDO-LIBERTY-REVIEWS)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1243533
[Bug 1243533] (RDO) Tracker: Review requests for new RDO Liberty packages
-- 
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 1259061] New: Review request: python-securepass - SecurePass Python library & tools

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1259061

Bug ID: 1259061
   Summary: Review request: python-securepass - SecurePass Python
library & tools
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: gpate...@gpaterno.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
Blocks: 177841 (FE-NEEDSPONSOR)



Spec URL: https://gpaterno.fedorapeople.org/python-securepass.spec
SRPM URL:
https://gpaterno.fedorapeople.org/python-securepass-0.4.2-1.el6.src.rpm

Description: 
The tools and python libraries for accessing SecurePass platform.
It uses the SecurePass public APIs.

Fedora Account System Username: gpaterno

The package is linked somehow to the NSS/PAM modules that are filed in the
bugzilla request 1162234.

The SPEC file defines a subpackage, as a matter of fact the binaries that comes
out of the python egg are split in a separate package.

Please help me reviewing the package.

Note: the subpackage "securepass-tools" produces an rpmlint warning because "
devel-file-in-non-devel-package /usr/bin/sp-config". sp-config is not actually
a development package, but rather a config facility for kickstart or
cloud-init.


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 1190269] Review Request: openstack-barbican - Secrets as a Service

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1190269

Haïkel Guémar  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+
   |needinfo?(karlthered@gmail. |
   |com)|



--- Comment #22 from Haïkel Guémar  ---
Only small change before importing the package
%doc LICENSE => %license LICENSE
For El6, you'd need the following macros as fallback: %{!?_licensedir:%global
license %%doc}

I hereby approve this package into Fedora Packages Collection. Please submit a
SCM
request.

I also suggest that you add apevec as owner of your package.



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: "Apache (v2.0)", "Unknown or generated", "*No copyright* Apache
 (v2.0)". 2 files have unknown license. Detailed output of licensecheck
 in /home/haikel/1190269-openstack-barbican/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 /etc/barbican/vassals
[x]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/lib/systemd,
 /usr/lib/systemd/system, /etc/logrotate.d, /etc/barbican,
 /etc/barbican/vassals
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[x]: Each %files section contains %defattr if rpm < 4.4
 Note: %defattr present but not needed
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[ ]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[x]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 61440 bytes in 3 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]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: %config files are marked noreplace or the reason is justified.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: No %config files under /usr.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

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

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate
 file from 

[Bug 1256359] Review Request: kf5-kalarmcal - The KAlarmCal Library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256359

Jan Grulich  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Jan Grulich  ---
1) License should be LGPLv2+

Only minor issue which can be fixed later or during import, 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 1256386] Review Request: kf5-kmbox - The KMbox library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256386

Daniel Vrátil  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Daniel Vrátil  ---
New Package SCM Request
===
Package Name: kf5-kmbox
Short Description: The MBox Library
Upstream URL: https://projects.kde.org/projects/kde/pim/kmbox
Owners: dvratil group::kde-sig
Branches: f22 f23
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 1256380] Review Request: kf5-kldap - The KLDAP Library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256380

Daniel Vrátil  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Daniel Vrátil  ---
New Package SCM Request
===
Package Name: kf5-kldap
Short Description: The KLDAP Library
Upstream URL: https://projects.kde.org/projects/kde/pim/kldap
Owners: dvratil group::kde-sig
Branches: f22 23
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 1242090] Review Request: nodejs-socks-client - A SOCKS proxy client supporting SOCKS 4, 4a, and 5

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1242090

Fedora Update System  changed:

   What|Removed |Added

 Status|CLOSED  |ON_QA
 Resolution|RAWHIDE |---
   Keywords||Reopened



--- Comment #12 from Fedora Update System  ---
nodejs-socks-client-1.1.4-3.fc22 has been pushed to the Fedora 22 testing
repository. If problems still persist, please make note of it in this bug
report.\nIf you want to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update nodejs-socks-client'. You can provide
feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14696

-- 
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 1258137] Review Request: nodejs-run-parallel-limit - Run functions in parallel with a limit on concurrent tasks

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258137

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #7 from Fedora Update System  ---
nodejs-run-parallel-limit-1.0.1-1.fc22 has been pushed to the Fedora 22 testing
repository. If problems still persist, please make note of it in this bug
report.\nIf you want to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update nodejs-run-parallel-limit'. You can provide
feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14712

-- 
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 1224800] Review Request: xfce-bluetooth - A bluetooth manager for Xfce

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1224800



--- Comment #24 from Fedora Update System  ---
xfce-bluetooth-0-0.4.20150130git.fc22 has been pushed to the Fedora 22 testing
repository. If problems still persist, please make note of it in this bug
report.\nIf you want to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update xfce-bluetooth'. You can provide feedback
for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14689

-- 
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 1228172] Review Request: jedis - A redis Java client

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1228172

Fedora Update System  changed:

   What|Removed |Added

 Status|CLOSED  |ON_QA
 Resolution|RAWHIDE |---
   Keywords||Reopened



--- Comment #14 from Fedora Update System  ---
jedis-2.7.2-2.fc22 has been pushed to the Fedora 22 testing repository. If
problems still persist, please make note of it in this bug report.\nIf you want
to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update jedis'. You can provide feedback for this
update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14667

-- 
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 1228203] Review Request: springframework-data-redis - Provides support to increase developer productivity in Java when using Redis

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1228203
Bug 1228203 depends on bug 1228172, which changed state.

Bug 1228172 Summary: Review Request: jedis - A redis Java client
https://bugzilla.redhat.com/show_bug.cgi?id=1228172

   What|Removed |Added

 Status|CLOSED  |ON_QA
 Resolution|RAWHIDE |---



-- 
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 1256472] Review Request: kdebugsettings - Application to configure debug output of KDE applications

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256472

Jan Grulich  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #3 from Jan Grulich  ---
Looks good now, 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 1257595] Review Request: libkmahjongg4 - Common code, backgrounds and tile sets for games using Mahjongg tiles

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1257595

Daniel Vrátil  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #3 from Daniel Vrátil  ---
New Package SCM Request
===
Package Name: libkmahjongg4
Short Description: Common code, backgrounds and tile sets for games using
Mahjongg tiles
Upstream URL: https://projects.kde.org/projects/kde/kdegames/libkmahjongg
Owners: dvratil group::kde-sig
Branches: f22 f23
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 1256359] Review Request: kf5-kalarmcal - The KAlarmCal Library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256359

Daniel Vrátil  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Daniel Vrátil  ---
New Package SCM Request
===
Package Name: kf5-kalarmcal
Short Description: The KAlarmCal Library
Upstream URL: https://projects.kde.org/projects/kde/pim/kalarmcal
Owners: dvratil group::kde-sig
Branches: f22 f23
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 1256347] Review Request: kf5-akonadi-search - The Akonadi Search library and indexing agent

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256347

Daniel Vrátil  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Daniel Vrátil  ---
New Package SCM Request
===
Package Name: kf5-akonadi-search
Short Description: The Akonadi Search Library and Indexing Agent
Upstream URL: https://projects.kde.org/projects/kde/pim/akonadi-search
Owners: dvratil group::kde-sig
Branches: f22 f23
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 1256362] Review Request: kf5-kblog - The KBlog Library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256362

Daniel Vrátil  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Daniel Vrátil  ---
New Package SCM Request
===
Package Name: kf5-kblog
Short Description: THe KBlog Library
Upstream URL: https://projects.kde.org/projects/kde/pim/kblog
Owners: dvratil group::kde-sig
Branches: f22 f23
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 1256341] Review Request: kf5-akonadi-calendar - The Akonadi Calendar client library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256341

Jan Grulich  changed:

   What|Removed |Added

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



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

[Bug 1256347] Review Request: kf5-akonadi-search - The Akonadi Search library and indexing agent

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256347

Jan Grulich  changed:

   What|Removed |Added

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



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

[Bug 1256362] Review Request: kf5-kblog - The KBlog Library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256362

Jan Grulich  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Jan Grulich  ---
1) License should be LGPLv2+

Only minor issue which can be fixed later or during import, 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 1256369] Review Request: kf5-kimap - The KIMAP Library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256369

Daniel Vrátil  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Daniel Vrátil  ---
New Package SCM Request
===
Package Name: kf5-kimap
Short Description: The KIMAP Library
Upstream URL: https://projects.kde.org/projects/kde/pim/kimap
Owners: dvratil group::kde-sig
Branches: f22 f23
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 1256383] Review Request: kf5-kmailtransport - The KMailTransport Library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256383

Daniel Vrátil  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Daniel Vrátil  ---
New Package SCM Request
===
Package Name: kf5-kmailtransport
Short Description: The KMailTransport Library
Upstream URL: https://projects.kde.org/projects/kde/pim/kmailtransport
Owners: dvratil group::kde-sig
Branches: f22 f23
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 1258430] Review Request: dolphin - KDE File Manager

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258430

Kevin Kofler  changed:

   What|Removed |Added

 Blocks||1258678




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1258678
[Bug 1258678] dolphin dependencies broken, kde-baseapps build failed
-- 
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 1257595] Review Request: libkmahjongg4 - Common code, backgrounds and tile sets for games using Mahjongg tiles

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1257595

Kevin Kofler  changed:

   What|Removed |Added

 Blocks||1258678




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1258678
[Bug 1258678] dolphin dependencies broken, kde-baseapps build failed
-- 
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 1256362] Review Request: kf5-kblog - The KBlog Library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256362

Jan Grulich  changed:

   What|Removed |Added

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



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

[Bug 1258159] Review Request: nodejs-lolex - Fake JavaScript timers

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258159

Piotr Popieluch  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #7 from Piotr Popieluch  ---
Thank you!

New Package SCM Request
===
Package Name: nodejs-lolex
Short Description: Fake JavaScript timers
Upstream URL: https://github.com/sinonjs/lolex
Owners: piotrp
Branches: f21 f22 f23 epel7

-- 
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 1256347] Review Request: kf5-akonadi-search - The Akonadi Search library and indexing agent

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256347

Jan Grulich  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Jan Grulich  ---
1) License should be GPLv2+ and LGPLv2+
2) Shouldn't you use %config for /etc/xdg/akonadi-search.categories?

Only minor issues which can be fixed later or during import, 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 1132670] Review Request: rubygem-compass-core - The Compass core stylesheet library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1132670

Vít Ondruch  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||rubygem-compass-core-1.0.1-
   ||4.fc22
 Resolution|--- |CURRENTRELEASE
Last Closed||2015-09-01 04:50:21



-- 
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 1256472] Review Request: kdebugsettings - Application to configure debug output of KDE applications

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1256472



--- Comment #2 from Daniel Vrátil  ---
Spec URL:
https://dvratil.fedorapeople.org/applications/review/kdebugsettings.spec
SRPM URL:
https://dvratil.fedorapeople.org/applications/review/kdebugsettings-15.08.0-2.fc22.src.rpm

* Tue Sep 01 2015 Daniel Vrátil  - 15.08.0-2
- fix license
- use %%license
- fix typo in summary
- use %%config for kde.categories

-- 
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 1258137] Review Request: nodejs-run-parallel-limit - Run functions in parallel with a limit on concurrent tasks

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258137



--- Comment #8 from Fedora Update System  ---
nodejs-run-parallel-limit-1.0.1-1.fc23 has been pushed to the Fedora 23 testing
repository. If problems still persist, please make note of it in this bug
report.\nIf you want to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update nodejs-run-parallel-limit'. You can provide
feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14709

-- 
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 1258151] Review Request: nodejs-async-cache - Cache your async lookups and don't fetch the same thing more than necessary

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258151

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #7 from Fedora Update System  ---
nodejs-async-cache-1.0.0-1.fc23 has been pushed to the Fedora 23 testing
repository. If problems still persist, please make note of it in this bug
report.\nIf you want to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update nodejs-async-cache'. You can provide
feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14705

-- 
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 1258241] Review Request: nodejs-basic-auth-parser - Parse Basic Auth Authorization HTTP headers

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1258241



--- Comment #9 from Fedora Update System  ---
nodejs-basic-auth-parser-0.0.2-1.fc23 has been pushed to the Fedora 23 testing
repository. If problems still persist, please make note of it in this bug
report.\nIf you want to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update nodejs-basic-auth-parser'. You can provide
feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14700

-- 
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 1242090] Review Request: nodejs-socks-client - A SOCKS proxy client supporting SOCKS 4, 4a, and 5

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1242090



--- Comment #13 from Fedora Update System  ---
nodejs-socks-client-1.1.4-3.fc23 has been pushed to the Fedora 23 testing
repository. If problems still persist, please make note of it in this bug
report.\nIf you want to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update nodejs-socks-client'. You can provide
feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14695

-- 
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 1223673] Review Request: llvm35 - The Low Level Virtual Machine

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1223673



--- Comment #8 from Jens Petersen  ---
(In reply to Milan Bouchet-Valat from comment #7)
> the unversioned .so
> files in the path, which is common with llvm and llvm34. I'm inclined to say
> that's OK for now, and we should try to fix this in the llvm package first..
> It's not like we're introducing a new bug...

Yeah, looking at it again further it does not seem that bad actually:

/etc/ld.so.conf.d/llvm-x86_64.conf
/usr/lib64/llvm/BugpointPasses.so
/usr/lib64/llvm/LLVMgold.so
/usr/lib64/llvm/libLLVM-3.5.0.so
/usr/lib64/llvm/libLLVM-3.5.so
/usr/lib64/llvm/libLTO.so
/usr/lib64/llvm/readline.so

LLVM.so is versioned (before .so) which I consider sufficient.
Then there are a number of modules which I think are also okay
(no idea what readline is doing there...).
Which just leaves libLTO.so, which seems to me is not
used by anything in llvm35?  (Not sure what it is for
to be honest.)

So I also feel we could waive this like was done for llvm34.

> - Minor point: I realized each subpackage creates its own directory
>   under /usr/share/doc/. Since they contain very few files and most apply
>   to all subpackages (e.g. LICENSE.txt), wouldn't it be better to put
> everthing
>   under a common llvm dir?

This is true and also true for many other packages I think.
I'd rather just leave it for simplicity - though in principle
I agree with you completely, but I feel this is more a deficiency
of rpm.

> [!]: Development (unversioned) .so files in -devel subpackage, if present.
>  Note: Unversioned so-files in private %_libdir subdirectory (see
>  attachment). Verify they are not in ld path.

See above

> [!]: Package must own all directories that it creates.
>  Note: Directories without known owners: /usr/lib64/llvm35

This is a fedora-review bug.

> [!]: %build honors applicable compiler flags or justifies otherwise.
>  Could you add a comment explaining why %{optflags} is ignored
>  and why --with-optimize-option=-O3 is a good idea?

Okay

> [!]: Patches link to upstream bugs/comments/lists or are otherwise
>  justified.
>  Could use a bit more verbose comments and links to upstream patches.

> Checking: llvm35-3.5.2-1.fc22.x86_64.rpm
>   llvm35-devel-3.5.2-1.fc22.x86_64.rpm
>   llvm35-doc-3.5.2-1.fc22.noarch.rpm
>   llvm35-libs-3.5.2-1.fc22.x86_64.rpm
>   llvm35-static-3.5.2-1.fc22.x86_64.rpm
>   llvm35-3.5.2-1.fc22.src.rpm
> llvm35.x86_64: W: spelling-error %description -l en_US runtime -> run time,
> run-time, rudiment
> Could fix this one.

Hmm, isn't "runtime" pretty standard?

> llvm35.x86_64: W: no-manual-page-for-binary llvm-diff-3.5
:
:
> llvm35-devel.x86_64: W: no-manual-page-for-binary llvm-config-64-3.5
> Why don't we have manpages for all those commands, even in the standard llvm
> package?

Good question - perhaps Fedora could grab a patch from Debian?
Again I think this should be done first for the llvm package.
I opened bug 1258760 for that.

> llvm35.src:177: E: hardcoded-library-path in
> %{_prefix}/lib/gcc/%{_target_cpu}*/*/include)
> Out of curiosity, why doesn't gcc install these files under %{_libdir}?

Historical I think - it is using target arch dirs instead of multilib.

-- 
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 1039301] Review Request: jackson-datatype-joda - Extension module to properly support full datatype set of Joda date-time library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1039301

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #14 from Fedora Update System  ---
jackson-datatype-joda-2.5.0-1.fc22 has been pushed to the Fedora 22 testing
repository. If problems still persist, please make note of it in this bug
report.\nIf you want to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update jackson-datatype-joda'. You can provide
feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14586

-- 
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 1247251] Review Request: xfce-polkit - Simple PolicyKit authentication agent for Xfce

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1247251



--- Comment #10 from Fedora Update System  ---
xfce-polkit-0-0.1.20130717gitbe888ee.fc22 has been pushed to the Fedora 22
testing repository. If problems still persist, please make note of it in this
bug report.\nIf you want to test the update, you can install it with \n su -c
'yum --enablerepo=updates-testing update xfce-polkit'. You can provide feedback
for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14583

-- 
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 1242091] Review Request: nodejs-smart-buffer - A smarter Buffer that keeps track of its own read and write positions while growing endlessly

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1242091



--- Comment #14 from Fedora Update System  ---
nodejs-smart-buffer-1.0.3-2.fc22 has been pushed to the Fedora 22 testing
repository. If problems still persist, please make note of it in this bug
report.\nIf you want to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update nodejs-smart-buffer'. You can provide
feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14594

-- 
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 1213184] Review Request: grantlee-qt5 - Qt string template engine based on the Django template system

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1213184

Fedora Update System  changed:

   What|Removed |Added

 Status|CLOSED  |ON_QA
 Resolution|RAWHIDE |---
   Keywords||Reopened



--- Comment #8 from Fedora Update System  ---
grantlee-qt5-5.0.0-2.fc23 has been pushed to the Fedora 23 testing repository.
If problems still persist, please make note of it in this bug report.\nIf you
want to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update grantlee-qt5'. You can provide feedback for
this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14650

-- 
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 1039301] Review Request: jackson-datatype-joda - Extension module to properly support full datatype set of Joda date-time library

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1039301



--- Comment #15 from Fedora Update System  ---
jackson-datatype-joda-2.5.0-1.fc23 has been pushed to the Fedora 23 testing
repository. If problems still persist, please make note of it in this bug
report.\nIf you want to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update jackson-datatype-joda'. You can provide
feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14584

-- 
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 1228203] Review Request: springframework-data-redis - Provides support to increase developer productivity in Java when using Redis

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1228203

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #11 from Fedora Update System  ---
springframework-data-redis-1.3.5-2.fc23 has been pushed to the Fedora 23
testing repository. If problems still persist, please make note of it in this
bug report.\nIf you want to test the update, you can install it with \n su -c
'yum --enablerepo=updates-testing update springframework-data-redis'. You can
provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-14589

-- 
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 1247251] Review Request: xfce-polkit - Simple PolicyKit authentication agent for Xfce

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1247251



--- Comment #11 from Fedora Update System  ---
xfce-polkit-0-0.1.20130717gitbe888ee.fc23 has been pushed to the Fedora 23
testing repository. If problems still persist, please make note of it in this
bug report.\nIf you want to test the update, you can install it with \n su -c
'yum --enablerepo=updates-testing update xfce-polkit'. You can provide feedback
for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14581

-- 
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 1246846] Review Request: jmh - Java Microbenchmark Harness

2015-09-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1246846

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #19 from Fedora Update System  ---
jmh-1.10.5-2.fc23 has been pushed to the Fedora 23 testing repository. If
problems still persist, please make note of it in this bug report.\nIf you want
to test the update, you can install it with \n su -c 'yum
--enablerepo=updates-testing update jmh'. You can provide feedback for this
update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14608

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

  1   2   3   >