[Bug 1329199] Review Request: concurrent-trees - Concurrent Trees for Java

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

gil cattaneo  changed:

   What|Removed |Added

 CC||tre...@redhat.com
  Flags||needinfo?(tre...@redhat.com
   ||)



--- Comment #5 from gil cattaneo  ---
@ Tomas can you do a PR?
https://github.com/nicoulaj/compile-command-annotations/issues/8

-- 
You are receiving this mail because:
You are on the CC list 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 1343224] Review Request: xenlism-wildfire - Minimalist theme for your desktop

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



--- Comment #8 from Fedora Update System  ---
xenlism-wildfire-0-0.1.20160511gitd3b9ad2.fc24 has been submitted as an update
to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-ccd8a281c5

-- 
You are receiving this mail because:
You are on the CC list 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 1343224] Review Request: xenlism-wildfire - Minimalist theme for your desktop

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

Fedora Update System  changed:

   What|Removed |Added

 Status|RELEASE_PENDING |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 1343224] Review Request: xenlism-wildfire - Minimalist theme for your desktop

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



--- Comment #9 from Fedora Update System  ---
xenlism-wildfire-0-0.1.20160511gitd3b9ad2.el7 has been submitted as an update
to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-958e4584be

-- 
You are receiving this mail because:
You are on the CC list 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 1343224] Review Request: xenlism-wildfire - Minimalist theme for your desktop

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



--- Comment #7 from Fedora Update System  ---
xenlism-wildfire-0-0.1.20160511gitd3b9ad2.fc23 has been submitted as an update
to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-2f6141369f

-- 
You are receiving this mail because:
You are on the CC list 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 1338553] Review Request: nitroshare - Transfer files from one device to another made extremely simple

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

Fedora Update System  changed:

   What|Removed |Added

 Status|RELEASE_PENDING |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 1338553] Review Request: nitroshare - Transfer files from one device to another made extremely simple

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

Raphael Groner  changed:

   What|Removed |Added

  Alias|nitroshare  |



-- 
You are receiving this mail because:
You are on the CC list 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 1338553] Review Request: nitroshare - Transfer files from one device to another made extremely simple

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



--- Comment #8 from Fedora Update System  ---
nitroshare-0.3.1-2.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-f0d64d86f6

-- 
You are receiving this mail because:
You are on the CC list 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 1338553] Review Request: nitroshare - Transfer files from one device to another made extremely simple

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



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

-- 
You are receiving this mail because:
You are on the CC list 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 1338553] Review Request: nitroshare - Transfer files from one device to another made extremely simple

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



--- Comment #10 from Fedora Update System  ---
nitroshare-0.3.1-2.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-9fe7e4e3c3

-- 
You are receiving this mail because:
You are on the CC list 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 1310294] Review Request: sqlcipher - An SQLite extension that provides 256 bit AES encryption of database files

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



--- Comment #10 from Raphael Groner  ---
What's your FAS account? Please use the same e-mail address and name for your
Bugzilla account. Otherwise, people have trouble to identify you.

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


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

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



--- Comment #11 from Raphael Groner  ---
(In reply to Siddharth Sharma from comment #4)
> (In reply to Raphael Groner from comment #2)
> > Hi,
> > is there any progress with this package? I could do the official review.
> 
> new SPEC and SRPM
> 
> SPEC: https://siddharths.fedorapeople.org/SPECS/sqlcipher.spec
> SRPM:
> https://siddharths.fedorapeople.org/SRPMS/sqlcipher-3.3.1-2.fc23.src.rpm

Review swap with bug #1343733?

-- 
You are receiving this mail because:
You are on the CC list 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 1202231] Review Request: jbehave - Behavior-Driven Development framework for Java and Groovy

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

Raphael Groner  changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
   Assignee|projects...@smart.ms|nob...@fedoraproject.org
  Flags|fedora-review?  |



--- Comment #8 from Raphael Groner  ---
Too bad. Wildfly is currently not in my business, sorry.

-- 
You are receiving this mail because:
You are on the CC list 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 1339730] Review Request: c-sundry - Collection of sundry convenience headers

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

Harald Hoyer  changed:

   What|Removed |Added

 CC||har...@redhat.com
   Assignee|nob...@fedoraproject.org|har...@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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1343895] New: Review Request: python-xunitmerge - utility for merging multiple XUnit xml reports into a single xml report

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

Bug ID: 1343895
   Summary: Review Request: python-xunitmerge - utility for
merging multiple XUnit xml reports into a single xml
report
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: abreg...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://bitbucket.org/abregman/bregman-rpms/downloads/python-xunitmerge.spec
SRPM URL:
https://bitbucket.org/abregman/bregman-rpms/downloads/python-xunitmerge-1.0-4.fc23.src.rpm

Description: The primary purpose of xunitmerge is to be able to merge XUnit
reports generated by nosetests (e.g. nosetests --with-xunit).
This is especially useful when nosetests needs to be run multiple times for
different parts of a project and then all reports need to be merged to a single
report.

Fedora Account System Username: abregman

-- 
You are receiving this mail because:
You are on the CC list 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 1343895] Review Request: python-xunitmerge - utility for merging multiple XUnit xml reports into a single xml report

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



--- Comment #1 from Arie Bregman  ---
Updated:

Spec URL:
https://bitbucket.org/abregman/bregman-rpms/downloads/python-xunitmerge.spec
SRPM URL:
https://bitbucket.org/abregman/bregman-rpms/downloads/python-xunitmerge-1.0-4.fc23.src.rpm

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


[Bug 1329199] Review Request: concurrent-trees - Concurrent Trees for Java

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

