[Bug 1312303] Review Request: perl-Net-LDAP-Server-Test - Test Net:: LDAP code

[bugzilla]

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

Jitka Plesnikova  changed:

   What|Removed |Added

 CC||jples...@redhat.com



--- Comment #1 from Jitka Plesnikova  ---
Please update perl-Net-LDAP-Server-Test to the latest version 0.21 and add
build-requires 'make' and 'perl-generators'.

-- 
You are receiving this mail because:
You are on the CC 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 1398949] Review Request: bear - Game engine and editors dedicated to creating great 2D games

[bugzilla]

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



--- Comment #9 from Jeremy Newton  ---
I get a build error when attempting to build in mock (F25):

bear-factory/bear-editor/src/bf/CMakeFiles/bear-editor.dir/build.make:65:
recipe for target
'bear-factory/bear-editor/src/bf/CMakeFiles/bear-editor.dir/code/accelerator_table.cpp.o'
failed
make[2]: Leaving directory
'/builddir/build/BUILD/bear-ac6be8bebf35cd1a4d4151773707c9ee313b154e'
/builddir/build/BUILD/bear-ac6be8bebf35cd1a4d4151773707c9ee313b154e/bear-factory/bear-editor/src/bf/code/accelerator_table.cpp:
In member function 'void bf::accelerator_table::on_key_pressed(wxKeyEvent&)':
/builddir/build/BUILD/bear-ac6be8bebf35cd1a4d4151773707c9ee313b154e/bear-factory/bear-editor/src/bf/code/accelerator_table.cpp:102:45:
error: 'virtual bool wxEvtHandler::ProcessEvent(wxEvent&)' is inaccessible
within this context
   m_event_handler.ProcessEvent( command );
 ^
In file included from /usr/include/wx-3.0/wx/window.h:18:0,
 from
/builddir/build/BUILD/bear-ac6be8bebf35cd1a4d4151773707c9ee313b154e/bear-factory/bear-editor/src/bf/../bf/accelerator_table.hpp:19,
 from
/builddir/build/BUILD/bear-ac6be8bebf35cd1a4d4151773707c9ee313b154e/bear-factory/bear-editor/src/bf/code/accelerator_table.cpp:14:
/usr/include/wx-3.0/wx/event.h:3355:18: note: declared here
 virtual bool ProcessEvent(wxEvent& event);
  ^~~~
make[2]: ***
[bear-factory/bear-editor/src/bf/CMakeFiles/bear-editor.dir/code/accelerator_table.cpp.o]
Error 1
make[1]: *** [bear-factory/bear-editor/src/bf/CMakeFiles/bear-editor.dir/all]
Error 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 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

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



--- Comment #17 from Jeremy Newton  ---
(In reply to Jeremy Newton from comment #16)
> (In reply to MartinKG from comment #15)
> for bear, where longhash is the bear git commit hash:
> https://github.com/j-jorge/bear/archive/%{longhash}.tar.gz

In other words, use something similar to what you did with the bear package you
have in review. The sources should be valid URLs where possible.
Obviously CMakeLists.txt would be exempt from this.

-- 
You are receiving this mail because:
You are on the CC 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 1402456] Review Request: python3-idna - Internationalized Domain Names in Applications (IDNA)

[bugzilla]

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

Orion Poplawski  changed:

   What|Removed |Added

 Blocks||1399337




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1399337
[Bug 1399337] Review Request: python3-requests - HTTP library, written in
Python, for human beings
-- 
You are receiving this mail because:
You are on the CC 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 1399337] Review Request: python3-requests - HTTP library, written in Python, for human beings

[bugzilla]

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

Orion Poplawski  changed:

   What|Removed |Added

 Depends On||1402456




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1402456
[Bug 1402456] Review Request: python3-idna - Internationalized Domain Names
in Applications (IDNA)
-- 
You are receiving this mail because:
You are on the CC 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 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

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



