[Bug 1543060] Review Request: php-phpunit-dbunit4 - Extension for database interaction testing for PHPUnit 7

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1543060



--- Comment #4 from Remi Collet  ---
Thanks again!

SCM request
https://pagure.io/releng/fedora-scm-requests/issue/4459

-- 
You are receiving this mail because:
You are on the CC 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 1542902] Review Request: php-pecl-redis4 - Extension for communicating with the Redis key-value store

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542902



--- Comment #3 from Remi Collet  ---
Temporarily disable s390x build
https://git.remirepo.net/cgit/rpms/php/pecl/php-pecl-redis4.git/commit/?id=cf6f083fe277e41c871e60da88c59fe06e2015a4

Will investigate later on the s390x issue (if upstream don't fix it before)

Scratch build
https://koji.fedoraproject.org/koji/taskinfo?taskID=24824179

Spec URL:
https://git.remirepo.net/cgit/rpms/php/pecl/php-pecl-redis4.git/plain/php-pecl-redis4.spec?id=cf6f083fe277e41c871e60da88c59fe06e2015a4
SRPM URL:
http://rpms.remirepo.net/SRPMS/php-pecl-redis4-4.0.0~RC1-3.remi.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 1542807] Review Request: gap-pkg-toric - Computations with toric varieties in GAP

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542807

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



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


[Bug 1542931] Review Request: cri-tools - CLI and validation tools for Container Runtime Interface

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542931



--- Comment #3 from Lokesh Mandvekar  ---
ha, thanks for the approval

-- 
You are receiving this mail because:
You are on the CC 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 1542931] Review Request: cri-tools - CLI and validation tools for Container Runtime Interface

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542931



--- Comment #2 from Lokesh Mandvekar  ---
added critest binary as well.

SRPM URL:
https://pagure.io/cri-tools/blob/master/f/SRPMS/cri-tools-1.0.0-2.alpha.0.git653cc8c.fc27.src.rpm

Spec URL unchanged.

-- 
You are receiving this mail because:
You are on the CC 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 1542902] Review Request: php-pecl-redis4 - Extension for communicating with the Redis key-value store

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542902



--- Comment #2 from Remi Collet  ---
(In reply to Robert-André Mauchin from comment #1)
>  - Group: is not needed in Fedora.

>  - The very last test fails:

Damned... I forgot to check new tests running a scratch build (and local mock
have network because of bug #1514028)

Fixed by
https://git.remirepo.net/cgit/rpms/php/pecl/php-pecl-redis4.git/commit/?id=ededdd74af7211f1fd5be84db712db58e0f9e2ab

Scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=24823796

I now need to fix the bigendian issue...

-- 
You are receiving this mail because:
You are on the CC 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 1542650] Review Request: QEverCloud - Unofficial Evernote Cloud API for Qt5

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542650

Ben Rosser  changed:

   What|Removed |Added

  Flags||fedora-review+



--- Comment #1 from Ben Rosser  ---
There seem to be some unit tests that aren't run. The README says:

> Also, if Qt4's QtTest or Qt5's Qt5Test modules are found during the pre-build 
> configuration, the unit tests are enabled and can be run with `make test` 
> command.

It looks as though the test binary "test_QEverCloud" is indeed getting built
according to the log file. Is it possible to run these in %check?

In addition, rpmlint says:

qevercloud.x86_64: E: library-without-ldconfig-postin
/usr/lib64/libqt5qevercloud.so.4.0.0
qevercloud.x86_64: E: library-without-ldconfig-postun
/usr/lib64/libqt5qevercloud.so.4.0.0

This was recently discussed on the devel list. The takeaway from the change
proposal page seems to be that this is fine if you only want to support Fedora
28+, but if you want to backport this to older releases, make sure to add the
ldconfig post scriplets (or use the new macros):

https://fedoraproject.org/wiki/Changes/Removing_ldconfig_scriptlets#Upgrade.2Fcompatibility_impact

Otherwise, the package looks fine. Since neither of the above issues are
blockers, I'm going to go ahead and approve it.

-- 
You are receiving this mail because:
You are on the CC 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 1492475] Review Request: aftertheflood-sparks-fonts - a font to display charts within text

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1492475



--- Comment #8 from Elliott Sales de Andrade  ---
Are not Bar and Dot separate families, and thus should be packaged separately? 

Can you still not build from source? I see some sources in the archive right
now.

There's also something wrong with either the metrics or LO, because if I type
out the example using Sparks Bar Medium (and *some* of the other weights), the
cursor ends up a little short (as does the highlight if I select everything.)

-- 
You are receiving this mail because:
You are on the CC 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 1539209] Review Request: ddgr - DuckDuckGo from the terminal

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1539209

Ben Rosser  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #4 from Ben Rosser  ---
Excellent, package approved.

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


[Bug 1539189] Review Request: golang-github-zyedidia-pty - PTY interface for Go

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1539189



--- Comment #3 from Elliott Sales de Andrade  ---
There's only really one significant commit; maybe you can try to convince the
dev to send the changes upstream?

-- 
You are receiving this mail because:
You are on the CC 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 1541566] Review Request: whipper - Python CD-DA ripper

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1541566



--- Comment #5 from Matthew  ---
Spec URL:
https://copr-be.cloud.fedoraproject.org/results/mruszczyk/whipper/fedora-rawhide-x86_64/00711948-whipper/whipper.spec

SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/mruszczyk/whipper/fedora-rawhide-x86_64/00711948-whipper/whipper-0.6.0-3.fc28.src.rpm

Description: CD-DA ripper focused on accuracy over speed.

Fedora Account System Username: mruszczyk

