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

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



--- Comment #3 from Elliott Sales de Andrade  ---
Fixed.

Spec URL: https://qulogic.fedorapeople.org/R-ncdf4.spec
SRPM URL: https://qulogic.fedorapeople.org/R-ncdf4-1.16-2.fc29.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 1684177] Review Request: perl-Alien-FFI - Make available libffi

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

Petr Pisar  changed:

   What|Removed |Added

 Status|POST|CLOSED
   Fixed In Version||perl-Alien-FFI-0.22-1.fc31
 Resolution|--- |RAWHIDE
Last Closed||2019-03-08 07:21:37



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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 1670656] Review Request: grafana - an open source, feature rich metrics dashboard and graph editor

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



--- Comment #30 from Elliott Sales de Andrade  ---
Builds are there for all versions:
https://koji.fedoraproject.org/koji/packageinfo?packageID=28148

If you're on copr, you probably need to wait for the Rawhide compose that
finally finished to be synced out to mirrors.

BTW, is there a way for the webpack to be unminified in the source tarball (and
then combined in the rpm build)?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1670656] Review Request: grafana - an open source, feature rich metrics dashboard and graph editor

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

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Depends On||1377227




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1377227
[Bug 1377227] Review Request: golang-github-go-stack-stack - Implements
utilities to capture, manipulate, and format call stacks
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1377227] Review Request: golang-github-go-stack-stack - Implements utilities to capture, manipulate, and format call stacks

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

Elliott Sales de Andrade  changed:

   What|Removed |Added

 CC||mgood...@redhat.com
 Blocks||1670656



--- Comment #8 from Elliott Sales de Andrade  ---
Mark is interested in this for Grafana.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1670656
[Bug 1670656] Review Request: grafana - an open source, feature rich metrics
dashboard and graph editor
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1670656] Review Request: grafana - an open source, feature rich metrics dashboard and graph editor

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

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Depends On||1376387




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1376387
[Bug 1376387] Review Request: golang-github-Unknwon-com - Commonly used
functions for the Go programming language
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1376387] Review Request: golang-github-Unknwon-com - Commonly used functions for the Go programming language

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

Elliott Sales de Andrade  changed:

   What|Removed |Added

 CC||mgood...@redhat.com
 Blocks||1670656



--- Comment #6 from Elliott Sales de Andrade  ---
Mark is interested in this for Grafana.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1670656
[Bug 1670656] Review Request: grafana - an open source, feature rich metrics
dashboard and graph editor
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1670441] Review Request: setconf - Tool for changing values of parameters in text files

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



--- Comment #35 from Jan Macku  ---
Miro, I decided to package alternatives (C) to Fedora. Once it will be ready, I
will create bug and add put you to the CC.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1671787

Jan Macku  changed:

   What|Removed |Added

  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 1670656] Review Request: grafana - an open source, feature rich metrics dashboard and graph editor

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



--- Comment #29 from Mark Goodwin  ---
Have updated to grafana-6.0.1-1 - this is now the current upstream stable
version.
For the Fedora spec build, have switched to a new branch
'grafana-native-spec-v6'
(retaining the 'native-rpm-spec' branch for the 5.4.3 version).

Have incorporated the review comments from Comment #25 Comment #26 and Comment
#27.

Spec URL:
https://raw.githubusercontent.com/goodwinos/grafana/grafana-native-spec-v6/packaging/rpm/spec/grafana.spec

SRPM URL:
https://mgoodwin.fedorapeople.org/grafana/grafana-6.0.1-1.fc28.src.rpm

COPR URL: https://copr.fedorainfracloud.org/coprs/mgoodwin/grafana/

Additional notes:

- golang vendor package unbundling efforts are being tracked in a spreadsheet
in google docs. 
  Reviewers have been invited to edit - let me know if you want to be added.

- the fc28 build fails because grafana-v6 requires golang >= 1.11
  Strangely, the el7 copr build is fine with this.

- the f29 build is fine.

- the rawhide build fails because golang-github-hashicorp-version seems to be
missing from the f30 chroot
  (Elliott, maybe a build is needed - rawhide builds have been pretty
unreliable lately)

Thanks

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686506

Neal Gompa  changed:

   What|Removed |Added

 CC||ngomp...@gmail.com



--- Comment #4 from Neal Gompa  ---
> # Use standard perms for /etc/wireguard
> sed -i 's|install -v -m 0700|install -v -m 0755|' src/tools/Makefile

Are you sure this is a good idea? I'm pretty sure that this contains the
interface configuration, and may be considered sensitive information.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684612] Review Request: f30-backgrounds - Fedora 30 default desktop background

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



--- Comment #17 from satelli...@gmail.com ---
Created attachment 1541980
  --> https://bugzilla.redhat.com/attachment.cgi?id=1541980=edit
anaconda with new installer in  branched
Fedora-MATE_Compiz-Live-x86_64-30-20190307.n.1.iso

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684612] Review Request: f30-backgrounds - Fedora 30 default desktop background

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

satelli...@gmail.com changed:

   What|Removed |Added

 CC||satelli...@gmail.com



