[Bug 1394167] Review Request: perl-Data-ICal-TimeZone - Time zones for Data::ICal

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394167



--- Comment #6 from Fedora Update System  ---
perl-Data-ICal-TimeZone-1.23-1.fc24 has been submitted as an update to Fedora
24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-9d8306c742

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1369708] Review Request: tpm2-tss - TPM2.0 Software Stack

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1369708



--- Comment #47 from yunying@intel.com ---
(In reply to Raphael Groner from comment #45)
> Please follow the official process to get sponsored as a packager:
> https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group

I've done some reviews, see comments in bug 1385331 and bug 1366047. Will try
to do more reviews in future.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394167] Review Request: perl-Data-ICal-TimeZone - Time zones for Data::ICal

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394167



--- Comment #5 from Fedora Update System  ---
perl-Data-ICal-TimeZone-1.23-1.fc25 has been submitted as an update to Fedora
25. https://bodhi.fedoraproject.org/updates/FEDORA-2016-d9ac25fe7a

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394167] Review Request: perl-Data-ICal-TimeZone - Time zones for Data::ICal

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394167



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394167] Review Request: perl-Data-ICal-TimeZone - Time zones for Data::ICal

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394167

Petr Pisar  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED
   Fixed In Version||perl-Data-ICal-TimeZone-1.2
   ||3-1.fc26



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1393899] Review Request: libdirq - C implementation of the simple directory queue algorithm

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1393899



--- Comment #11 from Fedora Update System  ---
libdirq-0.4-1.el6 has been submitted as an update to Fedora EPEL 6.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-7a13703365

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1369720] Review Request: tpm2-tools - a TPM2.0 testing tool build upon TPM2.0-TSS

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1369720



--- Comment #16 from yunying@intel.com ---
Upstream has released v1.1.0 at early Nov 2016. Here I updated this package
from v1.1-beta_1 to v1.1.0.

Updated SPEC:
https://raw.githubusercontent.com/yunyings/share/master/tpm2-tools.spec
Updated SRPM:
https://github.com/yunyings/share/blob/master/tpm2-tools-1.1.0-1.el7.src.rpm?raw=true

Succeeded COPR build:
https://copr.fedorainfracloud.org/coprs/yunyings/tpm2-tools/build/476799/

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1369708] Review Request: tpm2-tss - TPM2.0 Software Stack

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1369708



--- Comment #46 from yunying@intel.com ---
Upstream has released 1.0 at Nov 3 2016:
https://github.com/01org/TPM2.0-TSS/releases

I updated this package to 1.0 upstream as well, to make sure this package is up
to date.

Updated SPEC:
https://raw.githubusercontent.com/yunyings/share/master/tpm2-tss.spec
Updated SRPM:
https://github.com/yunyings/share/blob/master/tpm2-tss-1.0-1.el7.src.rpm?raw=true

Succeeded Koji build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=16456652
Succeeded COPR build:
https://copr.fedorainfracloud.org/coprs/yunyings/tpm2-tss/build/476765/

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1386938] Review Request: libprelude - Prelude Library

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1386938



--- Comment #9 from Thomas Andrejak  ---
Hello

Thanks for the review

Here is the new src file :
https://copr-be.cloud.fedoraproject.org/results/totol/Prelude/fedora-rawhide-x86_64/00476797-libprelude/libprelude-3.1.0-24.fc26.src.rpm
And here the new spec file :
https://www.prelude-siem.org/pkg/src/3.1.0/libprelude.spec

I have some questions :

- For soname at the end of the subpackage name, this is what other linux
distribution required, that's why I do it. It make you able to install multiple
libprelude versions in parallel. Do I really have to make only one subpackage
with all ".so" files and do not precise the soname in the subpackage name ?

- Can you tell which macro you are talking about ?

- %description is required when you use "%package -n"

OK for evrything else :) See the spec file

--

Thanks for sponsoring :) I will do some reviews this week

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1344276] Review Request: gdeploy - Tool to deploy GlusterFS clusters and other utilities

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1344276



--- Comment #11 from Ken Dreyer  ---
It seems like you're mixing the Release value and Version values here?

Release is a number that is "downstream" (Fedora) only. Other Fedora
maintainers, like ProvenPackagers or Fedora rel-eng, will sometimes bump this
Release integer, so it's not something you can completely keep in sync between
Fedora downstream and Gluster upstream.

It would be best to avoid having the Release number in your upstream tarballs.
To put it another way, "v%{version}-%{release}.tar.gz" should just be
"v%{version}.tar.gz"

The "/usr/local/bin/gluster-replace-node" file should not be in /usr/local (see
https://fedoraproject.org/wiki/Packaging:Guidelines#No_Files_or_Directories_under_.2Fsrv.2C_.2Fusr.2Flocal.2C_or_.2Fhome.2F.24USER)

The license seems unclear here, because the "LICENSE" file from upstream is the
GPLv2, but the .spec here says GPLv3. Which is it? Also, the LICENSE file
should be included in the package, via the %license directive.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1344276] Review Request: gdeploy - Tool to deploy GlusterFS clusters and other utilities

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1344276



--- Comment #12 from Ken Dreyer  ---
Would you please post raw direct links to the latest .spec and .src.rpm so that
the fedora-review tool can process them?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1393044] Review Request: su-exec - switch user and group id, setgroups and exec

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1393044

Elan Ruusamäe  changed:

   What|Removed |Added

 CC||g...@delfi.ee



--- Comment #5 from Elan Ruusamäe  ---
not that i use fedora (or derivates), but systemd-run and su-exec serve totally
different purpose. so-exec purpose was to just witch user without any pam,
systemd. having minimal executable size. it has no dependencies like running
systemd, so it could be used in docker containers, chroots.