(In reply to William Moreno from comment #4)
> Spec looks good, will run fedora-review soon.

Thank you very much for your time, I hope to hear from you soon.

I've had to update the spec file above because I was adding the incorrect
gobject dependency. It's fixed in the latest version on the copr repo and
linked above.

-- 
You are receiving this mail because:
You are on the CC 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 1542807] Review Request: gap-pkg-toric - Computations with toric varieties in GAP

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542807



--- Comment #3 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/gap-pkg-toric. You may commit to the branch
"f27" in about 10 minutes.

-- 
You are receiving this mail because:
You are on the CC 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 1542807] Review Request: gap-pkg-toric - Computations with toric varieties in GAP

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542807



--- Comment #2 from Jerry James  ---
Thank you!  I appreciate the review.

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


[Bug 1518463] Review Request: tinyobjloader - Tiny wavefront obj loader

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1518463

Rich Mattes  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2018-02-07 19:50:54



-- 
You are receiving this mail because:
You are on the CC 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 1411590] Review Request: ignition-msgs - Common messages for the ignition framework

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1411590

Rich Mattes  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2018-02-07 19:49:11



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


[Bug 972346] Review Request: ros-common_msgs - Common ROS Messages

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=972346

Rich Mattes  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |WONTFIX
Last Closed||2018-02-07 19:48:06



-- 
You are receiving this mail because:
You are on the CC 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 972348] Review Request: ros-actionlib - Interface for pre-emptible tasks

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=972348
Bug 972348 depends on bug 972346, which changed state.

Bug 972346 Summary: Review Request: ros-common_msgs - Common ROS Messages
https://bugzilla.redhat.com/show_bug.cgi?id=972346

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |WONTFIX



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


[Bug 928584] Review Request: ros-std_msgs - Standard ROS Messages

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=928584

Rich Mattes  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |WONTFIX
Last Closed||2018-02-07 19:47:03



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


[Bug 972348] Review Request: ros-actionlib - Interface for pre-emptible tasks

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=972348

Rich Mattes  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |WONTFIX
Last Closed||2018-02-07 19:47:35



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


[Bug 928584] Review Request: ros-std_msgs - Standard ROS Messages

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=928584
Bug 928584 depends on bug 927462, which changed state.

Bug 927462 Summary: Review Request: roscpp_core - The ROS C++ API
https://bugzilla.redhat.com/show_bug.cgi?id=927462

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |WONTFIX



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


[Bug 972346] Review Request: ros-common_msgs - Common ROS Messages

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=972346
Bug 972346 depends on bug 928584, which changed state.

Bug 928584 Summary: Review Request: ros-std_msgs - Standard ROS Messages
https://bugzilla.redhat.com/show_bug.cgi?id=928584

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |WONTFIX



-- 
You are receiving this mail because:
You are on the CC 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 927462] Review Request: roscpp_core - The ROS C++ API

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=927462

Rich Mattes  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |WONTFIX
Last Closed||2018-02-07 19:46:29



-- 
You are receiving this mail because:
You are on the CC 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 972345] Review Request: ros_comm - ROS communications-related packages

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=972345

Rich Mattes  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX
Last Closed||2018-02-07 19:45:48



-- 
You are receiving this mail because:
You are on the CC 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 972348] Review Request: ros-actionlib - Interface for pre-emptible tasks

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=972348
Bug 972348 depends on bug 927461, which changed state.

Bug 927461 Summary: Review Request: ros - The Robot Operating System
https://bugzilla.redhat.com/show_bug.cgi?id=927461

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX



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


[Bug 972345] Review Request: ros_comm - ROS communications-related packages

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=972345
Bug 972345 depends on bug 927461, which changed state.

Bug 927461 Summary: Review Request: ros - The Robot Operating System
https://bugzilla.redhat.com/show_bug.cgi?id=927461

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX



-- 
You are receiving this mail because:
You are on the CC 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 927461] Review Request: ros - The Robot Operating System

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=927461

Rich Mattes  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |WONTFIX
Last Closed||2018-02-07 19:45:13



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


[Bug 972346] Review Request: ros-common_msgs - Common ROS Messages

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=972346



--- Comment #13 from Rich Mattes  ---
I added you as admin to the ros packages that I got through the review process,
and I'm going to close out the review requests I still have open at this point. 

I'm also trying to add you to the "robotics-sig" group which is attached to a
bunch of the python ros packages, gazebo, etc., but I don't know how to do it
now that groups are managed through pagure.  I'll try to ask on IRC and get it
done.

Rich

-- 
You are receiving this mail because:
You are on the CC 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 1520585] Review Request: perl-Test-SQL-Data - Helps running SQL tests : database preparing and result matching

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1520585



--- Comment #14 from Fedora Update System  ---
perl-Test-SQL-Data-0.0.6-2.el7 has been submitted as an update to Fedora EPEL
7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2018-483d50018f

-- 
You are receiving this mail because:
You are on the CC 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 1525698] Review Request: jid - Json Incremental Digger

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1525698



--- Comment #5 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/jid. You may commit to the branch "f27" in
about 10 minutes.

-- 
You are receiving this mail because:
You are on the CC 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 1525698] Review Request: jid - Json Incremental Digger

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1525698

Elliott Sales de Andrade  changed:

   What|Removed |Added

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



--- Comment #4 from Elliott Sales de Andrade  ---
OK. APPROVED

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


[Bug 1539134] Review Request: bouncycastle1.58 - Bouncy Castle Cryptography APIs for Java

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1539134

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



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


[Bug 1539134] Review Request: bouncycastle1.58 - Bouncy Castle Cryptography APIs for Java

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1539134