--- Comment #16 from satelli...@gmail.com ---
fixed in Fedora-MATE_Compiz-Live-x86_64-30-20190307.n.1.iso  branched with
30.25.3-2.fc30

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686506



--- Comment #3 from Robert-André Mauchin  ---
(In reply to Lubomir Rintel from comment #2)
> 1.) Why do you call the package lowercase "wireguard"? Upstream seems to
> call the thing "WireGuard"

>In Fedora packaging, the maintainer uses their best judgement when considering 
>how to name the package. While case sensitivity is not a mandatory 
>requirement, case SHOULD only be used where necessary. Keep in mind to respect 
>the wishes of the upstream maintainers. If they refer to their application as 
>"ORBit", you SHOULD use "ORBit" as the package name, and not "orbit". However, 
>if they do not express any preference of case, you SHOULD default to lowercase 
>naming.
https://docs.fedoraproject.org/en-US/packaging-guidelines/Naming/#_case_sensitivity

In any all the other distros use lowercase.


> 
> 2.) What's the use of this in Fedora?
> 
> > Provides:   %{name}-kmod-common = %{version}
> > %if 0%{?fedora} || 0%{?rhel} > 7
> > Suggests:   %{name}-kmod >= %{version}
> > %endif
> 
> The Fedora kernel-core package seems to provide "kmod(.ko)" for the
> modules it ships. I guess that "kmod(wireguard.ko)" is what we should depend
> on? Though I suppose only a weak dependency would do until the kernel
> actually ships the module.

For now it' for compatibility with the module from RPMFusion. Once the module
is in mainline, it would be removed. Apparently this won't happen until at
least 5.2.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1684906



--- Comment #6 from Fedora Update System  ---
golang-github-glycerine-unsnap-stream-0-0.1.20190307gitf967730.fc29 has been
submitted as an update to Fedora 29.
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 1684906] Review Request: golang-github-glycerine-unsnap-stream - Small golang library for decoding the snappy streaming format

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



--- Comment #5 from Fedora Update System  ---
golang-github-glycerine-unsnap-stream-0-0.1.20190307gitf967730.fc30 has been
submitted as an update to Fedora 30.
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 1685930] Review Request: R-doParallel - Foreach Parallel Adaptor for the 'parallel' Package

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



--- Comment #10 from Fedora Update System  ---
R-doParallel-1.0.14-1.fc28 has been pushed to the Fedora 28 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-b5eb07c936

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1680307] Review Request: php-typo3-phar-stream-wrapper2 - Interceptors for PHP's native phar:// stream handling (v2)

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



--- Comment #11 from Fedora Update System  ---
drupal8-8.6.10-1.fc29, php-typo3-phar-stream-wrapper2-2.0.1-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 1643742] Review Request: golang-etcd-bbolt - An embedded key/value database for Go

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



--- Comment #6 from Fedora Update System  ---
golang-etcd-bbolt-1.3.2-2.fc29 has been submitted as an update to Fedora 29.
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 1666493] Review Request: OpenLiberty - Server runtime for Java Developers

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

Felix Schwarz  changed:

   What|Removed |Added

 CC||fschw...@fedoraproject.org



--- Comment #7 from Felix Schwarz  ---
(In reply to Michael.Zhang from comment #5)
> From your previous comment of:
>  - "It is forbidden to ship jar files directly:"
> 
> I'm assuming you wish for us to incorporate the jar building process into
> our travis build process. So there are 2 ways we can go about this:
>  
>(1) call rpmbuild - which would then have to call gradle inside to build
> straight from the source and package as rpm all in 1 rpm command
>(2) call gradle to build the necessary jars then call rpmbuild to package
> the contents into a rpm
> 
> We would much rather prefer the 2nd option as it is easier and more
> convenient on our end.

Not sure if I understood you correctly but Fedora's build process can not use
pre-built binaries (even if you build them transparently on Travis). The full
build process (including *all* dependencies) must be in Fedora. All sources
must be included in your source rpm (or rpm dependencies) you must not download
anything from the internet during your builds.

(And you judging from the size of the OpenLiberty download packaging it for
Fedora will be quite a bit of work.)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1643742

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #5 from Fedora Update System  ---
golang-etcd-bbolt-1.3.2-2.fc30 has been submitted as an update to Fedora 30.
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 1648584] Review Request: golang-github-dgryski-bitstream - Read and write bits from io.Reader and io.Writer streams

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



--- Comment #11 from Fedora Update System  ---
golang-github-dgryski-bitstream-0-0.1.20190307git3522498.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-a68e757b0b

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684929] Review Request: golang-github-huandu-xstrings - A collection of useful string functions in Go

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



--- Comment #8 from Fedora Update System  ---
golang-github-huandu-xstrings-1.2.0-1.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-3c5a235311

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684933] Review Request: golang-github-bradfitz-iter - Range over integers [0,n).

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



--- Comment #9 from Fedora Update System  ---
golang-github-bradfitz-iter-0-0.1.20190307git33e6a98.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-d42566b72f

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1644010] Review Request: golang-github-pierrec-lz4 - LZ4 compression and decompression in pure Go

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



