[Bug 1269056] Review Request: php-pecl-mongodb - MongoDB driver for PHP

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1269056



--- Comment #11 from Remi Collet  ---
Update to 1.1.6

Spec:
https://raw.githubusercontent.com/remicollet/remirepo/cc9e722b354cfd00cf9df337af9f2bf9504b052a/php/pecl/php-pecl-mongodb/php-pecl-mongodb.spec
Srpm: http://rpms.remirepo.net/SRPMS/php-pecl-mongodb-1.1.6-1.remi.src.rpm

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

[Bug 1321438] Review Request: python-breathe - Doxygen xml output to reStructuredText

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1321438



--- Comment #9 from Fedora Update System  ---
python-breathe-4.2.0-1.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-c26ee51214

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

[Bug 1321438] Review Request: python-breathe - Doxygen xml output to reStructuredText

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1321438



--- Comment #8 from Fedora Update System  ---
python-breathe-4.2.0-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-1ffe139ba7

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

[Bug 1321438] Review Request: python-breathe - Doxygen xml output to reStructuredText

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1321438

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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1321438] Review Request: python-breathe - Doxygen xml output to reStructuredText

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1321438



--- Comment #10 from Fedora Update System  ---
python-breathe-4.2.0-1.fc22 has been submitted as an update to Fedora 22.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-9eabd21ea8

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

[Bug 1319714] Review Request: golang-github-cheggaaa-pb - Console progress bar for Golang

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1319714



--- Comment #14 from Fedora Update System  ---
golang-github-cheggaaa-pb-0-0.1.gitda1f27a.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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1319726] Review Request: golang-github-kballard-go-shellquote - Go utilities for performing shell-like word splitting/joining

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1319726



--- Comment #13 from Fedora Update System  ---
golang-github-kballard-go-shellquote-0-0.1.gitd8ec1a6.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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1319715] Review Request: golang-github-mattn-go-runewidth - Functions for getting fixed width of the character or string

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1319715



--- Comment #14 from Fedora Update System  ---
golang-github-mattn-go-runewidth-0-0.1.gitd6bea18.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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1319720] Review Request: golang-github-akrennmair-gopcap - A simple wrapper around libpcap for the Go programming language

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1319720



--- Comment #14 from Fedora Update System  ---
golang-github-akrennmair-gopcap-0-0.1.git00e1103.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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1319724] Review Request: golang-github-coreos-gexpect - Pure golang expect library, for easily starting and controlling subprocesses

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1319724



--- Comment #14 from Fedora Update System  ---
golang-github-coreos-gexpect-0-0.1.git5173270.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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1319716] Review Request: golang-github-olekukonko-ts - Simple go Application to get Terminal Size

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1319716



--- Comment #14 from Fedora Update System  ---
golang-github-olekukonko-ts-0-0.1.gitecf753e.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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1308779] Review Request: git-tools - Assorted git-related scripts and tools

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1308779



--- Comment #32 from Fedora Update System  ---
git-tools-0-0.2.20160313gitd6d55b3.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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1319730] Review Request: golang-github-spacejam-loghisto - Counters and logarithmically bucketed histograms for distributed systems

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1319730



--- Comment #14 from Fedora Update System  ---
golang-github-spacejam-loghisto-0-0.1.git3233097.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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1307127] Review Request: mkdocs-bootswatch - Bootswatch themes for MkDocs

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1307127



--- Comment #3 from William Moreno  ---
Thanks for the review, I have updated the spec to the lastes upstream release
and fixfed the 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1323871] Review Request: phonon-qt5-backend-gstreamer - Gstreamer phonon-qt5 backend

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1323871

William Moreno  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #4 from William Moreno  ---
Looks good for me, aproved.

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

[Bug 1308479] Review Request: chck - Collection of C utilities

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1308479



--- Comment #5 from Denis Fateyev  ---
Package Review
==

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


Issues:
===
- Development (unversioned) .so files in -devel subpackage, if present.
  Note: Unversioned so-files directly in %_libdir.
  See: http://fedoraproject.org/wiki/Packaging/Guidelines#DevelPackages


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

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Unknown or generated", "zlib/libpng". 53 files have unknown
 license. Detailed output of licensecheck in
 /home/mock/sandbox/review/1308479-chck/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package requires other packages for directories it uses.
 Note: No known owner of /usr/include/chck