Tomas Repik  changed:

   What|Removed |Added

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



--- Comment #6 from Tomas Repik  ---
(In reply to gil cattaneo from comment #5)
> @ Tomas can you do a PR?
 Done
https://github.com/nicoulaj/compile-command-annotations/pull/9

-- 
You are receiving this mail because:
You are on the CC list 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 1343895] Review Request: python-xunitmerge - utility for merging multiple XUnit xml reports into a single xml report

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

Matthias Runge  changed:

   What|Removed |Added

 CC||mru...@redhat.com



--- Comment #2 from Matthias Runge  ---
While running tests, I get the following error:
ERROR: xunitmerge (unittest.loader._FailedTest)
--
ImportError: Failed to import test module: xunitmerge
Traceback (most recent call last):
  File "/usr/lib64/python3.5/unittest/loader.py", line 462, in _find_test_path
package = self._get_module_from_name(name)
  File "/usr/lib64/python3.5/unittest/loader.py", line 369, in
_get_module_from_name
__import__(name)
  File "/builddir/build/BUILD/xunitmerge-1.0/xunitmerge/__init__.py", line 1,
in 
from xmerge import *
ImportError: No module named 'xmerge'
--
Ran 1 test in 0.000s
FAILED (errors=1)
RPM build errors:
error: Bad exit status from /var/tmp/rpm-tmp.28f3WC (%check)
Bad exit status from /var/tmp/rpm-tmp.28f3WC (%check)
Child return code was: 1
EXCEPTION: [Error()]
Traceback (most recent call last):
  File "/usr/lib/python3.5/site-packages/mockbuild/trace_decorator.py", line
88, in trace
result = func(*args, **kw)
  File "/usr/lib/python3.5/site-packages/mockbuild/util.py", line 547, in do
raise exception.Error("Command failed. See logs for output.\n # %s" %
(command,), child.returncode)
mockbuild.exception.Error: Command failed. See logs for output.
 # bash --login -c /usr/bin/rpmbuild -bb --target x86_64 --nodeps
/builddir/build/SPECS/python-xunitmerge.spec
Mock Version: 1.2.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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1341672] Review Request: perl-Test2-AsyncSubtest - Tools for writing asynchronous subtest

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

Jitka Plesnikova  changed:

   What|Removed |Added

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


[Bug 1342839] Review Request: python-pycodestyle - Python style guide checker

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



--- Comment #7 from Miro Hrončok  ---
You should provide all -2, -2.7, -3, -3.5.

See http://python-rpm-porting.readthedocs.io/en/latest/tools.html#install and
the following section there as well for a good idea how to do it.

John: I find your lack of good review disturbing, please follow
https://fedoraproject.org/wiki/Packaging:ReviewGuidelines

-- 
You are receiving this mail because:
You are on the CC list 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 1342839] Review Request: python-pycodestyle - Python style guide checker

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



--- Comment #8 from Miro Hrončok  ---
It also seems like a good idea to provide /usr/bin/pep8* symlinks for backward
compatibility.

-- 
You are receiving this mail because:
You are on the CC list 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 1343895] Review Request: python-xunitmerge - utility for merging multiple XUnit xml reports into a single xml report

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



--- Comment #3 from Arie Bregman  ---
Updated:

Spec URL:
https://bitbucket.org/abregman/bregman-rpms/downloads/python-xunitmerge-1.0.4-1.fc23.src.rpm
SRPM URL:
https://bitbucket.org/abregman/bregman-rpms/downloads/python-xunitmerge.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 1341672] Review Request: perl-Test2-AsyncSubtest - Tools for writing asynchronous subtest

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

Jitka Plesnikova  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Jitka Plesnikova  ---
Summary is ok
License is ok
Description is ok
URL and Source0 are ok
All tests passed
BuildRequires are ok

$ rpm -qp --requires perl-Test2-AsyncSubtest-0.16-1.fc25.noarch.rpm | sort
| uniq -c
  1 perl(:MODULE_COMPAT_5.24.0)
  1 perl(Carp)
  1 perl(Exporter)
  1 perl(List::Util)
  1 perl(Scalar::Util)
  1 perl(Scope::Guard)
  1 perl(Test2::API) >= 1.302015
  1 perl(Test2::API::Context)
  1 perl(Test2::AsyncSubtest)
  1 perl(Test2::AsyncSubtest::Event::Attach)
  1 perl(Test2::AsyncSubtest::Event::Detach)
  1 perl(Test2::AsyncSubtest::Hub)
  1 perl(Test2::Event)
  1 perl(Test2::Event::Exception)
  1 perl(Test2::Hub::Subtest)
  1 perl(Test2::IPC)
  1 perl(Test2::Util)
  1 perl(Test2::Util::HashBase)
  1 perl(Test2::Util::Trace)
  1 perl(Time::HiRes)
  1 perl(base)
  1 perl(strict)
  1 perl(warnings)
  1 rpmlib(CompressedFileNames) <= 3.0.4-1
  1 rpmlib(FileDigests) <= 4.6.0-1
  1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  1 rpmlib(PayloadIsXz) <= 5.2-1
Binary requires are Ok.

$ rpm -qp --provides perl-Test2-AsyncSubtest-0.16-1.fc25.noarch.rpm | sort
| uniq -c
  1 perl(Test2::AsyncSubtest) = 0.16
  1 perl(Test2::AsyncSubtest::Event::Attach) = 0.16
  1 perl(Test2::AsyncSubtest::Event::Detach) = 0.16
  1 perl(Test2::AsyncSubtest::Hub) = 0.16
  1 perl(Test2::Tools::AsyncSubtest) = 0.16
  1 perl-Test2-AsyncSubtest = 0.16-1.fc25