--- Comment #11 from Fedora Update System  ---
golang-github-pierrec-lz4-2.1.0-1.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-d0b7fda363

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684862] Review Request: python-dask - Parallel PyData with Task Scheduling

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



--- Comment #9 from Fedora Update System  ---
python-dask-1.1.3-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-faf89234b9

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1648593] Review Request: golang-github-mattn-tty - Simple tty utility

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



--- Comment #6 from Fedora Update System  ---
golang-github-mattn-tty-0-0.1.20190306gite4f8711.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-a18ed89962

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684927] Review Request: golang-github-ryszard-goskiplist - A skip list implementation in Go

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



--- Comment #6 from Fedora Update System  ---
golang-github-ryszard-goskiplist-0-0.1.20190306git2dfbae5.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-7309946bbc

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1685930] Review Request: R-doParallel - Foreach Parallel Adaptor for the 'parallel' Package

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



--- Comment #9 from Fedora Update System  ---
R-doParallel-1.0.14-1.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-f976d0f35a

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684909] Review Request: golang-github-ttacon-chalk - Intuitive package for prettifying terminal/console output

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



--- Comment #7 from Fedora Update System  ---
golang-github-ttacon-chalk-0.1-0.1.20190306git22c06c8.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-9aeb951045

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684919] Review Request: golang-github-tinylib-msgp - A Go code generator for MessagePack

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



--- Comment #9 from Fedora Update System  ---
golang-github-tinylib-msgp-1.1.0-0.1.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-cef817bce7

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684908] Review Request: golang-github-philhofer-fwd - Buffered Reader/Writer

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



--- Comment #10 from Fedora Update System  ---
golang-github-philhofer-fwd-1.0.0-1.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-79fde1c534

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1685730] Review Request: golang-github-nrdcg-auroradns - Aurora DNS API client written in Go

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



--- Comment #8 from Fedora Update System  ---
golang-github-nrdcg-auroradns-1.0.0-1.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-11e5eba71e

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1643906] Review Request: golang-github-rican7-retry - A simple, stateless, functional mechanism to perform actions repetitively

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



--- Comment #8 from Fedora Update System  ---
golang-github-rican7-retry-0.1.0-1.20190306git272ad12.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-3a203be391

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684924] Review Request: golang-github-mschoch-smat - State Machine Assisted Testing

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



--- Comment #7 from Fedora Update System  ---
golang-github-mschoch-smat-0-0.1.20190306git90eadee.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-778a31c4a9

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1680307] Review Request: php-typo3-phar-stream-wrapper2 - Interceptors for PHP's native phar:// stream handling (v2)

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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2019-03-07 20:05:58



--- Comment #10 from Fedora Update System  ---
drupal8-8.6.10-1.fc28, php-typo3-phar-stream-wrapper2-2.0.1-1.fc28 has been
pushed to the Fedora 28 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 1686210] Review Request: perl-JSON-Validator - Validate data against a JSON schema

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

Emmanuel Seyman  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2019-03-07 19:59:49



--- Comment #3 from Emmanuel Seyman  ---
JFTR, I took into account all your comments before building for rawhide and
publishing updates for f30 and f29.

Thank you for the review, Jikta.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1684936



--- Comment #6 from Fedora Update System  ---
golang-github-anacrolix-envpprof-0-0.1.20190307git323002c.fc29 has been
submitted as an update to Fedora 29.
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 1686210] Review Request: perl-JSON-Validator - Validate data against a JSON schema

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



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1684936

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #5 from Fedora Update System  ---
golang-github-anacrolix-envpprof-0-0.1.20190307git323002c.fc30 has been
submitted as an update to Fedora 30.
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 1648584] Review Request: golang-github-dgryski-bitstream - Read and write bits from io.Reader and io.Writer streams

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #10 from Fedora Update System  ---
golang-github-dgryski-bitstream-0-0.1.20190307git3522498.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-a0fa315ddb

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684933] Review Request: golang-github-bradfitz-iter - Range over integers [0,n).

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System  ---
golang-github-bradfitz-iter-0-0.1.20190307git33e6a98.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-b3cb28a754

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684929] Review Request: golang-github-huandu-xstrings - A collection of useful string functions in Go

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #7 from Fedora Update System  ---
golang-github-huandu-xstrings-1.2.0-1.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-dd105f9391

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1644010] Review Request: golang-github-pierrec-lz4 - LZ4 compression and decompression in pure Go

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #10 from Fedora Update System  ---
golang-github-pierrec-lz4-2.1.0-1.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-95f122d39e

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1685930] Review Request: R-doParallel - Foreach Parallel Adaptor for the 'parallel' Package

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System  ---
R-doParallel-1.0.14-1.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-a28a339b26

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1685978] Review Request: R-ggplot2movies - Movies Data

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #5 from Fedora Update System  ---
R-ggplot2movies-0.0.1-1.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-fc4dca33a7

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684862] Review Request: python-dask - Parallel PyData with Task Scheduling

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System  ---
python-dask-1.1.3-3.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-66a6320a12

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1686032] Review Request: R-svglite - An 'SVG' Graphics Device

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #5 from Fedora Update System  ---
R-svglite-1.2.1-1.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-4c6daa9e15

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1685996] Review Request: R-dplyr - A Grammar of Data Manipulation

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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #5 from Fedora Update System  ---
R-dplyr-0.8.0.1-1.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-cb16f3b2c5

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1648584] Review Request: golang-github-dgryski-bitstream - Read and write bits from io.Reader and io.Writer streams

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



