[Bug 1352972] Review Request: python-twine - Collection of utilities for interacting with PyPI

[bugzilla]

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

William Moreno  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #19 from William Moreno  ---
Looks good, package 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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1310368] review request: rubygem-jekyll - A simple, blog aware, static site generator

[bugzilla]

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

Dennis Chen  changed:

   What|Removed |Added

 CC||barracks...@gmail.com



--- Comment #3 from Dennis Chen  ---
This isn't a review request. Each one of those packages needs to undergo it's
own package review. I'll be happy to sponsor you and review each one of the
individual packages, but please file a separate package review for each of the
dependencies and link them to this bugzilla for easy tracking.

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

[Bug 1208911] Review Request: doublecmd - Twin-panel (commander-style) file manager

[bugzilla]

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

Bug 1247925 Summary: add ppc64le and aarch64 support for fpc
https://bugzilla.redhat.com/show_bug.cgi?id=1247925

   What|Removed |Added

 Status|CLOSED  |NEW
 Resolution|EOL |---



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

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

[bugzilla]

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



--- Comment #25 from Fedora Update System  ---
php-pecl-mongodb-1.1.8-3.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-b855865eef

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

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

[bugzilla]

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



--- Comment #24 from Fedora Update System  ---
php-pecl-mongodb-1.1.8-3.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-e331dcb6c7

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

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

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #23 from Fedora Update System  ---
php-pecl-mongodb-1.1.8-3.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-539597afca

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

[Bug 1351092] Review Request: python-geoip2 - MaxMind GeoIP2 API

[bugzilla]

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



--- Comment #8 from Fedora Update System  ---
python-geoip2-2.4.0-1.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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1344115] Review Request: winetricks - Work around common problems in Wine

[bugzilla]

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



--- Comment #27 from Fedora Update System  ---
winetricks-20160709-1.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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1325378] Review Request: spasm-ng - A z80 assembler with extra features for TI calculators

[bugzilla]

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



--- Comment #1 from Ben Rosser  ---
Having thought about the include file situation further... I believe they
probably should be removed from the package:

Their authorship and copyright is difficult to assert (most of the community
written files are not licensed, and ti83plus.inc was originally written by TI
and then heavily modified), most authors of TI calculator assembly projects
bundle them with their code anyway, and spasm does not expect include files to
be available system-wide to begin with. The AUR package for Arch makes a
similar decision and does not include them.

I have also realized the package was not being built with system-wide ldflags,
so I fixed that and reuploaded the package.

Spec URL: https://tc01.fedorapeople.org/spasm/spasm-ng.spec
SRPM URL:
https://tc01.fedorapeople.org/spasm/spasm-ng-0.5-0.3.beta.2.fc23.src.rpm

I believe it is now ready 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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1082825] Review Request: mozilla-lightbeam - An add-on for visualizing HTTP requests between websites in real time

[bugzilla]

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

Ben Rosser  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED



--- Comment #6 from Ben Rosser  ---
Issues
==

* Version 1.3.1 seems to be available:
https://addons.mozilla.org/en-US/firefox/addon/lightbeam/; can you update the
package, and I'll re-review?

*
https://fedoraproject.org/wiki/User:Kalev/MozillaExtensionsDraft#Sample_Spec_File
seems to sugggest that XPIs should be unpacked? I'm assuming that's no longer
the case because of extension signing?

* This is very minor, but rpmlint is unhappy that your bundled font provides
are not versioned. The versioning information can be acquired by unzipping the
XPI and running ttname -a over the font files. It'd probably be nice to include
that information, though I wouldn't consider it a blocker for review.

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.
[-]: 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]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "BSD (3 clause)", "MIT/X11 (BSD like)", "*No copyright* MIT/X11
 (BSD like)", "Unknown or generated", "*No copyright* MPL (v2.0)". 5
 files have unknown license. Detailed output of licensecheck in
 /home/bjr/Programming/fedora/reviews/1082825-mozilla-
 lightbeam/licensecheck.txt
[x]: If the package is under multiple licenses, the licensing breakdown
 must be documented in the spec.
[x]: Package must own all directories that it creates.
 Note: Directories without known owners:
 /usr/share/mozilla/extensions/{ec8030f7-c20a-464f-9b0e-13a3a9e97384},
 /usr/share/mozilla/extensions/{92650c4d-4b8e-4d2a-b7eb-24ecf4f6b63a}
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: Package requires other packages for directories it uses.
[x]: Package 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]: 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, 

[Bug 1328868] Review Request: presto - Distributed SQL query engine for big data

[bugzilla]

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

Bug 1277450 Summary: libsvm(-java) remove versioned jars from %{_javadir}
https://bugzilla.redhat.com/show_bug.cgi?id=1277450

   What|Removed |Added

 Status|CLOSED  |NEW
 Resolution|EOL |---



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

[Bug 1223774] Review Request: rubygem-aes - An AES encrypt/ decrypt gem built ontop of OpenSSL.

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

 Status|CLOSED  |NEW
Version|22  |rawhide
 Resolution|EOL |---
   Keywords||Reopened



--- Comment #4 from Igor Gnatenko  ---
Something what should not be closed.

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