Binary provides are Ok.

$ rpmlint ./perl-Test2-AsyncSubtest*
perl-Test2-AsyncSubtest.noarch: W: spelling-error Summary(en_US) subtest ->
subtlest, sub test, sub-test
perl-Test2-AsyncSubtest.noarch: W: spelling-error %description -l en_US
subtests -> sub tests, sub-tests, subtexts
perl-Test2-AsyncSubtest.src: W: spelling-error Summary(en_US) subtest ->
subtlest, sub test, sub-test
perl-Test2-AsyncSubtest.src: W: spelling-error %description -l en_US subtests
-> sub tests, sub-tests, subtexts
2 packages and 1 specfiles checked; 0 errors, 4 warnings.

Rpmlint is ok


The package looks good.
Approved.

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


[Bug 1343950] New: Review Request: python-iso639 - ISO639-2 support for Python

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

Bug ID: 1343950
   Summary: Review Request: python-iso639 - ISO639-2 support for
Python
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ignate...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://ignatenkobrain.fedorapeople.org/for-review/python-iso639.spec
SRPM URL:
https://ignatenkobrain.fedorapeople.org/for-review/python-iso639-0.1.4-1.fc25.src.rpm
Description: A simple (really simple) library for working with ISO639-2
language codes.
Fedora Account System Username: ignatenkobrain

-- 
You are receiving this mail because:
You are on the CC list 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 1343895] Review Request: python-xunitmerge - utility for merging multiple XUnit xml reports into a single xml report

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



--- Comment #4 from Matthias Runge  ---
Could you please add
BuildRequires: python-six
BuildRequires: python-nose

BuildRequires: python3-six
BuildRequires: python3-nose

and then for tests:

%check  
%if 0%{?with_python3}
nosetests-%{python3_version} --with-doctest
%endif   
nosetests --with-doctest

-- 
You are receiving this mail because:
You are on the CC list 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 1343895] Review Request: python-xunitmerge - utility for merging multiple XUnit xml reports into a single xml report

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

Matthias Runge  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|mru...@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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1343977] New: Review Request: asciidoctor-mallard - A Project Mallard converter for AsciiDoc

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

Bug ID: 1343977
   Summary: Review Request:  asciidoctor-mallard - A Project
Mallard converter for AsciiDoc
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: dhanvi...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://copr-dist-git.fedorainfracloud.org/cgit/dhanvi/asciidoctor-mallard/rubygem-asciidoctor-mallard.git/tree/asciidoctor-mallard.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/dhanvi/asciidoctor-mallard/fedora-rawhide-x86_64/00322904-rubygem-asciidoctor-mallard/rubygem-asciidoctor-mallard-0.1.0.dev-1.fc25.src.rpm
Description: An extension for Asciidoctor that converts AsciiDoc documents to
Mallard 1.0
Fedora Account System Username: dhanvi

-- 
You are receiving this mail because:
You are on the CC list 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 1343895] Review Request: python-xunitmerge - utility for merging multiple XUnit xml reports into a single xml report

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



--- Comment #5 from Arie Bregman  ---
Updated:

Spec URL:
https://bitbucket.org/abregman/bregman-rpms/downloads/python-xunitmerge-1.0.4-1.fc23.src.rpm
SRPM URL:
https://bitbucket.org/abregman/bregman-rpms/downloads/python-xunitmerge.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 1242056] Review Request: rubygem-chake - serverless configuration management tool for chef

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

Vít Ondruch  changed:

   What|Removed |Added

 CC||vondr...@redhat.com