--- Comment #9 from Fedora Update System  ---
golang-github-dgryski-bitstream-0-0.1.20190307git3522498.fc29 has been
submitted as an update to Fedora 29.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-a68e757b0b

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1677989



--- Comment #8 from Petr Menšík  ---
It would be very good if library had its own configure, which could prepare
also makefiles for all examples. Examples could be installed into some place.
But if they can verify library actually works, they should be definitely
buildable from library sources itself. Jerry mentioned them already in comment
#1. Examples subpackage be installed in separate subpackage.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1648584] Review Request: golang-github-dgryski-bitstream - Read and write bits from io.Reader and io.Writer streams

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

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #8 from Fedora Update System  ---
golang-github-dgryski-bitstream-0-0.1.20190307git3522498.fc30 has been
submitted as an update to Fedora 30.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-a0fa315ddb

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1677989

Petr Menšík  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|pemen...@redhat.com



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1658153

Petr Menšík  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|pemen...@redhat.com



--- Comment #42 from Petr Menšík  ---
Created attachment 1541915
  --> https://bugzilla.redhat.com/attachment.cgi?id=1541915=edit
configure.in patch

adds support for pkg-config flags of eigen3 and freetype2. Also ads support for
--with-vcglib=path

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1658153

Petr Menšík  changed:

   What|Removed |Added

 Depends On||1677989



--- Comment #41 from Petr Menšík  ---
I prepared some patch that might help configuring it better. I tried to mimick
original detections, but allow better detection of system-wide flags. Used
pkg-config macro for eigen3 and freetype2.

./configure --with-vcglib=$VCGLIB
make -j4

compiled to me. I did not really tried to run it yet, but sounds promising.
VCGLIB variable is path to modified vcglib, as prepared by sed in bug #1677989
comment 5.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1677989
[Bug 1677989] Review Request: vcglib Visualization and Computer Graphics
Library
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1677989

Petr Menšík  changed:

   What|Removed |Added

 Blocks||1658153




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1658153
[Bug 1658153] Review Request: wdune - wdune (white_dune) is a graphical
VRML97/X3D editor
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686506



--- Comment #2 from Lubomir Rintel  ---
Thanks, this looks fairly good.

A couple of questions:

1.) Why do you call the package lowercase "wireguard"? Upstream seems to call
the thing "WireGuard"

2.) What's the use of this in Fedora?

> Provides:   %{name}-kmod-common = %{version}
> %if 0%{?fedora} || 0%{?rhel} > 7
> Suggests:   %{name}-kmod >= %{version}
> %endif

The Fedora kernel-core package seems to provide "kmod(.ko)" for the
modules it ships. I guess that "kmod(wireguard.ko)" is what we should depend
on? Though I suppose only a weak dependency would do until the kernel actually
ships the module.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686506

Lubomir Rintel  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|lkund...@v3.sk



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684929] Review Request: golang-github-huandu-xstrings - A collection of useful string functions in Go

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



--- Comment #6 from Fedora Update System  ---
golang-github-huandu-xstrings-1.2.0-1.fc29 has been submitted as an update to
Fedora 29. https://bodhi.fedoraproject.org/updates/FEDORA-2019-3c5a235311

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1677989



--- Comment #7 from Petr Menšík  ---
Releases should be the same for any distribution, you should not make different
releases for Fedora only. Just include all software related to vcglib inside,
it is always possible to not install it. Examples are useful however, I think
they should be included in doc files.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1677989



--- Comment #6 from Petr Menšík  ---
second sed command is missing -i, should be
sed -e '/^#include/ s,"../../eigenlib/\(.*\)",<\1>,' -i math/eigen.h

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684929] Review Request: golang-github-huandu-xstrings - A collection of useful string functions in Go

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

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #5 from Fedora Update System  ---
golang-github-huandu-xstrings-1.2.0-1.fc30 has been submitted as an update to
Fedora 30. https://bodhi.fedoraproject.org/updates/FEDORA-2019-dd105f9391

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1677989



--- Comment #5 from Petr Menšík  ---
Removed eigenlib prefix with this command:
pushd vcg
sed -e '/^#include/ s/,' math/eigen.h
popd

However, libary itself does not provide any way to configure or build examples,
does it? No Makefile is provided. Can it be tested just from wdune (bug
#1658153)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1685975] cmocka

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

Jaroslav Škarvada  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2019-03-07 17:10:56



--- Comment #4 from Jaroslav Škarvada  ---
Also please next time open the review request by using the template link, i.e.:

https://bugzilla.redhat.com/enter_bug.cgi?product=Fedora=fedora-review

to have the correct title of the bug.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1685975] cmocka

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