[Bug 1223777] Review Request: rubygem-digest-tiger - A Digest module implementing the Tiger hashing algorithm

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

 Status|CLOSED  |NEW
Version|22  |rawhide
 Resolution|EOL |---
   Keywords||Reopened



--- Comment #2 from Igor Gnatenko  ---
Something what should not be closed.

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

[Bug 1223780] Review Request: rubygem-twofish - Twofish symmetric cipher in pure Ruby

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

Version|22  |rawhide



--- Comment #4 from Igor Gnatenko  ---
Something what should not be closed.

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

[Bug 1318814] Review Request: burp2 - Network backup / restore program

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

 Status|CLOSED  |NEW
Version|22  |rawhide
 Resolution|EOL |---
   Keywords||Reopened



--- Comment #5 from Igor Gnatenko  ---
well, it was not reviewed/pushed, so reopening

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

[Bug 1351041] Review Request: mlt - A multimedia framework designed for television broadcasting

[bugzilla]

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



--- Comment #11 from Sergio Monteiro Basto  ---
No, we need open mlt-freeworld package review , but just for f25/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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1050744] Review Request: belle-sip - Linphone SIP stack

[bugzilla]

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



--- Comment #46 from Orion Poplawski  ---
Well, yeah.  They are different function with different arguments.

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

[Bug 1244658] Review Request: dgit - Integration between git and Debian-style archives

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

 Resolution|EOL |CURRENTRELEASE



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

[Bug 1351092] Review Request: python-geoip2 - MaxMind GeoIP2 API

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-07-19 18:22:49



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

[Bug 1351092] Review Request: python-geoip2 - MaxMind GeoIP2 API

[bugzilla]

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



--- Comment #7 from Fedora Update System  ---
python-geoip2-2.4.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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1344115] Review Request: winetricks - Work around common problems in Wine

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-07-19 18:21:49



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

[Bug 1344115] Review Request: winetricks - Work around common problems in Wine

[bugzilla]

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



--- Comment #26 from Fedora Update System  ---
winetricks-20160709-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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1050744] Review Request: belle-sip - Linphone SIP stack

[bugzilla]

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



--- Comment #45 from nucleo  ---
(In reply to Orion Poplawski from comment #44)
> readdir_r is deprecated, just use readdir everywhere.

After I replaced readdir_r with readdir:

belle_sip_utils.c: In function 'belle_sip_parse_directory':
belle_sip_utils.c:1241:8: error: too many arguments to function 'readdir'
  res = readdir(dir, ent, );
^~~
In file included from belle_sip_utils.c:31:0:
/usr/include/dirent.h:162:23: note: declared here
 extern struct dirent *readdir (DIR *__dirp) __nonnull ((1));
   ^~~
belle_sip_utils.c:1241:6: error: assignment makes integer from pointer without
a cast [-Werror=int-conversion]
  res = readdir(dir, ent, );
  ^
belle_sip_utils.c:1249:9: error: too many arguments to function 'readdir'
   res = readdir(dir, ent, );
 ^~~
In file included from belle_sip_utils.c:31:0:
/usr/include/dirent.h:162:23: note: declared here
 extern struct dirent *readdir (DIR *__dirp) __nonnull ((1));
   ^~~
belle_sip_utils.c:1249:7: error: assignment makes integer from pointer without
a cast [-Werror=int-conversion]
   res = readdir(dir, ent, );

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

[Bug 1358041] New: Review Request: python3-scipy - Scientific Tools for Python

[bugzilla]

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

Bug ID: 1358041
   Summary: Review Request: python3-scipy - Scientific Tools for
Python
   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-scipy.spec
SRPM URL:
http://www.cora.nwra.com/~orion/fedora/python3-scipy-0.17.1-1.el7.src.rpm
Description:
Scipy is open-source software for mathematics, science, and
engineering. The core library is NumPy which provides convenient and
fast N-dimensional array manipulation. The SciPy library is built to
work with NumPy arrays, and provides many user-friendly and efficient
numerical routines such as routines for numerical integration and
optimization. Together, they run on all popular operating systems, are
quick to install, and are free of charge. NumPy and SciPy are easy to
use, but powerful enough to be depended upon by some of the world's
leading scientists and engineers.

Fedora Account System Username: orion

This is for EPEL7 only.

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

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

[Bug 1050744] Review Request: belle-sip - Linphone SIP stack

[bugzilla]

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



--- Comment #44 from Orion Poplawski  ---
readdir_r is deprecated, just use readdir everywhere.

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

[Bug 1154750] Review Request: mozilla-privacy-badger - Protects your privacy by blocking spying ads and invisible trackers

[bugzilla]

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

Dominik 'Rathann' Mierzejewski  changed:

   What|Removed |Added

 Blocks|182235 (FE-Legal)   |



--- Comment #10 from Dominik 'Rathann' Mierzejewski  ---
No reason given for FE-Legal block, so lifting. Feel free to re-add with
rationale.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=182235
[Bug 182235] Fedora Legal 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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1324784] Review Request: pseudo - Advanced tool for simulating superuser privileges

[bugzilla]

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



