[Bug 1030755] Review Request: python-livereload - LiveReload server in Python

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1030755



--- Comment #6 from Pete Travis m...@petetravis.com ---
There have been a few releases since your last update, are you still interested
in pursuing this review, Christopher?  Nikola currently declares requirements
as livereload==2.3.2, fwiw.

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

[Bug 894609] Review Request: coin-or-OS - Optimization Services

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=894609

Antonio Trande anto.tra...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|anto.tra...@gmail.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 1194428] Review Request: fedfind - Fedora image finder

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194428

Pete Travis m...@petetravis.com changed:

   What|Removed |Added

 CC||awill...@redhat.com
  Flags||needinfo?(awilliam@redhat.c
   ||om)



--- Comment #5 from Pete Travis m...@petetravis.com ---
I looked over this today and don't see any problems ( besides the surmountable
ones mentioned in #3 ).  However, there are releases up to 1.1.2 tagged in your
repo, do you have an updated build for 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 1210983] New: Review Request: python-vcrpy - Automatically mock your HTTP interactions to simplify and speed up testing

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210983

Bug ID: 1210983
   Summary: Review Request: python-vcrpy - Automatically mock your
HTTP interactions to simplify and speed up testing
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: rb...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org




Spec URL: http://ralph.fedorapeople.org//python-vcrpy.spec
SRPM URL: http://ralph.fedorapeople.org//python-vcrpy-1.4.0-1.fc21.src.rpm

Description:
Simplify and speed up testing HTTP by recording all HTTP interactions and
saving them to cassette files, which are yaml files containing the contents
of your requests and responses.  Then when you run your tests again, they all
just hit the text files instead of the internet.  This speeds up your tests and
lets you work offline.

If the server you are testing against ever changes its API, all you need to do
is delete your existing cassette files, and run your tests again.  All of the
mocked responses will be updated with the new API.

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

[Bug 1210978] Review Request: python-contextlib2 - Backports and enhancements for the contextlib module

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210978

Ralph Bean rb...@redhat.com changed:

   What|Removed |Added

 Blocks||1210983




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1210983
[Bug 1210983] Review Request: python-vcrpy - Automatically mock your HTTP
interactions to simplify and speed up testing
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1210983] Review Request: python-vcrpy - Automatically mock your HTTP interactions to simplify and speed up testing

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210983

Ralph Bean rb...@redhat.com changed:

   What|Removed |Added

 Depends On||1210982, 1210978




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1210978
[Bug 1210978] Review Request: python-contextlib2 - Backports and
enhancements for the contextlib module
https://bugzilla.redhat.com/show_bug.cgi?id=1210982
[Bug 1210982] [patch] Add a python3 subpackage
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1210971] Review Request: libkface - A face recognition and detection library

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210971



--- Comment #2 from Mario Blättermann mario.blaetterm...@gmail.com ---
(In reply to Mario Blättermann from comment #1)
 AUTHORS and TODO are missing from %doc.

Sorry, AUTHORS is already present.

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

[Bug 1210978] Review Request: python-contextlib2 - Backports and enhancements for the contextlib module

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210978



--- Comment #1 from Ralph Bean rb...@redhat.com ---
This package built on koji: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=9462898

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

[Bug 1210978] New: Review Request: python-contextlib2 - Backports and enhancements for the contextlib module

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210978

Bug ID: 1210978
   Summary: Review Request: python-contextlib2 - Backports and
enhancements for the contextlib module
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: rb...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org




Spec URL: http://ralph.fedorapeople.org//python-contextlib2.spec
SRPM URL:
http://ralph.fedorapeople.org//python-contextlib2-0.4.0-1.fc21.src.rpm

Description:
contextlib2 is a backport of the standard library's contextlib module to
earlier Python versions.

It also serves as a real world proving ground for possible future
enhancements to the standard library version.

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

[Bug 1194428] Review Request: fedfind - Fedora image finder

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194428

awill...@redhat.com awill...@redhat.com changed:

   What|Removed |Added

  Flags|needinfo?(awilliam@redhat.c |
   |om) |



--- Comment #6 from awill...@redhat.com awill...@redhat.com ---
No, I've been busy with Beta work. I need to drop the bundled cached_property
now we package it, but haven't had time.

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

[Bug 1208904] Review Request: python-jenkins-job-builder - Manage Jenkins jobs with YAML

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1208904



--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
python-jenkins-job-builder-1.1.0-1.fc21 has been submitted as an update for
Fedora 21.
https://admin.fedoraproject.org/updates/python-jenkins-job-builder-1.1.0-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 1210971] Review Request: libkface - A face recognition and detection library

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210971



--- Comment #3 from Rex Dieter rdie...@math.unl.edu ---
Spec URL: https://rdieter.fedorapeople.org/rpms/kf5/libkface.spec
SRPM URL:
https://rdieter.fedorapeople.org/rpms/kf5/libkface-15.03.97-2.fc22.src.rpm

%changelog
* Sat Apr 11 2015 Rex Dieter rdie...@fedoraproject.org 15.03.97-2
- switch to publically-available 15.03.97
- fix missing %%description
- -devel: %%license COPYING.LIB COPYING-CMAKE_SCRIPTS

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

[Bug 1210971] Review Request: libkface - A face recognition and detection library

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210971

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

   What|Removed |Added

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



--- Comment #1 from Mario Blättermann mario.blaetterm...@gmail.com ---
%description is still empty.

AUTHORS and TODO are missing from %doc.

The cmake scripts are BSD licensed, those license text should be added to the
-devel package.

The file libkface/libkface_export.h is LGPL licensed, that's why COPYING.LIB
needs to go into -devel.

rpmlint complains about a non-existent download URL. Indeed, the access is
forbidden:

$ wget
http://download.kde.org/stable/applications/15.04.0/src/libkface-15.04.0.tar.xz
--2015-04-11 21:05:15-- 
http://download.kde.org/stable/applications/15.04.0/src/libkface-15.04.0.tar.xz
Auflösen des Hostnamens »download.kde.org (download.kde.org)« …
2001:67c:2178:7::71, 195.135.221.71
Verbindungsaufbau zu download.kde.org
(download.kde.org)|2001:67c:2178:7::71|:80 … verbunden.
HTTP-Anforderung gesendet, auf Antwort wird gewartet … 403 Forbidden
2015-04-11 21:05:15 FEHLER 403: Forbidden.

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

[Bug 1194798] Review Request: GeoIP-GeoLite-data - Free GeoLite IP geolocation country database

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194798



--- Comment #16 from Paul Howarth p...@city-fan.org ---
Ping?

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

[Bug 1210972] Review Request: libkgeomap - A wrapper around different world-map components, to browse and arrange photos over a map

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210972



--- Comment #1 from Rex Dieter rdie...@math.unl.edu ---
Spec URL: https://rdieter.fedorapeople.org/rpms/kf5/libkgeomap.spec
SRPM URL:
https://rdieter.fedorapeople.org/rpms/kf5/libkgeomap-15.03.97-2.fc22.src.rpm

%changelog
* Sat Apr 11 2015 Rex Dieter rdie...@fedoraproject.org 15.03.97-2
- switch to publically-available 15.03.97
- fix missing %%description
- -devel: %%license COPYING.LIB COPYING-CMAKE_SCRIPTS

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

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=921134



--- Comment #19 from Denis Arnaud denis.arnaud_fed...@m4x.org ---
EPEL 7: http://koji.fedoraproject.org/koji/taskinfo?taskID=9463024
The builds were successful for both x86-64 and ppc64 (there is an error at the
end in Koji, but it seems to be a Koji issue, as all the RPMs have successfully
been built)

-
So, ready for approval?

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

[Bug 1161099] Review Request: nodejs-wrappy - Callback wrapping utility

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1161099



--- Comment #14 from Zuzana Svetlikova zsvet...@redhat.com ---
The canonical method for shipping most node modules is tarballs from the npm
registry.
This method should be preferred to using checkouts from git or automatically
generated tarballs from GitHub.

Also, the github source might include things the developer doesn't want to have
in official tarball posted on npmjs.org

Given the tarballs are the same, the missing macros made the builds fail.

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

[Bug 1193730] Review Request: apache-jena - Java framework for building Semantic Web and Linked Data applications

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1193730

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 Depends On||1210958




Referenced Bugs:

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

[Bug 1210990] New: Review Request: Qt5 - WebChannel component

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210990

Bug ID: 1210990
   Summary: Review Request: Qt5 - WebChannel component
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: tay...@braun-jones.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://files.braun-jones.org/qt5-qtwebchannel.spec
SRPM URL:
http://files.braun-jones.org/qt5-qtwebchannel-5.4.1-1.el7.centos.src.rpm
Description: 
The Qt WebChannel module provides a library for seamless integration of C++
and QML applications with HTML/JavaScript clients. Any QObject can be
published to remote clients, where its public API becomes available.

Fedora Account System Username: nocnokneo

Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=9463055

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

[Bug 1193730] Review Request: apache-jena - Java framework for building Semantic Web and Linked Data applications

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1193730

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 Depends On||970285




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=970285
[Bug 970285] Review Request: shiro - A powerful and flexible open-source
security framework
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 970285] Review Request: shiro - A powerful and flexible open-source security framework

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=970285

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

 Blocks||1193730




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1193730
[Bug 1193730] Review Request: apache-jena - Java framework for building
Semantic Web and Linked Data applications
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=921134