--- Comment #3 from Jaroslav Škarvada  ---
(In reply to Jaroslav Škarvada from comment #2)
> (In reply to Petr Menšík from comment #0)
> > Description of problem:
> > Recent BIND depends on cmocka to run unit tests, instead of former libatf-c.
> > However cmocka is not yet even packaged in Fedora.
> > 
> What about:
> https://src.fedoraproject.org/rpms/cmocka
> ?

Agreed with the requester on side channel that there is nothing to do, so
closing.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1670441] Review Request: setconf - Tool for changing values of parameters in text files

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

Patrik Kopkan  changed:

   What|Removed |Added

  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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1658153



--- Comment #40 from Petr Menšík  ---
(In reply to J. Scheurich from comment #39)
> > Anyway, header paths guessing in configure.in is quite unfortunate. I would
> > recommend use of PKG_CHECK_MODULES macro or something similar for eigen3
> > [1]. Similar approach should be used for vcglib if possible.
> 
> vcglib is just huge collection of include files. The Problem with Eigen3 is
> that the fedora Eigen3 package is incompatible with
> vcglib's Eigen example: https://bugzilla.redhat.com/show_bug.cgi?id=1677989

I am afraid the project should be modified to work also with more recent
version. It should definitely work with latest version together. I understand
vcglib has different version. What parts are not compatible, it is known?

I understand it adds more work to adapt changes made by eigen upstream. It
drives Fedora forward however, makes it possible to cooperate and reuse single
source for many projects. It is required for a good reason.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686418

Robert-André Mauchin  changed:

   What|Removed |Added

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



--- Comment #2 from Robert-André Mauchin  ---
 - Squash the rpath:

R-ncdf4.x86_64: E: binary-or-shlib-defines-rpath
/usr/lib64/R/library/ncdf4/libs/ncdf4.so ['/usr/lib64']


Package Review
==

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


Issues:
===
- Package have the default element marked as %%doc :DESCRIPTION
- Package requires R-core.


= MUST items =

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

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]: 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]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Unknown or generated", "*No copyright* Public domain". 37
 files have unknown license. Detailed output of licensecheck in
 /home/bob/packaging/review/R-ncdf4/review-R-ncdf4/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: Package requires other packages for directories it uses.
[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]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 0 bytes in 0 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

R:
[x]: Package contains the mandatory BuildRequires.
[x]: The package has the standard %install section.

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

[Bug 1684933] Review Request: golang-github-bradfitz-iter - Range over integers [0,n).

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



--- Comment #7 from Fedora Update System  ---
golang-github-bradfitz-iter-0-0.1.20190307git33e6a98.fc29 has been submitted as
an update to Fedora 29.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-d42566b72f

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1685975] cmocka

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



--- Comment #2 from Jaroslav Škarvada  ---
(In reply to Petr Menšík from comment #0)
> Description of problem:
> Recent BIND depends on cmocka to run unit tests, instead of former libatf-c.
> However cmocka is not yet even packaged in Fedora.
> 
What about:
https://src.fedoraproject.org/rpms/cmocka
?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686410

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 #2 from Robert-André Mauchin  ---
Package approved.



Package Review
==

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


Issues:
===
- Package have the default element marked as %%doc :DESCRIPTION


= 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]: 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]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Unknown or generated". 21 files have unknown license. Detailed
 output of licensecheck in

/home/bob/packaging/review/R-nycflights13/review-R-nycflights13/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.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: No rpmlint messages.
[x]: Package requires other packages for directories it uses.
[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]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 0 bytes in 0 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

R:
[x]: Package contains the mandatory BuildRequires.
[x]: The package has the standard %install section.
[x]: Package requires R-core.

= 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 outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: Packager, Vendor, PreReq, Copyright 

[Bug 1684933] Review Request: golang-github-bradfitz-iter - Range over integers [0,n).

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

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #6 from Fedora Update System  ---
golang-github-bradfitz-iter-0-0.1.20190307git33e6a98.fc30 has been submitted as
an update to Fedora 30.
https://bodhi.fedoraproject.org/updates/FEDORA-2019-b3cb28a754

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1686210] Review Request: perl-JSON-Validator - Validate data against a JSON schema

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

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
FIX: Please update URL and source0 to the new location:
 URL: https://metacpan.org/release/JSON-Validator
 Source0:
https://cpan.metacpan.org/authors/id/J/JH/JHTHORSEN/JSON-Validator-%{version}.tar.gz

All tests passed
BuildRequires are ok

$ rpm -qp --requires perl-JSON-Validator-3.06-1.fc30.noarch.rpm | sort | uniq
-c
  1 perl(B)
  1 perl(base)
  1 perl(Carp)
  1 perl(constant)
  1 perl(Exporter)
  1 perl(JSON::Validator)
  1 perl(JSON::Validator::Error)
  1 perl(JSON::Validator::Formats)
  1 perl(JSON::Validator::Joi)
  1 perl(JSON::Validator::Ref)
  1 perl(:MODULE_COMPAT_5.28.1)
  1 perl(Mojo::Base)
  1 perl(Mojo::File)
  1 perl(Mojo::JSON)
  1 perl(Mojo::JSON::Pointer)
  1 perl(Mojolicious) >= 7.28
  1 perl(Mojo::Loader)
  1 perl(Mojo::URL)
  1 perl(Mojo::Util)
  1 perl(overload)
  1 perl(Scalar::Util)
  1 perl(Tie::Hash)
  1 perl(Tie::StdHash)
  1 perl(Time::Local)
  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