--- Comment #5 from Fedora Update System  ---
bouncycastle1.58-1.58-1.el6 canl-java-2.5.0-2.el6 jglobus-2.1.0-5.el6
voms-api-java-3.2.0-7.el6 voms-clients-java-3.0.7-6.el6 has been submitted as
an update to Fedora EPEL 6.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2018-71db8f6f28

-- 
You are receiving this mail because:
You are on the CC 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 1520572] Review Request: perl-Authen-DecHpwd - DEC VMS password hashing

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1520572



--- Comment #15 from Fedora Update System  ---
perl-Authen-DecHpwd-2.007-1.el7 has been submitted as an update to Fedora EPEL
7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2018-5480df179b

-- 
You are receiving this mail because:
You are on the CC 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 1520571] Review Request: perl-Data-Integer - Details of the native integer data type

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1520571



--- Comment #15 from Fedora Update System  ---
perl-Data-Integer-0.006-1.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2018-ea8e5d2eb5

-- 
You are receiving this mail because:
You are on the CC 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 1520570] Review Request: perl-Scalar-String - String aspects of scalars

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1520570



--- Comment #12 from Fedora Update System  ---
perl-Scalar-String-0.003-1.el7 has been submitted as an update to Fedora EPEL
7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2018-6c82f2d57f

-- 
You are receiving this mail because:
You are on the CC 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 1428202] Review Request: sirikali - GUI front end to encfs,cryfs, gocryptfs and securefs

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1428202



--- Comment #25 from Raphael Groner  ---
Spec URL: https://raphgro.fedorapeople.org/review/util/sirikali.spec
SRPM URL:
https://raphgro.fedorapeople.org/review/util/sirikali-1.3.2-2.fc27.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 1428202] Review Request: sirikali - GUI front end to encfs,cryfs, gocryptfs and securefs

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1428202



--- Comment #24 from Raphael Groner  ---
Sorry for the delay. I was busy with other work.

Spec URL: https://raphgro.fedorapeople.org/review/util/sirikali.spec
SRPM URL:
https://raphgro.fedorapeople.org/review/util/sirikali-1.3.2-1.fc27.src.rpm

%changelog
* Wed Feb 07 2018 Raphael Groner  - 1.3.2-2
- drop obsolete scriptlets
- drop explicit file permission
- include upstreamed patches

Test builds: https://koji.fedoraproject.org/koji/taskinfo?taskID=24811226

-- 
You are receiving this mail because:
You are on the CC 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 1520569] Review Request: perl-Crypt-UnixCrypt_XS - Perl xs interface for a portable traditional crypt function

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1520569



--- Comment #14 from Fedora Update System  ---
perl-Crypt-UnixCrypt_XS-0.11-2.el7 has been submitted as an update to Fedora
EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2018-095c3b7f59

-- 
You are receiving this mail because:
You are on the CC 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 1520568] Review Request: perl-Crypt-MySQL - Emulate MySQL PASSWORD() function

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1520568



--- Comment #11 from Fedora Update System  ---
perl-Crypt-MySQL-0.04-1.el7 has been submitted as an update to Fedora EPEL 7.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2018-4abaf72a99

-- 
You are receiving this mail because:
You are on the CC 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 1522917] Review Request: perl-Algorithm-Loops - Perl module for looping constructs

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1522917



--- Comment #13 from Fedora Update System  ---
perl-Algorithm-Loops-1.032-1.el7 has been submitted as an update to Fedora EPEL
7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2018-d314769f44

-- 
You are receiving this mail because:
You are on the CC 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 1536852] Review Request: molsketch - Molecular Structures Editor

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1536852



--- Comment #23 from Alexander Ploumistos  ---
New builds, after Hendrik's update.
I've also remembered to use %ldconfig_scriptlets.

Spec URL: https://alexpl.fedorapeople.org/packages/Molsketch/molsketch.spec
SRPM URL:
https://alexpl.fedorapeople.org/packages/Molsketch/molsketch-0.5.1-5.fc28.src.rpm

koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=24808294

and one for f27: https://koji.fedoraproject.org/koji/taskinfo?taskID=24808437

-- 
You are receiving this mail because:
You are on the CC 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 1540347] Review Request: hashcat - password recovery utility

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1540347



--- Comment #17 from Tomáš Korbař  ---
Next version available
SRPM: https://tkorbar.fedorapeople.org/hashcat-4.0.1-2.fc27.src.rpm
SPEC: https://tkorbar.fedorapeople.org/hashcat.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 1377631] Review Request: gnome-shell-extension-netspeed - A gnome-shell extension to show speed of the internet

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1377631



--- Comment #16 from mgans...@alice.de  ---
@Andrew I think we can go ahead with the review.

Spec URL:
https://martinkg.fedorapeople.org/Review/SPECS/gnome-shell-extension-netspeed.spec
SRPM URL:
https://martinkg.fedorapeople.org/Review/SRPMS/gnome-shell-extension-netspeed-3.26-0.1.20180207git02a51b2.fc27.src.rpm

%changelog
* Wed Feb 07 2018 Martin Gansser  -
3.26-0.1.20180207git02a51b2
- Update to recent git version 3.26-0.1.20180207git02a51b2

-- 
You are receiving this mail because:
You are on the CC 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 1542646] Review Request: patool - Portable command line archive file manager

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542646



--- Comment #7 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/patool. You may commit to the branch "f27"
in about 10 minutes.

-- 
You are receiving this mail because:
You are on the CC 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 1492475] Review Request: aftertheflood-sparks-fonts - a font to display charts within text

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1492475

