[Bug 1686418] Review Request: R-ncdf4 - Interface to Unidata netCDF (Version 4 or Earlier) Format Data Files

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686418



--- Comment #7 from Fedora Update System  ---
R-ncdf4-1.16-2.fc29 has been submitted as an update to Fedora 29.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-21647cf3b9

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686418] Review Request: R-ncdf4 - Interface to Unidata netCDF (Version 4 or Earlier) Format Data Files

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686418



--- Comment #8 from Fedora Update System  ---
R-ncdf4-1.16-2.fc28 has been submitted as an update to Fedora 28.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-44c5b5301f

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686418] Review Request: R-ncdf4 - Interface to Unidata netCDF (Version 4 or Earlier) Format Data Files

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686418

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #6 from Fedora Update System  ---
R-ncdf4-1.16-2.fc30 has been submitted as an update to Fedora 30.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-19854b8cc0

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686410] Review Request: R-nycflights13 - Flights that Departed NYC in 2013

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686410

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #4 from Fedora Update System  ---
R-nycflights13-1.0.0-1.fc30 has been submitted as an update to Fedora 30.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-d5c23c3ba9

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686402] Review Request: R-jqr - Client for 'jq', a 'JSON' Processor

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686402

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #4 from Fedora Update System  ---
R-jqr-1.1.0-1.fc30 has been submitted as an update to Fedora 30.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-b962bcffd5

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686244] Review Request: R-maps - Draw Geographical Maps

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686244



--- Comment #6 from Fedora Update System  ---
R-maps-3.3.0-1.fc28 has been submitted as an update to Fedora 28.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-846575e9b1

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686244] Review Request: R-maps - Draw Geographical Maps

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686244



--- Comment #5 from Fedora Update System  ---
R-maps-3.3.0-1.fc29 has been submitted as an update to Fedora 29.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-f2be008349

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686244] Review Request: R-maps - Draw Geographical Maps

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686244

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #4 from Fedora Update System  ---
R-maps-3.3.0-1.fc30 has been submitted as an update to Fedora 30.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-0d9b7a492b

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1685068] Review Request: libxls - Read binary Excel files from C/C++

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1685068



--- Comment #5 from Fedora Update System  ---
libxls-1.5.0-1.fc29 has been submitted as an update to Fedora 29.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-129e28c44d

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1685068] Review Request: libxls - Read binary Excel files from C/C++

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1685068

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #4 from Fedora Update System  ---
libxls-1.5.0-1.fc30 has been submitted as an update to Fedora 30.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-9c37a50dbd

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1683848] Review Request: python-cftime - Time-handling functionality from netcdf4-python

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1683848

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|ASSIGNED|POST
  Flags|fedora-review?  |fedora-review+
   |needinfo?(zebo...@gmail.com |
   |)   |



--- Comment #4 from Robert-André Mauchin  ---
Sorry, I didn't see the noticex

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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1683848] Review Request: python-cftime - Time-handling functionality from netcdf4-python

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1683848

Orion Poplawski  changed:

   What|Removed |Added

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



--- Comment #3 from Orion Poplawski  ---
Ping?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1658153] Review Request: wdune - wdune (white_dune) is a graphical VRML97/X3D editor

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1658153



--- Comment #47 from J. Scheurich  ---
(In reply to Petr Menšík from comment #44)

> Script, lets say named wdune-sound-editor could be set by default into
> configure. It could contain something like:
> 
> #!/bin/sh
> 
> KNOWN_SOUND_EDITORS="audacity rezound wavesurfer gnusound sweep soundeditor"

It looks like rezound, wavesurfer, gnusound and soundeditor are not part of
fedora 30

# dnf install rezound wavesurfer gnusound sweep soundeditor
Last metadata expiration check: 0:00:32 ago on Sat 09 Mar 2019 01:25:39 AM CET.
No match for argument: rezound
No match for argument: wavesurfer
No match for argument: gnusound
No match for argument: soundeditor
Error: Unable to find a match
#

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1683321] Review Request: plasma-pass - Plasma applet to access passwords from the pass password manager

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1683321

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2019-03-08 23:51:15



--- Comment #5 from Fedora Update System  ---
plasma-pass-1.0.0-1.fc29 has been pushed to the Fedora 29 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1677989] Review Request: vcglib Visualization and Computer Graphics Library

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1677989



--- Comment #11 from J. Scheurich  ---
Spec URL: ftp://ftp.ourproject.org/pub/wdune/vcglib.spec
SRPM URL: ftp://ftp.ourproject.org/pub/wdune/vcglib-1.0.1-1.fc30.src.rpm
Description: The Visualization and Computer Graphics Library (VCG for short) 
 is a open source portable C++ templated library for manipulation,
 processing and displaying with OpenGL of triangle and tetrahedral
 meshes.
Fedora Account System Username: mufti11