however, i think su-exec is not mature enough, looking at it's open bugreports

https://github.com/ncopa/su-exec/issues/10
https://github.com/ncopa/su-exec/issues/8
https://github.com/ncopa/su-exec/issues/6
https://github.com/ncopa/su-exec/issues/2

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394146] Review Request: perl-Convert-Base64 - Encoding and decoding of Base64 strings

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394146

Athos Ribeiro  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Athos Ribeiro  ---
BuildRequires are ok.

binary Requires are ok.

binary Provides are ok.

fedora-review complains about BuildRequires make and findutils, since those are
already included in the current minimum build environment.

Guidelines are vague on those since "RPM deps may change" and the definition of
"basic shell scripts" is not clear. So I believe those are up to the packager.

package looks good to me. Approved

Package Review
==

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

= 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]: If the package is under multiple licenses, the licensing breakdown
 must be documented in the spec.
[-]: Package does not own files or directories owned by other packages.
[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.
[x]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 bytes in 2 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: No rpmlint messages.
[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 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

Perl:
[x]: Package contains the mandatory BuildRequires and Requires:.
[x]: CPAN urls should be non-versioned.

= 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]: 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.
[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, 

[Bug 1394146] Review Request: perl-Convert-Base64 - Encoding and decoding of Base64 strings

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394146

Athos Ribeiro  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||athoscribe...@gmail.com
   Assignee|nob...@fedoraproject.org|athoscribe...@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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1389955] Review Request: gnome-shell-extension-no-topleft-hot-corner - Disable the "hot corner" in GNOME Shell

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1389955

terrycloth  changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |gnome-shell-extension-no-to |gnome-shell-extension-no-to
   |p-left-hot-corner - Disable |pleft-hot-corner - Disable
   |the "hot corner" in GNOME   |the "hot corner" in GNOME
   |Shell   |Shell



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394969] Review Request: python-pluginbase - A support library for building plugins systems

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394969

Athos Ribeiro  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED



--- Comment #1 from Athos Ribeiro  ---
As you can see in [1], ..."You should not include the directories
%{python3_sitearch}/__pycache__ or %{python3_sitelib}/__pycache__ because they
are already owned by the python3-libs package."

 The package owns

/usr/lib/python3.5/site-packages/__pycache__

when it should not.

I believe it would be better to be a little more verbose with the wildcards to
avoid these kinds of problem.

There is a %check section in the spec file, but during the build, tests are not
being executed at all.

If you BuildRequires make and pythonX-pytest you could run the test suite with
`make test`.

[1] https://fedoraproject.org/wiki/Packaging:Python#Byte_compiling


Here is a fedora-review checklist:

Package Review
==

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



= 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.
[!]: Package does not own files or directories owned by other packages.
 Note: Dirs in package are owned also by: /usr/lib/python3.5/site-
 packages/__pycache__(system-python-libs, python3-pyparsing)
[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.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 20480 bytes in 2 files.
[!]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: No rpmlint messages.
[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]: 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.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in 

[Bug 1391892] Review Request: python-vitrageclient - Python client for Virage REST API

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1391892



--- Comment #11 from Alan Pevec  ---
This OpenStack package has been imported into RDO:
https://review.rdoproject.org/r/#/q/topic:add-vitrageclient

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1279112] Review Request: kiss-fft - Fast Fourier Transform library

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279112



--- Comment #13 from Zbigniew Jędrzejewski-Szmek  ---
(In reply to František Dvořák from comment #11)
> There is difference between:
>   make CFLAGS=...
> and:
>   CFLAGS=... make
> 
> I would rather not change it. It's only cosmetic hint anyway, right?

Yes, both should work.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394969] Review Request: python-pluginbase - A support library for building plugins systems

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394969

Athos Ribeiro  changed:

   What|Removed |Added

 CC||athoscribe...@gmail.com
   Assignee|nob...@fedoraproject.org|athoscribe...@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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394969] Review Request: python-pluginbase - A support library for building plugins systems

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394969

Fabian Affolter  changed:

   What|Removed |Added

 Blocks||1394980 (wafw00f)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1394980
[Bug 1394980] Review Request: wafw00f - A tool to identifies and
fingerprints Web Application Firewall (WAF)
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394980] Review Request: wafw00f - A tool to identifies and fingerprints Web Application Firewall (WAF)

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394980

Fabian Affolter  changed:

   What|Removed |Added

 Blocks||563471 (FE-SECLAB)
 Depends On||1394969
  Alias||wafw00f




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=563471
[Bug 563471] Tracker: Review Requests for Fedora Security Lab related
packages
https://bugzilla.redhat.com/show_bug.cgi?id=1394969
[Bug 1394969] Review Request: python-pluginbase - A support library for
building plugins systems
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394980] New: Review Request: wafw00f - A tool to identifies and fingerprints Web Application Firewall (WAF)

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394980

Bug ID: 1394980
   Summary: Review Request: wafw00f - A tool to identifies and
fingerprints Web Application Firewall (WAF)
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: m...@fabian-affolter.ch
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://fab.fedorapeople.org/packages/SRPMS/wafw00f.spec
SRPM URL:
https://fab.fedorapeople.org/packages/SRPMS/wafw00f-0.9.4-1.fc24.src.rpm

Project URL: https://github.com/sandrogauci/wafw00f

Description:
WAFW00F identifies and fingerprints Web Application Firewall (WAF) products.

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

rpmlint output:
[fab@laptop016 SRPMS]$ rpmlint wafw00f-*
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

wafw00f.noarch: W: no-manual-page-for-binary wafw00f
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Fedora Account System Username: fab

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1391892] Review Request: python-vitrageclient - Python client for Virage REST API

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1391892