Robert-André Mauchin  changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |aftertheflood-spark-fonts   |aftertheflood-sparks-fonts
   |- a font to display charts  |- a font to display charts
   |within text |within text



--- Comment #7 from Robert-André Mauchin  ---
The font has been renamed to Sparks and version 2.0 has been released. License
has been added to the font metadata too.

SPEC URL:
https://raw.githubusercontent.com/eclipseo/packaging/8ecc6a5/aftertheflood-sparks-fonts.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/aftertheflood-sparks/fedora-rawhide-x86_64/00711730-aftertheflood-sparks-fonts/aftertheflood-sparks-fonts-2.0-1.fc28.src.rpm

SPEC diff:
https://github.com/eclipseo/packaging/commit/8ecc6a50ac8e276fb7198962f8c64df64c120893#diff-e272db7fccba87eda45a3aa43baacdc1

-- 
You are receiving this mail because:
You are on the CC 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 1538658] Review Request: python-anyconfig - common API to load and dump configuration files

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1538658



--- Comment #9 from Brett Lentz  ---
I've fixed the url and the buildrequires. File URLs haven't changed since
comment #7

-- 
You are receiving this mail because:
You are on the CC 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 1542931] Review Request: cri-tools - CLI and validation tools for Container Runtime Interface

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542931

Frantisek Kluknavsky  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Frantisek Kluknavsky  ---
Looks good to 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 1543060] Review Request: php-phpunit-dbunit4 - Extension for database interaction testing for PHPUnit 7

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1543060

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #3 from Robert-André Mauchin  ---
Package approved.


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: "BSD (2 clause)", "Unknown or generated", "*No copyright* BSD
 (3 clause)". 135 files have unknown license. Detailed output of
 licensecheck in /home/bob/packaging/review/php-phpunit-dbunit4/review-
 php-phpunit-dbunit4/licensecheck.txt
[x]: Package requires other packages for directories it uses.
 Note: No known owner of /usr/share/php/PHPUnit7
[x]: Package must own all directories that it creates.
 Note: Directories without known owners: /usr/share/php/PHPUnit7
[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: 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]: 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

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[x]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
 files.
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[x]: No file requires 

[Bug 1119197] Review Request: gnushogi - Shogi (Japanese Chess) AI engine

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1119197



--- Comment #21 from Kevin Fenzi  ---
Just a small drive by comment: 

(In reply to Ben Rosser from comment #20)
> copr is fine for scratch builds these days (it uses Koji behind the scenes).

copr does not use koji at all. ;) Scratch builds in koji are similar to copr
builds with the 'network access' turned off, but they aren't the exact same
systems doing the building. Just wanted to note it... carry on.

-- 
You are receiving this mail because:
You are on the CC 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 1542902] Review Request: php-pecl-redis4 - Extension for communicating with the Redis key-value store

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542902

Robert-André Mauchin  changed:

   What|Removed |Added

 CC||zebo...@gmail.com



--- Comment #1 from Robert-André Mauchin  ---
 - Group: is not needed in Fedora.

 - The very last test fails:

testConnectException   
Warning: Redis::connect(): php_network_getaddresses: getaddrinfo failed: Name
or service not known in
/builddir/build/BUILD/php-pecl-redis4-4.0.0~RC1/NTS/tests/RedisTest.php on line
5169
[FAILED]

Looking at the source, it tries to connect to Github, but Internet is not
available from Mock/Koji:

public function testConnectException() {
$redis = new Redis();
try {
$redis->connect('github.com', 6379, 0.01);
}  catch (Exception $e) {
$this->assertTrue(strpos($e, "timed out") !== false);
}
}


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]: 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:
[-]: 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)", "PHP (v3.01)", "Unknown or generated". 26
 files have unknown license. Detailed output of licensecheck in
 /home/bob/packaging/review/php-pecl-redis4/review-php-pecl-
 redis4/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/share/doc/pecl/redis
 (php-pecl-redis)
[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.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 133120 bytes in 4 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package 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]: %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
 

[Bug 1542671] Review Request: emojitwo-fonts - Color emoji font with a flat style

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542671



--- Comment #3 from Peter Oliver  ---
(In reply to Bastien Nocera from comment #2)

> Please make sure that fontconfig has the necessary
> changes to prefer emojitwo over emojione, but can still use both.

45-generic.conf and 60-generic.conf already mention Emoji Two before Emoji One,
so I think that means there's nothing more to do, there.

-- 
You are receiving this mail because:
You are on the CC 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 1543060] Review Request: php-phpunit-dbunit4 - Extension for database interaction testing for PHPUnit 7

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1543060



--- Comment #2 from Remi Collet  ---
Looks like I'm tired ;)

Spec URL:
https://git.remirepo.net/cgit/rpms/php/phpunit/php-phpunit-dbunit4.git/plain/php-phpunit-dbunit4.spec?id=aae4b0f3751703ac8c2e22d51bce401def2d3872
SRPM URL:
http://rpms.remirepo.net/SRPMS/php-phpunit-dbunit4-4.0.0-3.remi.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 1542807] Review Request: gap-pkg-toric - Computations with toric varieties in GAP

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542807

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Robert-André Mauchin  ---
Seems good, package approved.


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: "MIT/X11 (BSD like)", "Unknown or generated". 47 files have
 unknown license. Detailed output of licensecheck in
 /home/bob/packaging/review/gap-pkg-toric/review-gap-pkg-
 toric/licensecheck.txt
[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: 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]: 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

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[-]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
 files.
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded 

[Bug 1543060] Review Request: php-phpunit-dbunit4 - Extension for database interaction testing for PHPUnit 7

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1543060