--- Comment #5 from Dominik 'Rathann' Mierzejewski  ---
(In reply to Dominik 'Rathann' Mierzejewski from comment #4)
> (In reply to Raphael Groner from comment #1)
> > Can you use the name pseudoroot for package and binary? I think pseudo is a
> > too generic name and pseudoroot would be in conjunction with fakeroot what
> > does similiar things.
> 
> Makes sense. I'll ask upstream.

On second thought, I don't think it's an issue. Besides, Ubuntu is already
packaging it as "pseudo"[1] and upstream's rationale for the name is that it's
pronounced similar to "sudo".

[1] http://packages.ubuntu.com/xenial/pseudo

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

[Bug 1351041] Review Request: mlt - A multimedia framework designed for television broadcasting

[bugzilla]

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

Nicolas Chauvet (kwizart)  changed:

   What|Removed |Added

 CC||kwiz...@gmail.com



--- Comment #9 from Nicolas Chauvet (kwizart)  ---
Anyone has made a mlt-freeworld package ?

Because until mlt is made fully fonctional, the rpmfusion's mlt package will
replace the mlt one.

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

[Bug 1324784] Review Request: pseudo - Advanced tool for simulating superuser privileges

[bugzilla]

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



--- Comment #4 from Dominik 'Rathann' Mierzejewski  ---
(In reply to Raphael Groner from comment #1)
> Can you use the name pseudoroot for package and binary? I think pseudo is a
> too generic name and pseudoroot would be in conjunction with fakeroot what
> does similiar things.

Makes sense. I'll ask upstream.

> There's an alternative and some binary/script fakeroot-pseudo, is it a plugin?

This is actually a drop-in replacement for fakeroot.

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

[Bug 1351041] Review Request: mlt - A multimedia framework designed for television broadcasting

[bugzilla]

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



--- Comment #10 from Nicolas Chauvet (kwizart)  ---
I meant "replace the fedora's one"

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

[Bug 1324784] Review Request: pseudo - Advanced tool for simulating superuser privileges

[bugzilla]

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



--- Comment #3 from Dominik 'Rathann' Mierzejewski  ---
It looks like bug 134647 is taken already, but I'll happily review something
else for you.

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

[Bug 804125] Review Request: rdkit - A toolkit for cheminformatics and machine learning

[bugzilla]

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



--- Comment #43 from Antonio Trande  ---
Hi Gianluca.

Is this package for epel too ?

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

[Bug 804125] Review Request: rdkit - A toolkit for cheminformatics and machine learning

[bugzilla]

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

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

[Bug 1318814] Review Request: burp2 - Network backup / restore program

[bugzilla]

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

Fedora End Of Life  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |EOL
Last Closed||2016-07-19 16:00:57



--- Comment #4 from Fedora End Of Life  ---
Fedora 22 changed to end-of-life (EOL) status on 2016-07-19. Fedora 22 is
no longer maintained, which means that it will not receive any further
security or bug fix updates. As a result we are closing this bug.

If you can reproduce this bug against a currently maintained version of
Fedora please feel free to reopen this bug against that version. If you
are unable to reopen this bug, please file a new report against the
current release. If you experience problems, please add a comment to this
bug.

Thank you for reporting this bug and we are sorry it could not be fixed.

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

[Bug 1337348] Review Request: mediawiki-piwik - MediaWiki Piwik integration extension

[bugzilla]

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

Ricky Elrod  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX
Last Closed||2016-07-19 15:36: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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1223777] Review Request: rubygem-digest-tiger - A Digest module implementing the Tiger hashing algorithm

[bugzilla]

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

Fedora End Of Life  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |EOL
Last Closed||2016-07-19 15:34:52



--- Comment #1 from Fedora End Of Life  ---
Fedora 22 changed to end-of-life (EOL) status on 2016-07-19. Fedora 22 is
no longer maintained, which means that it will not receive any further
security or bug fix updates. As a result we are closing this bug.

If you can reproduce this bug against a currently maintained version of
Fedora please feel free to reopen this bug against that version. If you
are unable to reopen this bug, please file a new report against the
current release. If you experience problems, please add a comment to this
bug.

Thank you for reporting this bug and we are sorry it could not be fixed.

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

[Bug 1223780] Review Request: rubygem-twofish - Twofish symmetric cipher in pure Ruby

[bugzilla]

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

Fedora End Of Life  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |EOL
Last Closed||2016-07-19 15:35:00



--- Comment #3 from Fedora End Of Life  ---
Fedora 22 changed to end-of-life (EOL) status on 2016-07-19. Fedora 22 is
no longer maintained, which means that it will not receive any further
security or bug fix updates. As a result we are closing this bug.

If you can reproduce this bug against a currently maintained version of
Fedora please feel free to reopen this bug against that version. If you
are unable to reopen this bug, please file a new report against the
current release. If you experience problems, please add a comment to this
bug.

Thank you for reporting this bug and we are sorry it could not be fixed.

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

[Bug 1223774] Review Request: rubygem-aes - An AES encrypt/ decrypt gem built ontop of OpenSSL.

[bugzilla]

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

Fedora End Of Life  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |EOL
Last Closed||2016-07-19 15:34:43



--- Comment #3 from Fedora End Of Life  ---
Fedora 22 changed to end-of-life (EOL) status on 2016-07-19. Fedora 22 is
no longer maintained, which means that it will not receive any further
security or bug fix updates. As a result we are closing this bug.

If you can reproduce this bug against a currently maintained version of
Fedora please feel free to reopen this bug against that version. If you
are unable to reopen this bug, please file a new report against the
current release. If you experience problems, please add a comment to this
bug.

Thank you for reporting this bug and we are sorry it could not be fixed.

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

[Bug 1082825] Review Request: mozilla-lightbeam - An add-on for visualizing HTTP requests between websites in real time

[bugzilla]

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



--- Comment #5 from Dominik 'Rathann' Mierzejewski  ---
Sure, I'll review cmap-resources. Thanks!

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

[Bug 1350260] Review Request: cmap-resources - CMap Resources for Adobe' s public character collections

[bugzilla]

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

Dominik 'Rathann' Mierzejewski  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|domi...@greysector.net
  Flags||fedora-review?



--- Comment #1 from Dominik 'Rathann' Mierzejewski  ---
Taking 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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1354210] Review Request: xviewer - Fast and functional graphics viewer

[bugzilla]

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



--- Comment #8 from Mario Blättermann  ---
There is a general problem with /usr/share/help and all of its language-based
subdirectories. They are not owned by *any* package. Some ownerships on a real
system happen by accidence, but there's no regular owner, see
https://bugzilla.redhat.com/show_bug.cgi?id=1357974.

The bug in the Amharic translation has been fixed already in Launchpad:
https://translations.launchpad.net/linuxmint/latest/+pots/xviewer/am/+translate?batch=10=all=%E1%88%9D%E1%88%B5%E1%88%8E%E1%89%BD%3A%E1%89%B0%E1%8A%95%E1%88%B8%E1%88%AB%E1%89%B3%E1%89%BD+%E1%88%9B%E1%88%B3%E1%8B%AB%3A%E1%8A%95%E1%8B%B5%E1%8D%8E%E1%89%BD%3B

I've renamed the ChangeLog files amd added them to doc.

The user documentation has now its own subpackage. This behaves the same as in
Glabels (which I've used as template): The -doc package is independent, because
in fact it contains text files only. It will be pulled in automatically by the
main package, to avoid unexpected behavior when calling the help in the GUI.
The advantages are, the -doc package is noarch, and it is easier to declare the
license (CC-BY-SA).

Spec URL: https://mariobl.fedorapeople.org/Review/SPECS/xviewer.spec
SRPM URL:
https://mariobl.fedorapeople.org/Review/SRPMS/xviewer-1.0.4-2.fc24.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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1354202] Review Request: purple-hangouts - Adds support for Hangouts to libpurple-based clients

[bugzilla]

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



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

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

[Bug 1324204] Review Request: fast-vm - script for defining VMs from images provided in thin LVM pool.

[bugzilla]

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

Fedora End Of Life  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |EOL
Last Closed||2016-07-19 14:47:42



--- Comment #12 from Fedora End Of Life  ---
Fedora 22 changed to end-of-life (EOL) status on 2016-07-19. Fedora 22 is
no longer maintained, which means that it will not receive any further
security or bug fix updates. As a result we are closing this bug.

If you can reproduce this bug against a currently maintained version of
Fedora please feel free to reopen this bug against that version. If you
are unable to reopen this bug, please file a new report against the
current release. If you experience problems, please add a comment to this
bug.

Thank you for reporting this bug and we are sorry it could not be fixed.

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

[Bug 1328868] Review Request: presto - Distributed SQL query engine for big data

[bugzilla]

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

Bug 1277450 Summary: libsvm(-java) remove versioned jars from %{_javadir}
https://bugzilla.redhat.com/show_bug.cgi?id=1277450

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |EOL



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

[Bug 1357978] Review Request: php-consolidation-annotated-command - Initialize Symfony Console commands from annotated command class methods

[bugzilla]

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



--- Comment #1 from Shawn Iwinski  ---
NOTE: Not latest version 1.1.x because 1.0.x is required by drush 8.1.3
dependency (bug 1357097)

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

[Bug 1357978] Review Request: php-consolidation-annotated-command - Initialize Symfony Console commands from annotated command class methods

[bugzilla]

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

Shawn Iwinski  changed:

   What|Removed |Added

 Depends On||1357977
   ||(php-consolidation-output-f
   ||ormatters)
  Alias||php-consolidation-annotated
   ||-command




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1357977
[Bug 1357977] Review Request: php-consolidation-output-formatters - Format
text by applying transformations provided by plug-in formatters
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1357978] Review Request: php-consolidation-annotated-command - Initialize Symfony Console commands from annotated command class methods

[bugzilla]

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

Shawn Iwinski  changed:

   What|Removed |Added

 Blocks||1357097




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1357097
[Bug 1357097] drush-8.1.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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1357977] New: Review Request: php-consolidation-output-formatters - Format text by applying transformations provided by plug-in formatters