[!]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/include/chck
[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.
[-]: Package contains desktop file if it is a GUI application.
[x]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[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 10240 bytes in 1 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 %license.
[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:
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in chck-
 debuginfo
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: Scriptlets 

[Bug 1324270] Review Request: python-spur - Run commands locally or over SSH using the same interface

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324270

Orion Poplawski  changed:

   What|Removed |Added

 Blocks||1324667




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1324667
[Bug 1324667] Review Request: python-locket - File-based locks for Python
for Linux and Windows
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1324667] Review Request: python-locket - File-based locks for Python for Linux and Windows

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324667

Orion Poplawski  changed:

   What|Removed |Added

 Depends On||1324270




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1324270
[Bug 1324270] Review Request: python-spur - Run commands locally or over
SSH using the same interface
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1324667] New: Review Request: python-locket - File-based locks for Python for Linux and Windows

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324667

Bug ID: 1324667
   Summary: Review Request: python-locket - File-based locks for
Python for Linux and Windows
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: or...@cora.nwra.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://www.cora.nwra.com/~orion/fedora/python-locket.spec
SRPM URL:
http://www.cora.nwra.com/~orion/fedora/python-locket-0.2.0-1.fc25.src.rpm
Description:
Locket implements a lock that can be used by multiple processes provided they
use the same path.

Locks largely behave as (non-reentrant) Lock instances from the threading
module in the standard library. Specifically, their behaviour is:

 * Locks are uniquely identified by the file being locked, both in the same
   process and across different processes.
 * Locks are either in a locked or unlocked state.
 * When the lock is unlocked, calling acquire() returns immediately and
   changes the lock state to locked.
 * When the lock is locked, calling acquire() will block until the lock state
   changes to unlocked, or until the timeout expires.
 * If a process holds a lock, any thread in that process can call release()
   to change the state to unlocked.
 * Behaviour of locks after fork is undefined.

Fedora Account System Username: orion

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

[Bug 1304097] Review Request: ctk - the Common Toolkit for biomedical imaging

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1304097

Dmitry Mikhirev  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2016-04-06 18:20: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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1297977] Review Request: python3-Cython - A language for writing Python 3 extension modules

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1297977

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-04-06 17:57:35



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

[Bug 1297977] Review Request: python3-Cython - A language for writing Python 3 extension modules

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1297977



--- Comment #9 from Fedora Update System  ---
python3-Cython-0.23.5-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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1223887] Review Request: atomicapp - Reference implementation of the Nulecule container application Specification

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1223887

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #23 from Fedora Update System  ---
atomicapp-0.4.5-1.el7 has been pushed to the Fedora EPEL 7 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-5d93f1948c

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

[Bug 1318358] Review Request: copr-dist-git - Copr services for Dist Git server

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1318358



--- Comment #11 from Fedora Update System  ---
copr-dist-git-0.15-1.el7 has been pushed to the Fedora EPEL 7 testing
repository. If problems still persist, please make note of it in this bug
report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-c3816edcbf

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

[Bug 1324636] New: Review Request: python3-dbus - D-Bus Python 3 Bindings

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324636

Bug ID: 1324636
   Summary: Review Request: python3-dbus - D-Bus Python 3 Bindings
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: or...@cora.nwra.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://www.cora.nwra.com/~orion/fedora/python3-dbus.spec
SRPM URL:
http://www.cora.nwra.com/~orion/fedora/python3-dbus-1.2.4-1.el7.src.rpm
Description: 
D-Bus Python 3 Bindings.

Fedora Account System Username: orion

This is an EPEL only package.

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

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

[Bug 1294904] Review Request: python-rpm-macros - The unversioned Python RPM macros

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1294904

Orion Poplawski  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2016-04-06 16:39:42



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

[Bug 1323871] Review Request: phonon-qt5-backend-gstreamer - Gstreamer phonon-qt5 backend

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1323871