--- Comment #1 from Remi Collet  ---
Spec URL:
https://git.remirepo.net/cgit/rpms/php/phpunit/php-phpunit-dbunit4.git/plain/php-phpunit-dbunit4.spec?id=7afef6d855daa597b5d1a73ae1d0f10b2bb756cc
SRPM URL:
http://rpms.remirepo.net/SRPMS/php-phpunit-dbunit4-4.0.0-2.remi.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 1543060] Review Request: php-phpunit-dbunit4 - Extension for database interaction testing for PHPUnit 7

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1543060

Remi Collet  changed:

   What|Removed |Added

 Depends On||1541346




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1541346
[Bug 1541346] Review Request: phpunit7 - The PHP Unit Testing framework
-- 
You are receiving this mail because:
You are on the CC 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 1541346] Review Request: phpunit7 - The PHP Unit Testing framework

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1541346

Remi Collet  changed:

   What|Removed |Added

 Blocks||1543060




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1543060
[Bug 1543060] Review Request: php-phpunit-dbunit4 - Extension for database
interaction testing for PHPUnit 7
-- 
You are receiving this mail because:
You are on the CC 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 1543060] New: Review Request: php-phpunit-dbunit4 - Extension for database interaction testing for PHPUnit 7

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1543060

Bug ID: 1543060
   Summary: Review Request: php-phpunit-dbunit4 - Extension for
database interaction testing for PHPUnit 7
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fed...@famillecollet.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://git.remirepo.net/cgit/rpms/php/phpunit/php-phpunit-dbunit4.git/plain/php-phpunit-dbunit4.spec?id=a26c1a18a4fae4dafb0d4268f8c0c960b8dbcf30
SRPM URL:
http://rpms.remirepo.net/SRPMS/php-phpunit-dbunit4-4.0.0-1.remi.src.rpm
Description: 
Extension for database interaction testing for PHPUnit 7.

Autoloader: /usr/share/php/PHPUnit7/DbUnit/autoload.php

Fedora Account System Username: remi

-- 
You are receiving this mail because:
You are on the CC 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 1542890] Opera browser as rpm in the official repo's

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542890

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||zebo...@gmail.com
 Resolution|--- |ERRATA
Last Closed||2018-02-07 11:13:53



--- Comment #1 from Robert-André Mauchin  ---
Opera is not free software, it is proprietary, as such it can't be included in
Fedora. You can ask third-party repo like RPMFusion if they wish to repackage
it, it is already in their "Wishlist": https://rpmfusion.org/Wishlist

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

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542896

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||zebo...@gmail.com
 Resolution|--- |ERRATA
Last Closed||2018-02-07 11:06:00



--- Comment #1 from Robert-André Mauchin  ---

 - kdemultimedia-devel was split in multiple package in Fedora

 - This is not the place for requesting an update, this place is to review
packages for inclusion. I suggest you file a bug report against Soundkonverter
to ask the maintainer to update his package. Click on File a bug here:
https://apps.fedoraproject.org/packages/soundkonverter/bugs

-- 
You are receiving this mail because:
You are on the CC 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 1542646] Review Request: patool - Portable command line archive file manager

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542646

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

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



--- Comment #6 from Zbigniew Jędrzejewski-Szmek  ---
+ package name is OK
+ license is acceptable for Fedora (GPLv3+)
+ license is specified correctly
+ builds and installs OK
+ Requires/BuildRequires/Provides/Recommends look good
+ fedora-review finds no important issues
+ tests are present and pass

Package is APPROVED.

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


[Bug 1481630] Review Request: virtualbox-guest-additions - VirtualBox Guest Additions

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1481630