--- Comment #18 from Denis Arnaud denis.arnaud_fed...@m4x.org ---
Spec URL: http://denisarnaud.fedorapeople.org/boost/boost148/boost148.spec
SRPM URL:
http://denisarnaud.fedorapeople.org/boost/boost148/boost148-1.48.0-5.fc20.src.rpm


EPEL 7: http://koji.fedoraproject.org/koji/taskinfo?taskID=9462960
EPEL 6: http://koji.fedoraproject.org/koji/taskinfo?taskID=9462986 (the build
is fine, but the Koji build process fails at the end because of a lack of
resource)

===
(In reply to Rex Dieter from comment #16)
 May be worth reviewing later commits/fixes to base boost-1.48.0 pkg up to:
 http://pkgs.fedoraproject.org/cgit/boost.git/commit/
 ?id=dcde8a75575cb4c256ea982593c9710ac16b022c

Done. Integrated all the missing patches.

 1 MUST fix missing -math dep in main metapackage and missing scriptlets
 fixed in:
 http://pkgs.fedoraproject.org/cgit/boost.git/commit/
 ?id=8aad81e67cbfd196e040449b0bf2cd854642c447

Done


 2.  SHOULD consider using %{name} and %{?_isa} macros in place of
 hard-coding boost148 in dependencies. imo, that simplifies things keeps
 things consistent.  for example, replace
 Requires: boost148-chrono = %{version}-%{release}
 with
 Requires: %{name}-chrono%{?_isa} = %{version}-%{release}

Done


Did not check yet whether the fix for ppc64 works.
===

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

[Bug 505154] Tracker: Review Requests for Science and Technology related packages

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=505154
Bug 505154 depends on bug 894600, which changed state.

Bug 894600 Summary: Review Request: coin-or-cppad - A Package for 
Differentiation of C++ Algorithms
https://bugzilla.redhat.com/show_bug.cgi?id=894600

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |DUPLICATE



-- 
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 894609] Review Request: coin-or-OS - Optimization Services

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=894609

Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com changed:

   What|Removed |Added

 Depends On||1197488




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1197488
[Bug 1197488] RFE: Please add Provides: coin-or-cppad =
%{version}-%{release} and Provides: coin-or-cppad-devel =
%{version}-%{release}
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894600] Review Request: coin-or-cppad - A Package for Differentiation of C++ Algorithms

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=894600

Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |DUPLICATE
Last Closed||2015-04-11 14:21:02



--- Comment #10 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com 
---
I made extra requests for the cppad package as the initial
patch was done incorrectly, it failed to provide
coin-or-cppad-devel.
https://bugzilla.redhat.com/show_bug.cgi?id=1197488

But in the meantime I adjusted the package that would
require it to only require cppad-devel, so, the next
review for the coin-or stack is
https://bugzilla.redhat.com/show_bug.cgi?id=894609

coin-or-OS is actually the last one, to match the coin-or
monolithic tarball, after that, mostly optimizations and
optional packages could be added.

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

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

[Bug 894609] Review Request: coin-or-OS - Optimization Services

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=894609
Bug 894609 depends on bug 894600, which changed state.

Bug 894600 Summary: Review Request: coin-or-cppad - A Package for 
Differentiation of C++ Algorithms
https://bugzilla.redhat.com/show_bug.cgi?id=894600

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |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 894609] Review Request: coin-or-OS - Optimization Services

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=894609



--- Comment #3 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com ---
The package can be built now in rawhide.