--- Comment #15 from Vít Ondruch  ---
(In reply to Athos Ribeiro from comment #13)
> (In reply to Paulo Andrade from comment #12)
> >   Hi Athos,
> 
> Hi Paulo, thank you for the feedback.
> 
> > 
> >   Please comment on this:
> > W: simplecov not installed, we won't have a coverage report
> > maybe it needs this build requires?
> > rubygem-simplecov.noarch : Code coverage analysis tool for Ruby 1.9
> 
> simplecov is a gem to measure test coverage. I don't see any advantage on
> measuring test coverage during the build step since the output would be
> ignored. It would also generate a "coverege" directory which would have to
> be removed. see https://github.com/colszowka/simplecov for reference.

That is right. We typically omit code coverage runs from the test suite. This
is upstream stuff, we don't generally care about it.

Less dependencies for package build is always better.

> > 
> >   I see the -doc package is installing files under:
> > /usr/share/gems/gems/chake-0.13/
> > I believe this is incorrect. Are you sure the main package runs
> > without the files installed there?
> 
> Yes,
> 
> the -doc files under /usr/share/gems/gems/chake-0.13/ are
> 
> /usr/share/gems/gems/chake-0.13/examples
>   which is a directory with examples on how to use chake
> 
> /usr/share/gems/gems/chake-0.13/spec
>   which is a directory with integration tests
> 
> /usr/share/gems/gems/chake-0.13/ChangeLog.md
> /usr/share/gems/gems/chake-0.13/README.md
>   upstream changelog and readme files
> 
> /usr/share/gems/gems/chake-0.13/Rakefile
>   Which is a file for rake (make for ruby, so this would be something 
> like a
> Makefile)
> /usr/share/gems/gems/chake-0.13/Gemfile
>   File defining all dependencies of the project
> /usr/share/gems/gems/chake-0.13/chake.gemspec
>   File that defines the gem
> 
> These files usually go in the -doc in other packaged gems.


This list and the split between main package and -doc subpackage looks
reasonable to me.

> > 
> >   Please also comment on the directory:
> > /usr/share/gems/doc/chake-0.13/ri
> > is it really required by the -doc package? Either way, what is
> > installing in /usr/share/gems/doc/chake-0.13/ should be installed
> > in /usr/share/doc/chake
> 
> The ri directory contains the ri documentation for the gem. ri documentation
> can be browsed through ri calls on the command line.
> 
> About being installed in /usr/share/doc/chake:
> 
> As you can see here (f23):
> $ dnf repoquery --repoid=fedora -l rubygem-*-doc | grep -i
> "^/usr/share/gems/doc/" | cut -d/ -f6 | uniq | wc -l
> 531
> $ dnf repoquery --repoid=fedora rubygem-*-doc | grep rubygem | wc -l
> 533
> 
> only 2 out of 533 gems do not install documentation in /usr/share/gems/doc,
> shouldn't I also follow this pattern?


The generated documentation must be available at these locations, since the
rubygems tooling expects it to be there. For example "gem server" can server
the documentation from there. Similarly, ther "ri" command knows these location
to server the documentation properly.

> > 
> >   About the fonts, I believe the bug report is
> > https://bugzilla.redhat.com/show_bug.cgi?id=1224715 and it will
> > not get any attention, as the linked, related upstream report
> > is closed https://github.com/rdoc/rdoc/issues/186 as they
> > apparently had a different idea about it.
> 
> I see...
> 
> > Please check that just adding a 'Requires: lato-fonts' is not
> > enough to display the documentation, and if not enough, please
> > check what kind of patch could be done, apparently only in the
> > *.css files.
> > I understand it is replicated in more than 500 packages, but
> > that is not correct :(
> 
> I removed the generated fonts, required the two packages wich contain them
> and added links to these files, how about that? I am also checking with Ruby
> SIG if that approach would be feasible for all the other 500+ -doc packages.

While I agree this should be fixed, I don't think it is good idea to fix only
single gem. I would prefer all or nothing.


> > 
> >   Please check https://fedoraproject.org/wiki/Packaging:Ruby
> > the template there installs documentation under:
> > $ rpm -E %_defaultdocdir
> > /usr/share/doc
> > what should be done by the sample command in the sample spec:
> > rdoc --op %{buildroot}%{_defaultdocdir}/%{name}
> 
> The example shown is for ruby applications, there are other instructions for
> packaging gems there. As mentioned above, there is a macro defined with the
> path for gems documentations in
> https://fedoraproject.org/wiki/Packaging:Ruby#Macros

Agree, this is correct explanation.

> >   About the reviews with no longer srpm or spec, what you did
> > is fine, just comment about it

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

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

Tomas Repik  changed:

   What|Removed |Added

 Depends On|1340876 |



--- Comment #2 from Tomas Repik  ---
Spec URL: https://trepik.fedorapeople.org/cassandra/v1/cassandra.spec
SRPM URL:
https://trepik.fedorapeople.org/cassandra/v1/cassandra-3.5-0.fc23.src.rpm

Getting closer but still not final...


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1340876
[Bug 1340876] cant find symbol error
-- 
You are receiving this mail because:
You are on the CC list 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 1218758] Review Request: flmsg - Fast Light Message Amateur Radio Forms Manager

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

Richard Shaw  changed:

   What|Removed |Added

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



--- Comment #21 from Richard Shaw  ---
Sorry, I just got back from my son's summer camp.

(In reply to Eric Christensen from comment #19)
> I'm a little concerned about the missing-call-to-setgroups-before-setuid
> error that rpmlint is giving.  Also, the COPYING is GPLv2 and SPEC says
> GPLv3.

As noted in comment #14, this is part of the bundled mongoose but flmsg doesn't
make use of it so it shouldn't be a problem.

I dug into the license stuff a bit. I'll follow up with the author but yet, the
COPYING file should probably be updated to GPLv3+.

The GPLv2 stuff is just the mongoose files, since it's not "+" I can't upgrade
them to 3+ so I'll add an appropriate statement to the spec file and license
tag. The BSD stuff is part of the build system files so it doesn't apply to the
package.

> Issues:
> ===
> - All build dependencies are listed in BuildRequires, except for any that
>   are listed in the exceptions section of Packaging Guidelines.
>   Note: These BR are not needed: gcc-c++
>   See: http://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2

There was an update to the packaging guidelines to add this as they are working
on lowing the number default packages installed for mock. Granted I saw this on
an email update but haven't looked at the guidelines in the wiki lately...

-- 
You are receiving this mail because:
You are on the CC list 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 1341672] Review Request: perl-Test2-AsyncSubtest - Tools for writing asynchronous subtest

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



--- Comment #2 from Jon Ciesla  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/perl-Test2-AsyncSubtest

-- 
You are receiving this mail because:
You are on the CC list 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 1339760] Review Request: js-gl-matrix - Javascript module designed to perform vector and matrix operations very fast

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



--- Comment #9 from Fedora Update System  ---
js-gl-matrix-2.3.2-4.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-eac799b2a0

-- 
You are receiving this mail because:
You are on the CC list 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 1341672] Review Request: perl-Test2-AsyncSubtest - Tools for writing asynchronous subtest

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

Petr Pisar  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-Test2-AsyncSubtest-0.0
   ||00016-1.fc25
 Resolution|--- |RAWHIDE
Last Closed||2016-06-08 10:18:46



--- Comment #3 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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1339760] Review Request: js-gl-matrix - Javascript module designed to perform vector and matrix operations very fast

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

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 1339760] Review Request: js-gl-matrix - Javascript module designed to perform vector and matrix operations very fast

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



--- Comment #8 from Fedora Update System  ---
js-gl-matrix-2.3.2-4.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-25d43d1f0e