--- Comment #63 from Sergio Monteiro Basto  ---
(In reply to Hans de Goede from comment #62)
> (In reply to Sergio Monteiro Basto from comment #60)
> > (In reply to Hans de Goede from comment #51)
> > > - Latest rpmfusion Release is 2, set our Release field to 3
> > 
> > Maybe could be better bump Epoch, for example VirtualBox had a rebuild today
> > ...
> 
> No Epoch's are evil. We can simply bump the release a bit extra if needed
> and rpmfusion should stop building the guest-additions sub-package for
> F28/rawhide soon now. I plan to import and build the Fedora guest-additions
> package for rawhide as soon as rawhide gets a 4.16-rc0 or rc1 with the
> vboxguest driver enabled.

OK , no problem for me . 

But , shared folder will work ? 

> With the vboxsf kernel driver which I'm working to get upstream, yes.

but this will be ready for 4.16-rc1 ? i.e. or we will still need vboxsf.ko from 
akmod-VirtualBox ? 

you can use /usr/lib/modules-load.d/VirtualBox-guest.conf only with vboxsf and
still Requires:   %{name}-kmod = %{version}

-- 
You are receiving this mail because:
You are on the CC 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 1542646] Review Request: patool - Portable command line archive file manager

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542646



--- Comment #5 from Robert-André Mauchin  ---
Spec URL: https://github.com/eclipseo/packaging/blob/7f99098/patool.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/patool/fedora-rawhide-x86_64/00711629-patool/patool-1.12-1.fc28.src.rpm

SPEC diff:
https://github.com/eclipseo/packaging/commit/7f99098c0c1efbfbc8ab5cba59927288038caa1b#diff-978fea9b8642e77c470ed419b5f34bfd


I patched the test of Zopfli: it was testing decompression which the zopfli
binary doesn't do. It now tests the compression.

The other errors were related to Star: the program assumed the arguments were
the same as "tar" but in fact they are slightly different for verbosity and
compress-type. I rewrote this portion and now tests are successful.

Thanks again for the review.

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


[Bug 1542671] Review Request: emojitwo-fonts - Color emoji font with a flat style

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542671



--- Comment #2 from Bastien Nocera  ---
(In reply to Peter Oliver from comment #1)
> I think this should probably obsolete package eosrei-emojione-fonts. 
> Maintainer of that package, Bastien Nocera, do you have an opinion on that?

Sounds fine by me. Please make sure that fontconfig has the necessary changes
to prefer emojitwo over emojione, but can still use both.

-- 
You are receiving this mail because:
You are on the CC 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 1540347] Review Request: hashcat - password recovery utility

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1540347



--- Comment #16 from Jaroslav Škarvada  ---
(In reply to Jaroslav Škarvada from comment #15)
> It seems to work with pocl-1.0-3, but it's way too slow :) Just for the
> record comparison of GPU accelerated MD5 benchmark in comment 5 (201.4 MH/s)
> vs CPU only POCL running in one thread (24567.2 kH/s), i.e. more than 8x
> speedup on GPU, the POCL run:

Of course comment 7 :)

-- 
You are receiving this mail because:
You are on the CC 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 1540347] Review Request: hashcat - password recovery utility

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1540347



--- Comment #15 from Jaroslav Škarvada  ---
It seems to work with pocl-1.0-3, but it's way too slow :) Just for the record
comparison of GPU accelerated MD5 benchmark in comment 5 (201.4 MH/s) vs CPU
only POCL running in one thread (24567.2 kH/s), i.e. more than 8x speedup on
GPU, the POCL run:

$ hashcat -b --force -d 1 --opencl-platforms=2
hashcat (v4.0.1) starting in benchmark mode...

Benchmarking uses hand-optimized kernel code by default.
You can use it in your cracking session by setting the -O option.
Note: Using optimized kernel code limits the maximum supported password length.
To disable the optimized kernel code in benchmark mode, use the -w option.

install_dir: /usr/bin, resolved_install_folder: /usr/bin
/usr/share/hashcat
clGetDeviceIDs(): CL_DEVICE_NOT_FOUND

OpenCL Platform #1: Intel, skipped or no OpenCL compatible devices found.

OpenCL Platform #2: The pocl project

* Device #1: pthread-Intel(R) Core(TM) i7-4600U CPU @ 2.10GHz, 2048/5807 MB
allocatable, 4MCU

OpenCL Platform #3: Mesa, skipped or no OpenCL compatible devices found.

Benchmark relevant options:
===
* --force
* --opencl-devices=1
* --opencl-platforms=2
* --optimized-kernel-enable

Hashmode: 900 - MD4

Speed.Dev.#1.: 46218.2 kH/s (90.57ms)

Hashmode: 0 - MD5

Speed.Dev.#1.: 24567.2 kH/s (87.91ms)
...

-- 
You are receiving this mail because:
You are on the CC 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 1542583] Review Request: rubygem-em-websocket - EventMachine based WebSocket server

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542583

Jaroslav Prokop  changed:

   What|Removed |Added

 Status|POST|CLOSED
   Fixed In Version||rubygem-em-websocket-0.5.1-
   ||1.fc28
 Resolution|--- |RAWHIDE
Last Closed||2018-02-07 10:03:32



-- 
You are receiving this mail because:
You are on the CC 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 1542485] Review Request: phpcpd - Copy/Paste Detector (CPD) for PHP code

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542485
Bug 1542485 depends on bug 1541346, which changed state.

Bug 1541346 Summary: Review Request: phpunit7 - The PHP Unit Testing framework
https://bugzilla.redhat.com/show_bug.cgi?id=1541346

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |RAWHIDE



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


[Bug 1541346] Review Request: phpunit7 - The PHP Unit Testing framework

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1541346

Remi Collet  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2018-02-07 10:01:14



-- 
You are receiving this mail because:
You are on the CC 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 1542884] Review Request: compat-libvpx4 - Compat package with libvpx libraries

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542884

Pete Walter  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||compat-libvpx4-1.6.1-1.fc28
 Resolution|--- |RAWHIDE
Last Closed||2018-02-07 09:50:31



--- Comment #3 from Pete Walter  ---
Thanks Chris! Built as compat-libvpx4-1.6.1-1.fc28

-- 
You are receiving this mail because:
You are on the CC 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 1525698] Review Request: jid - Json Incremental Digger

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1525698



--- Comment #3 from Robert-André Mauchin  ---
Tests run:

Spec URL: https://raw.githubusercontent.com/eclipseo/packaging/f8adf5e/jid.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/jid/fedora-rawhide-x86_64/00711557-jid/jid-0.7.2-1.fc28.src.rpm

SPEC diff:
https://github.com/eclipseo/packaging/commit/f8adf5e408a3115d062c87b364e76ec5c501a69c#diff-a8e72bb705766bbb907845afe3166dd8


Thanks for the review.

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


[Bug 1542583] Review Request: rubygem-em-websocket - EventMachine based WebSocket server

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542583



--- Comment #3 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/rubygem-em-websocket

-- 
You are receiving this mail because:
You are on the CC 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 1540553] Review Request: glusterd2- new management daemon for GlusterFS

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1540553



--- Comment #2 from Kaushal  ---
(In reply to Robert-André Mauchin from comment #1)
>  - You're missing %{?dist} in Release
> 
>  - Doesn't this service require its own user and group?
> 
>  - Don't use:
> 
> Requires(post): systemd
> Requires(preun): systemd
> Requires(postun): systemd
> 
>Use the provided macro instead:
> 
> %{?systemd_requires}
> BuildRequires: systemd
> 
>  - Also building with vendored is a no go, you will need to unbundle each
> dependency in vendor. Yes, it's gonna be a big task. Use gofed to do most of
> the work.

Thanks for the comments. I will be updating the spec with the suggested
changes.

About the bundled vendored packages, that will require some work as we'll need
to ensure that we run with the versions Fedora has packaged.

-- 
You are receiving this mail because:
You are on the CC 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 1542884] Review Request: compat-libvpx4 - Compat package with libvpx libraries

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542884