All mandatory run-requires are listed.

TODO: Please consider adding following optional dependencies as Suggests:
  YAML::XS 0.67
  Term::ANSIColor
  Mojo::UserAgent
  Data::Validate::Domain
  Data::Validate::IP
  Net::IDN::Encode

$ rpm -qp --provides perl-JSON-Validator-3.06-1.fc30.noarch.rpm | sort | uniq
-c
  1 perl(JSON::Validator) = 3.06
  1 perl-JSON-Validator = 3.06-1.fc30
  1 perl(JSON::Validator::Error)
  1 perl(JSON::Validator::Formats)
  1 perl(JSON::Validator::Joi)
  1 perl(JSON::Validator::Ref)
Binary provides are Ok.

$ rpmlint ./perl-JSON-Validator*
perl-JSON-Validator.noarch: W: spelling-error %description -l en_US
programmatically -> pro grammatically, pro-grammatically, programmatic ally
perl-JSON-Validator.src: W: spelling-error %description -l en_US
programmatically -> pro grammatically, pro-grammatically, programmatic ally
2 packages and 1 specfiles checked; 0 errors, 2 warnings.
Rpmlint is ok

Please correct all 'FIX' issues and consider fixing 'TODO' items.

Otherwise, 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1658153



--- Comment #39 from J. Scheurich  ---

> Anyway, header paths guessing in configure.in is quite unfortunate. I would
> recommend use of PKG_CHECK_MODULES macro or something similar for eigen3
> [1]. Similar approach should be used for vcglib if possible.

vcglib is just huge collection of include files. The Problem with Eigen3 is
that the fedora Eigen3 package is incompatible with
vcglib's Eigen example: https://bugzilla.redhat.com/show_bug.cgi?id=1677989

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1658153



--- Comment #38 from J. Scheurich  ---
(In reply to J. Scheurich from comment #35)
> Spec URL: ftp://ftp.ourproject.org/pub/wdune/wdune.spec
> SRPM URL: ftp://ftp.ourproject.org/pub/wdune/wdune-0.99rc1414-2.fc30.src.rpm
> Description: wdune (white_dune) is a graphical VRML97/X3D editor,
> simple NURBS/Superformula 3D modeller, animation tool, and VRML97/X3DV
> commandline compiler in development.
> With white_dune you can create/change 3D objects and animate them (in a easy
> way if you choose the -4kids GUI).
> The result can be shown in any webgl enabled web browser or can be converted
> to the RIB format for movie creation. 
> Fedora Account System Username: mufti11

Sorry, wrong SRPM URL: the correct Url is
ftp://ftp.ourproject.org/pub/wdune/wdune-0.99-2.fc29.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 1686022] Review Request: WireGuard - Tools for estabilishing a WireGuard secure IP tunnel

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

Jan Macku  changed:

   What|Removed |Added

   Assignee|jama...@redhat.com  |nob...@fedoraproject.org



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1658153



--- Comment #37 from J. Scheurich  ---
Spec URL: ftp://ftp.ourproject.org/pub/wdune/wdune.spec
SRPM URL: ftp://ftp.ourproject.org/pub/wdune/wdune-0.99-2.fc29.src.rpm
Description: wdune (white_dune) is a graphical VRML97/X3D editor,
simple NURBS/Superformula 3D modeller, animation tool, and VRML97/X3DV
commandline compiler in development.
With white_dune you can create/change 3D objects and animate them (in a easy
way if you choose the -4kids GUI).
The result can be shown in any webgl enabled web browser or can be converted to
the RIB format for movie creation. 
Fedora Account System Username: mufti11

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1658153



--- Comment #36 from J. Scheurich  ---
| And side note hint: your server provides also http protocol for files. It
seems something is broken in ftp:// handling of fedora-review tool, it 
| would simplify your review if you would use http URL links instead. Reported
as bug #1686383, but HTTP would simplify reviews.

The server support https://, but the rules on this server at ourproject.org
forces me to public files via ftp://.
There is nothing i can do against it 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 1658153] Review Request: wdune - wdune (white_dune) is a graphical VRML97/X3D editor

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



--- Comment #35 from J. Scheurich  ---
Spec URL: ftp://ftp.ourproject.org/pub/wdune/wdune.spec
SRPM URL: ftp://ftp.ourproject.org/pub/wdune/wdune-0.99rc1414-2.fc30.src.rpm
Description: wdune (white_dune) is a graphical VRML97/X3D editor,
simple NURBS/Superformula 3D modeller, animation tool, and VRML97/X3DV
commandline compiler in development.
With white_dune you can create/change 3D objects and animate them (in a easy
way if you choose the -4kids GUI).
The result can be shown in any webgl enabled web browser or can be converted to
the RIB format for movie creation. 
Fedora Account System Username: mufti11

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686506

