[Bug 1327218] Review Request: libvterm - An abstract library implementation of a VT220/xterm/ ECMA-48 terminal emulator

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1327218

Igor Gnatenko  changed:

   What|Removed |Added

 Status|ASSIGNED|POST



-- 
You are receiving this mail because:
You are on the CC list 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 1365320] Review Request: simplenote - Simplenote.com desktop client

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1365320



--- Comment #1 from Igor Gnatenko  ---
cp -r
$RPM_BUILD_DIR/simplenote-electron-%{version}/release/Simplenote-linux-x64/*
$RPM_BUILD_ROOT%{_datadir}/simplenote/

doesn't seem like true open source.

-- 
You are receiving this mail because:
You are on the CC list 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 1324020] Review Request: cassandra - OpenSource database Apache Cassandra

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1324020

Tomas Repik  changed:

   What|Removed |Added

 Depends On|1348231 |




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1348231
[Bug 1348231] update python-cassandra-driver
-- 
You are receiving this mail because:
You are on the CC list 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 1362490] Review Request: libphutil - A collection of PHP utility classes

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1362490



--- Comment #5 from Remi Collet  ---
Of course, I mean : "Non-free software must NOT go in the repo, even in
.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 1362490] Review Request: libphutil - A collection of PHP utility classes

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1362490



--- Comment #4 from Remi Collet  ---
Created attachment 1189040
  --> https://bugzilla.redhat.com/attachment.cgi?id=1189040&action=edit
phpci.log

phpCompatInfo version 5.0.1 DB version 1.11.0 built Jul 26 2016 06:33:42 CEST

Generated by: phpcompatinfo analyser:run 
libphutil-5fd2cf9d5ddd38424a54a8fba02398d527639970/ --output phpci.log
--no-ansi

Results give good information but are not 100% reliable, need to be checked.

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


[Bug 1362490] Review Request: libphutil - A collection of PHP utility classes

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1362490

Remi Collet  changed:

   What|Removed |Added

 CC||fed...@famillecollet.com



--- Comment #3 from Remi Collet  ---
Quick look.


1/cp -a * %{buildroot}%{_datadir}/%{name}/

This look very ugly, I think lot of files are unneeded at runtime
(documentation, license, tests...)

2/ JsonLInt is available as system library: php-jsonlint-1.4.0

$jsonlint_root =
phutil_get_library_root('phutil').'/../externals/jsonlint';
require_once $jsonlint_root.'/src/Seld/JsonLint/JsonParser.php';
require_once $jsonlint_root.'/src/Seld/JsonLint/Lexer.php';
require_once $jsonlint_root.'/src/Seld/JsonLint/ParsingException.php';
require_once $jsonlint_root.'/src/Seld/JsonLint/Undefined.php';
=>
require_once '/usr/share/php/Seld/JsonLint/autoload.php';

3/ Requires:   php-common >= 5

See PHP Guidelines, and attached phpci.log (phpcompatinfo static analysis
result)
You must not use package names, but extensions name. Version doesn't make sense
(we don't have anything < 5)

Requires: php-reflection
Requires: php-simplexml
Requires: php-ctype
Requires: php-curl
Requires: php-date
...

apc, memcache, xhprof are very probably optional, need code check.

4/ you cannot keep jsonlint in the archive, you have to remove it. Non-free
software must go in the repo, even in .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 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585



--- Comment #23 from Parag AN(पराग)  ---
Also, I find you have attempted only one full package review in bug 1196037 but
I am not able to understand if you are able to understand RPM packaging so do
more full package reviews.

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


[Bug 673585] Review Request: perl-Apache-Htgroup - Manage Apache htgroup files

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=673585



--- Comment #22 from Parag AN(पराग)  ---
Also, in recent discussions on Fedora devel list, its suggested that new
contributors can use Copr project to build their packages and provide links to
it here.

Go to https://copr.fedorainfracloud.org/coprs and login with your FAS username
and password, create a New project there.

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


[Bug 1358870] Review Request: simplesamlphp - A PHP implementation of a SAML 2.0 service provider and identity provider , also compatible with Shibboleth 1.3 and 2.0

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1358870
Bug 1358870 depends on bug 1356595, which changed state.

Bug 1356595 Summary: Review Request: php-simplesamlphp-saml2_1 - SAML2 PHP 
library from SimpleSAMLphp (version 1)
https://bugzilla.redhat.com/show_bug.cgi?id=1356595

   What|Removed |Added

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



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


[Bug 1356595] Review Request: php-simplesamlphp-saml2_1 - SAML2 PHP library from SimpleSAMLphp ( version 1)

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356595



--- Comment #16 from Fedora Update System  ---
php-simplesamlphp-saml2_1-1.9-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 1356587] Review Request: php-simplesamlphp-saml2 - SAML2 PHP library from SimpleSAMLphp

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356587

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-08-09 00:50:45



-- 
You are receiving this mail because:
You are on the CC list 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 1356595] Review Request: php-simplesamlphp-saml2_1 - SAML2 PHP library from SimpleSAMLphp ( version 1)

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356595

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-08-09 00:50:39



-- 
You are receiving this mail because:
You are on the CC list 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 1356587] Review Request: php-simplesamlphp-saml2 - SAML2 PHP library from SimpleSAMLphp

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356587



--- Comment #16 from Fedora Update System  ---
php-simplesamlphp-saml2-2.2-2.fc24 has been pushed to the Fedora 24 stable
repository. If problems still persist, please make note of it in this bug
report.

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


[Bug 1350143] Review Request: fmt - Small, safe and fast formating library for C++

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1350143

Dave Johansen  changed:

   What|Removed |Added

  Flags|needinfo?(davejohansen@gmai |
   |l.com)  |



--- Comment #4 from Dave Johansen  ---
Sorry for the delay. We moved and I haven't had time to work on this sort of
thing.

(In reply to Antonio Trande from comment #2)
> [!]: Package requires other packages for directories it uses.
>  Note: No known owner of /usr/share/cmake/fmt, /usr/include/fmt
> [!]: Package must own all directories that it creates.
>  Note: Directories without known owners: /usr/share/cmake,
>  /usr/share/cmake/fmt, /usr/include/fmt

Fixed.

> [-]: If the package is a rename of another package, proper Obsoletes and
>  Provides are present.

This is actually a rename of cppformat but I believe that these are correct.

> Generic:
> [!]: Avoid bundling fonts in non-fonts packages.
>  Note: Package contains font files

This came up during the original review of cppformat and the general consensus
was that it wasn't an issue:
https://lists.fedoraproject.org/archives/list/de...@lists.fedoraproject.org/message/EKVX5GCQUFINUYBRGFD2JQHF2UXUBBVV/

> [!]: Patches link to upstream bugs/comments/lists or are otherwise
>  justified.

Fixed.

> fmt.x86_64: W: tag-in-description C Provides:
> fmt.x86_64: W: tag-in-description C Obsoletes:

Is there something wrong with my Provides/Obsoletes?

> fmt.x86_64: W: incoherent-version-in-changelog 3.0.0-1 ['3.0.0-0.1.fc25',
> '3.0.0-0.1']

Fixed.

> fmt-devel.x86_64: W: only-non-binary-in-usr-lib

I don't understand what this is warning about.

The updated .spec file with the fixes is at:
https://daveisfera.fedorapeople.org/fmt_3.0.0/fmt.spec

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


[Bug 1356657] Review Request: lxqt-wallet - Create a kwallet like functionality for LXQt

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356657



--- Comment #20 from Fedora Update System  ---
lxqt-wallet-3.0.0-1.el7, zulucrypt-5.0.0-2.20160802git064e9db.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-f08257b92c

-- 
You are receiving this mail because:
You are on the CC list 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 1359234] Review Request: auter - Prepare and apply updates

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359234



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

-- 
You are receiving this mail because:
You are on the CC list 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 1361212] Review Request: perl-Ref-Util - Utility functions for checking references

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1361212



--- Comment #10 from Fedora Update System  ---
perl-Ref-Util-0.020-2.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-f5b3935fb7

-- 
You are receiving this mail because:
You are on the CC list 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 1361212] Review Request: perl-Ref-Util - Utility functions for checking references

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1361212



--- Comment #9 from Fedora Update System  ---
perl-Ref-Util-0.020-2.el6 has been pushed to the Fedora EPEL 6 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-da4724d0a9

-- 
You are receiving this mail because:
You are on the CC list 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 1359234] Review Request: auter - Prepare and apply updates

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359234



--- Comment #13 from Fedora Update System  ---
auter-0.7-1.el6 has been pushed to the Fedora EPEL 6 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-71b7677709

-- 
You are receiving this mail because:
You are on the CC list 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 1356739] Review Request: zulucrypt - Qt GUI front end to cryptsetup

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356739

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #33 from Fedora Update System  ---
lxqt-wallet-3.0.0-1.fc24, zulucrypt-5.0.0-2.20160802git064e9db.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-0f2d6af926

-- 
You are receiving this mail because:
You are on the CC list 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 1353000] Review Request: gns3-server - Graphical Network Simulator 3

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1353000

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #16 from Fedora Update System  ---
gns3-gui-1.5.1-2.fc24, gns3-net-converter-1.3.0-2.fc24,
gns3-server-1.5.1-1.fc24 has been pushed to the Fedora 24 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-b0b7e1057b

-- 
You are receiving this mail because:
You are on the CC list 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 1359234] Review Request: auter - Prepare and apply updates

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359234



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

-- 
You are receiving this mail because:
You are on the CC list 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 1356657] Review Request: lxqt-wallet - Create a kwallet like functionality for LXQt

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356657

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #19 from Fedora Update System  ---
lxqt-wallet-3.0.0-1.fc24, zulucrypt-5.0.0-2.20160802git064e9db.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-0f2d6af926

-- 
You are receiving this mail because:
You are on the CC list 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 1353001] Review Request: gns3-gui - GNS3 graphical user interface

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1353001

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #11 from Fedora Update System  ---
gns3-gui-1.5.1-2.fc24, gns3-net-converter-1.3.0-2.fc24,
gns3-server-1.5.1-1.fc24 has been pushed to the Fedora 24 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-b0b7e1057b

-- 
You are receiving this mail because:
You are on the CC list 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 1352997] Review Request: gns3-net-converter - Convert old ini-style GNS3 topologies to v1+ JSON format

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1352997

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #6 from Fedora Update System  ---
gns3-gui-1.5.1-2.fc24, gns3-net-converter-1.3.0-2.fc24,
gns3-server-1.5.1-1.fc24 has been pushed to the Fedora 24 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2016-b0b7e1057b

-- 
You are receiving this mail because:
You are on the CC list 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 1359804] Review Request: golang-github-mvo5-goconfigparser - Golang implementation of Python ConfigParser

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359804



--- Comment #7 from Fedora Update System  ---
golang-github-mvo5-goconfigparser-0-0.1.gitefba970.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-0233975de8

-- 
You are receiving this mail because:
You are on the CC list 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 1361212] Review Request: perl-Ref-Util - Utility functions for checking references

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1361212



--- Comment #8 from Fedora Update System  ---
perl-Ref-Util-0.020-2.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-927d9ead09

-- 
You are receiving this mail because:
You are on the CC list 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 1321473] Review Request: diodon - Clipboard manager for GNOME, Cinnamon and other Gtk desktops

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1321473

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #23 from Fedora Update System  ---
diodon-1.5.0-1.fc23 has been pushed to the Fedora 23 testing repository. If
problems still persist, please make note of it in this bug report.
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-4ba44c90c3

-- 
You are receiving this mail because:
You are on the CC list 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 1359234] Review Request: auter - Prepare and apply updates

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359234

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



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

-- 
You are receiving this mail because:
You are on the CC list 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 1361212] Review Request: perl-Ref-Util - Utility functions for checking references

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1361212

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #7 from Fedora Update System  ---
perl-Ref-Util-0.020-2.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-6137093555

-- 
You are receiving this mail because:
You are on the CC list 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 1359804] Review Request: golang-github-mvo5-goconfigparser - Golang implementation of Python ConfigParser

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359804

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #6 from Fedora Update System  ---
golang-github-mvo5-goconfigparser-0-0.1.gitefba970.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-0137e29c1e

-- 
You are receiving this mail because:
You are on the CC list 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 1359092] Review Request: golang-gopkg-go-check-check - Rich testing extension for Go' s testing package

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359092



--- Comment #10 from Fedora Update System  ---
golang-gopkg-go-check-check-1-1.git4f90aea.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 1354378] Review Request: golang-github-urfave-cli - A simple, fast, and fun package for building command line apps in Go

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354378



--- Comment #10 from Fedora Update System  ---
golang-github-urfave-cli-1.17.0-0.1.git6011f16.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 1358248] Review Request: python-http-client - HTTP REST client, simplified for Python

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1358248



--- Comment #10 from Fedora Update System  ---
python-http-client-2.1.1-1.fc23 has been pushed to the Fedora 23 stable
repository. If problems still persist, please make note of it in this bug
report.

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


[Bug 1354398] Review Request: golang-github-golang-time - Go supplementary time packages

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354398



--- Comment #11 from Fedora Update System  ---
golang-github-golang-time-0-0.1.gita4bde12.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 1276834] Review Request: php-mongodb - MongoDB driver library

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1276834



--- Comment #15 from Fedora Update System  ---
php-mongodb-1.0.2-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 1354375] Review Request: golang-github-gengo-grpc-gateway - GRPC to JSON proxy generator

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354375



--- Comment #11 from Fedora Update System  ---
golang-github-gengo-grpc-gateway-0-0.1.gitdcb8443.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 1347147] Review Request: php-alcaeus-mongo-php-adapter - Mongo PHP Adapter

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1347147



--- Comment #17 from Fedora Update System  ---
php-alcaeus-mongo-php-adapter-1.0.5-2.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 1270405] Review Request: chromium-native_client - Google Native Client Toolchain

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1270405



--- Comment #24 from Fedora Update System  ---
chromium-52.0.2743.82-9.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 1356907] Review Request: rust - The Rust Programming Language

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1356907



--- Comment #23 from Josh Stone  ---
@tromey There's no existing path that's already in both auto-load safe-path and
python sys.path.  I do seem to need "directory" (-d) too, or else it says:

  warning: Missing auto-load script at offset 0 in section .debug_gdb_scripts

I think I figured out how to set all three automatically, by adding a new file
like /etc/gdbinit.d/rust.gdb containing:

  directory /usr/share/rust/etc
  add-auto-load-safe-path /usr/share/rust/etc
  python sys.path.append('/usr/share/rust/etc')

Does this look OK to you?


Maybe it would also be better to move this to "/usr/share/gdb/rust", so any
future "etc"-like items don't infect gdb's paths.

-- 
You are receiving this mail because:
You are on the CC list 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 1365320] New: Review Request: simplenote - Simplenote.com desktop client

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1365320

Bug ID: 1365320
   Summary: Review Request: simplenote - Simplenote.com desktop
client
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: rb...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://bostrt.fedorapeople.org/simplenote.spec
SRPM URL: https://bostrt.fedorapeople.org/simplenote-1.0.2-1.fc24.src.rpm
Description:

I've created a package for the simplenote.com Linux desktop client. I look
forward to a review and any improvement suggestions. 

simplenote is a note-taking app with storage hosted by Automattic.

Fedora Account System Username: bostrt

-- 
You are receiving this mail because:
You are on the CC list 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 1365319] New: Review Request: EPEL vcftools - newest version 0.1.14

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1365319

Bug ID: 1365319
   Summary: Review Request: EPEL vcftools - newest version 0.1.14
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: aram.minas...@yahoo.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: 
SRPM URL: https://copr-be.cloud.fedoraproject.org/results/aminasyan/vcftools/
Description: VCF file manipulation tools
Fedora Account System Username: aminasyan

While I'm not the maintainer of the package, I have built the latest release
version available for EPEL, please include this in the EPEL builds &
repositories.

-- 
You are receiving this mail because:
You are on the CC list 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 1308561] Review Request: sway - i3-compatible window manager for Wayland

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1308561



--- Comment #12 from Fabio Alessandro Locati  ---
I've improved a little bit the dependencies and so forth:


SPEC: https://fale.fedorapeople.org/rpms/sway.spec
SRPM: https://fale.fedorapeople.org/rpms/sway-0.9-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 970972] Review Request: cuisine - Chef-like functionnality for Fabric

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=970972

Athmane Madjoudj  changed:

   What|Removed |Added

 CC||athma...@gmail.com
   Assignee|nob...@fedoraproject.org|athma...@gmail.com
  Flags||fedora-review?



--- Comment #5 from Athmane Madjoudj  ---
Before doing full review, please consider the following points (You still need
a sponsor, sadly I'm not):

1) You dont need "%{__rm} -fr %{buildroot}%{python_sitelib}/paramiko" , since
the tarball does not bundle paramiko

2) You need to switch to python2 macros since Fabric is python2-only (Fabric2
will support py3 but that it built on top of Invoke/Paramiko)

3) You should run the test suite in %check section

4) if you are not planning to support EL5, you could remove
"%defattr(-,root,root,-)"

5) You must include a license file (using %license macro), already included in
the tarball => LICENSE

6) Not a blocker: after quick look into the source code, i did not see any dnf
support in package_* operations, which could make this package unusable in
Fedora if yum script (dnf wrapper) is removed from base distro.

-- 
You are receiving this mail because:
You are on the CC list 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 1362265] Review Request: yara - Malware identification tool

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1362265



--- Comment #16 from Michal Ambroz  ---
Hello Antonio

Sorry haven't paid attention closely to the fc25 version. Having static fonts
is not manifesting in FC23. It is apparently specifics of the newer version of
sphinx that it brings the theme sphinx_rtd_theme installed by default (and the
associated fonts). I have patched configuration of the doc degenration to use
the default theme (without embedded fonts) no matter what.

Updated package:
SPEC URL: https://rebus.fedorapeople.org/SPECS/yara.spec
SRPM URL: https://rebus.fedorapeople.org/SRPMS/yara-3.5.0-4.fc23.src.rpm

Build:
https://copr.fedorainfracloud.org/coprs/rebus/infosec-rebus/build/439570/

-- 
You are receiving this mail because:
You are on the CC list 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 1324863] Review Request: varnish-modules - A collection of modules extending varnish VCL

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1324863

Athmane Madjoudj  changed:

   What|Removed |Added

  Flags|fedora-review?  |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 1324863] Review Request: varnish-modules - A collection of modules extending varnish VCL

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1324863

Athmane Madjoudj  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED



--- Comment #1 from Athmane Madjoudj  ---


Package Review
==

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


Result:
===

APPROVED

Issues: (Not blocker)
===

1) You should package the latest version: 0.9.1

2) Patch varnish-modules-0.9.0-add_missing_TCP_CONGESTION_on_el5.patch should
be upstreamed if possible

3) The license seems OK, since only build system is GPL.


GPL (v2 or later)
-
varnish-modules-0.9.0/ltmain.sh

MIT/X11 (BSD like)
--
varnish-modules-0.9.0/install-sh





= MUST items =

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

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[!]: License field in the package spec file matches the actual license.
[x]: License file installed when any subpackage combination is installed.
[x]: %build honors applicable compiler flags or justifies otherwise.
[-]: 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: needed for EPEL5 compatibility
[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]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
[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.
[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]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

Generic:
[-]: Buildroot is not present
 Note: Needed for EPEL5 compatibility
[-]: 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 
[-]: Fully versioned dependency in subpackages if applicable.
[x]: Package functions as described.
[!]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[!]: Patches link to upstream bugs/comments/lists or are otherwise
 justified.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[?]: 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]: Reviewer should test th

[Bug 1363825] Review Request: perl-LWP-UserAgent-DNS-Hosts - Override LWP HTTP/HTTPS request' s host like /etc/hosts

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1363825



--- Comment #2 from Carl George  ---
Thanks for the feedback, I'm working on correcting those dings.  One question
though.  Regarding "Use plain 'perl' command", the current Perl packaging
guidelines [1] require the following line:

> Requires:  perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo 
> $version))

Should the guidelines be updated to s/%{__perl}/perl/ on that requirement?

[1]: https://fedoraproject.org/wiki/Packaging:Perl

-- 
You are receiving this mail because:
You are on the CC list 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 1354649] Review Request: nodejs-grunt-known-options - The known options used in Grunt

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354649



--- Comment #3 from Fedora Update System  ---
nodejs-grunt-known-options-1.1.0-1.fc24 has been submitted as an update to
Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-79c917ad5e

-- 
You are receiving this mail because:
You are on the CC list 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 1354649] Review Request: nodejs-grunt-known-options - The known options used in Grunt

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354649

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 1357977] Review Request: php-consolidation-output-formatters - Format text by applying transformations provided by plug-in formatters

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1357977



--- Comment #15 from Fedora Update System  ---
php-consolidation-output-formatters-1.0.0-1.el7 has been pushed to the Fedora
EPEL 7 stable repository. If problems still persist, please make note of it in
this bug report.

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

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1357875



--- Comment #15 from Fedora Update System  ---
php-dnoegel-php-xdg-base-dir-0.1-1.el7,
php-jakub-onderka-php-console-color-0.1-1.el7,
php-jakub-onderka-php-console-highlighter-0.3.2-1.el7, psysh-0.7.2-2.el7 has
been pushed to the Fedora EPEL 7 stable repository. If problems still persist,
please make note of it in this bug report.

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

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1357876



--- Comment #14 from Fedora Update System  ---
php-dnoegel-php-xdg-base-dir-0.1-1.el7,
php-jakub-onderka-php-console-color-0.1-1.el7,
php-jakub-onderka-php-console-highlighter-0.3.2-1.el7, psysh-0.7.2-2.el7 has
been pushed to the Fedora EPEL 7 stable repository. If problems still persist,
please make note of it in this bug report.

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

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1357978



--- Comment #15 from Fedora Update System  ---
php-consolidation-annotated-command-1.0.1-1.el7 has been pushed to the Fedora
EPEL 7 stable repository. If problems still persist, please make note of it in
this bug report.

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

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1357871



--- Comment #15 from Fedora Update System  ---
php-dnoegel-php-xdg-base-dir-0.1-1.el7,
php-jakub-onderka-php-console-color-0.1-1.el7,
php-jakub-onderka-php-console-highlighter-0.3.2-1.el7, psysh-0.7.2-2.el7 has
been pushed to the Fedora EPEL 7 stable repository. If problems still persist,
please make note of it in this bug report.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://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)

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1357873



--- Comment #14 from Fedora Update System  ---
php-dnoegel-php-xdg-base-dir-0.1-1.el7,
php-jakub-onderka-php-console-color-0.1-1.el7,
php-jakub-onderka-php-console-highlighter-0.3.2-1.el7, psysh-0.7.2-2.el7 has
been pushed to the Fedora EPEL 7 stable repository. If problems still persist,
please make note of it in this bug report.

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


[Bug 1354649] Review Request: nodejs-grunt-known-options - The known options used in Grunt

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354649



--- Comment #2 from Piotr Popieluch  ---
thank you 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 1324863] Review Request: varnish-modules - A collection of modules extending varnish VCL

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1324863

Athmane Madjoudj  changed:

   What|Removed |Added

 CC||athma...@gmail.com
   Assignee|nob...@fedoraproject.org|athma...@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 1354649] Review Request: nodejs-grunt-known-options - The known options used in Grunt

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354649



--- Comment #1 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/nodejs-grunt-known-options

-- 
You are receiving this mail because:
You are on the CC list 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 1359092] Review Request: golang-gopkg-go-check-check - Rich testing extension for Go' s testing package

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359092

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-08-08 16:29:28



-- 
You are receiving this mail because:
You are on the CC list 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 1358248] Review Request: python-http-client - HTTP REST client, simplified for Python

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1358248

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-08-08 16:29:33



-- 
You are receiving this mail because:
You are on the CC list 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 1359092] Review Request: golang-gopkg-go-check-check - Rich testing extension for Go' s testing package

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359092



--- Comment #9 from Fedora Update System  ---
golang-gopkg-go-check-check-1-1.git4f90aea.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 1358248] Review Request: python-http-client - HTTP REST client, simplified for Python

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1358248



--- Comment #9 from Fedora Update System  ---
python-http-client-2.1.1-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 1354398] Review Request: golang-github-golang-time - Go supplementary time packages

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354398



--- Comment #10 from Fedora Update System  ---
golang-github-golang-time-0-0.1.gita4bde12.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 1354378] Review Request: golang-github-urfave-cli - A simple, fast, and fun package for building command line apps in Go

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354378

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-08-08 16:28:27



-- 
You are receiving this mail because:
You are on the CC list 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 1347147] Review Request: php-alcaeus-mongo-php-adapter - Mongo PHP Adapter

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1347147
Bug 1347147 depends on bug 1276834, which changed state.

Bug 1276834 Summary: Review Request: php-mongodb - MongoDB driver library
https://bugzilla.redhat.com/show_bug.cgi?id=1276834

   What|Removed |Added

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



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


[Bug 1354398] Review Request: golang-github-golang-time - Go supplementary time packages

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354398

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-08-08 16:28:35



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


[Bug 1276834] Review Request: php-mongodb - MongoDB driver library

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1276834



--- Comment #14 from Fedora Update System  ---
php-mongodb-1.0.2-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 1354378] Review Request: golang-github-urfave-cli - A simple, fast, and fun package for building command line apps in Go

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354378



--- Comment #9 from Fedora Update System  ---
golang-github-urfave-cli-1.17.0-0.1.git6011f16.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 1276834] Review Request: php-mongodb - MongoDB driver library

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1276834

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-08-08 16:28:18



-- 
You are receiving this mail because:
You are on the CC list 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 1347147] Review Request: php-alcaeus-mongo-php-adapter - Mongo PHP Adapter

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1347147



--- Comment #16 from Fedora Update System  ---
php-alcaeus-mongo-php-adapter-1.0.5-2.fc24 has been pushed to the Fedora 24
stable repository. If problems still persist, please make note of it in this
bug report.

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


[Bug 1347147] Review Request: php-alcaeus-mongo-php-adapter - Mongo PHP Adapter

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1347147

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-08-08 16:27:08



-- 
You are receiving this mail because:
You are on the CC list 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 1354375] Review Request: golang-github-gengo-grpc-gateway - GRPC to JSON proxy generator

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354375



--- Comment #10 from Fedora Update System  ---
golang-github-gengo-grpc-gateway-0-0.1.gitdcb8443.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 1354375] Review Request: golang-github-gengo-grpc-gateway - GRPC to JSON proxy generator

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1354375

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-08-08 16:27:13



-- 
You are receiving this mail because:
You are on the CC list 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 1324020] Review Request: cassandra - OpenSource database Apache Cassandra

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1324020
Bug 1324020 depends on bug 1348231, which changed state.

Bug 1348231 Summary: update python-cassandra-driver
https://bugzilla.redhat.com/show_bug.cgi?id=1348231

   What|Removed |Added

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



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


[Bug 1364535] Review Request: ehcache2 - Java in-process cache

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1364535

gil cattaneo  changed:

   What|Removed |Added

 Blocks||182235 (FE-Legal)




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 1364535] Review Request: ehcache2 - Java in-process cache

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1364535



--- Comment #1 from gil cattaneo  ---
The core is under ASL 2.0 license
http://svn.terracotta.org/svn/ehcache/tags/ehcache-2.9.0/distribution/src/main/assembly/root/licenses/EHCACHE-CORE-LICENSE.txt
the rest use:
http://svn.terracotta.org/svn/ehcache/tags/ehcache-2.9.0/distribution/src/main/assembly/root/licenses/EHCACHE-TERRACOTTA-LICENSE.txt

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


[Bug 1362265] Review Request: yara - Malware identification tool

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1362265



--- Comment #15 from Antonio Trande  ---
(In reply to Michal Ambroz from comment #14)
> >[!]: Changelog in prescribed format.
> rpmlint yara.spec doesn't show any errors in the prescribed format of
> Changelog.

rpmlint is not able to recognize something like this.
Please, take a look to what guidelines say.
http://fedoraproject.org/wiki/Packaging:Guidelines#Changelogs

> 
> >[!]: Avoid bundling fonts in non-fonts packages.
> I guess this is some false positive - I am not aware of any fonts being
> bundled to yara package.

I have indicated where they are:
yara-doc-3.5.0-3.fc25.noarch.rpm/usr/share/doc/yara-doc/html/_static/fonts

-- 
You are receiving this mail because:
You are on the CC list 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 1364603] Review Request: python-XStatic-Angular-Schema-Form - Angular-Schema-Form JavaScript library packaged for setuptools (easy_install) / pip.

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1364603



--- Comment #9 from David Moreau Simard  ---
Spec URL:
https://raw.githubusercontent.com/dmsimard/python-XStatic-Angular-Schema-Form-distgit/master/python-XStatic-Angular-Schema-Form.spec
SRPM URL:
https://github.com/dmsimard/python-XStatic-Angular-Schema-Form-distgit/raw/master/python-XStatic-Angular-Schema-Form-0.8.13.0-1.fc24.src.rpm
Description: Angular-Schema-Form JavaScript library packaged for setuptools
(easy_install) / pip.

Generate forms from JSON schemas using AngularJS.
Fedora Account System Username: dmsimard

Koji scratch build available at
http://koji.fedoraproject.org/koji/taskinfo?taskID=15178436

Note: This last update adds Requires: on three dependencies, of which two are
in-flight with ongoing reviews in:
- python-XStatic-tv4: https://bugzilla.redhat.com/show_bug.cgi?id=1364620
- python-XStatic-objectpath:
https://bugzilla.redhat.com/show_bug.cgi?id=1364607

The dependencies are expressed here:
https://github.com/json-schema-form/angular-schema-form/blob/59e4e2c8d0c50ffb5d65671f28225a53e691bb24/package.json#L21
angular-sanitize is embedded in angular and is met by python-XStatic-Angular.

-- 
You are receiving this mail because:
You are on the CC list 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 1362265] Review Request: yara - Malware identification tool

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1362265



--- Comment #14 from Michal Ambroz  ---
>[!]: Changelog in prescribed format.
rpmlint yara.spec doesn't show any errors in the prescribed format of
Changelog.

>[!]: Avoid bundling fonts in non-fonts packages.
I guess this is some false positive - I am not aware of any fonts being bundled
to yara package.

>[!]: Package should not use obsolete m4 macros
Both old and new macros used in the upstream project. 
Issue reported upstream https://github.com/VirusTotal/yara/issues/491
As it is "should" not "must" requirement and currently doesn't represent build
issue to Fedora package on all supported releases I decided to not patch at
this point.

-- 
You are receiving this mail because:
You are on the CC list 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 1361340] Review Request: python-fadvise - Python interface to posix_fadvise(2)

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1361340



--- Comment #2 from Piotr Popieluch  ---
(In reply to Igor Gnatenko from comment #1)
> > %{python2_sitearch}
> don't own directory.

aj.. this should be better
> %{python2_sitearch}/*

spec/srpm updated, same url.

-- 
You are receiving this mail because:
You are on the CC list 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 1361806] Review Request: python-marrow-mailer - A light-weight modular mail delivery framework for Python

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1361806

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 1361806] Review Request: python-marrow-mailer - A light-weight modular mail delivery framework for Python

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1361806



--- Comment #6 from Fedora Update System  ---
python-marrow-mailer-4.0.1-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-eafaa37cd8

-- 
You are receiving this mail because:
You are on the CC list 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

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1343661

Jan Pokorný  changed:

   What|Removed |Added

 Status|ON_QA   |MODIFIED
   Fixed In Version|clufter-0.59.4-1.el7|clufter-0.59.5-1.el7



-- 
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 1079064] Review Request: btbuilder - Role-playing game construction set in the style of the Bard' s Tale Construction Set

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079064

Ben Rosser  changed:

   What|Removed |Added

 CC||rosser@gmail.com
 Depends On||1227972




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1227972
[Bug 1227972] Review Request: SDL_mng - Library to load MNG files for SDL
-- 
You are receiving this mail because:
You are on the CC list 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 1227972] Review Request: SDL_mng - Library to load MNG files for SDL

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1227972

Ben Rosser  changed:

   What|Removed |Added

 Blocks||1079064




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1079064
[Bug 1079064] Review Request: btbuilder - Role-playing game construction
set in the style of the Bard's Tale Construction Set
-- 
You are receiving this mail because:
You are on the CC list 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 1364603] Review Request: python-XStatic-Angular-Schema-Form - Angular-Schema-Form JavaScript library packaged for setuptools (easy_install) / pip.

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1364603



--- Comment #8 from David Moreau Simard  ---
Note: I will update this review shortly -- we identified that
angular-schema-form has runtime dependencies that need to be expressed in the
spec.

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


[Bug 1361806] Review Request: python-marrow-mailer - A light-weight modular mail delivery framework for Python

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1361806



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

-- 
You are receiving this mail because:
You are on the CC list 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 1353169] Review Request: python-nikola - A static website and blog generator

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1353169



--- Comment #8 from Neal Gompa  ---
If /usr/bin/nikola is functionally equivalent between py2 and py3, why not just
switch it all to py3? Especially since upstream prefers py3 anyway?

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


[Bug 1343738] Review Request: lxqt-admin - LXQt system administration tool

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1343738



--- Comment #11 from Raphael Groner  ---
GCC in el7 is too old, sources need C++11 features.

-- 
You are receiving this mail because:
You are on the CC list 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 1232186] Review Request: python-pysaml2 - Python implementation of SAML Version 2

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1232186

Raphael Groner  changed:

   What|Removed |Added

 CC||projects...@smart.ms



--- Comment #11 from Raphael Groner  ---
F22 is EOL. 

For epel7, there's a successful build. Can you file a new bug about your
request?
http://koji.fedoraproject.org/koji/buildinfo?buildID=703394

-- 
You are receiving this mail because:
You are on the CC list 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 1343738] Review Request: lxqt-admin - LXQt system administration tool

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1343738



--- Comment #10 from Fedora Update System  ---
lxqt-admin-0.10.0-3.20160729git2f95601.fc24 has been submitted as an update to
Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-767d2dbac4

-- 
You are receiving this mail because:
You are on the CC list 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 1343738] Review Request: lxqt-admin - LXQt system administration tool

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1343738

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|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 1361334] Review Request: rubygem-rails-controller-testing - Extracting `assigns` and `assert_template` from ActionDispatch

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1361334



--- Comment #6 from Jun Aruga  ---
Hi,  František Dvořák
Just moment. Tomorrow I will reply for your comment here.

-- 
You are receiving this mail because:
You are on the CC list 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 1361212] Review Request: perl-Ref-Util - Utility functions for checking references

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1361212



--- Comment #5 from Fedora Update System  ---
perl-Ref-Util-0.020-2.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-927d9ead09

-- 
You are receiving this mail because:
You are on the CC list 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 1361212] Review Request: perl-Ref-Util - Utility functions for checking references

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1361212



--- Comment #6 from Fedora Update System  ---
perl-Ref-Util-0.020-2.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-f5b3935fb7

-- 
You are receiving this mail because:
You are on the CC list 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 1364620] Review Request: python-XStatic-tv4 - tv4 - Tiny Validator ( for v4 JSON Schema) JavaScript library packaged for setuptools ( easy_install) / pip.

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1364620

Vadim Rutkovsky  changed:

   What|Removed |Added

 CC||vrutk...@redhat.com



--- Comment #5 from Vadim Rutkovsky  ---
Informal package review. Looks good to me

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.
[x]: License file installed when any subpackage combination is installed.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[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
[-]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

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

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[-]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
 files.
[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]: Packager, Vendor, PreReq, Copyright tags shoul

[Bug 1361212] Review Request: perl-Ref-Util - Utility functions for checking references

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1361212



--- Comment #4 from Fedora Update System  ---
perl-Ref-Util-0.020-2.el6 has been submitted as an update to Fedora EPEL 6.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-da4724d0a9

-- 
You are receiving this mail because:
You are on the CC list 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 1361212] Review Request: perl-Ref-Util - Utility functions for checking references

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1361212

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 1361212] Review Request: perl-Ref-Util - Utility functions for checking references

2016-08-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1361212



--- Comment #3 from Fedora Update System  ---
perl-Ref-Util-0.020-2.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-6137093555

-- 
You are receiving this mail because:
You are on the CC list 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


  1   2   3   >