--- Comment #2 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/compat-libvpx4

-- 
You are receiving this mail because:
You are on the CC 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 1541346] Review Request: phpunit7 - The PHP Unit Testing framework

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1541346



--- Comment #6 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/phpunit7

-- 
You are receiving this mail because:
You are on the CC 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 1538259] Review Request: ocaml-lambda-term - Terminal manipulation library for OCaml

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1538259



--- Comment #3 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/ocaml-lambda-term

-- 
You are receiving this mail because:
You are on the CC 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 1519719] Review Request: php-phpunit-mock-objects5 - Mock Object library for PHPUnit

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1519719

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #11 from Fedora Update System  ---
php-phpunit-mock-objects5-5.0.6-3.fc27 has been pushed to the Fedora 27 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-2018-5bba96e2fb

-- 
You are receiving this mail because:
You are on the CC 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 1514274] Review Request: twitter-twemoji-fonts - Twitter Emoji for everyone

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1514274

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #34 from Fedora Update System  ---
twitter-twemoji-fonts-2.4.0-1.fc27 has been pushed to the Fedora 27 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-2018-86ff582bf9

-- 
You are receiving this mail because:
You are on the CC 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 1509856] Review Request: php-phpunit-php-token-stream2 - Wrapper around PHP tokenizer extension

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1509856

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System  ---
php-phpunit-PHP-TokenStream-1.4.12-1.fc27,
php-phpunit-php-token-stream2-2.0.2-1.fc27 has been pushed to the Fedora 27
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-2018-96c0ff19a5

-- 
You are receiving this mail because:
You are on the CC 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 1541587] Review Request: python-kiwisolver - A fast implementation of the Cassowary constraint solver

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1541587



--- Comment #11 from Fedora Update System  ---
python-kiwisolver-1.0.1-1.fc27 has been pushed to the Fedora 27 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-2018-92155e9316

-- 
You are receiving this mail because:
You are on the CC 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 1541554] Review Request: podman - Manage Pods, Containers and Container Images

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1541554

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #7 from Fedora Update System  ---
podman-0-0.3.git367213a.fc27 has been pushed to the Fedora 27 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-2018-665045691d

-- 
You are receiving this mail because:
You are on the CC 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 1535207] Review Request: ravada - Remote Virtual Desktops Manager

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1535207



--- Comment #13 from Fedora Update System  ---
ravada-0.2.13-0.1.20180117gitf70dfbf.fc27 has been pushed to the Fedora 27
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-2018-b84388d863

-- 
You are receiving this mail because:
You are on the CC 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 1509119] Review Request: php-gecko-packages-gecko-php-unit3 - Additional PHPUnit asserts and constraints

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1509119



--- Comment #8 from Fedora Update System  ---
php-gecko-packages-gecko-php-unit3-3.1.1-1.fc27 has been pushed to the Fedora
27 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-2018-65feba0684

-- 
You are receiving this mail because:
You are on the CC 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 1513669] Review Request: php-phpmailer6 - Full-featured email creation and transfer class for PHP

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1513669



--- Comment #11 from Fedora Update System  ---
php-phpmailer6-6.0.3-1.fc27 has been pushed to the Fedora 27 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-2018-723d9c742c

-- 
You are receiving this mail because:
You are on the CC 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 1539209] Review Request: ddgr - DuckDuckGo from the terminal

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1539209



--- Comment #3 from Robert-André Mauchin  ---
Spec diff:
https://github.com/eclipseo/packaging/commit/ff0547c1873a4e0ef8b3b42ffba0aa953b14#diff-1de3ee05e65db39bd52da3b02c701966

Thanks for the reviews!

-- 
You are receiving this mail because:
You are on the CC 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 1539209] Review Request: ddgr - DuckDuckGo from the terminal

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1539209



--- Comment #2 from Robert-André Mauchin  ---
Spec URL:
https://raw.githubusercontent.com/eclipseo/packaging/ff0547c/ddgr.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/ddgr/fedora-rawhide-x86_64/00711537-ddgr/ddgr-1.2-1.fc28.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 1541587] Review Request: python-kiwisolver - A fast implementation of the Cassowary constraint solver

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1541587

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #10 from Fedora Update System  ---
python-kiwisolver-1.0.1-1.fc26 has been pushed to the Fedora 26 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-2018-f21bbd868d

-- 
You are receiving this mail because:
You are on the CC 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 1513669] Review Request: php-phpmailer6 - Full-featured email creation and transfer class for PHP

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1513669

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #10 from Fedora Update System  ---
php-phpmailer6-6.0.3-1.fc26 has been pushed to the Fedora 26 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-2018-225a8b6ce9

-- 
You are receiving this mail because:
You are on the CC 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 1535207] Review Request: ravada - Remote Virtual Desktops Manager

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1535207

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #12 from Fedora Update System  ---
ravada-0.2.13-0.1.20180117gitf70dfbf.fc26 has been pushed to the Fedora 26
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-2018-4ff2ff1802

-- 
You are receiving this mail because:
You are on the CC 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 1509119] Review Request: php-gecko-packages-gecko-php-unit3 - Additional PHPUnit asserts and constraints

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1509119

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #7 from Fedora Update System  ---
php-gecko-packages-gecko-php-unit3-3.1.1-1.fc26 has been pushed to the Fedora
26 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-2018-04d1985734