-- 
You are receiving this mail because:
You are on the CC list 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 1302504] Review Request: elog - Weblog server and client

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

Antonio Trande  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||anto.tra...@gmail.com
   Assignee|nob...@fedoraproject.org|anto.tra...@gmail.com
  Flags||fedora-review?



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


[Bug 1218758] Review Request: flmsg - Fast Light Message Amateur Radio Forms Manager

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



--- Comment #22 from Richard Shaw  ---
Ok, already heard back from upstream, he is updating the COPYING file to GPLv3+
and it will be in the next release.

I don't see this as a blocker, do you? The new release should be soon so I was
going to wait for it before importing the package.

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


[Bug 1343661] Rebase clufter component

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



--- Comment #1 from Jan Pokorný  ---
In fact, it should have been
"compared to previous RHEL minor release (7.2/clufter-0.50.4)"
hence following new features and bugfixes are forthcoming:

0.50.5:
http://oss.clusterlabs.org/pipermail/users/2015-December/001981.html

0.55.0:
http://oss.clusterlabs.org/pipermail/users/2015-December/001981.html

0.56.0:
http://oss.clusterlabs.org/pipermail/users/2016-February/002214.html

0.56.1:
http://oss.clusterlabs.org/pipermail/users/2016-February/002271.html

0.56.2:
http://oss.clusterlabs.org/pipermail/users/2016-March/002534.html

-- 
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 1315801] Review Request: rubygem-nio4r - New IO for Ruby

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



--- Comment #5 from Vít Ondruch  ---
> > * Bundled library
> >   - It seems that this package bundles libev. Is there any chance to use the
> > system version of libev instead [1]? Or in the worst case provide the
> > "bundled" virtual provide.
> 
> I tried to separate bundled libev and using system libev (libev-devel). And
> I suceeded to compile the nio4r with system libev, and to pass almost all
> the rspec test cases.
> But when I compared the bundled libev (version 4.22) and original source
> [1], I found that the bundled libev had individually modified. [2][3]
> 
> So, finally I made a choice to use "bundled() = ".

Once this gets into Fedora, could you please mention this virtual provide here:

https://fedoraproject.org/wiki/Bundled_Libraries_Virtual_Provides

Seems there was some changes in this policy like 6 weeks ago.




Also a few more nits:

* Please bump the release
  - It is good habit to bump the release every iteration, update %changelog
as well as provide updated SRPM. That way, I can always check the progress
since last time.(In reply to Jun Aruga from comment #3)

* Better inline comments
  - I would suggest to replace your comment:

  # Fix the issue for rpmlint
  # I reported it to upstream, and its fix was merged to master branch.
  # https://github.com/celluloid/nio4r/pull/86

by something more explanatory, e.g.:

  # Remove useless shebang.
  # https://github.com/celluloid/nio4r/pull/86

I don't think that reference to rpmlint is useful there.

  - This note applies generally, also to other similar comments ;)

* License
  - Due to bundled libev, I think the license field should be updated. The
libev license appears to be (BSD or GPLv2+) while the code of the rest of
the package is MIT, the license tag should be:

  MIT and (BSD or GPLv2+)

-- 
You are receiving this mail because:
You are on the CC list 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 1343814] Review Request: vagrant-sshfs - A Vagrant synced folder plugin that mounts folders via SSHFS.

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

Randy Barlow  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|bowlofeggs@electronsweatsho
   ||p.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 1339738] Review Request: libbus1 - Bus1 IPC Library

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

Kay Sievers  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX
Last Closed||2016-06-08 12:23:23



--- Comment #1 from Kay Sievers  ---
This will be submitted later. There are still some things to sort out, before
it is ready for distribution packaging. Sorry for the noise.

-- 
You are receiving this mail because:
You are on the CC list 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 1343814] Review Request: vagrant-sshfs - A Vagrant synced folder plugin that mounts folders via SSHFS.

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

Randy Barlow  changed:

   What|Removed |Added

 CC||dustym...@redhat.com
  Flags||needinfo?(dustymabe@redhat.
   ||com)



--- Comment #1 from Randy Barlow  ---
The Source0 should be a URL to the source, and for Ruby is often a link to
where the gem can be downloaded from rubygems.org. Something like this I think
will work:

Source0: https://rubygems.org/gems/%{vagrant_plugin_name}-%{version}.gem

Also this SRPM does not build on Rawhide, as you also need rubygems-devel:

BuildRequires: rubygems-devel

Can you fix those two things and let me know when you have a new SRPM and spec
ready?

It's also a good idea to try a koji build and past the link here. You can do
that with something like this:

koji build --scratch rawhide /path/to/srpm

-- 
You are receiving this mail because:
You are on the CC list 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 1331818] Review Request: singularity - Portable application stack packaging and runtime utility

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



--- Comment #4 from Dave Love  ---
I think it's worth restarting review with v2.0:

Spec URL: https://loveshack.fedorapeople.org/review/singularity.spec
SRPM URL:
https://loveshack.fedorapeople.org/review/singularity-2.0-4.el6.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 1302504] Review Request: elog - Weblog server and client

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



--- Comment #2 from Antonio Trande  ---
Package Review
==

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


Issues:
===
- Large documentation must go in a -doc subpackage. Large could be size
  (~1MB) or number of files.
  Note: Documentation size is 128 bytes in 24 files.
  See:
  http://fedoraproject.org/wiki/Packaging/Guidelines#PackageDocumentation