Rex Dieter  changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |phonon-qt5-gstreamer -  |phonon-qt5-backend-gstreame
   |Gstreamer phonon-qt5|r - Gstreamer phonon-qt5
   |backend |backend
  Alias|phonon-qt5-gstreamer|phonon-qt5-backend-gstreame
   ||r



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

[Bug 1323871] Review Request: phonon-qt5-gstreamer - Gstreamer phonon-qt5 backend

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1323871

Rex Dieter  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED



--- Comment #3 from Rex Dieter  ---
SPEC Url:
https://rdieter.fedorapeople.org/rpms/phonon-qt5/phonon-qt5-backend-gstreamer.spec

SRPM Url:
https://rdieter.fedorapeople.org/rpms/phonon-qt5/phonon-qt5-backend-gstreamer-4.8.2-2.fc24.src.rpm


%changelog
* Wed Apr 06 2016 Rex Dieter  - 4.8.2-2
- rename phonon-backend-gstreamer => phonon-qt5-backend-gstreamer

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

[Bug 1310294] Review Request: sqlcipher - An SQLite extension that provides 256 bit AES encryption of database files

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1310294

Siddharth Sharma  changed:

   What|Removed |Added

  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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1316762] Review Request: erlang-iconv - Fast encoding conversion library for Erlang / Elixir

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1316762

Randy Barlow  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||erlang-iconv-1.0.0-4.fc25
 Resolution|--- |RAWHIDE
Last Closed||2016-04-06 15:36:29



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

[Bug 1221781] Review Request: zipios - C++ library for reading and writing Zip files

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1221781



--- Comment #44 from Richard Shaw  ---
I'm not sure why but my build completed:

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

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

[Bug 1323871] Review Request: phonon-qt5-gstreamer - Gstreamer phonon-qt5 backend

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1323871



--- Comment #2 from William Moreno  ---
Package Review
==

1- Package installs properly.
   Note: Installation errors (see attachment)
   See: https://fedoraproject.org/wiki/Packaging:Guidelines
   [!]: Rpmlint is run on all installed packages.
   I know this is intended to be a epel only package

2- Spec file name must match the spec package %{name}, in the format
   %{name}.spec.
   There is a package phonon-backend-gstreamer in Fedora, if this is a epel