Spec URL: http://pcpa.fedorapeople.org/coin-or/coin-or-OS.spec
SRPM URL: http://pcpa.fedorapeople.org/coin-or/coin-or-OS-2.9.2-1.fc23.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 505154] Tracker: Review Requests for Science and Technology related packages

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=505154

Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com changed:

   What|Removed |Added

 Depends On||1197488




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1197488
[Bug 1197488] RFE: Please add Provides: coin-or-cppad =
%{version}-%{release} and Provides: coin-or-cppad-devel =
%{version}-%{release}
-- 
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 1208904] Review Request: python-jenkins-job-builder - Manage Jenkins jobs with YAML

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1208904

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 1208904] Review Request: python-jenkins-job-builder - Manage Jenkins jobs with YAML

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1208904



--- Comment #7 from Fedora Update System upda...@fedoraproject.org ---
python-jenkins-job-builder-1.1.0-1.fc22 has been submitted as an update for
Fedora 22.
https://admin.fedoraproject.org/updates/python-jenkins-job-builder-1.1.0-1.fc22

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

[Bug 1210430] Review Request: yumex-dnf - graphical package manager powered by dnf

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210430



--- Comment #2 from Tim Lauridsen t...@rasmil.dk ---
Thanks for reviewing !

Spec URL:
https://timlau.fedorapeople.org/files/packages/yumex-dnf/yumex-dnf.spec
SRPM
URL:https://timlau.fedorapeople.org/files/packages/yumex-dnf/yumex-dnf-4.0.10-2.fc22.src.rpm

fixed rpmlint errors, make flags and Using both %{buildroot} and
$RPM_BUILD_ROOT

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

[Bug 1210430] Review Request: yumex-dnf - graphical package manager powered by dnf

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210430



--- Comment #3 from Tim Lauridsen t...@rasmil.dk ---
koji scratchbuid:
http://koji.fedoraproject.org/koji/taskinfo?taskID=9461220

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

[Bug 1141807] Review Request: golang-github-coreos-go-etcd - The official etcd v0.2 client library for Go

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1141807

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

   What|Removed |Added

   Fixed In Version|golang-github-coreos-go-etc |golang-github-coreos-go-etc
   |d-0.4.6-0.2.rc1.git2af7a95. |d-0.4.6-0.2.rc1.git2af7a95.
   |fc21|fc20



--- Comment #26 from Fedora Update System upda...@fedoraproject.org ---
golang-github-coreos-go-etcd-0.4.6-0.2.rc1.git2af7a95.fc20 has been pushed to
the Fedora 20 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 1085881] Review Request: golang-github-spf13-cobra - A Commander for modern go CLI interactions

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1085881

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

   What|Removed |Added

   Fixed In Version|golang-github-spf13-cobra-0 |golang-github-spf13-cobra-0
   |-0.7.git79bd93d.fc21|-0.7.git79bd93d.fc20



--- Comment #31 from Fedora Update System upda...@fedoraproject.org ---
golang-github-spf13-cobra-0-0.7.git79bd93d.fc20 has been pushed to the Fedora
20 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 1210430] Review Request: yumex-dnf - graphical package manager powered by dnf

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210430



--- Comment #4 from Florian der-flo Lehner d...@der-flo.net ---
Hi Tim!

There are still issues:

[ ] Sources used to build the package match the upstream source, as provided in
the spec URL.
See: http://fedoraproject.org/wiki/Packaging/SourceURL

Only in
/home/flo/review/1210430-yumex-dnf/srpm-unpacked/yumex-dnf-4.0.10.tar.gz-extract:
yumex-dnf-4.0.10
Only in /home/flo/review/1210430-yumex-dnf/upstream-unpacked/Source0:
yumex-dnf-yumex-dnf-4.0.10
Source checksums

https://github.com/timlau/yumex-dnf/archive/yumex-dnf-4.0.10.tar.gz :
CHECKSUM(SHA256) this package :
d7822fcb50821a4d1ae5d3bf2a9672f1f5493caf69f5260d0af15409a41f4d85
CHECKSUM(SHA256) upstream package :
3f720e90e02cad75939f196e00ece5b3b57932f9d627b0bddf288519fbc31e9c

[ ] incoherent-version-in-changelog
It should look something like:
* Thu Apr 9 2015 Tim Lauridsen tim...@fedoraproject.org 4.0.10-3

Cheers,
 Florian

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

[Bug 1210941] New: Review Request: main package name here - short summary here

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210941

Bug ID: 1210941
   Summary: Review Request: main package name here - short
summary here
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ville.sky...@iki.fi
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



https://scop.fedorapeople.org/packages/isrcsubmit.spec
https://scop.fedorapeople.org/packages/isrcsubmit-2.0.0-1.fc21.src.rpm

Description: isrcsubmit extracts ISRCs from audio cds and submits them to
MusicBrainz. Features: read and submit ISRCs from disc, search for
releases with the TOC of the disc, submit discIds / TOCs, display
release information from MusicBrainz, duplicate ISRC detection (local
and on server), keyring support for login information.

Fedora Account System Username: scop

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

[Bug 1210941] Review Request: isrcsubmit - Script to submit ISRCs from disc to MusicBrainz

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210941

Ville Skyttä ville.sky...@iki.fi changed:

   What|Removed |Added

Summary|Review Request: main   |Review Request: isrcsubmit
   |package name here - short |- Script to submit ISRCs
   |summary here   |from disc to MusicBrainz



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

[Bug 1210939] New: Review Request: vdr-skindesigner - A VDR skinning engine that displays XML based Skins

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210939

Bug ID: 1210939
   Summary: Review Request: vdr-skindesigner - A VDR skinning
engine that displays XML based Skins
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: mgans...@alice.de
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/vdr-skindesigner.spec
SRPM URL:
https://martinkg.fedorapeople.org/Review/SRPMS/vdr-skindesigner-0.4.2-2.fc21.src.rpm

Description: SkinDesigner is a powerfull tool to create VDR Skins based on
Skindesigner specific XML Code. The following documentation shows the
SkinDesigner internals so that new Skinners get easily an overview how
Skindesigner works.
Hopefully all your open questions are answerd, if not, feel free to ask in
VDR Portal.

Fedora Account System Username: martinkg

The package has been available on rpmfusion so far.
http://cvs.rpmfusion.org/viewvc/rpms/vdr-skindesigner/devel/?root=free

It seems that it has no dependencies that are only in RPM Fusion and a
license problem does not exist either, so we could get back the package
to Fedora ?

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