Update:
Removed Eigen3 from library (incompatible with fedoras more recent Eigen3 lib)
Added  /usr/share/pkgconfig/vcglib.pc

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1684906] Review Request: golang-github-glycerine-unsnap-stream - Small golang library for decoding the snappy streaming format

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1684906



--- Comment #8 from Fedora Update System  ---
golang-github-glycerine-unsnap-stream-0-0.1.20190307gitf967730.fc29 has been
pushed to the Fedora 29 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-2019-cf607ef326

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1684936] Review Request: golang-github-anacrolix-envpprof - Allows run time configuration of Go's pprof features and default HTTP mux

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1684936



--- Comment #8 from Fedora Update System  ---
golang-github-anacrolix-envpprof-0-0.1.20190307git323002c.fc29 has been pushed
to the Fedora 29 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-2019-77b958a281

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1643742] Review Request: golang-etcd-bbolt - An embedded key/value database for Go

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1643742



--- Comment #8 from Fedora Update System  ---
golang-etcd-bbolt-1.3.2-2.fc29 has been pushed to the Fedora 29 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-2019-9495d9893b

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1572826] Review Request: python-Pallets-Sphinx-Themes - Sphinx themes for Pallets and related projects

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1572826

François Cami  changed:

   What|Removed |Added

 CC||f...@fcami.net,
   ||rel...@redhat.com
  Flags||needinfo?(rel...@redhat.com
   ||)



--- Comment #2 from François Cami  ---
Hi Rick,
It looks like this was never built. 
Can you build?
Also I've noticed 1.1.3 was out.
Cheers

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1687010] New: Review Request: golang-github-koofr-koofrclient - Go Koofr client

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1687010

Bug ID: 1687010
   Summary: Review Request: golang-github-koofr-koofrclient - Go
Koofr client
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: zebo...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/rclone1.45/fedora-29-x86_64/00865872-golang-github-koofr-koofrclient/golang-github-koofr-go-koofrclient.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/rclone1.45/fedora-29-x86_64/00865872-golang-github-koofr-koofrclient/golang-github-koofr-koofrclient-0-0.1.20190308git7f32759.fc29.src.rpm

Description:
Go Koofr client.

Fedora Account System Username: eclipseo

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1687008] New: Review Request: golang-github-koofr-httpclient - Go HTTP client

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1687008

Bug ID: 1687008
   Summary: Review Request: golang-github-koofr-httpclient - Go
HTTP client
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: zebo...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/rclone1.45/fedora-29-x86_64/00865868-golang-github-koofr-httpclient/golang-github-koofr-httpclient.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/rclone1.45/fedora-29-x86_64/00865868-golang-github-koofr-httpclient/golang-github-koofr-httpclient-0-0.1.20190308git0378617.fc29.src.rpm

Description:
Go HTTP client.

Fedora Account System Username: eclipseo

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1687007] New: Review Request: golang-github-hpcloud-tail - Go package for reading from continously updated files

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1687007

Bug ID: 1687007
   Summary: Review Request: golang-github-hpcloud-tail - Go
package for reading from continously updated files
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: zebo...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/rclone1.45/fedora-29-x86_64/00865864-golang-github-hpcloud-tail/golang-github-hpcloud-tail.specSRPM
URL:
https://copr-be.cloud.fedoraproject.org/results/eclipseo/rclone1.45/fedora-29-x86_64/00865864-golang-github-hpcloud-tail/golang-github-hpcloud-tail-1.0.0-1.20190308gita1dbeea.fc29.src.rpm

Description:
Go package for reading from continously updated files.

Fedora Account System Username: eclipseo

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686506] Review Request: wireguard - Fast, modern, secure VPN tunnel

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686506



--- Comment #8 from Jason A. Donenfeld  ---
It's unfortunate you had that misguided discussion without the input of
upstream. I disagree with the conclusion there of WireGuard being merely, "a
piece of software, which just happens to require a kernel module in order to
operate." We rather expect that many different types of software will be formed
from the small basic building block of a WireGuard tunnel.

The name of the tools is wireguard-tools. As soon as WireGuard is upstreamed,
the name of the repo will be wireguard-tools. The tarballs of that software
will be called wireguard-tools-1.0.tar.xz. Extremely common nomenclature for
tools that operate particular network interface types is -tools.

So I urge you to both stick with the naming conventions used by both numerous
upstream kernel networking projects and by the WireGuard project itself.

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1384917] Review Request: python-pykwalify - Python lib/cli for JSON/YAML schema validation

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1384917

Fedora Update System  changed:

   What|Removed |Added

 Resolution|NEXTRELEASE |ERRATA



--- Comment #11 from Fedora Update System  ---
python-pykwalify-1.7.0-1.el7 has been pushed to the Fedora EPEL 7 stable
repository. If problems still persist, please make note of it in this bug
report.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686506] Review Request: wireguard - Fast, modern, secure VPN tunnel

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686506