[bugzilla]

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

Bug ID: 1357977
   Summary: Review Request: php-consolidation-output-formatters -
Format text by applying transformations provided by
plug-in formatters
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: sh...@iwin.ski
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://raw.githubusercontent.com/siwinski/rpms/4858e809df9cad5c7992b0e40120b83f540eda16/php-consolidation-output-formatters/php-consolidation-output-formatters.spec

SRPM URL:
https://siwinski.fedorapeople.org/SRPMS/php-consolidation-output-formatters-1.0.0-1.fc24.src.rpm

Description: Format text by applying transformations provided by plug-in
formatters.

Fedora Account System Username: siwinski

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

[Bug 1270405] Review Request: native_client - Google Native Client Toolchain

[bugzilla]

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

Orion Poplawski  changed:

   What|Removed |Added

 CC||or...@cora.nwra.com



--- Comment #9 from Orion Poplawski  ---
It's also a terribly lousy name by virtue of being too generic.  Can't wait for
"native_server".

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

[Bug 1357977] Review Request: php-consolidation-output-formatters - Format text by applying transformations provided by plug-in formatters

[bugzilla]

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

Shawn Iwinski  changed:

   What|Removed |Added

 Blocks||1357097




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1357097
[Bug 1357097] drush-8.1.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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1357977] Review Request: php-consolidation-output-formatters - Format text by applying transformations provided by plug-in formatters