- Upstream reports a GPLv3 license (https://midas.psi.ch/elog/#license)
  In the source archive, exists a GPLv2 COPYING file ??
  Some .c and .h files are release under GPLv2+ and GPLv3+

  If main license is GPLv2, the RPMs should be release with
  "GPLv2 and GPLv3+"

  If main license is GPLv3, the RPMs should be release with
  GPLv3+

- elog-client is a standalone package without an own license file

- Binary files are "No full Relro" and "No PIE"
  Probably LDFLAGS are not properly set

- "rm -rf %{buildroot}" present but not required

- PREFIX=%{_usr}  ??

- Systemd scriptlets are missing
 
https://fedoraproject.org/wiki/Packaging:Scriptlets?rd=Packaging:ScriptletSnippets#Systemd

= MUST items =

C/C++:
[?]: Provides: bundled(gnulib) in place as required.
 Note: Sources not installed
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Header files in -devel subpackage, if present.
[x]: 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.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "GPL (v2 or later) (with incorrect FSF address)", "GPL (v3 or
 later)", "Unknown or generated". 19 files have unknown license.
 Detailed output of licensecheck in
 /home/sagitter/1302504-elog/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/lib/systemd/system,
 /usr/lib/systemd
[!]: %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: rm -rf %{buildroot} present but not required
[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.
[!]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[x]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: Package requires other packages for directories it uses.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: %config files are marked noreplace or the reason is justified.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: No %config files under /usr.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: Spec file name must match the spec pa

[Bug 1242056] Review Request: rubygem-chake - serverless configuration management tool for chef

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



--- Comment #16 from Athos Ribeiro  ---
Thank you for the comments, Vít.

Should I go ahead and revert the latest modifications which remove and link the
fonts? I will wait to see if Paulo has any comments on that.

Maybe removing and linking fonts could be done in the end of %{gem_install}
with some checks to find out if the files are present, how about that?

-- 
You are receiving this mail because:
You are on the CC list 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 1342839] Review Request: python-pycodestyle - Python style guide checker

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



--- Comment #9 from Luke Macken  ---
(In reply to Miro Hrončok from comment #7)
> You should provide all -2, -2.7, -3, -3.5.
> 
> See http://python-rpm-porting.readthedocs.io/en/latest/tools.html#install
> and the following section there as well for a good idea how to do it.

Thanks for the link, that's a great resource. I'll work on fixing this up at
some point today.

-- 
You are receiving this mail because:
You are on the CC list 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 1339760] Review Request: js-gl-matrix - Javascript module designed to perform vector and matrix operations very fast

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #10 from Fedora Update System  ---
js-gl-matrix-2.3.2-4.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-25d43d1f0e

-- 
You are receiving this mail because:
You are on the CC list 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 1338553] Review Request: nitroshare - Transfer files from one device to another made extremely simple

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #11 from Fedora Update System  ---
nitroshare-0.3.1-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-f0d64d86f6

-- 
You are receiving this mail because:
You are on the CC list 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 1341013] Review Request: compile-command-annotations - Hotspot compile command annotations

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



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

-- 
You are receiving this mail because:
You are on the CC list 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 1342737] Review Request: btrfs-sxbackup - Incremental btrfs snapshot backups with push/ pull support via SSH

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #11 from Fedora Update System  ---
btrfs-sxbackup-0.6.6-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-e58d159972

-- 
You are receiving this mail because:
You are on the CC list 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 1329199] Review Request: concurrent-trees - Concurrent Trees for Java

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



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

-- 
You are receiving this mail because:
You are on the CC list 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 1331914] Review Request: csvjdbc - Java JDBC driver for reading comma-separated-value files

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System  ---
csvjdbc-1.0.29-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-6256cfdabc

-- 
You are receiving this mail because:
You are on the CC list 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 1342747] Review Request: limnoria - A modified version of Supybot ( an IRC bot) with enhancements and bug fixes

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #9 from Fedora Update System  ---
limnoria-20160506-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-c23d543bc9

-- 
You are receiving this mail because:
You are on the CC list 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 1343177] Review Request: jboss-jaxrs-2.0-api - JAX-RS 2.0: The Java API for RESTful Web Services

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System  ---
jboss-jaxrs-2.0-api-1.0.0-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-0c0242092b

-- 
You are receiving this mail because:
You are on the CC list 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 1343224] Review Request: xenlism-wildfire - Minimalist theme for your desktop

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #10 from Fedora Update System  ---
xenlism-wildfire-0-0.1.20160511gitd3b9ad2.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-ccd8a281c5

-- 
You are receiving this mail because:
You are on the CC list 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 1315852] Review Request: gstreamermm1 - C++ wrapper for GStreamer library

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

Bruno Wolff III  changed:

   What|Removed |Added

 CC||br...@wolff.to



--- Comment #4 from Bruno Wolff III  ---
However you guys checked dependencies is wrong and this change broke
lordsofwar. Are yiu guys planning on doing a compat package for 0.10? Or
provide info on how to convert a program to switch from 0.10 to 1.4?

-- 
You are receiving this mail because:
You are on the CC list 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 1344094] New: Review Request: vagrant-digitalocean - Vagrant plugin for having Digital Ocean as an provider

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

Bug ID: 1344094
   Summary: Review Request: vagrant-digitalocean - Vagrant plugin
for having Digital Ocean as an provider
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: m...@kushaldas.in
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://kushal.fedorapeople.org/packages/vagrant-digitalocean.spec
SRPM URL:
https://kushal.fedorapeople.org/packages/vagrant-digitalocean-0.7.10-1.fc24.src.rpm
Description: It is a Vagrant provider plugin that supports the management of
DigitalOcean
droplets (instances).
Fedora Account System Username: kushal