Robert-André Mauchin  changed:

   What|Removed |Added

 CC||lkund...@v3.sk



--- Comment #1 from Robert-André Mauchin  ---
*** Bug 1686022 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1686022] Review Request: WireGuard - Tools for estabilishing a WireGuard secure IP tunnel

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

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |DUPLICATE
Last Closed||2019-03-07 15:26:32



--- Comment #4 from Robert-André Mauchin  ---
Discussed with head honcho at RPMFusion, suggested to close as duplicate. I
have fusionned both SPECs, hopefully it will work fine:

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

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1684177] Review Request: perl-Alien-FFI - Make available libffi

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



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1684906



--- Comment #4 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/golang-github-glycerine-unsnap-stream

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1643742



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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] New: Review Request: wireguard - Fast, modern, secure VPN tunnel

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

Bug ID: 1686506
   Summary: Review Request: wireguard - Fast, modern, secure VPN
tunnel
   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://eclipseo.fedorapeople.org/wireguard.spec
SRPM URL:
https://eclipseo.fedorapeople.org/wireguard-0.0.20190227-2.fc31.src.rpm

Description:
WireGuard is a novel VPN that runs inside the Linux Kernel and utilizes 
state-of-the-art cryptography. It aims to be faster, simpler, leaner, 
and more useful than IPSec, while avoiding the massive headache. It intends 
to be considerably more performant than OpenVPN. WireGuard is designed as a 
general purpose VPN for running on embedded interfaces and super computers 
alike, fit for many different circumstances. It runs over UDP.

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 1686402] Review Request: R-jqr - Client for 'jq', a 'JSON' Processor

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

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 #2 from Robert-André Mauchin  ---
Package approved.



Package Review
==

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


Issues:
===
- Package have the default element marked as %%doc :doc, DESCRIPTION
- Package requires R-core.


= MUST items =

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

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
[x]: License file installed when any subpackage combination is installed.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: Package requires other packages for directories it uses.
[x]: Package 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]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 0 bytes in 0 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

R:
[x]: Package contains the mandatory BuildRequires.
[x]: The package has the standard %install section.

= 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).
[-]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in R-jqr-
 debuginfo , R-jqr-debugsource
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported 

[Bug 1644010] Review Request: golang-github-pierrec-lz4 - LZ4 compression and decompression in pure Go

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



--- Comment #9 from Fedora Update System  ---
golang-github-pierrec-lz4-2.1.0-1.fc29 has been submitted as an update to
Fedora 29. https://bodhi.fedoraproject.org/updates/FEDORA-2019-d0b7fda363

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1686022] Review Request: WireGuard - Tools for estabilishing a WireGuard secure IP tunnel

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



--- Comment #3 from Robert-André Mauchin  ---
> esp. once the kernel parts hit mainline kernel

Do you know when is it supposed to happen? This package by itself is useless
without the kernel module, isn't it?

I would like to comaintain it at least if you plan to package it here.

 - Regarding the Review, you're missing the SystemD macros.

%{?systemd_requires}
BuildRequires:  systemd-rpm-macros

[…]

%post
%systemd_post wg-quick@.service


%preun
%systemd_preun wg-quick@.service


%postun
%systemd_postun_with_restart wg-quick@.service

 - Add the examples to the %doc. (remove the .gitignore files there)

 - Use %set_build_flags to use all the default flags.

 - make %{?_smp_mflags} → %make_build

 - Use standard perms for /etc/wireguard instead of 0700 in the Makefile


I don't know how to coordinate between RPMFusion and Fedora. On what branches
do you want to package it?

Would you be okay to add Release:2%{?dist} to supersede the RPMFusion
package?

Would you add:

Provides:   %{name}-kmod-common = %{version}


The problem is I need to sync the kmod package to the tools package and if we
are handling this separately, it's gonna be problematic.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1644010] Review Request: golang-github-pierrec-lz4 - LZ4 compression and decompression in pure Go

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

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #8 from Fedora Update System  ---
golang-github-pierrec-lz4-2.1.0-1.fc30 has been submitted as an update to
Fedora 30. https://bodhi.fedoraproject.org/updates/FEDORA-2019-95f122d39e

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1686210] Review Request: perl-JSON-Validator - Validate data against a JSON schema

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

Jitka Plesnikova  changed:

   What|Removed |Added

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



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
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 1684177] Review Request: perl-Alien-FFI - Make available libffi

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

Robert-André Mauchin  changed:

   What|Removed |Added

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



--- Comment #4 from Robert-André Mauchin  ---
You kick asses! 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 1686307] Review Request: python-distributed - Distributed scheduler for Dask

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

Robert-André Mauchin  changed:

   What|Removed |Added

 CC||zebo...@gmail.com



--- Comment #2 from Robert-André Mauchin  ---
CC 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 1686244] Review Request: R-maps - Draw Geographical Maps

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

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 #2 from Robert-André Mauchin  ---
 - ask upstream for a license file

 Package approved.



Package Review
==

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


Issues:
===
- Package have the default element marked as %%doc :DESCRIPTION
- Package requires R-core.