[bugzilla]

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

Shawn Iwinski  changed:

   What|Removed |Added

 Blocks||1357978
   ||(php-consolidation-annotate
   ||d-command)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1357978
[Bug 1357978] Review Request: php-consolidation-annotated-command -
Initialize Symfony Console commands from annotated command class methods
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1357977] Review Request: php-consolidation-output-formatters - Format text by applying transformations provided by plug-in formatters

[bugzilla]

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

Shawn Iwinski  changed:

   What|Removed |Added

  Alias||php-consolidation-output-fo
   ||rmatters



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

[Bug 1357978] New: Review Request: php-consolidation-annotated-command - Initialize Symfony Console commands from annotated command class methods

[bugzilla]

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

Bug ID: 1357978
   Summary: Review Request: php-consolidation-annotated-command -
Initialize Symfony Console commands from annotated
command class methods
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: sh...@iwin.ski
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://raw.githubusercontent.com/siwinski/rpms/4d8d91a0a108bddf0c4cdf94b65eea16c2ec47c3/php-consolidation-annotated-command/php-consolidation-annotated-command.spec

SRPM URL:
https://siwinski.fedorapeople.org/SRPMS/php-consolidation-annotated-command-1.0.1-1.fc24.src.rpm

Description: Initialize Symfony Console commands from annotated command class
methods.

Fedora Account System Username: siwinski

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

[Bug 1050744] Review Request: belle-sip - Linphone SIP stack

[bugzilla]

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



--- Comment #43 from nucleo  ---
(In reply to Tom "spot" Callaway from comment #41)
> With the patch in Comment 40 applied, this package builds cleanly on Fedora
> 24 (the patch fixes an obvious typo in a comparison that wasn't caught by
> earlier gcc revisions).
> 
> All the items noted in Chris's previous review are resolved in 1.4.2-1, so
> I'm approving this package (please be sure to apply that patch before
> committing).

Thanks for the patch but it builds fine only in F24 and fails in Rawhide:

belle_sip_utils.c: In function 'belle_sip_parse_directory':
belle_sip_utils.c:1241:2: error: 'readdir_r' is deprecated
[-Werror=deprecated-declarations]
  res = readdir_r(dir, ent, );
  ^~~