only package there will be no conflic but remane the package to se same name
please.
   https://admin.fedoraproject.org/pkgdb/package/rpms/phonon-backend-gstreamer/
   [!]: Package does not generate any conflict.
   [!]: Spec file according to URL is the same as in SRPM.
   phonon-backend-gstreamer.src: E: invalid-spec-name

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

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
[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.
[-]: 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 obeys FHS, except libexecdir and /usr/target.
[x]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: 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 %license.
[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 is not relocatable.
[x]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: File names are valid UTF-8.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 0 bytes in 0 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =
Generic:
[-]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Patches link to upstream bugs/comments/lists or are otherwise
 justified.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[-]: %check 

[Bug 1324367] Review Request: libkml - Reference implementation of OGC KML 2.2

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324367

Roman Tsisyk  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||ro...@tsisyk.com
   Assignee|nob...@fedoraproject.org|ro...@tsisyk.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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1246792] Review Request: ndctl - Manage "libnvdimm" subsystem devices (Non-volatile Memory)

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1246792



--- Comment #26 from Dan Williams  ---
Here's the result of a fedora-review run with the rest of the template filled
out and a note about ndctl-devel rpmlint warnings.

C/C++:
[-]: Provides: bundled(gnulib) in place as required.
 Note: Sources not installed
[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.
[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.
[-]: Package contains desktop file if it is a GUI application.
[x]: 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]: Useful -debuginfo package or justification otherwise.
[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 10240 bytes in 1 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 %license.
[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:
[x]: Sources can be downloaded from URI in Source: tag
[-]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: Scriptlets must be sane, if used.
[x]: SourceX tarball generation or download is documented.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[x]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
 files.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Reviewer should test that 

[Bug 1318358] Review Request: copr-dist-git - Copr services for Dist Git server

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1318358



--- Comment #10 from Fedora Update System  ---
copr-dist-git-0.15-1.fc23 has been pushed to the Fedora 23 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-49331c598d

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

[Bug 1287837] Review Request: borgbackup - A deduplicating backup program

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1287837



--- Comment #27 from Fedora Update System  ---
borgbackup-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.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-bd1515f44d

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

[Bug 1324590] Review Request: hfi1-psm - Intel PSM Libraries

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324590



--- Comment #1 from paul.j.re...@intel.com ---
My Koji build is:

https://koji.fedoraproject.org/koji/taskinfo?taskID=13576497

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

[Bug 1323466] Review Request: python-orderedmultidict - Ordered Multivalue Dictionary

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1323466

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #4 from Fedora Update System  ---
python-furl-0.4.93-1.fc24, python-orderedmultidict-0.7.6-1.fc24 has been pushed
to the Fedora 24 testing repository. If problems still persist, please make
note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-cb83d1d06c

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

[Bug 1323468] Review Request: python-furl - URL manipulation made simple

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1323468

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #4 from Fedora Update System  ---
python-furl-0.4.93-1.fc24, python-orderedmultidict-0.7.6-1.fc24 has been pushed
to the Fedora 24 testing repository. If problems still persist, please make
note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-cb83d1d06c

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

[Bug 1318358] Review Request: copr-dist-git - Copr services for Dist Git server

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1318358

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #9 from Fedora Update System  ---
copr-dist-git-0.15-1.fc24 has been pushed to the Fedora 24 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-371842c47d

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

[Bug 1287837] Review Request: borgbackup - A deduplicating backup program

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1287837

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #26 from Fedora Update System  ---
borgbackup-1.0.0-1.fc24 has been pushed to the Fedora 24 testing repository. If
problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-4ffaff98a7

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

[Bug 1324590] New: Review Request: hfi1-psm - Intel PSM Libraries

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324590

Bug ID: 1324590
   Summary: Review Request: hfi1-psm - Intel PSM Libraries
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: paul.j.re...@intel.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://github.com/01org/opa-psm2/releases/tag/10_1
SRPM URL: https://github.com/01org/opa-psm2/releases/tag/10_1
Description: The PSM Messaging API, or PSM API, is the low-level
user-level communications interface for the Intel(R) OPA
family of products. PSM users are enabled with mechanisms
necessary to implement higher level communications
interfaces in parallel environments.
Fedora Account System Username: pjreger

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

[Bug 1316912] Review Request: php-sabre-xml - XML library that you may not hate

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1316912

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed|2016-03-29 15:35:27 |2016-04-06 13:21: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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1297411] Review Request: pdc-client - Client library and console client for Product Definition Center

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1297411

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed|2016-03-09 09:28:21 |2016-04-06 13:21:14



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

[Bug 1316912] Review Request: php-sabre-xml - XML library that you may not hate

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1316912



--- Comment #20 from Fedora Update System  ---
php-sabre-xml-1.4.1-1.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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1297411] Review Request: pdc-client - Client library and console client for Product Definition Center

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1297411



--- Comment #19 from Fedora Update System  ---
pdc-client-0.9.0-1.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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1291558] Review Request: mariadb-java-client - MariaDB connector for java

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1291558

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-04-06 13:20: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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1291558] Review Request: mariadb-java-client - MariaDB connector for java

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1291558



--- Comment #20 from Fedora Update System  ---
mariadb-java-client-1.3.7-1.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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1317306] Review Request: legofy - Make images look as LEGO blocks

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1317306

William Moreno  changed:

   What|Removed |Added

 CC||omarberroteran...@gmail.com
  Flags||needinfo?(omarberroteranlkf
   ||@gmail.com)



--- Comment #8 from William Moreno  ---
Any update?

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

[Bug 1308561] Review Request: sway - i3-compatible window manager for Wayland

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1308561



--- Comment #1 from Fabio Alessandro Locati  ---
I've packaged the version 0.3 (current release):

SPEC: https://fale.fedorapeople.org/rpms/sway.spec
SRPM: https://fale.fedorapeople.org/rpms/sway-0.3-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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1308480] Review Request: wlc - Wayland compositor library

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1308480



--- Comment #1 from Fabio Alessandro Locati  ---
I upgraded to the current released version:

SPEC: https://fale.fedorapeople.org/rpms/wlc.spec
SRPM: https://fale.fedorapeople.org/rpms/wlc-0.0.1-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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1249094] Review Request: nrn - A flexible and powerful simulator of neurons and networks

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1249094



--- Comment #2 from Ankur Sinha (FranciscoD)  ---
I do plan to, but I'm caught up with work at the moment with Nest - sort of why
iv has stalled - I don't have enough cycles right 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1308479] Review Request: chck - Collection of C utilities

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1308479



--- Comment #4 from Fabio Alessandro Locati  ---
Hello,

Sorry for the delay.

I've packed a new version:
SPEC: https://fale.fedorapeople.org/rpms/chck.spec
SRPM:
https://fale.fedorapeople.org/rpms/chck-0-1.20160406git2efd6cd.fc23.src.rpm

I've tried to get a an "official" release from upstream, but they are still
skeptical, but they have released 0.1 version for wlc and sway (the two
packages that depend on this one), so I guess a release for chck will arrive at
some point in future, but I would prefer not to wait it.

As for the test problem, I'm not able to reproduce it with the current 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1310128] Review Request: zpaq - Incremental journaling back-up archiver

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1310128

Petr Pisar  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||zpaq-7.05-1.fc25
 Resolution|--- |RAWHIDE
Last Closed||2016-04-06 11:42:20



--- Comment #16 from Petr Pisar  ---
Thank you for the review and the repository.

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

[Bug 1249094] Review Request: nrn - A flexible and powerful simulator of neurons and networks

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1249094

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 CC||zbys...@in.waw.pl



--- Comment #1 from Zbigniew Jędrzejewski-Szmek  ---
I'm pretty sure the package should be called neuron. I know the tarball is
called nrn, but afaik, everybody thinks of the project as "neuron".

BTW., do you still plan to work on this? The iv review seems stalled.

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

[Bug 1316186] Review Request: python-librosa - a python package for music and audio analysis

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1316186



--- Comment #19 from Miro Hrončok  ---
I think it's ok. Also delete .buildinfo.

Once done, I'll recheck all other issues, but other than that, it seems OK.
Didn't try to build it 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1305650] Review Request: uispec4j - Java/Swing GUI testing made simple

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1305650

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed|2016-03-24 18:47:51 |2016-04-06 10:08: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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1305650] Review Request: uispec4j - Java/Swing GUI testing made simple

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1305650



--- Comment #26 from Fedora Update System  ---
uispec4j-2.5-0.2.20150412gitfdc0b42.fc24 has been pushed to the Fedora 24
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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1305365] Review Request: concurrentunit - Toolkit for testing multi-threaded and asynchronous applications

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1305365

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-04-06 10:06:53



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

[Bug 1246790] Review Request: mdp - Minimalist password safe

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1246790

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed|2016-03-23 10:16:22 |2016-04-06 10:06:59



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

[Bug 1305365] Review Request: concurrentunit - Toolkit for testing multi-threaded and asynchronous applications

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1305365



--- Comment #19 from Fedora Update System  ---
concurrentunit-0.4.2-2.fc24 has been pushed to the Fedora 24 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1246790] Review Request: mdp - Minimalist password safe

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1246790



--- Comment #41 from Fedora Update System  ---
mdp-0.7.4-4.fc24 has been pushed to the Fedora 24 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1305547] Review Request: lyra - High availability RabbitMQ client

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1305547
Bug 1305547 depends on bug 1305365, which changed state.

Bug 1305365 Summary: Review Request: concurrentunit - Toolkit for testing 
multi-threaded and asynchronous applications
https://bugzilla.redhat.com/show_bug.cgi?id=1305365

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA



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

[Bug 1316186] Review Request: python-librosa - a python package for music and audio analysis

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1316186



--- Comment #18 from Dominika Krejčí  ---
Is it ok just delete fonts after the build of the documentation?

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

[Bug 1322168] Review Request: ibacm - InfiniBand Communication Manager Assistant

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1322168

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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1322168] Review Request: ibacm - InfiniBand Communication Manager Assistant

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1322168