-- 
You are receiving this mail because:
You are on the CC list 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 1344094] Review Request: vagrant-digitalocean - Vagrant plugin for having Digital Ocean as an provider

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



--- Comment #1 from kushal...@gmail.com  ---
Tested with my personal digital ocean account.

-- 
You are receiving this mail because:
You are on the CC list 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 1344101] New: Review Request: rubygem-sinatra-cross_origin - Cross Origin Resource Sharing helper for Sinatra

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

Bug ID: 1344101
   Summary: Review Request: rubygem-sinatra-cross_origin - Cross
Origin Resource Sharing helper for Sinatra
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: val...@civ.zcu.cz
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://scientific.zcu.cz/fedora/rubygem-sinatra-cross_origin-0.3.2-1/rubygem-sinatra-cross_origin.spec
SRPM URL:
http://scientific.zcu.cz/fedora/rubygem-sinatra-cross_origin-0.3.2-1/rubygem-sinatra-cross_origin-0.3.2-1.fc25.src.rpm
Description: Cross Origin Resource Sharing helper for Sinatra
Fedora Account System Username: valtri

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

-- 
You are receiving this mail because:
You are on the CC list 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 1218758] Review Request: flmsg - Fast Light Message Amateur Radio Forms Manager

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



--- Comment #23 from Richard Shaw  ---
SPEC: https://hobbes1069.fedorapeople.org/flmsg.spec
SRPM: https://hobbes1069.fedorapeople.org/flmsg-3.00.00-1.fc23.src.rpm

* Wed Jun  8 2016 Richard Shaw  - 3.00.00-1
- Update to latest upstream release.
- Update license tag to include GPLv2 (embedded mongoose server).

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


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

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

Raphael Groner  changed:

   What|Removed |Added

 Whiteboard||Trivial



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


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

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

Bug ID: 1344115
   Summary: Review Request: winetricks -  Work around common
problems in Wine
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: projects...@smart.ms
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://raphgro.fedorapeople.org/review/util/winetricks.spec
SRPM URL: 
Description: Work around common problems in Wine
Winetricks is an easy way to work around common problems in Wine.

It has a menu of supported games/apps for which it can do all the
workarounds automatically. It also lets you install missing DLLs
or tweak various Wine settings individually.

Fedora Account System Username: raphgro

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=14425757

-- 
You are receiving this mail because:
You are on the CC list 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 1344094] Review Request: vagrant-digitalocean - Vagrant plugin for having Digital Ocean as an provider

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

Randy Barlow  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|bowlofeggs@electronsweatsho
   ||p.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 1344094] Review Request: vagrant-digitalocean - Vagrant plugin for having Digital Ocean as an provider

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

Randy Barlow  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #2 from Randy Barlow  ---
There are a few !'s in the MUST section below. Make sure you fix those before
putting it in Koji. The !'s in the other sections are optional, so it's up to
you. Nice work!


Package Review
==

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



= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Unknown or generated". 32 files have unknown license. Detailed
 output of licensecheck in /home/rbarlow/reviews/1344094-vagrant-
 digitalocean/licensecheck.txt
[!]: License file installed when any subpackage combination is installed.
 bowlofeggs: The -doc package doesn't have the license file. My own
 Vagrant package also had this problem, so I'll fix it
 there too.
[!]: Package requires other packages for directories it uses.
 Note: No known owner of /usr/share/vagrant/gems/gems/vagrant-
 digitalocean-0.7.10/box
 bowlofeggs: I think you probably need %{vagrant_plugin_instdir}/box
 in your %files section to fix this.
[!]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/share/vagrant/gems/doc,
 /usr/share/vagrant/gems/gems/vagrant-digitalocean-0.7.10/box,
 /usr/share/vagrant/gems, /usr/share/vagrant
 bowlofeggs: Same note as directly above.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: 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]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 0 bytes in 0 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

Generic:
[!]: Avoid bundling fonts in non-fonts packages.
 Note: Package contains font files
 bowlofeggs: You can declare that you have bundled fonts. Here's an
 example where I did that in one of my packages:

http://pkgs.fedoraproject.org/cgit/rpms/python-crane.git/tree/python-crane.spec#n34
 Alternatively, y

[Bug 1342693] Review Request: python2-typing - Typing defines a standard notation for type annotations

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

kushal...@gmail.com  changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |python2-typing -Typing  |python2-typing - Typing
   |defines a standard notation |defines a standard notation
   |for type annotations|for type annotations



-- 
You are receiving this mail because:
You are on the CC list 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 1344094] Review Request: vagrant-digitalocean - Vagrant plugin for having Digital Ocean as an provider

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



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

-- 
You are receiving this mail because:
You are on the CC list 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 1342693] Review Request: python2-typing - Typing defines a standard notation for type annotations

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



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

-- 
You are receiving this mail because:
You are on the CC list 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 1344094] Review Request: vagrant-digitalocean - Vagrant plugin for having Digital Ocean as an provider

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



--- Comment #4 from Fedora Update System  ---
vagrant-digitalocean-0.9.0-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-c020f5a02c

-- 
You are receiving this mail because:
You are on the CC list 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 1344094] Review Request: vagrant-digitalocean - Vagrant plugin for having Digital Ocean as an provider

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

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 1129023] Review Request: yara - Malware identification tool

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

Michal Ambroz  changed:

   What|Removed |Added

 Blocks||563471 (FE-SECLAB)



--- Comment #3 from Michal Ambroz  ---
Hello,
as there is no update from Christopher for nearly 2 years I would like to join
in this request as a second requestor, possibly co-maintaining with
Christopher.