[Bug 1174030] Review Request: golang-github-appc-spec - Schema defs and tools for app container specification

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1174030
Bug 1174030 depends on bug 1174021, which changed state.

Bug 1174021 Summary: Review Request: golang-github-coreos-go-semver - Go 
semantic versioning library
https://bugzilla.redhat.com/show_bug.cgi?id=1174021

   What|Removed |Added

 Status|ON_QA   |CLOSED
 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 1174021] Review Request: golang-github-coreos-go-semver - Go semantic versioning library

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1174021

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2015-04-11 02:02:38



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

[Bug 1063052] Review Request: rubygem-middleman - A static site generator

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1063052

Nitesh Narayan Lal niteshnarayanlal...@gmail.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||niteshnarayanlalleo@gmail.c
   ||om
 Resolution|--- |WONTFIX
  Flags||fedora-review+
Last Closed||2015-04-11 04:14:26



--- Comment #4 from Nitesh Narayan Lal niteshnarayanlal...@gmail.com ---
Apologies for the delayed response, I won't be able to work on this due to lack
of time.
@Christopher I am not a new to Fedora but you must think before you write.
Putting it in a gentle way if you don't want to do it, then don't do it.

Remember no one is perfect and we learn as we do.

@Rex  @Haikel Sorry guys, I am closing this for now. Hopefully will pick it up
again once I get some time.

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

[Bug 1082734] Review Request: golang-github-godbus-dbus - Go client bindings for D-Bus

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1082734

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

   What|Removed |Added

   Fixed In Version|golang-github-godbus-dbus-2 |golang-github-godbus-dbus-2
   |-0.1.git939230d.fc21|-0.1.git939230d.fc20



--- Comment #21 from Fedora Update System upda...@fedoraproject.org ---
golang-github-godbus-dbus-2-0.1.git939230d.fc20 has been pushed to the Fedora
20 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 1196363] Review Request: golang-github-docker-spdystream - A multiplexed stream library using spdy

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196363

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

   What|Removed |Added

   Fixed In Version|golang-github-docker-spdyst |golang-github-docker-spdyst
   |ream-0-0.3.gite731c8f.fc21  |ream-0-0.3.gite731c8f.fc20



--- Comment #19 from Fedora Update System upda...@fedoraproject.org ---
golang-github-docker-spdystream-0-0.3.gite731c8f.fc20 has been pushed to the
Fedora 20 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 1164152] Review Request: golang-github-emicklei-go-restful - Package for building REST-style Web Services using Google Go

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1164152

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

   What|Removed |Added

   Fixed In Version|golang-github-emicklei-go-r |golang-github-emicklei-go-r
   |estful-1.1.3-0.1.git5e1952e |estful-1.1.3-0.1.git5e1952e
   |.fc21   |.fc20



--- Comment #22 from Fedora Update System upda...@fedoraproject.org ---
golang-github-emicklei-go-restful-1.1.3-0.1.git5e1952e.fc20 has been pushed to
the Fedora 20 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 1210430] Review Request: yumex-dnf - graphical package manager powered by dnf

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210430



--- Comment #5 from Tim Lauridsen t...@rasmil.dk ---
Spec URL:
https://timlau.fedorapeople.org/files/packages/yumex-dnf/yumex-dnf.spec
SRPM URL:
https://timlau.fedorapeople.org/files/packages/yumex-dnf/yumex-dnf-4.0.10-3.fc22.src.rpm

- forgot to update upstream source, done now, so checksums should match now
- fixed incoherent-version-in-changelog

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

[Bug 1201925] Review Request: gnome-shell-extension-touchpad-indicator - Minimalistic Touchpad management extension for the Gnome Shell

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1201925



--- Comment #2 from Ville Skyttä ville.sky...@iki.fi ---
Good job with the review, thanks. License tag fixed and comment added:

https://scop.fedorapeople.org/packages/gnome-shell-extension-touchpad-indicator.spec
https://scop.fedorapeople.org/packages/gnome-shell-extension-touchpad-indicator-0-2.20150221gitf2f30f4.fc23.src.rpm

The comment and %files section breakdown etc are in the ...actual
implementation of this is left to the maintainer. Some suggested
implementations include... category, so I think the comment is enough (they're
suggested, not mandatory, and not all of them need to be done anyway).

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

[Bug 1161483] Review Request: o3dgc - an open 3D graphics compression library

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1161483

Alexander Ploumistos alex.ploumis...@gmail.com changed:

   What|Removed |Added

 CC||alex.ploumis...@gmail.com



--- Comment #2 from Alexander Ploumistos alex.ploumis...@gmail.com ---
Hello,

This is an unofficial review.

I don't know if anyone more qualified than me will take issue with the source
(re)packaging, which gives rpmlint something to complain about, but the
rationale seems obvious.

The patch is also pretty much self-explanatory, perhaps someone would prefer a
more explicit comment someplace.

There does seem to be an issue with the licenses though:

BSD (2 clause)
--
o3dgc/src/o3dgc_common_lib/inc/o3dgcArithmeticCodec.h
o3dgc/src/o3dgc_common_lib/src/o3dgcArithmeticCodec.cpp

MIT/X11 (BSD like)
--
o3dgc/js/o3dgc.js
o3dgc/src/o3dgc_common_lib/inc/o3dgcAdjacencyInfo.h
o3dgc/src/o3dgc_common_lib/inc/o3dgcBinaryStream.h
o3dgc/src/o3dgc_common_lib/inc/o3dgcCommon.h
o3dgc/src/o3dgc_common_lib/inc/o3dgcDVEncodeParams.h
o3dgc/src/o3dgc_common_lib/inc/o3dgcDynamicVector.h
o3dgc/src/o3dgc_common_lib/inc/o3dgcFIFO.h
o3dgc/src/o3dgc_common_lib/inc/o3dgcIndexedFaceSet.h
o3dgc/src/o3dgc_common_lib/inc/o3dgcSC3DMCEncodeParams.h
o3dgc/src/o3dgc_common_lib/inc/o3dgcTimer.h
o3dgc/src/o3dgc_common_lib/inc/o3dgcTriangleFans.h
o3dgc/src/o3dgc_common_lib/inc/o3dgcVector.h
o3dgc/src/o3dgc_common_lib/src/o3dgcTools.cpp
o3dgc/src/o3dgc_common_lib/src/o3dgcTriangleFans.cpp
o3dgc/src/o3dgc_decode_lib/inc/o3dgcDynamicVectorDecoder.h
o3dgc/src/o3dgc_decode_lib/inc/o3dgcSC3DMCDecoder.h
o3dgc/src/o3dgc_decode_lib/inc/o3dgcTriangleListDecoder.h
o3dgc/src/o3dgc_decode_lib/src/o3dgcDynamicVectorDecoder.cpp
o3dgc/src/o3dgc_encode_lib/inc/o3dgcDynamicVectorEncoder.h
o3dgc/src/o3dgc_encode_lib/inc/o3dgcSC3DMCEncoder.h
o3dgc/src/o3dgc_encode_lib/inc/o3dgcTriangleListEncoder.h
o3dgc/src/o3dgc_encode_lib/src/o3dgcDynamicVectorEncoder.cpp
o3dgc/src/test/src/main.cpp