--- Comment #12 from Fedora Update System  ---
ibacm-1.2.0-3.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-256afff9e8

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

[Bug 1322168] Review Request: ibacm - InfiniBand Communication Manager Assistant

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1322168



--- Comment #13 from Fedora Update System  ---
ibacm-1.2.0-3.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-a5f422b859

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

[Bug 1319451] Review Request: mbassador - a light-weight and high-performance message (event) bus implementation

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1319451



--- Comment #10 from Fedora Update System  ---
mbassador-1.2.4.1-2.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-edbdcb1e7f

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

[Bug 1319451] Review Request: mbassador - a light-weight and high-performance message (event) bus implementation

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1319451

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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1319451] Review Request: mbassador - a light-weight and high-performance message (event) bus implementation

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1319451



--- Comment #9 from Fedora Update System  ---
mbassador-1.2.4.1-2.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-bc3faf67d8

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

[Bug 1324477] Review Request: rubygem-cucumber-wire - Wire protocol for Cucumber

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324477



--- Comment #2 from Vít Ondruch  ---
Please note that the rubygem-cucumber-core was already imported into 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1324063] Review Request: rubygem-cucumber-core - Core library for the Cucumber BDD app

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324063

Vít Ondruch  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||rubygem-cucumber-core-1.4.0
   ||-1.fc25
 Resolution|--- |RAWHIDE
Last Closed||2016-04-06 09:10: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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1310092] Review Request: cryptobone - Secure Communication Under Your Control

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1310092

Richard Shaw  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|hobbes1...@gmail.com



--- Comment #36 from Richard Shaw  ---
Too late but to make it correct, I'm assigning the bug to myself.

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

[Bug 1324063] Review Request: rubygem-cucumber-core - Core library for the Cucumber BDD app

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324063



--- Comment #4 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/rubygem-cucumber-core

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

[Bug 1310128] Review Request: zpaq - Incremental journaling back-up archiver

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1310128



--- Comment #15 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/zpaq

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

[Bug 1310092] Review Request: cryptobone - Secure Communication Under Your Control

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1310092



--- Comment #35 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/cryptobone

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

[Bug 1321438] Review Request: python-breathe - Doxygen xml output to reStructuredText

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1321438



--- Comment #7 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/python-breathe

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

[Bug 1324477] Review Request: rubygem-cucumber-wire - Wire protocol for Cucumber

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324477



--- Comment #1 from Vít Ondruch  ---
I should probably provide the link to plain text .spec file, so here it is:

https://fedorapeople.org/cgit/vondruch/public_git/rubygem-cucumber-wire.git/plain/rubygem-cucumber-wire.spec?id=5b37e754717ceeaa505ff1a27d5be1ebb2a03c8c

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

[Bug 1316186] Review Request: python-librosa - a python package for music and audio analysis

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1316186



--- Comment #17 from T. Orsava  ---
Besides that, everything else checks out?

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

[Bug 1324477] Review Request: rubygem-cucumber-wire - Wire protocol for Cucumber

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324477

Vít Ondruch  changed:

   What|Removed |Added

 Blocks||1124273




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1124273
[Bug 1124273] rubygem-cucumber-2.3.3 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1324477] New: Review Request: rubygem-cucumber-wire - Wire protocol for Cucumber

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324477

Bug ID: 1324477
   Summary: Review Request: rubygem-cucumber-wire - Wire protocol
for Cucumber
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: vondr...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://fedorapeople.org/cgit/vondruch/public_git/rubygem-cucumber-wire.git/tree/rubygem-cucumber-wire.spec?id=5b37e754717ceeaa505ff1a27d5be1ebb2a03c8c
SRPM URL:
http://people.redhat.com/vondruch/rubygem-cucumber-wire-0.0.1-1.fc25.src.rpm
Description: Wire protocol for Cucumber
Fedora Account System Username: vondruch

Please note that this has circular dependency with cucumber, so you'll need its
updated version together with its dependencies for testing. You might find
these scratch builds useful for testing:

http://koji.fedoraproject.org/koji/taskinfo?taskID=13564836
http://koji.fedoraproject.org/koji/taskinfo?taskID=13574243

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