--- Comment #7 from Robert-André Mauchin  ---
(In reply to Jason A. Donenfeld from comment #6)
> Somebody asked me to chime in here about naming. The WireGuard project has
> long had an "official" set of packages maintained by jdoss on copr, and the
> usage there is fairly high. I think it'd be prudent to stick with the
> existing package names. You can see how it boils down by distro here:
> https://www.wireguard.com/install/
> 
> Packages that install the tools (wg, wg-quick, man pages, etc) should be
> called "wireguard-tools".
> 
> Packages that install a dkms module should be called "wireguard-dkms".
> 
> Packages that install a binary module should be called "wireguard-kmod".
> 
> Packages that install all of the above or pull in multiple as dependencies
> or are otherwise some sort of meta-package should be called "wireguard".

We already had this discussion over at RPMFusion and it had then be decided to
drop the "tools" suffix: https://bugzilla.rpmfusion.org/show_bug.cgi?id=5020#c2

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1684906] Review Request: golang-github-glycerine-unsnap-stream - Small golang library for decoding the snappy streaming format

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1684906

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #7 from Fedora Update System  ---
golang-github-glycerine-unsnap-stream-0-0.1.20190307gitf967730.fc30 has been
pushed to the Fedora 30 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-2019-790d41043a

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1684936] Review Request: golang-github-anacrolix-envpprof - Allows run time configuration of Go's pprof features and default HTTP mux

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1684936

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #7 from Fedora Update System  ---
golang-github-anacrolix-envpprof-0-0.1.20190307git323002c.fc30 has been pushed
to the Fedora 30 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-2019-3b426a79fc

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1643742] Review Request: golang-etcd-bbolt - An embedded key/value database for Go

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1643742

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #7 from Fedora Update System  ---
golang-etcd-bbolt-1.3.2-2.fc30 has been pushed to the Fedora 30 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-2019-8ae4c94e3b

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1672833] Review Request: python38 - Version 3.8 of the Python interpreter

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1672833

Miro Hrončok  changed:

   What|Removed |Added

 Blocks||1686977




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1686977
[Bug 1686977] Python 3.8
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686506] Review Request: wireguard - Fast, modern, secure VPN tunnel

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686506

Jason A. Donenfeld  changed:

   What|Removed |Added

 CC||ja...@zx2c4.com,
   ||j...@solidadmin.com
  Flags||needinfo?(j...@solidadmin.co
   ||m)



--- Comment #6 from Jason A. Donenfeld  ---
Somebody asked me to chime in here about naming. The WireGuard project has long
had an "official" set of packages maintained by jdoss on copr, and the usage
there is fairly high. I think it'd be prudent to stick with the existing
package names. You can see how it boils down by distro here:
https://www.wireguard.com/install/

Packages that install the tools (wg, wg-quick, man pages, etc) should be called
"wireguard-tools".

Packages that install a dkms module should be called "wireguard-dkms".

Packages that install a binary module should be called "wireguard-kmod".

Packages that install all of the above or pull in multiple as dependencies or
are otherwise some sort of meta-package should be called "wireguard".

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686024] Review Request: pforth - Portable ANS-like Forth

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686024

Lubomir Rintel  changed:

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2019-03-08 16:56:14



--- Comment #3 from Lubomir Rintel  ---
Imported and built.

Thank you!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686024] Review Request: pforth - Portable ANS-like Forth

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686024



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

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686841] Review Request: perl-GIS-Distance-Fast - C implementation of GIS::Distance formulas

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686841

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  ---
URL and Source addresses are Ok.
Source archive (SHA-256:
4b026ca45329f889a512f5e6f610219b293116aca752d18f981abcc7f4e62631) is original.
Ok.
Summary verified from lib/GIS/Distance/Fast.pm. Ok.
Description verified from lib/GIS/Distance/Fast.pm. Ok.
License verified from lib/GIS/Distance/Fast.pm and README.md. Ok.
XS code, BuildArch is Ok.


$ rpmlint perl-GIS-Distance-Fast.spec
review-perl-GIS-Distance-Fast/results/perl-GIS-Distance-Fast-0.10-1.fc31.src.rpm
review-perl-GIS-Distance-Fast/results/perl-GIS-Distance-Fast-0.10-1.fc31.x86_64.rpm
 
perl-GIS-Distance-Fast.src: W: spelling-error %description -l en_US
reimplements -> re implements, re-implements, implements
perl-GIS-Distance-Fast.x86_64: W: spelling-error %description -l en_US
reimplements -> re implements, re-implements, implements
2 packages and 1 specfiles checked; 0 errors, 2 warnings.