--- Comment #16 from Jeremy Newton  ---
(In reply to MartinKG from comment #15)
> @Jeremy,
> Now I have managed to build a running asgp, but with a bundled bear-engine
> package. Can you have a look at it.
> 
> Spec URL:
> https://martinkg.fedorapeople.org/Review/asgp-1.0.16/test1/asgp.spec
> SRPM URL:
> https://martinkg.fedorapeople.org/Review/asgp-1.0.16/test1/asgp-1.0.16-1.
> fc25.src.rpm
> 
> changelog
> * Tue Nov 22 2016 Martin Gansser  - 1.0.16-1
> - Update to 1.0.16
> - Update Source URL

I gave it a shot, but it seems cmake is having trouble finding libclaw (I'm
using f25 with mock):

CMake Error at /usr/share/cmake/libclaw/libclaw-config.cmake:377 (MESSAGE):
  Could not find Claw library

Also, why are you using git clone to get the sources, when you can just use the
two following download links?

for bear, where longhash is the bear git commit hash:
https://github.com/j-jorge/bear/archive/%{longhash}.tar.gz

and for asgp:
https://github.com/j-jorge/%{name}/archive/%{version}.tar.gz

As well, I've noticed that asgp 1.18 was released 3 days ago. I would encourage
you to target that if possible.

-- 
You are receiving this mail because:
You are on the CC 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 1401961] Review Request: madplay - MPEG audio decoder and player

[bugzilla]

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



--- Comment #10 from Fedora Update System  ---
madplay-0.15.2b-11.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-47bf47857f

-- 
You are receiving this mail because:
You are on the CC 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 1398369] Review Request: python-idstools - Snort and Suricata Rule and Event Utilities

[bugzilla]

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



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

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

[bugzilla]

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



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

-- 
You are receiving this mail because:
You are on the CC 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 1401961] Review Request: madplay - MPEG audio decoder and player

[bugzilla]

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



--- Comment #9 from Fedora Update System  ---
madplay-0.15.2b-11.el6 has been pushed to the Fedora EPEL 6 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-36d08ed9b0

-- 
You are receiving this mail because:
You are on the CC 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 1403082] Review Request: nom-tam-fits - Java library for reading and writing FITS files

[bugzilla]

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

gil cattaneo  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||punto...@libero.it
   Assignee|nob...@fedoraproject.org|punto...@libero.it
  Flags||fedora-review+



--- Comment #1 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.
[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". 268 files have unknown license.
 Detailed output of licensecheck in /home/gil/1403082-nom-tam-
 fits/licensecheck.txt

 All java file include this header:
 /*
 * #%L
 * nom.tam FITS library
 * %%
 * Copyright (C) 2004 - 2015 nom-tam-fits
 * %%
 * This is free and unencumbered software released into the public domain.
 * 
 * Anyone is free to copy, modify, publish, use, compile, sell, or
 * distribute this software, either in source code form or as a compiled
 * binary, for any purpose, commercial or non-commercial, and by any
 * means.
 * 
 * In jurisdictions that recognize copyright laws, the author or authors
 * of this software dedicate any and all copyright interest in the
 * software to the public domain. We make this dedication for the benefit
 * of the public at large and to the detriment of our heirs and
 * successors. We intend this dedication to be an overt act of
 * relinquishment in perpetuity of all present and future rights to this
 * software under copyright law.
 * 
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
 * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
 * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
 * IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR
 * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
 * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
 * OTHER DEALINGS IN THE SOFTWARE.
 * #L%
 */


[x]: License file installed when any subpackage combination is installed.
[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 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 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 

[Bug 1399337] Review Request: python3-requests - HTTP library, written in Python, for human beings

[bugzilla]

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



--- Comment #5 from Tim Orling  ---
So, do we need a python3-idna package in order to package python3-requests?

Not sure how to proceed. Cannot install (for mock build) without python34-idna.

-- 
You are receiving this mail because:
You are on the CC 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 1378526] Review Request: gap-pkg-hapcryst - Integral cohomology computations of Bieberbach groups

[bugzilla]

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

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 1378526] Review Request: gap-pkg-hapcryst - Integral cohomology computations of Bieberbach groups

[bugzilla]

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



--- Comment #3 from Fedora Update System  ---
gap-pkg-hapcryst-0.1.11-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-64d4930cdd

-- 
You are receiving this mail because:
You are on the CC 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 1399365] Review Request: gap-pkg-xmod - Crossed Modules and Cat1-Groups for GAP

[bugzilla]

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

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 1399365] Review Request: gap-pkg-xmod - Crossed Modules and Cat1-Groups for GAP

[bugzilla]

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



--- Comment #5 from Fedora Update System  ---
gap-pkg-xmod-2.58-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-d14870fe5b

-- 
You are receiving this mail because:
You are on the CC 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 1403082] Review Request: nom-tam-fits - Java library for reading and writing FITS files

[bugzilla]

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

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 Blocks||1403030




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1403030
[Bug 1403030] Review Request: hdfview - Java HDF5 Object viewer
-- 
You are receiving this mail because:
You are on the CC 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 1403030] Review Request: hdfview - Java HDF5 Object viewer