Alan Pevec  changed:

   What|Removed |Added

 Blocks||1342987




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1342987
[Bug 1342987] Review Request: openstack-vitrage - OpenStack RCA (Root Cause
Analysis) Engine
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394969] New: Review Request: python-pluginbase - A support library for building plugins systems

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394969

Bug ID: 1394969
   Summary: Review Request: python-pluginbase - A support library
for building plugins systems
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: m...@fabian-affolter.ch
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://fab.fedorapeople.org/packages/SRPMS/python-pluginbase.spec
SRPM URL:
https://fab.fedorapeople.org/packages/SRPMS/python-pluginbase-0.5-1.fc24.src.rpm

Project URL: https://github.com/mitsuhiko/pluginbase

Description:
PluginBase is a module for Python that enables the development of flexible
plugin systems in Python.

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

rpmlint output:
[fab@laptop016 noarch]$ rpmlint python*-plu*
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

[fab@laptop016 SRPMS]$ rpmlint python3-pluginbase*
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

Fedora Account System Username: fab

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1279104] Review Request: python-pbkdf2 - password-based key derivation in pure Python

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279104

Jonny Heggheim  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed|2016-04-12 15:08:47 |2016-11-14 16:49:17



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1187084] Review Request: electrum - a lightweight Bitcoin client

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1187084
Bug 1187084 depends on bug 1279104, which changed state.

Bug 1279104 Summary: Review Request: python-pbkdf2 - password-based key 
derivation in pure Python
https://bugzilla.redhat.com/show_bug.cgi?id=1279104

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NEXTRELEASE



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1187084] Review Request: electrum - a lightweight Bitcoin client

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1187084
Bug 1187084 depends on bug 1187082, which changed state.

Bug 1187082 Summary: Review Request: python-slowaes - An Implementation of AES 
in python
https://bugzilla.redhat.com/show_bug.cgi?id=1187082

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |NEXTRELEASE



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1187082] Review Request: python-slowaes - An Implementation of AES in python

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1187082

Jonny Heggheim  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2016-11-14 16:49:02



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394962] New: Review Request: clevis - Automated decryption framework

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394962

Bug ID: 1394962
   Summary: Review Request: clevis - Automated decryption
framework
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: npmccal...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://copr-dist-git.fedorainfracloud.org/cgit/npmccallum/clevis/clevis.git/plain/clevis.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/npmccallum/clevis/fedora-rawhide-x86_64/00476674-clevis/clevis-1-1.fc26.src.rpm
Description: Automated decryption framework
Fedora Account System Username:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1279112] Review Request: kiss-fft - Fast Fourier Transform library

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279112



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1279112] Review Request: kiss-fft - Fast Fourier Transform library

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279112

František Dvořák  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|zbys...@in.waw.pl



--- Comment #11 from František Dvořák  ---
(In reply to Zbigniew Jędrzejewski-Szmek from comment #10)
> Please remove the definition of %version: just put the version in Version,
> rpm defines the macro %version automatically.
> 

Right, it will be better to not override %{version} macro. Updated.

> All %descriptions have an empty line at the beggining, it should be removed.
> 

Nice catch, updated.

> I'd change:
> for type in float double int16 int32; do
> mkdir build_${type}
> pushd build_${type}
> CFLAGS="%{optflags}" LDFLAGS="%{?__global_ldflags}" \
> DATATYPE=${type} \
> make %{?_smp_mflags} -f %{SOURCE1} libdir=%{_libdir} srcdir=..
> popd
> done
> →
> for type in float double int16 int32; do
> mkdir build_${type}
> %make_build -f %{SOURCE1} libdir=%{_libdir} srcdir=.. \
> CFLAGS="%{optflags}" LDFLAGS="%{?__global_ldflags}" \
> DATATYPE=${type}
> done
> (only because it's simpler...)
> 

There is difference between:

  make CFLAGS=...

and:

  CFLAGS=... make

I would rather not change it. It's only cosmetic hint anyway, right?

> %{!?_licensedir:%global license %doc}
> is not necessary, %license is now defined in all Fedora and EPEL versions.
>

OK, removed.

> Please add a comment that fftw is only used for benchmarking during build
> (FFTW is GPLv2).
> 

Comment added.

Thanks for the review!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1386774] Review Request: kf5-libktorrent - Library providing torrent downloading code

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1386774

Randy Barlow  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||ra...@electronsweatshop.com
   Assignee|nob...@fedoraproject.org|ra...@electronsweatshop.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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1393996] Review Request: python-novajoin - Nova integration to enroll IPA clients

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1393996



--- Comment #3 from Rob Crittenden  ---
Spec URL: https://rcritten.fedorapeople.org/python-novajoin.spec
SRPM URL:
https://rcritten.fedorapeople.org/python-novajoin-1.0.7-1.fc23.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394952] Review Request: python-avocado - Framework with tools and libraries for Automated Testing

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394952

Stephen Gallagher  changed:

   What|Removed |Added

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



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1383781] Review Request: sunflow - A rendering system for photo-realistic image synthesis

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1383781

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-11-14 16:01:35



--- Comment #22 from Fedora Update System  ---
sunflow-0.07.3-8097f6d.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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1389784] Review Request: python-scour - An SVG scrubber

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1389784



--- Comment #13 from Fedora Update System  ---
python-scour-0.35-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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394952] New: Review Request: python-avocado - Framework with tools and libraries for Automated Testing

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394952

Bug ID: 1394952
   Summary: Review Request: python-avocado - Framework with tools
and libraries for Automated Testing
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: mmath...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://merlinm.fedorapeople.org/python-avocado.spec
SRPM URL: https://merlinm.fedorapeople.org/python-avocado-43.0-1.fc24.src.rpm
COPR build URL:
https://copr.fedoraproject.org/coprs/merlinm/avocado/build/476635/
Description:

Avocado is a set of tools and libraries (what people call
these days a framework) to perform automated testing.

Fedora Account System Username: merlinm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1382859] Review Request: kelbt - Backtracking LR Parsing

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1382859



--- Comment #6 from Fedora Update System  ---
kelbt-0.16-2.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-457ec9286f

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1382859] Review Request: kelbt - Backtracking LR Parsing

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1382859



--- Comment #5 from Fedora Update System  ---
kelbt-0.16-2.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-b0389dc0da

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1382859] Review Request: kelbt - Backtracking LR Parsing

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1382859

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1387201] Review Request: golang-github-spf13-afero - A FileSystem Abstraction System for Go

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1387201

Athos Ribeiro  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||athoscribe...@gmail.com
 Resolution|--- |DUPLICATE
Last Closed||2016-11-14 13:54:24



--- Comment #2 from Athos Ribeiro  ---


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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1386625] Review Request: gotun - Tool to run tests on OpenStack

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1386625
Bug 1386625 depends on bug 1387201, which changed state.

Bug 1387201 Summary: Review Request: golang-github-spf13-afero - A FileSystem 
Abstraction System for Go
https://bugzilla.redhat.com/show_bug.cgi?id=1387201

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |DUPLICATE



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1206886] Review Request: golang-github-spf13-afero - A FileSystem Abstraction System for Go

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1206886

Athos Ribeiro  changed:

   What|Removed |Added

 Blocks||1386625



--- Comment #2 from Athos Ribeiro  ---
*** Bug 1387201 has been marked as a duplicate of this bug. ***


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1386625
[Bug 1386625] Review Request: gotun - Tool to run tests on OpenStack
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1386625] Review Request: gotun - Tool to run tests on OpenStack

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1386625

Athos Ribeiro  changed:

   What|Removed |Added

 Depends On||1206886




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1206886
[Bug 1206886] Review Request: golang-github-spf13-afero - A FileSystem
Abstraction System for Go
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394923] New: Review Request: golang-github-fortytw2-leaktest - Goroutine Leak Detector

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394923

Bug ID: 1394923
   Summary: Review Request: golang-github-fortytw2-leaktest -
Goroutine Leak Detector
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: athoscribe...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://athoscr.fedorapeople.org/packaging/golang-github-fortytw2-leaktest.spec
SRPM URL:
https://athoscr.fedorapeople.org/packaging/golang-github-fortytw2-leaktest-0-0.1.git0db74e8.fc26.src.rpm

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

Description:
Refactored, tested variant of the goroutine leak detector found in both
net/http tests and the cockroachdb source tree.

Fedora Account System Username: athoscr

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1366862] Review Request: gimp-focusblur-plugin - Simulate an out-of-focus blur

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1366862



--- Comment #4 from Fedora Update System  ---
gimp-focusblur-plugin-3.2.6-1.fc25 has been submitted as an update to Fedora
25. https://bodhi.fedoraproject.org/updates/FEDORA-2016-b0d9130199

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1366862] Review Request: gimp-focusblur-plugin - Simulate an out-of-focus blur

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1366862



--- Comment #5 from Fedora Update System  ---
gimp-focusblur-plugin-3.2.6-1.fc24 has been submitted as an update to Fedora
24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-fd38fa3162

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1366862] Review Request: gimp-focusblur-plugin - Simulate an out-of-focus blur

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1366862

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1370533] Review Request: python-django-pgjson - PostgreSQL json field support for Django

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1370533

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1370533] Review Request: python-django-pgjson - PostgreSQL json field support for Django

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1370533



--- Comment #7 from Fedora Update System  ---
python-django-pgjson-0.3.1-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-599b249678

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1206877] Review Request: golang-github-dchest-cssmin - Go port of Ryan Grove's cssmin.

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1206877

Athos Ribeiro  changed:

   What|Removed |Added

 CC||athoscribe...@gmail.com



--- Comment #3 from Athos Ribeiro  ---
There's a new commit with a LICENSE file [1]

https://github.com/dchest/cssmin/commit/fb8d9b44afdc258bfff6052d3667521babcb2239

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1353615] Review Request: apfloat-java - High performance arbitrary precision arithmetic library for Java

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1353615

gil cattaneo  changed:

   What|Removed |Added

 Status|NEW |CLOSED
   Fixed In Version||apfloat-java-1.8.2-1.fc26
 Resolution|--- |RAWHIDE
Last Closed||2016-11-14 13:13:53



--- Comment #11 from gil cattaneo  ---
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=16451116

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1389016] Review Request: libxsmm - Library for small matrix-matrix multiplications on Intel x86_64 (e.g. for cp2k)

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1389016



--- Comment #32 from Fedora Update System  ---
libxsmm-1.5.2-2.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-041c46cdc9

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1389016] Review Request: libxsmm - Library for small matrix-matrix multiplications on Intel x86_64 (e.g. for cp2k)

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1389016



--- Comment #30 from Fedora Update System  ---
libxsmm-1.5.2-2.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-02fb3b4515

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1389016] Review Request: libxsmm - Library for small matrix-matrix multiplications on Intel x86_64 (e.g. for cp2k)

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1389016



--- Comment #31 from Fedora Update System  ---
libxsmm-1.5.2-2.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-120603edb4

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1389016] Review Request: libxsmm - Library for small matrix-matrix multiplications on Intel x86_64 (e.g. for cp2k)

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1389016

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1389016] Review Request: libxsmm - Library for small matrix-matrix multiplications on Intel x86_64 (e.g. for cp2k)

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1389016