$ rpm -q -lv -p perl-GIS-Distance-Fast-0.10-1.fc31.x86_64.rpm 
drwxr-xr-x2 rootroot0 mars  8 17:15
/usr/lib/.build-id
drwxr-xr-x2 rootroot0 mars  8 17:15
/usr/lib/.build-id/4d
lrwxrwxrwx1 rootroot   70 mars  8 17:15
/usr/lib/.build-id/4d/d33a433614dce6fa66541a2499bdf9fb2a05a8 ->
../../../../usr/lib64/perl5/vendor_perl/auto/GIS/Distance/Fast/Fast.so
drwxr-xr-x2 rootroot0 mars  8 17:15
/usr/lib64/perl5/vendor_perl/GIS
drwxr-xr-x2 rootroot0 mars  8 17:15
/usr/lib64/perl5/vendor_perl/GIS/Distance
drwxr-xr-x2 rootroot0 mars  8 17:15
/usr/lib64/perl5/vendor_perl/GIS/Distance/Fast
-rw-r--r--1 rootroot 1327 mars  8 00:17
/usr/lib64/perl5/vendor_perl/GIS/Distance/Fast.pm
-rw-r--r--1 rootroot  729 mars  8 00:17
/usr/lib64/perl5/vendor_perl/GIS/Distance/Fast/Cosine.pm
-rw-r--r--1 rootroot  750 mars  8 00:17
/usr/lib64/perl5/vendor_perl/GIS/Distance/Fast/Haversine.pm
-rw-r--r--1 rootroot  902 mars  8 00:17
/usr/lib64/perl5/vendor_perl/GIS/Distance/Fast/Vincenty.pm
drwxr-xr-x2 rootroot0 mars  8 17:15
/usr/lib64/perl5/vendor_perl/auto/GIS
drwxr-xr-x2 rootroot0 mars  8 17:15
/usr/lib64/perl5/vendor_perl/auto/GIS/Distance
drwxr-xr-x2 rootroot0 mars  8 17:15
/usr/lib64/perl5/vendor_perl/auto/GIS/Distance/Fast
-rwxr-xr-x1 rootroot20600 mars  8 17:15
/usr/lib64/perl5/vendor_perl/auto/GIS/Distance/Fast/Fast.so
drwxr-xr-x2 rootroot0 mars  8 17:15
/usr/share/doc/perl-GIS-Distance-Fast
-rw-r--r--1 rootroot 1120 mars  8 00:17
/usr/share/doc/perl-GIS-Distance-Fast/Changes
-rw-r--r--1 rootroot 1261 mars  8 00:17
/usr/share/doc/perl-GIS-Distance-Fast/README.md
drwxr-xr-x2 rootroot0 mars  8 17:15
/usr/share/licenses/perl-GIS-Distance-Fast
-rw-r--r--1 rootroot18421 mars  8 00:17
/usr/share/licenses/perl-GIS-Distance-Fast/LICENSE
-rw-r--r--1 rootroot 1592 mars  8 17:15
/usr/share/man/man3/GIS::Distance::Fast.3pm.gz
-rw-r--r--1 rootroot 1284 mars  8 17:15
/usr/share/man/man3/GIS::Distance::Fast::Cosine.3pm.gz
-rw-r--r--1 rootroot 1287 mars  8 17:15
/usr/share/man/man3/GIS::Distance::Fast::Haversine.3pm.gz
-rw-r--r--1 rootroot 1367 mars  8 17:15
/usr/share/man/man3/GIS::Distance::Fast::Vincenty.3pm.gz
File permissions and layout are Ok.


$ rpm -q --requires -p perl-GIS-Distance-Fast-0.10-1.fc31.x86_64.rpm | sort -f
| uniq -c
  1 libm.so.6()(64bit)
  1 libm.so.6(GLIBC_2.2.5)(64bit)
  1 libperl.so.5.28()(64bit)
  1 libpthread.so.0()(64bit)
  1 libpthread.so.0(GLIBC_2.2.5)(64bit)
  1 perl(:MODULE_COMPAT_5.28.1)
  1 perl(:VERSION) >= 5.8.1
  1 perl(GIS::Distance::Constants) >= 0.11
  1 perl(GIS::Distance::Fast)
  1 perl(namespace::clean) >= 0.24
  1 perl(strictures) >= 2
  1 perl(XSLoader)
  1 rpmlib(CompressedFileNames) <= 3.0.4-1
  1 rpmlib(FileDigests) <= 4.6.0-1
  1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  1 rpmlib(PayloadIsXz) <= 5.2-1
  1 rtld(GNU_HASH)
Binary requires are Ok.


$ rpm -q --provides -p 

[Bug 1686023] Review Request: console-setup - Debian console setup

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686023

Lubomir Rintel  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NOTABUG
Last Closed||2019-03-08 16:16:03



--- Comment #5 from Lubomir Rintel  ---
How could I miss this. Thank you and sorry for the noise.

Created a PR for an existing package instead:
https://src.fedoraproject.org/rpms/console-setup/pull-request/1

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1659556] Review Request: cockpit-composer - Composer GUI for use with Cockpit

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1659556