[bugzilla]

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

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

   See Also|https://bugzilla.redhat.com |
   |/show_bug.cgi?id=1048951|
 Blocks||1048951
 Depends On||1403082



--- Comment #1 from Zbigniew Jędrzejewski-Szmek  ---
Spec URL: http://in.waw.pl/~zbyszek/hdfview/hdfview.spec
SRPM URL: http://in.waw.pl/~zbyszek/hdfview/hdfview-2.13.0-1.fc26.src.rpm

Now with appdata.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1048951
[Bug 1048951] hdfview is missing icons in the toolbar
https://bugzilla.redhat.com/show_bug.cgi?id=1403082
[Bug 1403082] Review Request: nom-tam-fits - Java library for reading and
writing FITS files
-- 
You are receiving this mail because:
You are on the CC 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 1403082] New: Review Request: nom-tam-fits - Java library for reading and writing FITS files

[bugzilla]

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

Bug ID: 1403082
   Summary: Review Request: nom-tam-fits - Java library for
reading and writing FITS files
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: zbys...@in.waw.pl
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://in.waw.pl/~zbyszek/fedora/nom-tam-fits/nom-tam-fits.spec
SRPM URL:
http://in.waw.pl/~zbyszek/fedora/nom-tam-fits/nom-tam-fits-1.15.1-1.fc26.src.rpm
Description:
FITS (Flexible Image Transport System) is the standard data format in
astronomy used for the transport, analysis, and archival storage of
scientific data sets.

This library provides efficient I/O for FITS images and binary
tables. All basic FITS formats and GZIP compressed files are
supported.

Fedora Account System Username: zbyszek

Note: This spec file was created by gil. I updated it to 1.15.1 and I'm
submitting it for review with his permission.

-- 
You are receiving this mail because:
You are on the CC 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 1381029] Review Request: python-more-itertools - Python library for efficient use of itertools utility in python

[bugzilla]

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

marcindulak  changed:

   What|Removed |Added

 CC||marcin.du...@gmail.com



--- Comment #18 from marcindulak  ---
Please see bug #1336097 - it may be possible to build your package for EPEL7

-- 
You are receiving this mail because:
You are on the CC 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 1403030] Review Request: hdfview - Java HDF5 Object viewer

[bugzilla]

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

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

   See Also||https://bugzilla.redhat.com
   ||/show_bug.cgi?id=1048951



-- 
You are receiving this mail because:
You are on the CC 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 1403030] New: Review Request: hdfview - Java HDF5 Object viewer

[bugzilla]

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

Bug ID: 1403030
   Summary: Review Request: hdfview - Java HDF5 Object viewer
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: zbys...@in.waw.pl
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://in.waw.pl/~zbyszek/hdfview.spec
SRPM URL: http://in.waw.pl/~zbyszek/hdfview-2.13.0-1.fc26.src.rpm
Description:
HDF is a versatile data model that can represent very complex data objects
and a wide variety of meta-data. It is a completely portable file format
with no limit on the number or size of data objects in the collection.

This package provides a HDF4/HDF5 viewer.

Fedora Account System Username: zbyszek

Notes:
This package was a subpackage of the jhdf5 package, and was split out upstream.
Splitting this out allows us to update jhdf5 to the latest version.

To build this, it's most likely that the updated jhdf5 is necessary; it hasn't
been built yet in rawhide; I'm waiting for this review ;)

Also, tam-nom-fits package is necessary. Gil Cattaneo has created a spec file,
but it hasn't been submitted for review yet. I'll query whether he prefers to
submit it himself, otherwise I'll do it myself.

When this was built from jhdf5, three separate jars were provided: jhdfobj,
jdf4obj, jhdf5obj. The first two contained the other two and some other stuff.
I don't think this is useful, so I'm only providing jhdfobj.

Finally, I know that the appinfo metadata is missing. I'm working on it, but
it's not required, and I'd like to get this package ready as quickly as
possible.

-- 
You are receiving this mail because:
You are on the CC 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 1289717] Review Request: viennacl - Linear algebra and solver library using CUDA, OpenCL, and OpenMP

[bugzilla]

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



--- Comment #28 from Ilya Gradina  ---
(In reply to Sandro Mani from comment #23)
> @Ilya Please build for rawhide also

sorry but I can not yet.
https://pagure.io/releng/issue/6555

-- 
You are receiving this mail because:
You are on the CC 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 1397089] Review Request: dnscrypt-proxy-gui - GUI wrapper for dnscrypt-proxy