= 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:
[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]: 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]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Unknown or generated". 151 files have unknown license.
 Detailed output of licensecheck in
 /home/bob/packaging/review/R-maps/review-R-maps/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: Package requires other packages for directories it uses.
[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]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 0 bytes in 0 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

R:
[x]: Package contains the mandatory BuildRequires.
[x]: The package has the standard %install section.

= 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).
[-]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: 

[Bug 1684177] Review Request: perl-Alien-FFI - Make available libffi

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



--- Comment #3 from Petr Pisar  ---
I managed to locate and fix the bug in the tests by a series of scratch builds
in Koji. A fixed packages is available on the same addresses.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1684906

Robert-André Mauchin  changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |golang-github-glycerine-go- |golang-github-glycerine-uns
   |unsnap-stream - Small   |nap-stream - Small golang
   |golang library for decoding |library for decoding the
   |the snappy streaming format |snappy streaming format



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1686418



--- Comment #1 from Elliott Sales de Andrade  ---
This package built on koji: 
https://koji.fedoraproject.org/koji/taskinfo?taskID=33267674

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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] New: Review Request: R-ncdf4 - Interface to Unidata netCDF (Version 4 or Earlier) Format Data Files

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

Bug ID: 1686418
   Summary: Review Request: R-ncdf4 - Interface to Unidata netCDF
(Version 4 or Earlier) Format Data Files
   Product: Fedora
   Version: rawhide
Status: NEW
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: quantum.anal...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora




Spec URL: https://qulogic.fedorapeople.org//R-ncdf4.spec
SRPM URL: https://qulogic.fedorapeople.org//R-ncdf4-1.16-1.fc29.src.rpm

Description:
Provides a high-level R interface to data files written using Unidata's netCDF
library (version 4 or earlier), which are binary data files that are portable
across platforms and include metadata information in addition to the data sets.
Using this package, netCDF files (either version 4 or "classic" version 3) can
be opened and data sets read in easily. It is also easy to create new netCDF
dimensions, variables, and files, in either version 3 or 4 format, and
manipulate existing netCDF files.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1658153



--- Comment #34 from Petr Menšík  ---
And side note hint: your server provides also http protocol for files. It seems
something is broken in ftp:// handling of fedora-review tool, it would simplify
your review if you would use http URL links instead. Reported as bug #1686383,
but HTTP would simplify reviews.

Anyway, header paths guessing in configure.in is quite unfortunate. I would
recommend use of PKG_CHECK_MODULES macro or something similar for eigen3 [1].
Similar approach should be used for vcglib if possible.

I think it might look like this:
# Modify configure.in, then run autoreconf

PKG_CHECK_MODULES(EIGEN, [eigen3 eigen], [], AC_MSG_ERROR([eigenlib was not
found]) )

CPPFLAGS="$CPPFLAGS $EIGEN_CFLAGS"
LIBS="$LIBS $EIGEN_LIBS"

# Alternatively, use --with-vcglib=/some/path. [2]
AC_MSG_CHECKING([for vcglib])
AC_ARG_WITH(vcglib, AS_HELP_STRING([--with-vcglib], [use vcglib]),
use_vcglib="$withval", use_vcglib="yes")


# this allows --with-vcglib=/mypath/vcglib-1.0.1
# would use automagically installed vcglib and allows custom location at the
same time.
case "$use_vcglib" in
  yes)
   for d in /usr /usr/local $SELFDIR/vcglib; do
  if [ -f "$d/include/vcg/math/base.h" ]; then
VCGLIB_CFLAGS="-I$d/include"
  elif [ -f "$d/vcg/math/base.h" ]; then
VCGLIB_CFLAGS="-I$d"
  else
AC_MSG_ERROR([vcglib headers not found])
  fi
   done
   AC_MSG_RESULT([yes])
   ;;
  no)
   AC_MSG_ERROR([vcglib is mandatory])
   ;;
  *)
   VCGLIB_CFLAGS="-I$use_vcglib"
   AC_MSG_RESULT([$use_vcglib])
   ;;
esac

CPPFLAGS="$CPPFLAGS $VCGLIB_CFLAGS"

Such configuration would prefer installed packages over custom, but would allow
easy override if required.

1. https://autotools.io/pkgconfig/pkg_check_modules.html
2.
https://www.gnu.org/software/autoconf/manual/autoconf-2.60/html_node/External-Software.html

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1659556

Robert-André Mauchin  changed:

   What|Removed |Added

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



--- Comment #16 from Robert-André Mauchin  ---
Ok, package approved then.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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] New: Review Request: R-nycflights13 - Flights that Departed NYC in 2013

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

Bug ID: 1686410
   Summary: Review Request: R-nycflights13 - Flights that Departed
NYC in 2013
   Product: Fedora
   Version: rawhide
Status: NEW
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: quantum.anal...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora




Spec URL: https://qulogic.fedorapeople.org//R-nycflights13.spec
SRPM URL: https://qulogic.fedorapeople.org//R-nycflights13-1.0.0-1.fc29.src.rpm

Description:
Airline on-time data for all flights departing NYC in 2013. Also includes
useful 'metadata' on airlines, airports, weather, and planes.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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


  1   2   >