--- Comment #21 from Miroslav Suchý  ---
> But I suppose that happens at "fedpkg request-repo" time?

Yes, or any time later he can go to 
  src.fedoraproject.org/rpms/cockpit-composer
click on the settings tab and add anyone as co-maintainer.

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1659556] Review Request: cockpit-composer - Composer GUI for use with Cockpit

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1659556



--- Comment #20 from Martin Pitt  ---
@Miroslav: Fixed the description in
https://github.com/weldr/welder-web/pull/588 , thanks for pointing out!

Anthony, Miroslav: We need the "cockpit" Fedora user to be able to push to this
dist-git, and preferably also me (martinpitt) as a co-maintainer. But I suppose
that happens at "fedpkg request-repo" time?

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686418] Review Request: R-ncdf4 - Interface to Unidata netCDF (Version 4 or Earlier) Format Data Files

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686418



--- Comment #5 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/R-ncdf4

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686244] Review Request: R-maps - Draw Geographical Maps

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686244



--- Comment #3 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/R-maps

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686402] Review Request: R-jqr - Client for 'jq', a 'JSON' Processor

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686402



--- Comment #3 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/R-jqr

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686410] Review Request: R-nycflights13 - Flights that Departed NYC in 2013

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686410



--- Comment #3 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/R-nycflights13

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1685922] Review Request: packit - A set of tools to integrate upstream open source projects into Fedora operating system

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1685922



--- Comment #11 from Frantisek Lachman  ---
> Upstream issue created: https://github.com/packit-service/packit/issues/132

Fixed on master: https://github.com/packit-service/packit/pull/134

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686788] Review Request: perl-GIS-Distance - Calculate geographic distances

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686788

Petr Pisar  changed:

   What|Removed |Added

 Blocks||1686841




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1686841
[Bug 1686841] Review Request: perl-GIS-Distance-Fast - C implementation of
GIS::Distance formulas
-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686841] Review Request: perl-GIS-Distance-Fast - C implementation of GIS::Distance formulas

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686841

Petr Pisar  changed:

   What|Removed |Added

 Depends On||1686788




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1686788
[Bug 1686788] Review Request: perl-GIS-Distance - Calculate geographic
distances
-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686841] New: Review Request: perl-GIS-Distance-Fast - C implementation of GIS::Distance formulas

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686841

Bug ID: 1686841
   Summary: Review Request: perl-GIS-Distance-Fast - C
implementation of GIS::Distance formulas
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ppi...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://ppisar.fedorapeople.org/perl-GIS-Distance-Fast/perl-GIS-Distance-Fast.spec
SRPM URL:
https://ppisar.fedorapeople.org/perl-GIS-Distance-Fast/perl-GIS-Distance-Fast-0.10-1.fc31.src.rpm
Description:
This Perl module reimplements some, but not all, of the formulas that
come with GIS::Distance in the C programming language. C code is generally
much faster than the Perl equivalent.

Fedora Account System Username: ppisar

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1685922] Review Request: packit - A set of tools to integrate upstream open source projects into Fedora operating system

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1685922



--- Comment #10 from Frantisek Lachman  ---
Spec URL:
https://github.com/packit-service/packit/raw/9e2b2a9d56673a8815aedf90729265c16a71e9c6/packit.spec
SRPM URL:
https://github.com/packit-service/packit/raw/9e2b2a9d56673a8815aedf90729265c16a71e9c6/packit-0.0.1-1.fc30.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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1685922] Review Request: packit - A set of tools to integrate upstream open source projects into Fedora operating system

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1685922



--- Comment #9 from Frantisek Lachman  ---
Thank you for the feedback.

> python3-packit.noarch: E: non-executable-script 
> /usr/lib/python3.7/site-packages/packit/cli/watch_sg_pr.py 644 
> /usr/bin/python3 
> python3-packit.noarch: E: non-executable-script 
> /usr/lib/python3.7/site-packages/packit/cli/watch_upstream_release.py 644 
> /usr/bin/python3 
> If the script has shebang, then it should be executable script by setting 
> executable bit (and either be in /usr/bin/ or /usr/libexec) or when it does 
> not have executable bit then > it should not have shebang.

Upstream issue created: https://github.com/packit-service/packit/issues/132

I've resolved other issues about summary/description and will send new links
soon.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1658153] Review Request: wdune - wdune (white_dune) is a graphical VRML97/X3D editor

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1658153



--- Comment #46 from J. Scheurich  ---

> Well it does requires it as dependency. Bundled eigenlib must not be used if
> there is already package for the same project. If vcglib had pkgconfig file
> installed, it could have use Requires: eigen3 inside.
> They you would have to only use pkg-config --cflags vcglib, and it would
> include also all cflags required from eigen3 automatically. 
> 
> vcglib-devel should have Requires: eigen3-devel