Unknown or generated

o3dgc/install/run.py
o3dgc/js/index.php
o3dgc/js/three.min.js


You need to specify all licenses in the spec file and it would be nice to sort
files in the %files section by their license. See

https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/LicensingGuidelines#Multiple_Licensing_Scenarios

I am away from my main system and I don't have koji setup here, so I could only
test locally for x86_64. Have you tried building the package for other
architectures?

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

[Bug 1129429] Review Request: ccnet - A framework for writing networked applications in C

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1129429



--- Comment #3 from Nikos Roussos comzer...@fedoraproject.org ---
Update to 4.1.4

SPEC: https://comzeradd.fedorapeople.org/specs/ccnet.spec
SRPM: https://comzeradd.fedorapeople.org/srpms/ccnet-4.1.4-1.fc21.src.rpm

Also address some things from above comment:

2. Use release tag instead of commit
3. Packaging guidelines discourage use of %makeinstall
https://fedoraproject.org/wiki/Packaging:Guidelines#Why_the_.25makeinstall_macro_should_not_be_used
4. Enabled hardened build
5. Use %license for the LICENSE text

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

[Bug 1161483] Review Request: o3dgc - an open 3D graphics compression library

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1161483



--- Comment #3 from David Tardon dtar...@redhat.com ---
Spec URL: https://dtardon.fedorapeople.org/rpm/o3dgc.spec
SRPM URL: https://dtardon.fedorapeople.org/rpm/o3dgc-0-3.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 1209971] Review Request: usbguard - A tool for implementing USB device usage policy

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1209971



--- Comment #17 from Daniel Kopeček dkope...@redhat.com ---
Added dependencies on json-static and spdlog-static. Bundled libraries are
removed in %prep.

New SRPM is at:
https://fedorapeople.org/~dkopecek/usbguard/usbguard-0.3p2-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 1210969] New: Review Request: python-munch - A dot-accessible dictionary (a la JavaScript objects)

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210969

Bug ID: 1210969
   Summary: Review Request: python-munch - A dot-accessible
dictionary (a la JavaScript objects)
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: rb...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org




Spec URL: http://ralph.fedorapeople.org//python-munch.spec
SRPM URL: http://ralph.fedorapeople.org//python-munch-2.0.2-1.fc21.src.rpm

Description:
munch is a fork of David Schoonover's **Bunch** package, providing similar
functionality. 99% of the work was done by him, and the fork was made
mainly for lack of responsiveness for fixes and maintenance on the original
code.

Munch is a dictionary that supports attribute-style access, a la
JavaScript.

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

[Bug 1210826] Review Request: rubygem-pathspec - Use to match path patterns such as gitignore

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210826



--- Comment #3 from Ken Dreyer ktdre...@ktdreyer.com ---
I imported your SRPM into a Git repo so it's clearer. Here's how I recommend
handling the tests:
https://fedorapeople.org/cgit/ktdreyer/public_git/rubygem-pathspec.git/commit/?id=2957c1fb52216a43e9b6fc94d4fd7d02cf6738d8

I use a small shell script (rubygem-pathspec-generate-tarball.sh) to download
the upstream source zipfile from a GitHub tag, then strip down that zipfile so
it only contains the tests. At that point, %setup can decompress Source2 so
that it appears to be part of the tree.

To fix this upstream, I've submitted
https://github.com/highb/pathspec-ruby/pull/4

Ruby's RSpec library has made some API breaks between 2.x and 3.x, so I've also
updated the .spec there to handle both APIs depending on the %{fedora} version.
This allows the package to build on Fedora 21, 22, and 23.

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

[Bug 1210828] Review Request: rubygem-semantic - Utility class for parsing, storing, and comparing versions

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210828

Ken Dreyer ktdre...@ktdreyer.com changed:

   What|Removed |Added

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



--- Comment #1 from Ken Dreyer ktdre...@ktdreyer.com ---
Some initial comments...

- License is set as ASL 2.0, but upstream's license is really MIT

- The code in %check uses rspec, so you'll need to BuildRequire:
rubygems(rspec)

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

[Bug 1210971] New: Review Request: libkface - A face recognition and detection library

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210971

Bug ID: 1210971
   Summary: Review Request: libkface - A face recognition and
detection library
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: rdie...@math.unl.edu
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://rdieter.fedorapeople.org/rpms/kf5/libkface.spec
SRPM URL:
https://rdieter.fedorapeople.org/rpms/kf5/libkface-15.04.0-1.fc22.src.rpm
Description: A face recognition and detection library
Fedora Account System Username: rdieter

scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=9462465

Needed for newer digikam builds.

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

[Bug 1210969] Review Request: python-munch - A dot-accessible dictionary (a la JavaScript objects)

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210969



--- Comment #1 from Ralph Bean rb...@redhat.com ---
Scratch build:  http://koji.fedoraproject.org/koji/taskinfo?taskID=9462321

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

[Bug 1160672] Review Request: seafile-client - Seafile cloud storage desktop client

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1160672



--- Comment #1 from Nikos Roussos comzer...@fedoraproject.org ---
updated to 4.1.4

Spec: https://comzeradd.fedorapeople.org/specs/seafile-client.spec
SRPM:
https://comzeradd.fedorapeople.org/srpms/seafile-client-4.1.4-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 1160671] Review Request: seafile - Cloud storage system

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1160671



--- Comment #4 from Nikos Roussos comzer...@fedoraproject.org ---
Update to 4.1.4

SPEC: https://comzeradd.fedorapeople.org/specs/seafile.spec
SRPM: https://comzeradd.fedorapeople.org/srpms/seafile-4.1.4-1.fc21.src.rpm

Also address some things from above comment:

2. Use release tag instead of commit
3. Packaging guidelines discourage use of %makeinstall
https://fedoraproject.org/wiki/Packaging:Guidelines#Why_the_.25makeinstall_macro_should_not_be_used
4. Enabled hardened build
5. Use %license for the LICENSE text
6. .desktop is part of the seafile-client package
7. Fixed man page naming

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

[Bug 1210969] Review Request: python-munch - A dot-accessible dictionary (a la JavaScript objects)

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210969

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 CC||pin...@pingoured.fr



--- Comment #2 from Pierre-YvesChibon pin...@pingoured.fr ---
You can drop the python2_sitearch definition at the top since you don't need
it.

Otherwise spec file looks clean

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

[Bug 1201925] Review Request: gnome-shell-extension-touchpad-indicator - Minimalistic Touchpad management extension for the Gnome Shell

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1201925



--- Comment #3 from Alexander Ploumistos alex.ploumis...@gmail.com ---
(In reply to Ville Skyttä from comment #2)
 The comment and %files section breakdown etc are in the ...actual
 implementation of this is left to the maintainer. Some suggested
 implementations include... category, so I think the comment is enough
 (they're suggested, not mandatory, and not all of them need to be done
 anyway).

Fine by me, no need for nitpicking.

Have you by any chance tested the extension with libinput?

https://fedoraproject.org/wiki/Changes/LibinputForXorg

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

[Bug 1210972] New: Review Request: libkgeomap - A wrapper around different world-map components, to browse and arrange photos over a map

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210972

Bug ID: 1210972
   Summary: Review Request: libkgeomap - A wrapper around
different world-map components, to browse and arrange
photos over a map
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: rdie...@math.unl.edu
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://rdieter.fedorapeople.org/rpms/kf5/libkgeomap.spec
SRPM URL:
https://rdieter.fedorapeople.org/rpms/kf5/libkgeomap-15.04.0-1.fc22.src.rpm
Description: A wrapper around different world-map components, to browse and
arrange photos over a map
Fedora Account System Username: rdieter

scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=9462446

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

[Bug 1210972] Review Request: libkgeomap - A wrapper around different world-map components, to browse and arrange photos over a map

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210972

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

   What|Removed |Added

 Blocks||656997 (kde-reviews)
  Alias||libkgeomap




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=656997
[Bug 656997] kde-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 1210971] Review Request: libkface - A face recognition and detection library

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210971

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

   What|Removed |Added

 Blocks||656997 (kde-reviews)
  Alias||libkface




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=656997
[Bug 656997] kde-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 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=921134



--- Comment #23 from Denis Arnaud denis.arnaud_fed...@m4x.org ---
(In reply to Denis Arnaud from comment #21)
 (In reply to Rex Dieter from comment #20)
  The el6 failure is an undefined macro:
  + '%{_mpich2_load}'
  /var/tmp/rpm-tmp.pLSGl7: line 97: fg: no job control
 
 The only thing I do not understand, then, is why it used to work before. For
 instance,
 http://denisarnaud.fedorapeople.org/boost/boost148/boost148-1.48.0-4.spec
 contains the exact same macro (%{_mpich2_load})...


As a matter of fact, MPICH2 has been upgraded into MPICH v3. So, the macro is
now simply named _mpich_load (the '2' suffix has been dropped).

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

[Bug 1210969] Review Request: python-munch - A dot-accessible dictionary (a la JavaScript objects)

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210969

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Docs Contact||pin...@pingoured.fr
  Flags||fedora-review?



--- Comment #3 from Pierre-YvesChibon pin...@pingoured.fr ---
* Using the sources from upstream:
SRPM: 68fd192e64a22a69d445a74d84da640cb2314e9571114836864a7b5c06d1454d  
Upstream: 68fd192e64a22a69d445a74d84da640cb2314e9571114836864a7b5c06d1454d

* License MIT is valid, present in the sources and installed correctly in the
spec file

* Spec file looks good

! rpmlint output can be improved:
  python-munch.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/munch/__init__.py 0644L /usr/bin/env
  python3-munch.noarch: E: non-executable-script
/usr/lib/python3.4/site-packages/munch/__init__.py 0644L /usr/bin/env
  3 packages and 0 specfiles checked; 2 errors, 0 warnings.

Maybe
https://fedoraproject.org/wiki/Packaging_tricks#Remove_shebang_from_Python_libraries
could help

After this I'll approve the 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 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=921134



--- Comment #24 from Denis Arnaud denis.arnaud_fed...@m4x.org ---
Successful build on EPEL 6:
http://koji.fedoraproject.org/koji/taskinfo?taskID=9463281

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

[Bug 1210993] Review Request: boost157 - The free peer-reviewed portable C++ source libraries

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210993



--- Comment #1 from Denis Arnaud denis.arnaud_fed...@m4x.org ---
Successful build on EPEL 7:
http://koji.fedoraproject.org/koji/taskinfo?taskID=9463282

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

[Bug 1129429] Review Request: ccnet - A framework for writing networked applications in C

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1129429



--- Comment #4 from Nikos Roussos comzer...@fedoraproject.org ---
(In reply to Vladimir Stackov from comment #2)
 Your packages (seafile and ccnet) both introduces build time requirements
 (cross-package BuildRequires) so you should setup boostraping properly:
 https://fedoraproject.org/wiki/Packaging:Guidelines#Bootstrapping

I'm not familiar with Bootsrapping. Both seafile and ccnet depend on libsearpc.
How this is a circular dependency?

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

[Bug 1205376] Review Request: spooky-c - C port of Bob Jenkins' spooky hash algorithm

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1205376



--- Comment #15 from Jeff Layton jlay...@poochiereds.net ---
I haven't forgotten about this, but I haven't been able to get Andi to respond
to my efforts to clarify the licensing yet. I'm hoping that it's just because
he's busy.

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

[Bug 1210430] Review Request: yumex-dnf - graphical package manager powered by dnf

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210430

Florian der-flo Lehner d...@der-flo.net changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #6 from Florian der-flo Lehner d...@der-flo.net ---
Hi Tim!

Looks good :)

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:
 *No copyright* GPL (v2 or later), GPL (v2 or later), Unknown or
 generated. 2 files have unknown license.
[x]: Package requires other packages for directories it uses.
 Note: No known owner of /usr/share/locale/cmn,
 /usr/share/locale/cmn/LC_MESSAGES
[x]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/share/locale/cmn,
 /usr/share/dbus-1, /usr/share/locale/cmn/LC_MESSAGES,
 /usr/share/dbus-1/services
[x]: Package does not own files or directories owned by other packages.
 Note: Dirs in package are owned also by:
 /usr/share/icons/hicolor/scalable/apps(hicolor-icon-theme, fedora-logos),
 /usr/share/icons/hicolor(hicolor-icon-theme, fedora-logos),
 /usr/share/icons/hicolor/48x48(hicolor-icon-theme, fedora-logos),
 /usr/share/icons/hicolor/scalable(hicolor-icon-theme, fedora-logos),
 /usr/share/icons/hicolor/48x48/apps(hicolor-icon-theme, fedora-logos)
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: The spec file handles locales properly.
[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]: update-desktop-database is invoked in %post and %postun if package
 contains desktop file(s) with a MimeType: entry.
 Note: desktop file(s) with MimeType entry in yumex-dnf
[x]: gtk-update-icon-cache is invoked in %postun and %posttrans if package
 contains icons.
 Note: icons in yumex-dnf
[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 30720 bytes in 2 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]: 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 contains desktop file if it is a GUI application.
[x]: Package installs a %{name}.desktop using desktop-file-install or desktop-
 file-validate if there is such a file.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

Python:
[x]: Python eggs must not download any dependencies during the build process.
[x]: A package which is used by another package via an egg 

[Bug 1161483] Review Request: o3dgc - an open 3D graphics compression library

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1161483



--- Comment #4 from David Tardon dtar...@redhat.com ---
(In reply to Alexander Ploumistos from comment #2)
 I am away from my main system and I don't have koji setup here, so I could
 only test locally for x86_64. Have you tried building the package for other
 architectures?

http://copr.fedoraproject.org/coprs/dtardon/pending-review/build/83444/

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

[Bug 1129224] Review Request: libsearpc - A simple and easy-to-use C language RPC framework

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1129224



--- Comment #10 from Nikos Roussos comzer...@fedoraproject.org ---
Updated to use release tag instead of commit:
SPEC: https://comzeradd.fedorapeople.org/specs/libsearpc.spec
SRPM: https://comzeradd.fedorapeople.org/srpms/libsearpc-3.0-3.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 1208842] Review Request: gdouros-symbola-fonts - A symbol font

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1208842



--- Comment #7 from Alexander Ploumistos alex.ploumis...@gmail.com ---
And another one:
https://bugzilla.redhat.com/show_bug.cgi?id=1161483

I am a little worried that so far I've only noticed mostly licensing issues...

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

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=921134

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

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #20 from Rex Dieter rdie...@math.unl.edu ---
The el6 failure is an undefined macro:
+ '%{_mpich2_load}'
/var/tmp/rpm-tmp.pLSGl7: line 97: fg: no job control

I won't treat it as a review blocker here, since the epel-7 build succeeds, but
obviously this needs to be fixed if you want to support epel-6


Thanks.


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 1210990] Review Request: Qt5 - WebChannel component

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210990

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

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||rdie...@math.unl.edu
   Assignee|nob...@fedoraproject.org|rdie...@math.unl.edu
  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 1210990] Review Request: qt5-qtwebchannel - Qt5 WebChannel component

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210990

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

   What|Removed |Added

 Blocks||928937 (qt-reviews)
Summary|Review Request: Qt5 -   |Review Request:
   |WebChannel component|qt5-qtwebchannel - Qt5
   ||WebChannel component
  Alias||qt5-qtwebchannel




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 1210993] New: Review Request: boost157 - The free peer-reviewed portable C++ source libraries

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210993