[bugzilla]

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



--- Comment #22 from greg.helli...@gmail.com ---
I think it's worth noting that the package does not compile against f26. I'm
not certain if this is a fault of this package or of its dependencies. This is
the output I see when trying to build for Rawhide targets:

There is no installed groups file.
Maybe run: yum groups mark convert (see man yum)
Error: libcrypt conflicts with libcrypt-nss-2.24.90-21.fc26.x86_64
Error: libcrypt-nss conflicts with libcrypt-2.24.90-21.fc26.x86_64
 You could try using --skip-broken to work around the problem
 You could try running: rpm -Va --nofiles --nodigest
ERROR:
Exception(/home/vagrant/1397089-dnscrypt-proxy-gui/srpm-unpacked/dnscrypt-proxy-gui-1.2.3-3.fc26.src.rpm)
Config(f26-candidate-x86_64) 0 minutes 18 second

Can you briefly look into that to discern if it's a problem with this package
or with the libcrypt packages? That's my last outstanding concern.

-- 
You are receiving this mail because:
You are on the CC 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 1395255] Review Request: python3-zope-interface - Zope 3 Interface Infrastructure

[bugzilla]

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

Randy Barlow  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #3 from Randy Barlow  ---
Package Review
==

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



= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Development (unversioned) .so files in -devel subpackage, if present.
 Note: Unversioned so-files in private %_libdir subdirectory (see
 attachment). Verify they are not in ld path.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Unknown or generated". 81 files have unknown license. Detailed
 output of licensecheck in /home/rbarlow/reviews/1395255-python3-zope-
 interface/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package does not own files or directories owned by other packages.
 Note: Dirs in package are owned also by: /usr/lib64/python3.5/site-
 packages/zope(python3-zope-i18nmessageid, python3-zope-interface)
 randy: This seems to be how zope is doing things.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 204800 bytes in 16 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]: 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.
[-]: A package which is used by another package via an egg interface should
 provide egg info.
[x]: Package meets the Packaging Guidelines::Python
[x]: Package contains BR: python2-devel or python3-devel
[x]: Binary eggs must be removed in %prep

= SHOULD items =

Generic:
[-]: If the source package does not include 

[Bug 1398369] Review Request: python-idstools - Snort and Suricata Rule and Event Utilities

[bugzilla]

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



--- Comment #10 from Fedora Update System  ---
python-idstools-0.5.4-1.fc25 has been pushed to the Fedora 25 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-a0684e943f

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

[bugzilla]

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



--- Comment #43 from Fedora Update System  ---
libxsmm-1.6.1-1.fc25 has been pushed to the Fedora 25 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-0d598f3556

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

[bugzilla]

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



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

-- 
You are receiving this mail because:
You are on the CC 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 1398369] Review Request: python-idstools - Snort and Suricata Rule and Event Utilities

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #9 from Fedora Update System  ---
python-idstools-0.5.4-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-736b58c885

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

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



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

-- 
You are receiving this mail because:
You are on the CC 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 1379765] Review Request: dolphin-emu - GameCube / Wii / Triforce Emulator

[bugzilla]

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



--- Comment #7 from Jeremy Newton  ---
(In reply to MartinKG from comment #6)
> please don't forget to mark the file font-licenses.txt as %license in the
> final package.
> 
> Package APPROVED
> 
> Issues:
> ===
> - 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.
>   Note: License file font-licenses.txt is not marked as %license
>   See:
>   http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text

Thanks! I'll make sure I include that before importing

-- 
You are receiving this mail because:
You are on the CC 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 1397842] Review Request: python-cccolutils - Python Kerberos Credential Cache Collection Utilities

[bugzilla]

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2016-12-08 13:21:44



--- Comment #20 from Fedora Update System  ---
python-cccolutils-1.4-1.fc25 has been pushed to the Fedora 25 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 1395255] Review Request: python3-zope-interface - Zope 3 Interface Infrastructure

[bugzilla]

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

Aurelien Bompard  changed:

   What|Removed |Added

  Flags|needinfo?(aurelien@bompard. |
   |org)|



--- Comment #2 from Aurelien Bompard  ---
Ah, my older version of rpmlint did not report that, but now that I'm on F25 it
does, nice :-)

Thanks for the review, I fixed both issues.

Spec URL:
https://abompard.fedorapeople.org/reviews/python3-zope-interface/python3-zope-interface.spec
SRPM URL:
https://abompard.fedorapeople.org/reviews/python3-zope-interface/python3-zope-interface-4.3.2-1.el7.centos.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 1398400] Review Request: superlu_dist - Solution of large, sparse, nonsymmetric systems of linear equations