The example code shipped in the orginal vcglib is not compatible with the 
fedora eigen3-devel (misssing include file).

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1677989] Review Request: vcglib Visualization and Computer Graphics Library

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1677989



--- Comment #10 from Petr Menšík  ---
Found by coincidence, that meshlab project [1] already bundles this library.
That is a another reason to produce separate package that can be reused also by
it. Is seems to be high-quality project without separate compilation support.

Maintainers of meshlab, are you willing to help to prepare vcglib to reuse from
meshlab?

1. https://src.fedoraproject.org/rpms/meshlab

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686506] Review Request: wireguard - Fast, modern, secure VPN tunnel

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686506



--- Comment #5 from Robert-André Mauchin  ---
I'm not sure, it shows up as an error en Fedora-review.


Spec URL: https://eclipseo.fedorapeople.org/wireguard.spec
SRPM URL:
https://eclipseo.fedorapeople.org/wireguard-0.0.20190227-2.fc31.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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1685922] Review Request: packit - A set of tools to integrate upstream open source projects into Fedora operating system

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1685922



--- Comment #8 from Miroslav Suchý  ---
echo A set of tools to integrate upstream open source projects into Fedora
operating system |wc -c
87
The summary should be less than 80 characters:
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_summary_and_description

Summary:%{summary}
This is ... ehm lazy? :)
I guess something like:
  Python libraries for Packit
will work

packit.noarch: W: incoherent-version-in-changelog 0.1.0-1 ['0.0.1-1.fc29',
'0.0.1-1']
The Y and Z from X.Y.Z-R does not match.

python3-packit.noarch: E: non-executable-script
/usr/lib/python3.7/site-packages/packit/cli/watch_sg_pr.py 644 /usr/bin/python3 
python3-packit.noarch: E: non-executable-script
/usr/lib/python3.7/site-packages/packit/cli/watch_upstream_release.py 644
/usr/bin/python3 
If the script has shebang, then it should be executable script by setting
executable bit (and either be in /usr/bin/ or /usr/libexec) or when it does not
have executable bit then it should not have shebang.

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686788] Review Request: perl-GIS-Distance - Calculate geographic distances

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686788

Jitka Plesnikova  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



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

$ rpm -qp --requires perl-GIS-Distance-0.11-1.fc30.noarch.rpm | sort | uniq -c
  1 perl(Carp)
  1 perl(Class::Measure::Length)
  1 perl(Const::Fast) >= 0.014
  1 perl(Exporter)
  1 perl(GIS::Distance::Constants)
  1 perl(Math::Trig)
  1 perl(:MODULE_COMPAT_5.28.1)
  1 perl(namespace::clean) >= 0.24
  1 perl(strictures) >= 2
  1 perl(:VERSION) >= 5.8.1
  1 rpmlib(CompressedFileNames) <= 3.0.4-1
  1 rpmlib(FileDigests) <= 4.6.0-1
  1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  1 rpmlib(PayloadIsXz) <= 5.2-1
Binary requires are Ok.

$ rpm -qp --provides perl-GIS-Distance-0.11-1.fc30.noarch.rpm | sort | uniq -c
  1 perl(GIS::Distance) = 0.11
  1 perl-GIS-Distance = 0.11-1.fc30
  1 perl(GIS::Distance::Constants) = 0.11
  1 perl(GIS::Distance::Cosine) = 0.11
  1 perl(GIS::Distance::GreatCircle) = 0.11
  1 perl(GIS::Distance::Haversine) = 0.11
  1 perl(GIS::Distance::MathTrig) = 0.11
  1 perl(GIS::Distance::Polar) = 0.11
  1 perl(GIS::Distance::Vincenty) = 0.11
Binary provides are Ok.

$ rpmlint ./perl-GIS-Distance*
2 packages and 1 specfiles checked; 0 errors, 0 warnings.
Rpmlint is ok

The package looks good.
Approved

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1658153] Review Request: wdune - wdune (white_dune) is a graphical VRML97/X3D editor

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1658153



--- Comment #45 from Petr Menšík  ---
Usually subpackage sections of %package docs and %description docs follows
after main %description, before %prep section. %files docs should follow after
%files. Best practice for defining %global macros is to place them before all
tags like Summary. 

%global extraver pl1414
%global upstream_version %{version}%{?extraver}
...
Version: 0.99
Release: 2%{?extraver}%{?dist}
...
Source: ftp://ftp.ourproject.org/pub/wdune/wdune-%{upstream_version}.tar.bz2

I would recommend also using macro in --with-about="wdune-%{upstream_version}" 
in %configure

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686788] Review Request: perl-GIS-Distance - Calculate geographic distances

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686788

Jitka Plesnikova  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jples...@redhat.com
  Flags||fedora-review?



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686753] Review Request: perl-Class-Measure - Create, compare and convert units of measurement

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686753

Jitka Plesnikova  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



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