Bug ID: 1210993
   Summary: Review Request: boost157 - The free peer-reviewed
portable C++ source libraries
   Product: Fedora EPEL
   Version: epel7
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: denis.arnaud_fed...@m4x.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://denisarnaud.fedorapeople.org/boost/boost157/boost157.spec
SRPM URL:
http://denisarnaud.fedorapeople.org/boost/boost157/boost157-1.57.0-1.src.rpm

Description:
A few packages require a fairly recent Boost version, which EPEL does not
provide. EPEL 5 and 6 have had Boost-1.41 (and EPEL Boost-1.53) for a while (as
a parallel installation for EPEL 5), thanks to Robert Scheck and the Zarafa
project. The very same way they built Boost-1.41 for EPEL 5, I therefore
propose to build Boost-1.57 for EPEL 6 and 7, as a parallel installation to the
currently officially supported Boost packages (Boost-1.31 on EPEL 5, Boost-1.41
on EPEL 6 and Boost-1.53 on EPEL 7).

To minimise the work (and the support!) to be required, I re-start from the
Boost-1.57 package (on Fedora 22), which cleanly builds on EPEL 7 without any
modification. For EPEL 5 and its MPI-related issues, I re-inject the
work-arounds of Boost141 and Boost148 (on EPEL 5). For EPEL 6, work-arounds of
Boost148 may have to be re-injected.

References:
* Review request for Boost-1.48 on EPEL 5 and 6:
http://bugzilla.redhat.com/show_bug.cgi?id=921134
* Review request for Boost-1.41 on EPEL 5:
http://bugzilla.redhat.com/show_bug.cgi?id=673839
* Feature request for Boost-1.57 on Fedora:
https://fedoraproject.org/wiki/Changes/F22Boost158

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

[Bug 1210990] Review Request: qt5-qtwebchannel - Qt5 WebChannel component

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210990

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

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Rex Dieter rdie...@math.unl.edu ---
$ rpmlint qt5-qtwebchannel qt5-qtwebchannel-devel
qt5-qtwebchannel.x86_64: W: incoherent-version-in-changelog 5.4.0-1
['5.4.1-1.fc22', '5.4.1-1']
qt5-qtwebchannel.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libQt5WebChannel.so.5.4.1 /lib64/libQt5Network.so.5
qt5-qtwebchannel.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libQt5WebChannel.so.5.4.1 /lib64/libpthread.so.0
qt5-qtwebchannel.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libQt5WebChannel.so.5.4.1 /lib64/libm.so.6
qt5-qtwebchannel.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libQt5WebChannel.so.5.4.1 /lib64/libgcc_s.so.1
qt5-qtwebchannel-devel.x86_64: W: only-non-binary-in-usr-lib
qt5-qtwebchannel-devel.x86_64: W: no-documentation
2 packages and 0 specfiles checked; 0 errors, 7 warnings.

mostly harmless

naming: ok, matches standard naming of other qt5- modules in distro

license: ok

sources: ok
$ md5sum *.xz
bb914134c164c6758a0074748cc678ff  qtwebchannel-opensource-src-5.4.1.tar.xz