-- 
You are receiving this mail because:
You are on the CC 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 1540347] Review Request: hashcat - password recovery utility

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1540347



--- Comment #14 from Tomáš Korbař  ---
I will upload new version later today. Sorry for the delay.

-- 
You are receiving this mail because:
You are on the CC 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 1542561] Review Request: conu - library which makes it easy to write tests for your containers

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542561

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |ERRATA
Last Closed||2018-02-07 08:18:11



--- Comment #4 from Fedora Update System  ---
conu-0.2.0-1.fc27 has been pushed to the Fedora 27 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 1526056] Review Request: rubygem-ascii_binder - An AsciiDoc-based system for authoring and publishing documentation

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1526056

Vít Ondruch  changed:

   What|Removed |Added

 Status|POST|CLOSED
   Fixed In Version||rubygem-ascii_binder-0.1.14
   ||-2.fc28
 Resolution|--- |RAWHIDE
Last Closed||2018-02-07 08:15: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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1526717] Review Request: python-giacpy - Python binding for Giac

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1526717



--- Comment #20 from Fedora Update System  ---
giac-1.4.9.45-2.fc26, python-giacpy-0.6.5-3.fc26, qcas-0.5.3-5.fc26 has been
pushed to the Fedora 26 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 1542931] Review Request: cri-tools - CLI and validation tools for Container Runtime Interface

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542931

Lokesh Mandvekar  changed:

   What|Removed |Added

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



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


[Bug 1542931] New: Review Request: cri-tools - CLI and validation tools for Container Runtime Interface

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542931

Bug ID: 1542931
   Summary: Review Request: cri-tools - CLI and validation tools
for Container Runtime Interface
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: l...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://pagure.io/cri-tools/raw/master/f/cri-tools.spec
SRPM URL:
https://pagure.io/cri-tools/blob/master/f/SRPMS/cri-tools-1.0.0-1.alpha.0.gitf1a58d6.fc27.src.rpm

Description: %{summary}

Fedora Account System Username: lsm5


$ rpmlint cri-tools.spec
SRPMS/cri-tools-1.0.0-1.alpha.0.gitf1a58d6.fc27.src.rpm RPMS/x86_64/cri-tools-*
cri-tools.x86_64: W: only-non-binary-in-usr-lib
cri-tools-debugsource.x86_64: W: no-documentation
4 packages and 1 specfiles checked; 0 errors, 2 warnings.


koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=24791797

-- 
You are receiving this mail because:
You are on the CC 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 1542583] Review Request: rubygem-em-websocket - EventMachine based WebSocket server

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542583

Vít Ondruch  changed:

   What|Removed |Added

 Blocks||1470699




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1470699
[Bug 1470699] New version of rubygem-jekyll is available
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1481630] Review Request: virtualbox-guest-additions - VirtualBox Guest Additions

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1481630



--- Comment #62 from Hans de Goede  ---
(In reply to Sergio Monteiro Basto from comment #60)
> (In reply to Hans de Goede from comment #51)
> > - Latest rpmfusion Release is 2, set our Release field to 3
> 
> Maybe could be better bump Epoch, for example VirtualBox had a rebuild today
> ...

No Epoch's are evil. We can simply bump the release a bit extra if needed and
rpmfusion should stop building the guest-additions sub-package for F28/rawhide
soon now. I plan to import and build the Fedora guest-additions package for
rawhide as soon as rawhide gets a 4.16-rc0 or rc1 with the vboxguest driver
enabled.

(In reply to Sergio Monteiro Basto from comment #61)
> (In reply to Hans de Goede from comment #51)
> 
> > - Use pkgconfig to get include/libs instead of hardcoding (rhbz#1534595)
> 
> we can't use pkgconf, for example: 
> 
> pkgconf --cflags libxml-2.0
> 
> -I/usr/include/libxml2 
> 
> when we have to use [1] and result with pkgconf is [2] , which doesn't work .
> 
> [1]
> SDK_VBOX_LIBXML2_INCS=/usr/include/libxml2
> 
> [2]
> SDK_VBOX_LIBXML2_INCS=-I/usr/include/libxml2

Right, so this still worked because the additions don't care about any of the
includes I was setting this way, so for the next release  I'm simply setting
them all to "".

-- 
You are receiving this mail because:
You are on the CC 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 1542583] Review Request: rubygem-em-websocket - EventMachine based WebSocket server

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1542583



--- Comment #2 from Jaroslav Prokop  ---
Thanks for review!

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


[Bug 1540347] Review Request: hashcat - password recovery utility

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1540347



--- Comment #13 from Jaroslav Škarvada  ---
Tomas could you fix the spec according to the above comments (e.g.
ExclusiveArch/ExcludeArch, Prefix, dropping devel package and packaging the
missing files)? Also please add "Requires: ocl-icd". Then please bump a
release, add changelog (e.g. with the "Implemented changes according to the
package review" or similar text), create SRPM and upload new versions. Also
please resubmit the links to new version in the new comment.

It seems POCL is still only about x86/ARM, so it seems the point 2 from comment
4 is resolved. We will probably have to create ppc64, s390x bugs as stated in
the [1] later, once the package will get into Fedora.

Now the lzma bundling remains. I will take a look later.

[1]
https://fedoraproject.org/wiki/Packaging:Guidelines#Architecture_Build_Failures

-- 
You are receiving this mail because:
You are on the CC 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 1540347] Review Request: hashcat - password recovery utility

2018-02-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1540347



--- Comment #12 from Jaroslav Škarvada  ---
Ah, it seems the f26 pocl is prehistoric, I will try the latest.

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


  1   2   >