--- Comment #29 from Fedora Update System  ---
libxsmm-1.5.2-2.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-e9e3962e27

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1370611] Review Request: python-serpy - Ridiculously fast object serialization

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1370611



--- Comment #4 from Fedora Update System  ---
python-serpy-0.1.1-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-cd7ac9f5c4

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1370611] Review Request: python-serpy - Ridiculously fast object serialization

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1370611



--- Comment #3 from Fedora Update System  ---
python-serpy-0.1.1-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-15798a6a8e

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1370611] Review Request: python-serpy - Ridiculously fast object serialization

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1370611

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1362130] Review Request: olpcau-abc123-fonts - A nice font for kids/ readability

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1362130

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

  Flags||fedora-review?



--- Comment #7 from Zbigniew Jędrzejewski-Szmek  ---
developed for literacy John Greatorex → missing "by"

The package looks good. I'll approve the package once you are sponsored into
the packagers group. Any chance you could review some other packages?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1376867] Review Request: python3-pycurl - Python interface to libcurl for Python 3

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1376867



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1366862] Review Request: gimp-focusblur-plugin - Simulate an out-of-focus blur

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1366862



--- Comment #3 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/gimp-focusblur-plugin

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1382859] Review Request: kelbt - Backtracking LR Parsing

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1382859



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1387250] Review Request: python-lazr-smtptest - Test framework for SMTP-based applications

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1387250

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 Status|ASSIGNED|POST
  Flags|fedora-review?  |fedora-review+



--- Comment #3 from Zbigniew Jędrzejewski-Szmek  ---
+ latest version
+ package name is OK
+ license is acceptable (LGPLv3)
+ license is specified correctly
+ builds and install OK
+ provides/requires look correct
+ fedora-review is happy
+ no scriptlets
+ python packaging macros are used
+ %python_provide is used
+ %check passes

Nice. Package is APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1367819] Review Request: znc-clientbuffer - ZNC module for client specific buffers

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1367819



--- Comment #4 from Fedora Update System  ---
znc-clientbuffer-0-0.1gitfe0f368.fc25 has been submitted as an update to Fedora
25. https://bodhi.fedoraproject.org/updates/FEDORA-2016-bb95d1dd68

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1367819] Review Request: znc-clientbuffer - ZNC module for client specific buffers

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1367819

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1367819] Review Request: znc-clientbuffer - ZNC module for client specific buffers

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1367819



--- Comment #3 from Fedora Update System  ---
znc-clientbuffer-0-0.1gitfe0f368.fc24 has been submitted as an update to Fedora
24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-c61fb204df

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1187084] Review Request: electrum - a lightweight Bitcoin client

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1187084



--- Comment #26 from gil cattaneo  ---
Issues:

[?]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "MIT/X11 (BSD like)", "BSD (2 clause)", "*No copyright* MIT/X11
 (BSD like)", "Unknown or generated". 107 files have unknown license.
 Detailed output of licensecheck in
 /home/gil/1187084-electrum/licensecheck.txt

BSD (2 clause)
--
Electrum-2.7.12/lib/socks.py

The following source files are without license headers.

Unknown or generated

Electrum-2.7.12/gui/kivy/i18n.py
Electrum-2.7.12/gui/kivy/main_window.py
Electrum-2.7.12/gui/kivy/nfc_scanner/scanner_android.py
Electrum-2.7.12/gui/kivy/nfc_scanner/scanner_dummy.py
Electrum-2.7.12/gui/kivy/uix/combobox.py
Electrum-2.7.12/gui/kivy/uix/context_menu.py
Electrum-2.7.12/gui/kivy/uix/dialogs/amount_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/bump_fee_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/checkbox_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/choice_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/fee_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/fx_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/installwizard.py
Electrum-2.7.12/gui/kivy/uix/dialogs/label_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/nfc_transaction.py
Electrum-2.7.12/gui/kivy/uix/dialogs/password_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/qr_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/qr_scanner.py
Electrum-2.7.12/gui/kivy/uix/dialogs/question.py
Electrum-2.7.12/gui/kivy/uix/dialogs/seed_options.py
Electrum-2.7.12/gui/kivy/uix/dialogs/settings.py
Electrum-2.7.12/gui/kivy/uix/dialogs/tx_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/wallets.py
Electrum-2.7.12/gui/kivy/uix/drawer.py
Electrum-2.7.12/gui/kivy/uix/gridview.py
Electrum-2.7.12/gui/kivy/uix/menus.py
Electrum-2.7.12/gui/kivy/uix/qrcodewidget.py
Electrum-2.7.12/gui/kivy/uix/screens.py
Electrum-2.7.12/gui/qt/amountedit.py
Electrum-2.7.12/gui/qt/console.py
Electrum-2.7.12/gui/qt/icons_rc.py
Electrum-2.7.12/gui/qt/installwizard.py
Electrum-2.7.12/gui/qt/qrcodewidget.py
Electrum-2.7.12/gui/qt/qrtextedit.py
Electrum-2.7.12/gui/qt/util.py
Electrum-2.7.12/gui/stdio.py
Electrum-2.7.12/gui/text.py
Electrum-2.7.12/lib/msqr.py
Electrum-2.7.12/lib/paymentrequest_pb2.py
Electrum-2.7.12/lib/simple_config.py
Electrum-2.7.12/lib/tests/test_account.py
Electrum-2.7.12/lib/tests/test_bitcoin.py
Electrum-2.7.12/lib/tests/test_interface.py
Electrum-2.7.12/lib/tests/test_mnemonic.py
Electrum-2.7.12/lib/tests/test_simple_config.py
Electrum-2.7.12/lib/tests/test_transaction.py
Electrum-2.7.12/lib/tests/test_util.py
Electrum-2.7.12/lib/tests/test_wallet.py
Electrum-2.7.12/lib/version.py
Electrum-2.7.12/plugins/audio_modem/qt.py
Electrum-2.7.12/plugins/exchange_rate/exchange_rate.py
Electrum-2.7.12/plugins/exchange_rate/kivy.py
Electrum-2.7.12/plugins/exchange_rate/qt.py
Electrum-2.7.12/plugins/keepkey/client.py
Electrum-2.7.12/plugins/keepkey/cmdline.py
Electrum-2.7.12/plugins/keepkey/keepkey.py
Electrum-2.7.12/plugins/keepkey/qt.py
Electrum-2.7.12/plugins/labels/kivy.py
Electrum-2.7.12/plugins/labels/labels.py
Electrum-2.7.12/plugins/labels/qt.py
Electrum-2.7.12/plugins/ledger/cmdline.py
Electrum-2.7.12/plugins/ledger/ledger.py
Electrum-2.7.12/plugins/ledger/qt.py
Electrum-2.7.12/plugins/plot/qt.py
Electrum-2.7.12/plugins/trezor/client.py
Electrum-2.7.12/plugins/trezor/clientbase.py
Electrum-2.7.12/plugins/trezor/cmdline.py
Electrum-2.7.12/plugins/trezor/plugin.py
Electrum-2.7.12/plugins/trezor/qt.py
Electrum-2.7.12/plugins/trezor/qt_generic.py
Electrum-2.7.12/plugins/trezor/trezor.py
Electrum-2.7.12/plugins/trustedcoin/cmdline.py
Electrum-2.7.12/plugins/virtualkeyboard/qt.py
Electrum-2.7.12/scripts/bip70
Electrum-2.7.12/scripts/block_headers
Electrum-2.7.12/scripts/estimate_fee
Electrum-2.7.12/scripts/get_chunk
Electrum-2.7.12/scripts/get_donation_address
Electrum-2.7.12/scripts/get_history
Electrum-2.7.12/scripts/peers
Electrum-2.7.12/scripts/servers
Electrum-2.7.12/scripts/txradar
Electrum-2.7.12/scripts/util.py
Electrum-2.7.12/scripts/watch_address
Electrum-2.7.12/setup-release.py