$ rpm -qp --requires perl-Class-Measure-0.06-1.fc30.noarch.rpm | sort | uniq -c
  1 perl(base)
  1 perl(Carp)
  1 perl(Class::Measure)
  1 perl(:MODULE_COMPAT_5.28.1)
  1 perl(overload)
  1 perl(strict)
  1 perl(Sub::Exporter) >= 0.982 
  1 perl(warnings)
  1 rpmlib(CompressedFileNames) <= 3.0.4-1
  1 rpmlib(FileDigests) <= 4.6.0-1
  1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
  1 rpmlib(PayloadIsXz) <= 5.2-1
Binary requires are Ok.

$ rpm -qp --provides perl-Class-Measure-0.06-1.fc30.noarch.rpm | sort | uniq -c
  1 perl(Class::Measure) = 0.06
  1 perl-Class-Measure = 0.06-1.fc30
  1 perl(Class::Measure::Length) = 0.06
Binary provides are Ok.

$ rpmlint ./perl-Class-Measure*
2 packages and 1 specfiles checked; 0 errors, 0 warnings.
Rpmlint is ok

The package looks good.
Approved

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686753] Review Request: perl-Class-Measure - Create, compare and convert units of measurement

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686753

Jitka Plesnikova  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jples...@redhat.com
  Flags||fedora-review?



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1658153] Review Request: wdune - wdune (white_dune) is a graphical VRML97/X3D editor

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1658153



--- Comment #44 from Petr Menšík  ---
I think I found possible improvement for default programs support.
I think runtime detection is required, compile-time detection of supported
programs is far from user-friendly.

Script, lets say named wdune-sound-editor could be set by default into
configure. It could contain something like:

#!/bin/sh

KNOWN_SOUND_EDITORS="audacity rezound wavesurfer gnusound sweep soundeditor"
FOUND=
for APP in ${KNOWN_SOUND_EDITORS}; do
FOUND="`command -v $APP`"
if test -n "$FOUND"
then
$FOUND $@
exit $?
fi
done

if test -z "$FOUND"
then
echo "ERROR: No supported editor was found!" 2>&1
exit 1
fi


It would run first program in list, similar to configure script. But would not
require reconfiguration when editors available are changed. EDITORS could be
initialized by configure variables if required.

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1659556] Review Request: cockpit-composer - Composer GUI for use with Cockpit

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1659556



--- Comment #19 from Miroslav Suchý  ---
the %description can be more descriptive. E.g., the first paragraph from README
is just fine. And it should end with a dot (you are missing it there).
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_summary_and_description

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1658153] Review Request: wdune - wdune (white_dune) is a graphical VRML97/X3D editor

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1658153



--- Comment #43 from Petr Menšík  ---
> white_dune do not use Eigen3, it only uses vcglib without any eigen
> requirement. Maybe it is not a good idea to split the vcglib include
> files in new fedora package.

Well it does requires it as dependency. Bundled eigenlib must not be used if
there is already package for the same project. If vcglib had pkgconfig file
installed, it could have use Requires: eigen3 inside.
They you would have to only use pkg-config --cflags vcglib, and it would
include also all cflags required from eigen3 automatically. 

vcglib-devel should have Requires: eigen3-devel

Because no pkgconfig file is shipped by vcglib-devel, wdune has to mention it
explicitly to compile. Fix it by shipping vcglib.pc in vcglib-devel and it can
be used in more simple way. Until that, just include eigen3 in build flags.

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1659556] Review Request: cockpit-composer - Composer GUI for use with Cockpit

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1659556

Miroslav Suchý  changed:

   What|Removed |Added

 CC||msu...@redhat.com
   Assignee|zebo...@gmail.com   |msu...@redhat.com



--- Comment #18 from Miroslav Suchý  ---
I can sponsor Anthony. Anthony, I will contact you via email with more details.

Robert - thank you for the review, it made things easier 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1677989] Review Request: vcglib Visualization and Computer Graphics Library

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1677989



--- Comment #9 from Petr Menšík  ---
comment #1 states main package vcglib should be empty. It does not mean you
have to remove examples from tarball. I think on contrary, it should not be
deleted from source archive. We expect you would release one source archive for
all distributions, always the same. It can be chosen in spec file, which part
would be packaged and which should not be packaged. If you have to delete some
parts, delete them in build process only. Deleted from sources should be just
parts with incompatible license, nothing else.

I am not sure about that. Main package usually contains only compiled library,
check [1] for information. However, this library does not compile any shared
library that can be reused. That might be fixed later, I think it does not work
as library now but should. Instead, examples could have subpackage examples,
with more runtime dependencies on them, especially is they are visually
atractive and not simple to compile yourself. Ok, proposal is this. Includes
would not contain eigenlib, it has its own project. 

Because vcglib as its own project on github [2], it definitely requires its own
package and must NOT be part of wdune package.