I have switched to the current up-stream repository
(https://github.com/VirusTotal/yara) repackaged the current stable release
(3.4.0).

SPEC and SRPM can be found here:
SPEC URL: https://rebus.fedorapeople.org/SPECS/yara.spec
SRPM URL: https://rebus.fedorapeople.org/SRPMS/yara-3.4.0-1.fc23.src.rpm
Fedora Account System Username: rebus cicku

Best regards
Michal Ambroz


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=563471
[Bug 563471] Tracker: Review Requests for Fedora Security Lab related
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
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org


[Bug 1330429] Review Request: php-iamcal-lib-autolink - Adds anchors to urls in a text

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



--- Comment #18 from Fedora Update System  ---
php-iamcal-lib-autolink-1.7-1.fc23 has been pushed to the Fedora 23 stable
repository. If problems still persist, please make note of it in this bug
report.

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


[Bug 1330429] Review Request: php-iamcal-lib-autolink - Adds anchors to urls in a text

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-06-08 18:54:04



-- 
You are receiving this mail because:
You are on the CC list 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 1272255] Review Request: nodejs-lcov-parse - Parse lcov results files and return JSON

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



--- Comment #12 from Fedora Update System  ---
nodejs-lcov-parse-0.0.10-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 1272255] Review Request: nodejs-lcov-parse - Parse lcov results files and return JSON

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

Fedora Update System  changed:

   What|Removed |Added

 Resolution|NEXTRELEASE |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 1272255] Review Request: nodejs-lcov-parse - Parse lcov results files and return JSON

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



--- Comment #13 from Fedora Update System  ---
nodejs-lcov-parse-0.0.10-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 1233240] Review Request: orocos-bfl - A framework for inference in Dynamic Bayesian Networks

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



--- Comment #24 from Fedora Update System  ---
orocos-bfl-0.8.99-5.20160503gitc1b18e3.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 1129023] Review Request: yara - Malware identification tool

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



--- Comment #4 from Michal Ambroz  ---
Link to the Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=14430039

-- 
You are receiving this mail because:
You are on the CC list 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 1272255] Review Request: nodejs-lcov-parse - Parse lcov results files and return JSON

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



--- Comment #14 from Fedora Update System  ---
nodejs-lcov-parse-0.0.10-2.el6 has been pushed to the Fedora EPEL 6 stable
repository. If problems still persist, please make note of it in this bug
report.

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


[Bug 1330429] Review Request: php-iamcal-lib-autolink - Adds anchors to urls in a text

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



--- Comment #19 from Fedora Update System  ---
php-iamcal-lib-autolink-1.7-1.fc22 has been pushed to the Fedora 22 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 1340269] Review Request: keepassx0 - Cross-platform password manager

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



--- Comment #8 from Parag AN(पराग)  ---
I thought limb will request SCM admin and get this package built in Fedora but
that has not happened. Let me request this package in pkgdb now. 

Note though I will be getting maintainership here by requesting this package, I
will orphan or pass ownership to limb or anyone else interested once upgrade
issue is fixed.

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


[Bug 1338553] Review Request: nitroshare - Transfer files from one device to another made extremely simple

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



--- Comment #12 from Fedora Update System  ---
nitroshare-0.3.1-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-9fe7e4e3c3

-- 
You are receiving this mail because:
You are on the CC list 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 1343224] Review Request: xenlism-wildfire - Minimalist theme for your desktop

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



--- Comment #11 from Fedora Update System  ---
xenlism-wildfire-0-0.1.20160511gitd3b9ad2.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-958e4584be

-- 
You are receiving this mail because:
You are on the CC list 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 1339760] Review Request: js-gl-matrix - Javascript module designed to perform vector and matrix operations very fast

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



--- Comment #11 from Fedora Update System  ---
js-gl-matrix-2.3.2-4.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-eac799b2a0

-- 
You are receiving this mail because:
You are on the CC list 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 1342747] Review Request: limnoria - A modified version of Supybot ( an IRC bot) with enhancements and bug fixes

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



--- Comment #10 from Fedora Update System  ---
limnoria-20160506-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-80d3861400

-- 
You are receiving this mail because:
You are on the CC list 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 1342737] Review Request: btrfs-sxbackup - Incremental btrfs snapshot backups with push/ pull support via SSH

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



--- Comment #12 from Fedora Update System  ---
btrfs-sxbackup-0.6.6-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-aca792d485

-- 
You are receiving this mail because:
You are on the CC list 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 1343224] Review Request: xenlism-wildfire - Minimalist theme for your desktop

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



--- Comment #12 from Fedora Update System  ---
xenlism-wildfire-0-0.1.20160511gitd3b9ad2.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-2f6141369f

-- 
You are receiving this mail because:
You are on the CC list 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 1338553] Review Request: nitroshare - Transfer files from one device to another made extremely simple

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



--- Comment #13 from Fedora Update System  ---
nitroshare-0.3.1-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-8f9c517c78

-- 
You are receiving this mail because:
You are on the CC list 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 1342737] Review Request: btrfs-sxbackup - Incremental btrfs snapshot backups with push/ pull support via SSH

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



--- Comment #13 from Fedora Update System  ---
btrfs-sxbackup-0.6.6-2.fc22 has been pushed to the Fedora 22 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-635a338e80

-- 
You are receiving this mail because:
You are on the CC list 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 1343895] Review Request: python-xunitmerge - utility for merging multiple XUnit xml reports into a single xml report

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



--- Comment #6 from Matthias Runge  ---
Builds fine on koji:
http://koji.fedoraproject.org/koji/watchlogs?taskID=14422509

The spec looks good to me, for completeness, *my* review is currently blocked
by https://bugzilla.redhat.com/show_bug.cgi?id=1247644

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