[bugzilla]

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

Antonio Trande  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #20 from Antonio Trande  ---
Use parallel Make if possible.
Package approved.

Package Review
==

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



= MUST items =

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

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "BSD (3 clause)", "Unknown or generated". 276 files have
 unknown license. Detailed output of licensecheck in
 /home/sagitter/1398400-superlu_dist/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[x]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: 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 must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: 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:
[?]: Uses parallel make %{?_smp_mflags} macro.
[-]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
 superlu_dist-openmpi , superlu_dist-openmpi-devel , superlu_dist-doc ,
 superlu_dist-mpich , superlu_dist-mpich-devel , superlu_dist-debuginfo
[x]: Package functions as described.
[x]: Latest version 

[Bug 1395255] Review Request: python3-zope-interface - Zope 3 Interface Infrastructure

[bugzilla]

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

Randy Barlow  changed:

   What|Removed |Added

 CC||aurel...@bompard.org
  Flags||needinfo?(aurelien@bompard.
   ||org)



--- Comment #1 from Randy Barlow  ---
To pass this, fedora-review wants the COPYRIGHT.txt file to be marked with
%license as well.

Optionally, you could remove the .gitignore file that rpmlint is marking as an
error, and then you'd have a clean rpmlint.

-- 
You are receiving this mail because:
You are on the CC 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 1395255] Review Request: python3-zope-interface - Zope 3 Interface Infrastructure

[bugzilla]

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

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 1394275] Review Request: librdkafka - C/ C++ library implementation of the Apache Kafka protocol

[bugzilla]

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

Michael Schwendt  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)
 CC||de...@rocknsm.io



--- Comment #20 from Michael Schwendt  ---
*** Bug 1390595 has been marked as a duplicate of this bug. ***


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 1390595] Review Request: librdkafka - The Apache Kafka C Library

[bugzilla]

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

Michael Schwendt  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |DUPLICATE
Last Closed||2016-12-08 11:22:30



--- Comment #5 from Michael Schwendt  ---
I advise comparing the spec files for the two packages. This one contains a
multitude of packaging mistakes. It would have needed a lot of work to get it
through the review process.


> When doing a review, the reviewers use fedora-review -b
> which automates some tasks.

Packagers ought to do that, too.

"fedora-review -b 1390595" would try to retrieve the latest spec file and
src.rpm package from this ticket, perform local test builds and run lots of
checks on the result.

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

-- 
You are receiving this mail because:
You are on the CC 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 1394275] Review Request: librdkafka - C/ C++ library implementation of the Apache Kafka protocol

[bugzilla]

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



--- Comment #19 from Michael Schwendt  ---
> Name: librdkafka  
> Group:Development/Libraries

The base Group for runtime library packages has been "System
Environment/Libraries" for many years back to very old releases of Red Hat
Linux.

-- 
You are receiving this mail because:
You are on the CC 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 1398400] Review Request: superlu_dist - Solution of large, sparse, nonsymmetric systems of linear equations

[bugzilla]

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



--- Comment #19 from Zbigniew Jędrzejewski-Szmek  ---
No objection from me.

-- 
You are receiving this mail because:
You are on the CC 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 1398400] Review Request: superlu_dist - Solution of large, sparse, nonsymmetric systems of linear equations

[bugzilla]

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



--- Comment #18 from Dave Love  ---
I changed the compilation and linkage flags and kept the name, as at least some
-devel people think the guidelines mean what they say about upstream names.

SRPM URL:
https://loveshack.fedorapeople.org/review/superlu_dist-5.1.2-3.el6.src.rpm
SPEC URL: https://loveshack.fedorapeople.org/review/superlu_dist.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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1176273] Review Request: andy-super-great-park - 2D arcade game

[bugzilla]

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



--- Comment #15 from MartinKG  ---
@Jeremy,
Now I have managed to build a running asgp, but with a bundled bear-engine
package. Can you have a look at it.

Spec URL: https://martinkg.fedorapeople.org/Review/asgp-1.0.16/test1/asgp.spec
SRPM URL:
https://martinkg.fedorapeople.org/Review/asgp-1.0.16/test1/asgp-1.0.16-1.fc25.src.rpm

changelog
* Tue Nov 22 2016 Martin Gansser  - 1.0.16-1
- Update to 1.0.16
- Update Source URL

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