builds/installs: ok

macros: ok

scriptlet: ok

looks like the qt5-qtwebsocket build dependency is optional and only for some
examples, but using it here is mostly harmless.  Probably worth adding a .spec
comment about this, in case it helps bootstrapping new archs.


Otherwise, thanks, looks good, 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 1177038] Review Request: qt5-qtwebsockets - Qt5 - WebSockets component

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1177038



--- Comment #16 from Fedora Update System upda...@fedoraproject.org ---
qt5-qtwebsockets-5.4.1-1.fc22 has been submitted as an update for Fedora 22.
https://admin.fedoraproject.org/updates/qt5-qtwebsockets-5.4.1-1.fc22

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

[Bug 1177038] Review Request: qt5-qtwebsockets - Qt5 - WebSockets component

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1177038



--- Comment #17 from Fedora Update System upda...@fedoraproject.org ---
qt5-qtwebsockets-5.4.1-1.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/qt5-qtwebsockets-5.4.1-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 1177038] Review Request: qt5-qtwebsockets - Qt5 - WebSockets component

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1177038



--- Comment #18 from Fedora Update System upda...@fedoraproject.org ---
qt5-qtwebsockets-5.4.1-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/qt5-qtwebsockets-5.4.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 1210990] Review Request: qt5-qtwebchannel - Qt5 WebChannel component

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210990



--- Comment #2 from Rex Dieter rdie...@math.unl.edu ---
If you don't mind, please add group:kde-sig as comaintainer for this package
when submitting scm request.

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

[Bug 1210990] Review Request: qt5-qtwebchannel - Qt5 WebChannel component

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210990



--- Comment #3 from Taylor Braun-Jones tay...@braun-jones.org ---
(In reply to Rex Dieter from comment #1)
 looks like the qt5-qtwebsocket build dependency is optional and only for
 some examples, but using it here is mostly harmless.  Probably worth adding
 a .spec comment about this, in case it helps bootstrapping new archs.

Fixed with comment explanation.

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

[Bug 1194798] Review Request: GeoIP-GeoLite-data - Free GeoLite IP geolocation country database

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1194798



--- Comment #17 from Philip Prindeville phil...@redfish-solutions.com ---
(In reply to Paul Howarth from comment #16)
 Ping?

Just reran the review tool, and the sizes changed again.  Looks like you ran
this on 1 April, and upstream updated again on 5 April.  Real moving target!

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

[Bug 1210990] Review Request: qt5-qtwebchannel - Qt5 WebChannel component

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210990

Taylor Braun-Jones tay...@braun-jones.org changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #4 from Taylor Braun-Jones tay...@braun-jones.org ---
New Package SCM Request
===
Package Name: qt5-qtwebchannel
Short Description: Qt5 WebChannel component
Upstream URL: http://qt.io
Owners: nocnokneo group:kde-sig
Branches: f20 f21 f22 el6 epel7
InitialCC:

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

[Bug 1210990] Review Request: qt5-qtwebchannel - Qt5 WebChannel component

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210990



--- Comment #6 from Rex Dieter rdie...@math.unl.edu ---
New Package SCM Request
===
Package Name: qt5-qtwebchannel
Short Description: Qt5 WebChannel component
Upstream URL: http://qt.io
Owners: nocnokneo group::kde-sig
Branches: f20 f21 f22 el6 epel7
InitialCC:

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

[Bug 1210990] Review Request: qt5-qtwebchannel - Qt5 WebChannel component

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1210990



--- Comment #5 from Rex Dieter rdie...@math.unl.edu ---
Sorry, checking other reviews, looks like the correct syntax is group::kde-sig

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

[Bug 1161483] Review Request: o3dgc - an open 3D graphics compression library

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1161483



--- Comment #5 from Alexander Ploumistos alex.ploumis...@gmail.com ---
The latest link to the spec file still points to the one from 0-2, not the one
in your source rpm and the scripts will whine about that.

I did the exact same thing a few days ago, that's why I checked :)

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

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=921134



--- Comment #21 from Denis Arnaud denis.arnaud_fed...@m4x.org ---
(In reply to Rex Dieter from comment #20)
 The el6 failure is an undefined macro:
 + '%{_mpich2_load}'
 /var/tmp/rpm-tmp.pLSGl7: line 97: fg: no job control

Thanks!

The only thing I do not understand, then, is why it used to work before. For
instance,
http://denisarnaud.fedorapeople.org/boost/boost148/boost148-1.48.0-4.spec
contains the exact same macro (%{_mpich2_load})...

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

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=921134

Denis Arnaud denis.arnaud_fed...@m4x.org changed:

   What|Removed |Added

  Flags||fedora-cvs+



--- Comment #22 from Denis Arnaud denis.arnaud_fed...@m4x.org ---
New Package SCM Request
===
Package Name: boost148
Short Description: The free peer-reviewed portable C++ source libraries
Upstream URL: http://www.boost.org/users/history/version_1_48_0.html
Owners: denisarnaud
Branches: f20 f21 f22 el5 el6 epel7
InitialCC: rdieter

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

[Bug 921134] Review Request: boost148 - The free peer-reviewed portable C++ source libraries

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=921134

Denis Arnaud denis.arnaud_fed...@m4x.org 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 1202146] Review Request: python-munkres - A Munkres algorithm for Python

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1202146

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

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #9 from Fedora Update System upda...@fedoraproject.org ---
python-munkres-1.0.7-1.fc22 has been pushed to the Fedora 22 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 1177038] Review Request: qt5-qtwebsockets - Qt5 - WebSockets component

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1177038

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||qt5-qtwebsockets-5.4.0-1.el
   ||7
 Resolution|--- |ERRATA
Last Closed||2015-04-11 22:44:37



--- Comment #19 from Fedora Update System upda...@fedoraproject.org ---
qt5-qtwebsockets-5.4.0-1.el7 has been pushed to the Fedora EPEL 7 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 1177038] Review Request: qt5-qtwebsockets - Qt5 - WebSockets component

2015-04-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1177038

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

   What|Removed |Added

   Fixed In Version|qt5-qtwebsockets-5.4.0-1.el |qt5-qtwebsockets-5.4.0-1.el
   |7   |6



--- Comment #20 from Fedora Update System upda...@fedoraproject.org ---
qt5-qtwebsockets-5.4.0-1.el6 has been pushed to the Fedora EPEL 6 stable
repository.  If problems still persist, please make note of it in this bug
report.

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