Please, ask to upstream to confirm the licensing of code and/or content/s,
and to add license headers
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/LicensingGuidelines#License_Clarification

Non blocking issues:
[!]: Spec file according to URL is the same as in SRPM.
 Note: Spec file as given by url is not the same as in SRPM (see
 attached diff).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1376867] Review Request: python3-pycurl - Python interface to libcurl for Python 3

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1376867

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 Status|NEW |POST
   Assignee|nob...@fedoraproject.org|zbys...@in.waw.pl
  Flags||fedora-review+



--- Comment #6 from Zbigniew Jędrzejewski-Szmek  ---
> Shouldn't this request be with Product Fedora EPEL? 
Please correct me if I'm wrong, but from what I've seen, there's only a single
"Package review" component that is used both for Fedora proper and EPEL. 

> There's already python-pycurl in Fedora (with python2 and python3 
> subpackages).
Centos/redhat has python-pycurl. This review is to add python3-pycurl package
which provides the python 3 version.

Review:
+ package name is OK
+ no conflict with existing packages
+ builds and installs OK
+ provides/requires look correct
  (/bin/sh is pulled in because of tests, this seems OK)
+ license is OK
+ version is the same as in stable Fedora, so OK for EPEL
+ tests pass

rpmlint:
python34-pycurl.x86_64: E: wrong-script-interpreter
/usr/share/doc/python34-pycurl-7.43.0/tests/setopt_string_test.py /usr/bin/env
python
...
Those are test files, doesn't matter.

Package is APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1187084] Review Request: electrum - a lightweight Bitcoin client

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1187084



--- Comment #25 from gil cattaneo  ---
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.
[?]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "MIT/X11 (BSD like)", "BSD (2 clause)", "*No copyright* MIT/X11
 (BSD like)", "Unknown or generated". 107 files have unknown license.
 Detailed output of licensecheck in
 /home/gil/1187084-electrum/licensecheck.txt

BSD (2 clause)
--
Electrum-2.7.12/lib/socks.py

Unknown or generated