Ok, examples are not ready to use system-wide eigen3. But here we can reuse
pkgconfig as well [3]. Just append to all used .pro files:

cat > fedora.unix << QMAKE
unix {
CONFIG += link_pkgconfig
PKGCONFIG += eigen3
}
QMAKE
find -name '*.pro' | while read P; do cat fedora.unix >> "$P"; done

Then tools can be built. It seems they are not only samples, but also useful
tools. They should be packed definitely. utils subpackage could be used for
them, with all dependencies required to install them. I think it is strange
there is no script to build all those tools in single command, it should be
proposed to the project. It can be solved in spec file until that happens.
Generated makefiles have no working install target unfortunately.

I think such modifications should be recommended to the project in general, not
just for Fedora.

1.
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_libraries_and_applications
2. https://github.com/cnr-isti-vclab/vcglib
3. http://qt.shoutwiki.com/wiki/Using_pkg-config_with_qmake

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686788] Review Request: perl-GIS-Distance - Calculate geographic distances

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686788

Petr Pisar  changed:

   What|Removed |Added

 Blocks||1686667




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1686667
[Bug 1686667] perl-Geo-Distance-0.21 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686753] Review Request: perl-Class-Measure - Create, compare and convert units of measurement

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686753

Petr Pisar  changed:

   What|Removed |Added

 Blocks||1686788




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1686788
[Bug 1686788] Review Request: perl-GIS-Distance - Calculate geographic
distances
-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686788] Review Request: perl-GIS-Distance - Calculate geographic distances

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686788

Petr Pisar  changed:

   What|Removed |Added

 Depends On||1686753




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1686753
[Bug 1686753] Review Request: perl-Class-Measure - Create, compare and convert
units of measurement
-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686788] New: Review Request: perl-GIS-Distance - Calculate geographic distances

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686788

Bug ID: 1686788
   Summary: Review Request: perl-GIS-Distance - Calculate
geographic distances
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ppi...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://ppisar.fedorapeople.org/perl-GIS-Distance/perl-GIS-Distance.spec
SRPM URL:
https://ppisar.fedorapeople.org/perl-GIS-Distance/perl-GIS-Distance-0.11-1.fc31.src.rpm
Description:
This Perl module calculates distances between geographic points on, at the
moment, planet Earth. Various "FORMULAS" are available that provide different
levels of accuracy versus speed.

Fedora Account System Username: ppisar

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1671790] Review Request: python-vistir - python library including utility functions

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1671790



--- Comment #18 from Miro Hrončok  ---
What changed in the specfile?

If upstream merges a patch (or even when it is proposed) it is good to add a
link to the comment. Liek this:

# https://github.com/sarugaku/vistir/issues/59
Patch0: python_to_sys.executable.patch

Also, have you tested the upstream's patch instead of yours? I remember there
were a lot of little changes.

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1671790] Review Request: python-vistir - python library including utility functions

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1671790



--- Comment #17 from Patrik Kopkan  ---
Spec URl:
https://copr-be.cloud.fedoraproject.org/results/pkopkan/python-vistir/fedora-29-x86_64/00865704-python-vistir/python-vistir.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/pkopkan/python-vistir/fedora-29-x86_64/00865704-python-vistir/python3-vistir-0.3.0-1.fc29.noarch.rpm

upstream agreed on patch.

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1671787] Review Request: python-cursor - python package for hiding terminal cursor

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1671787



--- Comment #17 from Miro Hrončok  ---
Don't be :) It's perfectly OK to not now everything from the beginning (or
ever).

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1671787] Review Request: python-cursor - python package for hiding terminal cursor

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1671787



--- Comment #16 from Jan Macku  ---
I'm sorry.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686753] New: Review Request: perl-Class-Measure - Create, compare and convert units of measurement

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686753

Bug ID: 1686753
   Summary: Review Request: perl-Class-Measure - Create, compare
and convert units of measurement
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ppi...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://ppisar.fedorapeople.org/perl-Class-Measure/perl-Class-Measure.spec
SRPM URL:
https://ppisar.fedorapeople.org/perl-Class-Measure/perl-Class-Measure-0.06-1.fc31.src.rpm
Description:
This Perl module allows you to create, compare and convert units of
measurement.

Fedora Account System Username: ppisar

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1686418] Review Request: R-ncdf4 - Interface to Unidata netCDF (Version 4 or Earlier) Format Data Files

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686418

Robert-André Mauchin  changed:

   What|Removed |Added

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



--- Comment #4 from Robert-André Mauchin  ---
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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1671787] Review Request: python-cursor - python package for hiding terminal cursor

2019-03-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1671787

Miro Hrončok  changed:

   What|Removed |Added

  Flags|fedora-review+  |



--- Comment #15 from Miro Hrončok  ---
Until sponsored, please don't set those flags.

-- 
You are receiving this mail because:
You are on the CC 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
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org