[Bug 1398369] Review Request: python-idstools - Snort and Suricata Rule and Event Utilities

[bugzilla]

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



--- Comment #8 from Fedora Update System  ---
python-idstools-0.5.4-1.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-9f3d307043

-- 
You are receiving this mail because:
You are on the CC 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 1402886] New: Review Request: kio-gdrive - KDE I/O Google Drive

[bugzilla]

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

Bug ID: 1402886
   Summary: Review Request: kio-gdrive - KDE I/O Google Drive
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: vasc...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://raw.githubusercontent.com/RussianFedora/kio-gdrive/master/kio-gdrive.spec
SRPM URL:
http://koji.russianfedora.pro/kojifiles/packages/kio-gdrive/1.0.4/1.fc26/src/kio-gdrive-1.0.4-1.fc26.src.rpm

Description:
Google Drive for KDE file applications such as Dolphin, Gwenview etc.


It will be good has this in repo because Google still not present any linux
client software.

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

[bugzilla]

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



--- Comment #25 from Jeremy Newton  ---
If I understand correctly, lollypop should require lollypop-portal, not the
other way around.

As well, is it possible to subpackage the CLI?

-- 
You are receiving this mail because:
You are on the CC 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 1398369] Review Request: python-idstools - Snort and Suricata Rule and Event Utilities

[bugzilla]

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



--- Comment #7 from Fedora Update System  ---
python-idstools-0.5.4-1.fc24 has been submitted as an update to Fedora 24.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-736b58c885

-- 
You are receiving this mail because:
You are on the CC 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 1398369] Review Request: python-idstools - Snort and Suricata Rule and Event Utilities

[bugzilla]

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

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 1398369] Review Request: python-idstools - Snort and Suricata Rule and Event Utilities

[bugzilla]

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



--- Comment #6 from Fedora Update System  ---
python-idstools-0.5.4-1.fc25 has been submitted as an update to Fedora 25.
https://bodhi.fedoraproject.org/updates/FEDORA-2016-a0684e943f

-- 
You are receiving this mail because:
You are on the CC 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 1398369] Review Request: python-idstools - Snort and Suricata Rule and Event Utilities

[bugzilla]

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



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

-- 
You are receiving this mail because:
You are on the CC 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 1399365] Review Request: gap-pkg-xmod - Crossed Modules and Cat1-Groups for GAP

[bugzilla]

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



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

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

[bugzilla]

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



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

%changelog
* Thu Dec 08 2016 Martin Gansser  - 0.9.222-1
- Update to 0.9.222
- Set correct file permission
- Add subpkg lollypop-portal

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

[bugzilla]

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

Ville Skyttä  changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
 CC||ville.sky...@iki.fi
 Blocks||496968 (DebugInfo)
 Resolution|ERRATA  |---
   Keywords||Reopened



--- Comment #18 from Ville Skyttä  ---
(In reply to Alejandro Alvarez from comment #1)
> MUST
> 
> 
> [OK] -debuginfo package or justification otherwise.
> [OK] %build honors applicable compiler flags or justifies otherwise.

The -debuginfo package is not ok, it lacks sources and the package is not built
with $RPM_OPT_FLAGS (which is quite probably the reason for the faulty
-debuginfo). This needs to be fixed.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=496968
[Bug 496968] Tracking bug for packages with debuginfo problems
-- 
You are receiving this mail because:
You are on the CC 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 1379765] Review Request: dolphin-emu - GameCube / Wii / Triforce Emulator

[bugzilla]

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

MartinKG  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #6 from MartinKG  ---
please don't forget to mark the file font-licenses.txt as %license in the final
package.

Package APPROVED

Issues:
===
- 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.
  Note: License file font-licenses.txt is not marked as %license
  See:
  http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text

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

[bugzilla]

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

Tomas Repik  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2016-12-08 05:00:54



--- Comment #39 from Tomas Repik  ---
http://koji.fedoraproject.org/koji/buildinfo?buildID=823599

-- 
You are receiving this mail because:
You are on the CC 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 1384242] Review Request: python-astropy-helpers - Utilities for building and installing Astropy and affiliated package

[bugzilla]

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

Yin.JianHong  changed:

   What|Removed |Added

 Depends On||1356123



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

[bugzilla]

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

Bertrand  changed:

   What|Removed |Added

 CC||bra...@redhat.com
Summary|Review Request: |Review Request:
   |python-vitrageclient -  |python-vitrageclient -
   |Python client for Virage|Python client for Vitrage
   |REST API|REST API



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