[Bug 1316186] Review Request: python-librosa - a python package for music and audio analysis

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1316186



--- Comment #16 from Miro Hrončok  ---
It would be better to remove the bundled fonts, not to add their license to the
metadata.

See
https://fedoraproject.org/wiki/Packaging:Guidelines#Avoid_bundling_of_fonts_in_other_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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1324063] Review Request: rubygem-cucumber-core - Core library for the Cucumber BDD app

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324063

Vít Ondruch  changed:

   What|Removed |Added

 Blocks||1124273




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1124273
[Bug 1124273] rubygem-cucumber-2.3.3 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1324063] Review Request: rubygem-cucumber-core - Core library for the Cucumber BDD app

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324063



--- Comment #3 from Vít Ondruch  ---
(In reply to Pavel Valena from comment #2)
> The package looks good altogether. I have just one note:
> 
> I would not include the %{gem_instdir}/spec into the -doc package,
> as some of the tests are failing.

The test suite functionality was never the measure for its inclusion. We
include test suites even if it is not executed at all and nobody knows if it
works. We are also doing various changes to test suites to make them work, but
then don't include the changes in the package, because it serves just as a
documentation, not as a executable code.


Otherwise, thx for 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1304632] Review Request: libmlx5 - Mellanox Connect-IB InfiniBand HCA Userspace Driver

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1304632

Neil Horman  changed:

   What|Removed |Added

  Flags||needinfo?(kh...@redhat.com)



--- Comment #15 from Neil Horman  ---
ok, you're good, you can continue this by requesting git branches for the
package as per the review process

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

[Bug 1134342] Review Request: sasquatch - Modified unsquashfs utility supporting many SquashFS implementations

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1134342



--- Comment #9 from Charalampos Stratakis  ---
(In reply to Petr Viktorin from comment #8)
> It seems Matej and Robert have other priorities than Fedora now.
> 
> Harris, are you interested in this package?

I will take a look at it by next week and decide. Will not clear the needinfo
flag 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1324063] Review Request: rubygem-cucumber-core - Core library for the Cucumber BDD app

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324063

Pavel Valena  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #2 from Pavel Valena  ---
The package looks good altogether. I have just one note:

I would not include the %{gem_instdir}/spec into the -doc package,
as some of the tests are failing.

But that is not an issue, therefore

I APPROVE this package.


Package Review
==

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


Issues:
===
- None


= MUST items =

Generic:
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[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.
[x]: License file installed when any subpackage combination is installed.
[x]: Package must own all directories that it creates.
[x]: All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Package complies to the Packaging Guidelines
[x]: Rpmlint is run on all rpms the build produces.
[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 %license.
[x]: Package requires other packages for directories it uses.
[x]: Package does not own files or directories owned by other packages.
[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]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 0 bytes in 0 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

Ruby:
[x]: Platform dependent files must all go under %{gem_extdir_mri}, platform
 independent under %{gem_dir}.
[x]: Gem package must not define a non-gem subpackage
[x]: Macro %{gem_extdir} is deprecated.
[x]: Gem package is named rubygem-%{gem_name}
[x]: Package contains BuildRequires: rubygems-devel.
[x]: Gem package must define %{gem_name} macro.
[x]: Pure Ruby package must be built as noarch
[x]: Package does not contain Requires: ruby(abi).

= SHOULD items =

Generic:
[x]: Avoid bundling fonts in non-fonts packages.
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: %check is present and all tests pass.
  - Unindent dependency is not in fedora, therefore some tests
are failing.
[x]: Packages should try to preserve timestamps of original installed
 files.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded from URI in Source: tag
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: SourceX is a working URL.
[x]: Spec use %global instead of %define unless justified.

Ruby:
[x]: Gem package should exclude cached Gem.
[x]: Gem should use %gem_install macro.
[x]: Specfile should use macros from rubygem-devel package.
[x]: Test suite should not be run by rake.
[x]: Test suite of the library should be 

[Bug 1319451] Review Request: mbassador - a light-weight and high-performance message (event) bus implementation

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1319451

Michal Schmidt  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+
   |needinfo?(mschmidt@redhat.c |
   |om) |



--- Comment #7 from Michal Schmidt  ---
Looks good now. 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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1323128] Review Request: python-virtualenv-api - An API for virtualenv/pip

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1323128



--- Comment #3 from Petr Viktorin  ---
Almost good to go!

- The licence file should be specified with %licence, not %doc.

- Please add a comment for %Patch0, with either a link to upstream bug or an
explanation why you didn't propose the fix upstream

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

[Bug 1134342] Review Request: sasquatch - Modified unsquashfs utility supporting many SquashFS implementations

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1134342

Petr Viktorin  changed:

   What|Removed |Added

 CC||cstra...@redhat.com,
   ||pvikt...@redhat.com
  Flags||needinfo?(cstratak@redhat.c
   ||om)



--- Comment #8 from Petr Viktorin  ---
It seems Matej and Robert have other priorities than Fedora now.

Harris, are you interested in this package?

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

[Bug 1128480] Review Request: pipsi - Wraps pip and virtualenv to install scripts

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1128480

Petr Viktorin  changed:

   What|Removed |Added

 CC||cstra...@redhat.com,
   ||pvikt...@redhat.com
  Flags||needinfo?(cstratak@redhat.c
   ||om)



--- Comment #10 from Petr Viktorin  ---
It seems Matej and Robert have other priorities than Fedora now.

Harris, are you interested in this package?

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

[Bug 1134342] Review Request: sasquatch - Modified unsquashfs utility supporting many SquashFS implementations

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1134342

Jan Beran  changed:

   What|Removed |Added

   Assignee|rku...@redhat.com   |python-ma...@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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1128480] Review Request: pipsi - Wraps pip and virtualenv to install scripts

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1128480