In file included from belle_sip_utils.c:31:0:
/usr/include/dirent.h:183:12: note: declared here
 extern int readdir_r (DIR *__restrict __dirp,
^
belle_sip_utils.c:1249:3: error: 'readdir_r' is deprecated
[-Werror=deprecated-declarations]
   res = readdir_r(dir, ent, );
   ^~~
In file included from belle_sip_utils.c:31:0:
/usr/include/dirent.h:183:12: note: declared here
 extern int readdir_r (DIR *__restrict __dirp,
^

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

[Bug 1208911] Review Request: doublecmd - Twin-panel (commander-style) file manager

[bugzilla]

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

Bug 1247925 Summary: add ppc64le and aarch64 support for fpc
https://bugzilla.redhat.com/show_bug.cgi?id=1247925

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |EOL



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

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

[bugzilla]

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



--- Comment #22 from Fedora Update System  ---
php-pecl-mongodb-1.1.8-3.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-e331dcb6c7

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

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

[bugzilla]

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



--- Comment #20 from Fedora Update System  ---
php-pecl-mongodb-1.1.8-3.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-539597afca

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

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

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



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

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

[bugzilla]

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



--- Comment #21 from Fedora Update System  ---
php-pecl-mongodb-1.1.8-3.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-b855865eef

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

[Bug 1244658] Review Request: dgit - Integration between git and Debian-style archives

[bugzilla]

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

Fedora End Of Life  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |EOL
Last Closed||2016-07-19 13:05:23



--- Comment #7 from Fedora End Of Life  ---
Fedora 22 changed to end-of-life (EOL) status on 2016-07-19. Fedora 22 is
no longer maintained, which means that it will not receive any further
security or bug fix updates. As a result we are closing this bug.

If you can reproduce this bug against a currently maintained version of
Fedora please feel free to reopen this bug against that version. If you
are unable to reopen this bug, please file a new report against the
current release. If you experience problems, please add a comment to this
bug.

Thank you for reporting this bug and we are sorry it could not be fixed.

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

[Bug 1343661] Rebase clufter component

[bugzilla]

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

Jan Pokorný  changed:

   What|Removed |Added

 Depends On||1357945




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1357945
[Bug 1357945] [clufter integration] clufter is distribution-sensitive wrt.
new features so pass the current one on cluster.conf/corosync.conf match
and allow user's override
-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1270322] Review Request: chromium - A WebKit (Blink) powered web browser

[bugzilla]

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



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

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

[Bug 1352553] Review Request: php-zendframework-zend-expressive-twigrenderer - Twig integration for Expressive

[bugzilla]

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



--- Comment #6 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-zendframework-zend-expressive-twigrenderer

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

[Bug 1352536] Review Request: php-zendframework-zend-expressive - PSR-7 Middleware Microframework based on Stratigility

[bugzilla]

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



--- Comment #8 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-zendframework-zend-expressive

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

[Bug 1352549] Review Request: php-zendframework-zend-expressive-platesrenderer - Plates integration for Expressive

[bugzilla]

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



--- Comment #6 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-zendframework-zend-expressive-platesrenderer

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

[Bug 1352539] Review Request: php-zendframework-zend-expressive-zendviewrenderer - zend-view PhpRenderer integration for Expressive

[bugzilla]

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



--- Comment #6 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-zendframework-zend-expressive-zendviewrenderer

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

[Bug 1352538] Review Request: php-zendframework-zend-expressive-helpers - Helper/ Utility classes for Expressive

[bugzilla]

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



--- Comment #6 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-zendframework-zend-expressive-helpers

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

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

[bugzilla]

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



--- Comment #19 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-pecl-mongodb

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

[Bug 1352516] Review Request: php-zendframework-zend-expressive-aurarouter - Aura.Router integration for Expressive

[bugzilla]

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



--- Comment #6 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-zendframework-zend-expressive-aurarouter

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

[Bug 1352528] Review Request: php-zendframework-zend-expressive-zendrouter - zend-mvc router support for Expressive

[bugzilla]

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



--- Comment #5 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-zendframework-zend-expressive-zendrouter

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

[Bug 1352520] Review Request: php-zendframework-zend-expressive-fastroute - FastRoute integration for Expressive

[bugzilla]

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



--- Comment #6 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-zendframework-zend-expressive-fastroute

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

[Bug 1352468] Review Request: php-aura-router - A web router implementation

[bugzilla]

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



--- Comment #6 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-aura-router

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

[Bug 1352475] Review Request: php-zendframework-zend-stratigility - Middleware for PHP

[bugzilla]

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



--- Comment #6 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-zendframework-zend-stratigility

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

[Bug 1352471] Review Request: php-zendframework-zend-expressive-template - Template subcomponent for Expressive

[bugzilla]

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



--- Comment #6 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-zendframework-zend-expressive-template

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

[Bug 1352469] Review Request: php-zendframework-zend-expressive-router - Router subcomponent for Expressive

[bugzilla]

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



--- Comment #6 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-zendframework-zend-expressive-router

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

[Bug 1352466] Review Request: php-aura-di - Dependency injection container system

[bugzilla]

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



--- Comment #6 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/php-aura-di

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

[Bug 1356048] Review Request: rtlsdr-scanner - Frequency scanning GUI for RTL2832 based DVB-T dongles

[bugzilla]

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



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

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

[Bug 1050744] Review Request: belle-sip - Linphone SIP stack

[bugzilla]

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



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

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

[Bug 1355830] Review Request: hidrd - HID report descriptor I/ O library and conversion tool

[bugzilla]

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

Jaroslav Škarvada  changed:

   What|Removed |Added

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



--- Comment #3 from Jaroslav Škarvada  ---
Thanks for the review, hopefully all fixed, new files:
Spec URL: https://jskarvad.fedorapeople.org/hidrd/hidrd.spec
SRPM URL: hidrd-0.2.0-2.20160712git1abf7033.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://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1323186] Review Request: opa-fmgui - Intel OPA Fabric GUI

[bugzilla]

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



--- Comment #80 from Michal Schmidt  ---
(In reply to Neil Horman from comment #79)
> Thank you Rick.  You're approved as a sponsor now

Correction: s/sponsor/packager/

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

[Bug 1323186] Review Request: opa-fmgui - Intel OPA Fabric GUI

[bugzilla]

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

Rick Tierney  changed:

   What|Removed |Added

  Flags|needinfo?(rick.tierney@inte |needinfo?(nhor...@redhat.co
   |l.com)  |m)



--- Comment #81 from Rick Tierney  ---
Got it, no problem! I'll be working on that next.
Thanks!

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

[Bug 1346249] Review Request: resultsdb - Results store for automated tasks

[bugzilla]

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

Jaroslav Škarvada  changed:

   What|Removed |Added

 CC||jskar...@redhat.com



--- Comment #2 from Jaroslav Škarvada  ---
(In reply to Jiri Kulda from comment #1)
> - If settings.py.example is a configuration file than it should be installed
> in /etc/* without .example suffix

And executable files shouldn't be in /etc, nor in /usr/share/*. Also .pyc,
.pyo. shouldn't be in /usr/share/* (you will have to workaround python
autocompile directories).

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

[Bug 1341099] Review Request: taskotron-trigger - Triggering Taskotron jobs on fedmsgs

[bugzilla]

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

Jaroslav Škarvada  changed:

   What|Removed |Added

 CC||jskar...@redhat.com



--- Comment #3 from Jaroslav Škarvada  ---
(In reply to Jiri Kulda from comment #2)
> - I think python will download extra dependencies listed in setup.py:
>   moksha.hub - probably
> fedmsg - probably not because it is in Requires
> requests   - Same case as fedmsg
>   If not they need to be added into requires...

I think these are runtime deps, IMHO they should be added to the requires.

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

[Bug 1346249] Review Request: resultsdb - Results store for automated tasks

[bugzilla]

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



--- Comment #1 from Jiri Kulda  ---
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: "GPL (v2 or later)", "Unknown or generated". 18 files have
 unknown license. 
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[x]: 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.
[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.
[x]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 bytes in 4 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

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

= SHOULD items =

Generic:
[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.
[!]: Final provides and requires are sane (see attachments).
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: 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.
[!]: 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]: Reviewer should test that the package builds in mock.
[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.

[Bug 1341099] Review Request: taskotron-trigger - Triggering Taskotron jobs on fedmsgs

[bugzilla]

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



--- Comment #2 from Jiri Kulda  ---
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: "GPL (v2 or later)", "Unknown or generated". 20 files have
 unknown license. 
[x]: Package must own all directories that it creates.
 Note: Directories without known owners: /etc/logrotate.d
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[!]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf %{buildroot} present but not required
[x]: Sources contain only permissible code or content.
[x]: 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.
[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.
[x]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 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 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]: %config files are marked noreplace or the reason is justified.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: No %config files under /usr.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

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

= SHOULD items =

Generic:
[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]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[?]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[x]: %check is present and all tests pass.
[!]: 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]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean 

[Bug 1215071] Review Request: querydsl - Type-safe queries for Java

[bugzilla]

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

Bug 1213455 Summary: Enable java jdbc driver
https://bugzilla.redhat.com/show_bug.cgi?id=1213455

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |EOL



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

[Bug 739323] Review Request: mozilla-https-everywhere - HTTPS/ HSTS enforcement extension for Mozilla browsers

[bugzilla]

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

Bug 1154321 Summary: Fedora Packaging Rules violation: No source for 
defaults/rulesets.sqlite
https://bugzilla.redhat.com/show_bug.cgi?id=1154321

   What|Removed |Added

 Status|CLOSED  |NEW
 Resolution|EOL |---



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

[Bug 1323186] Review Request: opa-fmgui - Intel OPA Fabric GUI

[bugzilla]

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

Neil Horman  changed:

   What|Removed |Added

  Flags|needinfo?(nhor...@redhat.co |needinfo?(rick.tierney@inte
   |m)  |l.com)



--- Comment #79 from Neil Horman  ---
Thank you Rick.  You're approved as a sponsor now, so you should be able to
complete the import section of the packaging process and close this bz

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

[Bug 1208001] Review Request: python3-bokeh - < please provide short description>

[bugzilla]

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

Fedora End Of Life  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |EOL
Last Closed||2016-07-19 09:19:50



--- Comment #4 from Fedora End Of Life  ---
Fedora 22 changed to end-of-life (EOL) status on 2016-07-19. Fedora 22 is
no longer maintained, which means that it will not receive any further
security or bug fix updates. As a result we are closing this bug.

If you can reproduce this bug against a currently maintained version of
Fedora please feel free to reopen this bug against that version. If you
are unable to reopen this bug, please file a new report against the
current release. If you experience problems, please add a comment to this
bug.

Thank you for reporting this bug and we are sorry it could not be fixed.

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

[Bug 1357875] Review Request: php-dnoegel-php-xdg-base-dir - Implementation of the XDG Base Directory Specification for PHP

[bugzilla]

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

Shawn Iwinski  changed:

   What|Removed |Added

 Blocks||1357876 (psysh)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1357876
[Bug 1357876] Review Request: psysh - A runtime developer console,
interactive debugger and REPL for PHP
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1357873] Review Request: php-jakub-onderka-php-console-highlighter - Highlight PHP code in console (terminal)

[bugzilla]

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

Shawn Iwinski  changed:

   What|Removed |Added

 Blocks||1357876 (psysh)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1357876
[Bug 1357876] Review Request: psysh - A runtime developer console,
interactive debugger and REPL for PHP
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1357876] Review Request: psysh - A runtime developer console, interactive debugger and REPL for PHP

[bugzilla]

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

Shawn Iwinski  changed:

   What|Removed |Added

 Blocks||1357097
 Depends On||1357873
   ||(jakub-onderka-php-console-
   ||highlighter), 1357875
   ||(php-dnoegel-php-xdg-base-d
   ||ir)
  Alias||psysh




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1357097
[Bug 1357097] drush-8.1.3 is available
https://bugzilla.redhat.com/show_bug.cgi?id=1357873
[Bug 1357873] Review Request: php-jakub-onderka-php-console-highlighter -
Highlight PHP code in console (terminal)
https://bugzilla.redhat.com/show_bug.cgi?id=1357875
[Bug 1357875] Review Request: php-dnoegel-php-xdg-base-dir - Implementation
of the XDG Base Directory Specification for PHP
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1357876] New: Review Request: psysh - A runtime developer console, interactive debugger and REPL for PHP

[bugzilla]

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

Bug ID: 1357876
   Summary: Review Request: psysh - A runtime developer console,
interactive debugger and REPL for PHP
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: sh...@iwin.ski
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://raw.githubusercontent.com/siwinski/rpms/0a2a1552bffbda9d50cb1ae24cf86de10a8c388f/psysh/psysh.spec

SRPM URL: https://siwinski.fedorapeople.org/SRPMS/psysh-0.7.2-1.fc24.src.rpm

Description: A runtime developer console, interactive debugger and REPL for
PHP.

Fedora Account System Username: siwinski

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

[Bug 1357873] Review Request: php-jakub-onderka-php-console-highlighter - Highlight PHP code in console (terminal)

[bugzilla]

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

Shawn Iwinski  changed:

   What|Removed |Added

 Depends On||1357871
   ||(php-jakub-onderka-php-cons
   ||ole-color)
  Alias||jakub-onderka-php-console-h
   ||ighlighter




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1357871
[Bug 1357871] Review Request: php-jakub-onderka-php-console-color - Simple
library for creating colored console output
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1357875] Review Request: php-dnoegel-php-xdg-base-dir - Implementation of the XDG Base Directory Specification for PHP

[bugzilla]

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

Shawn Iwinski  changed:

   What|Removed |Added

  Alias||php-dnoegel-php-xdg-base-di
   ||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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1357875] New: Review Request: php-dnoegel-php-xdg-base-dir - Implementation of the XDG Base Directory Specification for PHP

[bugzilla]

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

Bug ID: 1357875
   Summary: Review Request: php-dnoegel-php-xdg-base-dir -
Implementation of the XDG Base Directory Specification
for PHP
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: sh...@iwin.ski
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://raw.githubusercontent.com/siwinski/rpms/3cdf23e59945a7f9b2c9fa0d9dc3e5a971e7d0ce/php-dnoegel-php-xdg-base-dir/php-dnoegel-php-xdg-base-dir.spec

SRPM URL:
https://siwinski.fedorapeople.org/SRPMS/php-dnoegel-php-xdg-base-dir-0.1-1.fc24.src.rpm

Description: Implementation of the XDG Base Directory Specification for PHP.


Fedora Account System Username: siwinski

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

[Bug 1357871] Review Request: php-jakub-onderka-php-console-color - Simple library for creating colored console output

[bugzilla]

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

Shawn Iwinski  changed:

   What|Removed |Added

 Blocks||1357873
   ||(jakub-onderka-php-console-
   ||highlighter)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1357873
[Bug 1357873] Review Request: php-jakub-onderka-php-console-highlighter -
Highlight PHP code in console (terminal)
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1357873] New: Review Request: php-jakub-onderka-php-console-highlighter - Highlight PHP code in console (terminal)

[bugzilla]

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

Bug ID: 1357873
   Summary: Review Request:
php-jakub-onderka-php-console-highlighter - Highlight
PHP code in console (terminal)
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: sh...@iwin.ski
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://raw.githubusercontent.com/siwinski/rpms/057d2be065ed67e4d04ae86f303b3312c5d11ce1/php-jakub-onderka-php-console-highlighter/php-jakub-onderka-php-console-highlighter.spec

SRPM URL:
https://siwinski.fedorapeople.org/SRPMS/php-jakub-onderka-php-console-highlighter-0.3.2-1.fc24.src.rpm

Description: Highlight PHP code in console (terminal).

Fedora Account System Username: siwinski

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

[Bug 1357871] Review Request: php-jakub-onderka-php-console-color - Simple library for creating colored console output

[bugzilla]

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

Shawn Iwinski  changed:

   What|Removed |Added

  Alias||php-jakub-onderka-php-conso
   ||le-color



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

[Bug 1357871] New: Review Request: php-jakub-onderka-php-console-color - Simple library for creating colored console output

[bugzilla]

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

Bug ID: 1357871
   Summary: Review Request: php-jakub-onderka-php-console-color -
Simple library for creating colored console output
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: sh...@iwin.ski
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://raw.githubusercontent.com/siwinski/rpms/96a1a2520097c1f99952102079d0171e692131ca/php-jakub-onderka-php-console-color/php-jakub-onderka-php-console-color.spec

SRPM URL:
https://siwinski.fedorapeople.org/SRPMS/php-jakub-onderka-php-console-color-0.1-1.fc24.src.rpm

Description: Simple library for creating colored console output.

Fedora Account System Username: siwinski

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