Electrum-2.7.12/gui/kivy/i18n.py
Electrum-2.7.12/gui/kivy/main_window.py
Electrum-2.7.12/gui/kivy/nfc_scanner/scanner_android.py
Electrum-2.7.12/gui/kivy/nfc_scanner/scanner_dummy.py
Electrum-2.7.12/gui/kivy/uix/combobox.py
Electrum-2.7.12/gui/kivy/uix/context_menu.py
Electrum-2.7.12/gui/kivy/uix/dialogs/amount_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/bump_fee_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/checkbox_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/choice_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/fee_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/fx_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/installwizard.py
Electrum-2.7.12/gui/kivy/uix/dialogs/label_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/nfc_transaction.py
Electrum-2.7.12/gui/kivy/uix/dialogs/password_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/qr_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/qr_scanner.py
Electrum-2.7.12/gui/kivy/uix/dialogs/question.py
Electrum-2.7.12/gui/kivy/uix/dialogs/seed_options.py
Electrum-2.7.12/gui/kivy/uix/dialogs/settings.py
Electrum-2.7.12/gui/kivy/uix/dialogs/tx_dialog.py
Electrum-2.7.12/gui/kivy/uix/dialogs/wallets.py
Electrum-2.7.12/gui/kivy/uix/drawer.py
Electrum-2.7.12/gui/kivy/uix/gridview.py
Electrum-2.7.12/gui/kivy/uix/menus.py
Electrum-2.7.12/gui/kivy/uix/qrcodewidget.py
Electrum-2.7.12/gui/kivy/uix/screens.py
Electrum-2.7.12/gui/qt/amountedit.py
Electrum-2.7.12/gui/qt/console.py
Electrum-2.7.12/gui/qt/icons_rc.py
Electrum-2.7.12/gui/qt/installwizard.py
Electrum-2.7.12/gui/qt/qrcodewidget.py
Electrum-2.7.12/gui/qt/qrtextedit.py
Electrum-2.7.12/gui/qt/util.py
Electrum-2.7.12/gui/stdio.py
Electrum-2.7.12/gui/text.py
Electrum-2.7.12/lib/msqr.py
Electrum-2.7.12/lib/paymentrequest_pb2.py
Electrum-2.7.12/lib/simple_config.py
Electrum-2.7.12/lib/tests/test_account.py
Electrum-2.7.12/lib/tests/test_bitcoin.py
Electrum-2.7.12/lib/tests/test_interface.py
Electrum-2.7.12/lib/tests/test_mnemonic.py
Electrum-2.7.12/lib/tests/test_simple_config.py
Electrum-2.7.12/lib/tests/test_transaction.py
Electrum-2.7.12/lib/tests/test_util.py
Electrum-2.7.12/lib/tests/test_wallet.py
Electrum-2.7.12/lib/version.py
Electrum-2.7.12/plugins/audio_modem/qt.py
Electrum-2.7.12/plugins/exchange_rate/exchange_rate.py
Electrum-2.7.12/plugins/exchange_rate/kivy.py
Electrum-2.7.12/plugins/exchange_rate/qt.py
Electrum-2.7.12/plugins/keepkey/client.py
Electrum-2.7.12/plugins/keepkey/cmdline.py
Electrum-2.7.12/plugins/keepkey/keepkey.py
Electrum-2.7.12/plugins/keepkey/qt.py
Electrum-2.7.12/plugins/labels/kivy.py
Electrum-2.7.12/plugins/labels/labels.py
Electrum-2.7.12/plugins/labels/qt.py
Electrum-2.7.12/plugins/ledger/cmdline.py
Electrum-2.7.12/plugins/ledger/ledger.py
Electrum-2.7.12/plugins/ledger/qt.py
Electrum-2.7.12/plugins/plot/qt.py
Electrum-2.7.12/plugins/trezor/client.py
Electrum-2.7.12/plugins/trezor/clientbase.py
Electrum-2.7.12/plugins/trezor/cmdline.py
Electrum-2.7.12/plugins/trezor/plugin.py
Electrum-2.7.12/plugins/trezor/qt.py
Electrum-2.7.12/plugins/trezor/qt_generic.py
Electrum-2.7.12/plugins/trezor/trezor.py
Electrum-2.7.12/plugins/trustedcoin/cmdline.py
Electrum-2.7.12/plugins/virtualkeyboard/qt.py
Electrum-2.7.12/scripts/bip70
Electrum-2.7.12/scripts/block_headers
Electrum-2.7.12/scripts/estimate_fee
Electrum-2.7.12/scripts/get_chunk
Electrum-2.7.12/scripts/get_donation_address
Electrum-2.7.12/scripts/get_history
Electrum-2.7.12/scripts/peers
Electrum-2.7.12/scripts/servers
Electrum-2.7.12/scripts/txradar
Electrum-2.7.12/scripts/util.py
Electrum-2.7.12/scripts/watch_address
Electrum-2.7.12/setup-release.py



[?]: Package requires other packages for directories it uses.
 Note: No known owner of /usr/share/locale/hy_AM,
 /usr/share/locale/eo_UY, /usr/share/locale/ky_KG/LC_MESSAGES,
 /usr/share/locale/ar_SA/LC_MESSAGES,
 /usr/share/locale/eo_UY/LC_MESSAGES, /usr/share/locale/ar_SA,
 /usr/share/locale/ky_KG, /usr/share/locale/hy_AM/LC_MESSAGES
[?]: Package must own all directories that it creates.
 Note: Directories without known owners:
 

[Bug 1373218] Review Request: zstd - Zstd compression library

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1373218

Igor Gnatenko  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed|2016-10-03 12:46:14 |2016-11-14 12:27:11



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1130098] Review Request: python-ejson - Extensible json serializer/ deserializer library

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1130098

Igor Gnatenko  changed:

   What|Removed |Added

 CC|ignate...@redhat.com|pa...@hubbitus.info
  Flags||needinfo?(pa...@hubbitus.in
   ||fo)



--- Comment #3 from Igor Gnatenko  ---
ping?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1382859] Review Request: kelbt - Backtracking LR Parsing

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1382859

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1393796] Review Request: golang-github-go-errors-error - Package errors provides errors that have stack-traces

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1393796



--- Comment #1 from Dennis Chen  ---
I don't think we'll be targeting EPEL6, at least not with the arduino releases,
could you simplify the SPEC to remove of all the unnecessary macros?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1277260] Review Request: rofi - A window switcher, run dialog and dmenu replacement

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1277260

Igor Gnatenko  changed:

   What|Removed |Added

 Blocks|177841 (FE-NEEDSPONSOR) |
  Flags|needinfo?(murmansksity@gmai |
   |l.com)  |



--- Comment #15 from Igor Gnatenko  ---
Okay, nevermind. I can take care about this package.

Till, can you do review?


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1246974] Review Request: rubygem-shortURL - very simple library to use URL shortening services.

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1246974

Igor Gnatenko  changed:

   What|Removed |Added

 CC|ignate...@redhat.com|



--- Comment #19 from Igor Gnatenko  ---
ping?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1075218] Review Request: pam_ldap2krb - password migration tool ldap to kerberos

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1075218