Jan Beran  changed:

   What|Removed |Added

   Assignee|rku...@redhat.com   |python-ma...@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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1304632] Review Request: libmlx5 - Mellanox Connect-IB InfiniBand HCA Userspace Driver

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1304632

kamal heib  changed:

   What|Removed |Added

  Flags|needinfo?(kh...@redhat.com) |



--- Comment #14 from kamal heib  ---
(In reply to Neil Horman from comment #12)
> Kamal, Can you please confirm for me what your fedora account system
> username is?  I need that to sponsor you into the packager group

Hi Neil,

My fedora account system username is "kheib".

Thanks,
Kamal

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

[Bug 1324020] Review Request: cassandra - OpenSource database Apache Cassandra

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1324020

Tomas Repik  changed:

   What|Removed |Added

 Depends On|1306575 |




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1306575
[Bug 1306575] Review Request: disruptor-thrift-server - Thrift Server
implementation backed by LMAX Disruptor
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1306575] Review Request: disruptor-thrift-server - Thrift Server implementation backed by LMAX Disruptor

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1306575

Tomas Repik  changed:

   What|Removed |Added

 Blocks|652183 (FE-JAVASIG),|
   |1324020 |




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker bug
https://bugzilla.redhat.com/show_bug.cgi?id=1324020
[Bug 1324020] Review Request: cassandra - OpenSource database Apache
Cassandra
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1323128] Review Request: python-virtualenv-api - An API for virtualenv/pip

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1323128



--- Comment #2 from Michal Cyprian  ---
There was not special reason, I agree, Github archive is better choice in this
case. I have updated both files:
Spec URL: https://mcyprian.fedorapeople.org/python-virtualenv-api.spec
SRPM URL:
https://mcyprian.fedorapeople.org/python-virtualenv-api-2.1.8-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
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1316186] Review Request: python-librosa - a python package for music and audio analysis

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1316186



--- Comment #15 from Dominika Krejčí  ---
Hi, this should work.

I fixed licences of ogg file and fonts, used macro for version, removed hidden
file, created subpackage for docs and corrected names of Python2 packages
(joblib added).


Spec URL:
https://raw.githubusercontent.com/Krejdom/librosa-specfile/master/python-librosa.spec

SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/dkrejci/librosa/fedora-rawhide-x86_64/00172584-python-librosa/python-librosa-0.4.2-1.fc25.src.rpm

Copr link:
https://copr.fedorainfracloud.org/coprs/dkrejci/librosa/build/172584/

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