--- Comment #15 from Igor Gnatenko  ---
ping?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1187084] Review Request: electrum - a lightweight Bitcoin client

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1187084

gil cattaneo  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1393796] Review Request: golang-github-go-errors-error - Package errors provides errors that have stack-traces

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1393796

Dennis Chen  changed:

   What|Removed |Added

 CC||barracks...@gmail.com
   Assignee|nob...@fedoraproject.org|barracks...@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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394193] Review Request: arduino-builder - A command line tool for compiling Arduino sketches

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394193

Dennis Chen  changed:

   What|Removed |Added

 CC||barracks...@gmail.com
   Assignee|nob...@fedoraproject.org|barracks...@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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1276632] Review Request: geronimo-jcdi-1.1-api - Apache Geronimo JCDI Spec 1.1

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1276632

gil cattaneo  changed:

   What|Removed |Added

 Status|NEW |CLOSED
   Fixed In Version||geronimo-jcdi-1.1-api-1.0-1
   ||.fc26
 Resolution|--- |RAWHIDE
Last Closed||2016-11-14 11:47:34



--- Comment #7 from gil cattaneo  ---
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=16450476

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1389985] Review Request: http-builder - HTTP client framework for Groovy

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1389985

gil cattaneo  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||http-builder-0.7.2-1.fc26
 Resolution|--- |RAWHIDE
Last Closed||2016-11-14 11:37:54



--- Comment #6 from gil cattaneo  ---
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=16450386

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1382152] Review Request: orpie - scientific calculator

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1382152

Jared Wallace  changed:

   What|Removed |Added

  Flags|needinfo?(jared-wallace@us. |
   |ibm.com)|



--- Comment #12 from Jared Wallace  ---
No problem.

Spec URL:
https://bitbucket.org/jared-wallace/orpie/raw/19e6466752aac51f6edde20f069d296df1c45cb0/SPECS/orpie.spec
SRPM URL:
https://bitbucket.org/jared-wallace/orpie/raw/19e6466752aac51f6edde20f069d296df1c45cb0/SRPMS/orpie-1.5.2-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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1382995] Review Request: unit-api - JSR 363 - Units of Measurement API

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1382995

gil cattaneo  changed:

   What|Removed |Added

 Status|NEW |CLOSED
   Fixed In Version||unit-api-1.0-1.fc26
 Resolution|--- |RAWHIDE
Last Closed||2016-11-14 11:28:52



--- Comment #11 from gil cattaneo  ---
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=16450358

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394046] Review Request: python3-netaddr - A pure Python network address representation and manipulation library

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394046



--- Comment #5 from Fedora Update System  ---
python3-netaddr-0.7.18-2.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-58bea500fe

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394046] Review Request: python3-netaddr - A pure Python network address representation and manipulation library

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394046

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1290342] Review Request: ecc-25519-java - Java library to use Ed25519 and Curve25518

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1290342



--- Comment #8 from gil cattaneo  ---
@Nicolas can you continue with the review?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1290342] Review Request: ecc-25519-java - Java library to use Ed25519 and Curve25518

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1290342

gil cattaneo  changed:

   What|Removed |Added

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



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1382152] Review Request: orpie - scientific calculator

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1382152

Randy Barlow  changed:

   What|Removed |Added

  Flags||needinfo?(jared-wallace@us.
   ||ibm.com)



--- Comment #11 from Randy Barlow  ---
Hello Jared! The new spec looks good. When making changes, it's customary to
also rebuild the SRPM when rebuilding the spec, and then to make a formatted BZ
comment here so that fedora-review can find and compare them:

Spec URL: 
SRPM URL: 

Can you rebuild the SRPM with the new spec and link them in a comment here? I
think this spec is in pretty good shape. I'll take a look at the reviews you
did soon.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1376867] Review Request: python3-pycurl - Python interface to libcurl for Python 3

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1376867



--- Comment #5 from Orion Poplawski  ---
(In reply to Zbigniew Jędrzejewski-Szmek from comment #3)
> SRPM gives 404 ;(

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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1359914] Review Request: lollypop - Music player for GNOME

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1359914



--- Comment #14 from MartinKG  ---
Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/lollypop.spec
SRPM URL:
https://martinkg.fedorapeople.org/Review/SRPMS/lollypop-0.9.216-1.fc25.src.rpm

%changelog
* Mon Nov 14 2016 Martin Gansser  - 0.9.216-1
* Update to 0.9.216

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394046] Review Request: python3-netaddr - A pure Python network address representation and manipulation library

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394046



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1370533] Review Request: python-django-pgjson - PostgreSQL json field support for Django

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1370533



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1389985] Review Request: http-builder - HTTP client framework for Groovy

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1389985



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1387250] Review Request: python-lazr-smtptest - Test framework for SMTP-based applications

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1387250



--- Comment #2 from Aurelien Bompard  ---
I've made these changes, thanks for the review.

Spec URL:
https://abompard.fedorapeople.org/reviews/lazr.smtptest/python-lazr-smtptest.spec
SRPM URL:
https://abompard.fedorapeople.org/reviews/lazr.smtptest/python-lazr-smtptest-2.0.3-1.fc23.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1369535] Review Request: sassc - Wrapper around libsass to compile CSS stylesheet

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1369535



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394046] Review Request: python3-netaddr - A pure Python network address representation and manipulation library

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394046



--- Comment #3 from Orion Poplawski  ---
Changes made here and in Fedora.  Thanks for the review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1385331] Review Request: hot-tux - Graphical CPU utilization monitoring utility

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1385331



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1394167] Review Request: perl-Data-ICal-TimeZone - Time zones for Data::ICal

2016-11-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1394167



--- Comment #3 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/perl-Data-ICal-TimeZone

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


  1   2   >