[Bug 1695139] Review Request: xonsh - A general purpose, Python-ish shell
https://bugzilla.redhat.com/show_bug.cgi?id=1695139
--- Comment #16 from Miro Hrončok ---
https://github.com/xonsh/xonsh/archive/%{version}/%{name}-%{version}.tar.gz
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1695139] Review Request: xonsh - A general purpose, Python-ish shell
https://bugzilla.redhat.com/show_bug.cgi?id=1695139 --- Comment #15 from Miro Hrončok --- Nice work! > Should we wait for upstream to release the fixes? Nah, just keep them in mind so you can unignore some tests when the changes are merged and released. Also, you should be able to use github tagged tarball to workaround missing 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 1696924] Review Request: python-prettyprinter - Syntax-highlighting, declarative and composable pretty printer
https://bugzilla.redhat.com/show_bug.cgi?id=1696924 --- Comment #5 from Fedora Update System --- python-prettyprinter-0.17.0-1.fc29 has been submitted as an update to Fedora 29. https://bodhi.fedoraproject.org/updates/FEDORA-2019-17702e6b9f -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696924] Review Request: python-prettyprinter - Syntax-highlighting, declarative and composable pretty printer
https://bugzilla.redhat.com/show_bug.cgi?id=1696924 --- Comment #4 from Fedora Update System --- python-prettyprinter-0.17.0-1.fc30 has been submitted as an update to Fedora 30. https://bodhi.fedoraproject.org/updates/FEDORA-2019-79d3094c4e -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696692] Review Request: ocaml-ocp-indent - A simple tool to indent OCaml programs
https://bugzilla.redhat.com/show_bug.cgi?id=1696692 --- Comment #10 from Fedora Update System --- ocaml-ocp-indent-1.7.0-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-0aa4b65b9e -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696048] Review Request: udisks2-qt5 - Qt5 binding for udisks2
https://bugzilla.redhat.com/show_bug.cgi?id=1696048 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #4 from Fedora Update System --- udisks2-qt5-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-cf958470b9 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696692] Review Request: ocaml-ocp-indent - A simple tool to indent OCaml programs
https://bugzilla.redhat.com/show_bug.cgi?id=1696692 --- Comment #9 from Fedora Update System --- ocaml-ocp-indent-1.7.0-2.fc30 has been pushed to the Fedora 30 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2019-777ccd6a43 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696874] Review Request: butt - Broadcast using this tool
https://bugzilla.redhat.com/show_bug.cgi?id=1696874 --- Comment #8 from Fedora Update System --- butt-0.1.17-2.fc30 has been submitted as an update to Fedora 30. https://bodhi.fedoraproject.org/updates/FEDORA-2019-1f1a6dbd50 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696692] Review Request: ocaml-ocp-indent - A simple tool to indent OCaml programs
https://bugzilla.redhat.com/show_bug.cgi?id=1696692 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System --- ocaml-ocp-indent-1.7.0-2.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-85ad7afba8 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1690634] Review Request: bifcl - A built-in-function (BIF) compiler/generator
https://bugzilla.redhat.com/show_bug.cgi?id=1690634 --- Comment #11 from Fedora Update System --- bifcl-1.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 1692070] Review Request: python-pystemd - A thin Cython-based wrapper on top of libsystemd
https://bugzilla.redhat.com/show_bug.cgi?id=1692070 --- Comment #3 from Chris Caron --- > This changelog format is compatible in all current Fedora releases. > It's being used in other packages now too: > https://src.fedoraproject.org/rpms/libsolv/blob/master/f/libsolv.spec#_304 > It does not work on EL7, but this package is literally not compatible with > EL7 systemd. I can change it if you insist... I don't see the harm in leaving it; I was just basing it on what was outlined in the current standards. It wouldn't hurt to change just to remain forwards compatible; who knows when you might generate some rpmlint error in the future. That all said, no harm is done if you don't; it's definitely not a show stopper by any means :) As per your other things, I've got no additional comments. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1692190] Review Request: ccls - C/C++/ObjC language server
https://bugzilla.redhat.com/show_bug.cgi?id=1692190 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2019-04-08 00:01:22 --- Comment #9 from Fedora Update System --- ccls-0.20190314-1.fc30 has been pushed to the Fedora 30 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 1665541] Review Request: python-resumable-urlretrieve - This is a drop-in replacement for urllib.request.urlretrieve
https://bugzilla.redhat.com/show_bug.cgi?id=1665541 --- Comment #18 from Fedora Update System --- python-resumable-urlretrieve-0.1.6-1.fc30 has been pushed to the Fedora 30 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 1689493] Review Request: python-zarr - An implementation of chunked, compressed, N-dimensional arrays for Python
https://bugzilla.redhat.com/show_bug.cgi?id=1689493 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2019-04-08 00:01:14 --- Comment #15 from Fedora Update System --- python-zarr-2.3.1-2.fc30 has been pushed to the Fedora 30 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 1689494] Review Request: python-xarray - N-D labeled arrays and datasets in Python
https://bugzilla.redhat.com/show_bug.cgi?id=1689494 Bug 1689494 depends on bug 1689493, which changed state. Bug 1689493 Summary: Review Request: python-zarr - An implementation of chunked, compressed, N-dimensional arrays for Python https://bugzilla.redhat.com/show_bug.cgi?id=1689493 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org 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 1692133] Review Request: crossguid2 - Lightweight cross platform C++ GUID/UUID library
https://bugzilla.redhat.com/show_bug.cgi?id=1692133 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2019-04-08 00:01:20 --- Comment #8 from Fedora Update System --- crossguid2-0.2.2-1.20190126gitb151b7d.fc30 has been pushed to the Fedora 30 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 1694735] Review Request: js8call - Amateur Radio message passing using FT8 modulation
https://bugzilla.redhat.com/show_bug.cgi?id=1694735 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2019-04-08 00:01:12 --- Comment #13 from Fedora Update System --- js8call-1.0.0-2.fc30 has been pushed to the Fedora 30 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 1690634] Review Request: bifcl - A built-in-function (BIF) compiler/generator
https://bugzilla.redhat.com/show_bug.cgi?id=1690634 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2019-04-08 00:01:01 --- Comment #10 from Fedora Update System --- bifcl-1.1-1.fc30 has been pushed to the Fedora 30 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 1692070] Review Request: python-pystemd - A thin Cython-based wrapper on top of libsystemd
https://bugzilla.redhat.com/show_bug.cgi?id=1692070
Neal Gompa changed:
What|Removed |Added
Status|ON_QA |ASSIGNED
--- Comment #2 from Neal Gompa ---
ON_QA status is for Bodhi...
> - Changelog format needs to be updated slightly; see
> - https://docs.fedoraproject.org/en-US/packaging-guidelines/#changelogs
>
This changelog format is compatible in all current Fedora releases.
It's being used in other packages now too:
https://src.fedoraproject.org/rpms/libsolv/blob/master/f/libsolv.spec#_304
It does not work on EL7, but this package is literally not compatible with EL7
systemd. I can change it if you insist...
> - python3-%{pypi_name} package entry is missing Requires: references.
> I presume you will require systemd and/or dbus to be availble?
The runtime dependencies will be autogenerated through the Python dependency
generator or the regular library dependency generator.
> - you got a lot of /usr/bin/env python3 references in files that are clearly
> not meant to be executable. Removing these should clear up a good number of
> the rpmlint checks identified at the bottom
I'll clean those up.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1696874] Review Request: butt - Broadcast using this tool
https://bugzilla.redhat.com/show_bug.cgi?id=1696874 Carl George changed: What|Removed |Added Status|POST|CLOSED Resolution|--- |RAWHIDE Last Closed||2019-04-07 23:43:53 --- Comment #7 from Carl George --- https://koji.fedoraproject.org/koji/buildinfo?buildID=1246464 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696874] Review Request: butt - Broadcast using this tool
https://bugzilla.redhat.com/show_bug.cgi?id=1696874 --- Comment #6 from Carl George --- For posterity, the guidelines currently state: > Fedora packages which use pkg-config to build against a library (e.g. 'foo') > on which they depend, SHOULD express their build dependency correctly as > pkgconfig(foo). butt doesn't use pkg-config to build against any of those other libraries. I understand the intent of trying to guard against future name changes, but if it's not using pkg-config then a dependency on pkgconfig(foo) would be incorrect. Of course if the guidelines are updated to the effect of "use the pkgconfig(foo) form whenever possible, even if the software isn't using pkg-config to build against the library", I'll happily update the spec file as needed. As it stands, this is just a SHOULD item, so I don't think we need to come to an agreement on it. As always Robert thanks for your review and your input. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1692070] Review Request: python-pystemd - A thin Cython-based wrapper on top of libsystemd
https://bugzilla.redhat.com/show_bug.cgi?id=1692070 Chris Caron changed: What|Removed |Added Status|NEW |ON_QA -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1691610] Review Request: python-elpy - Backend for the elpy Emacs mode
https://bugzilla.redhat.com/show_bug.cgi?id=1691610 Chris Caron changed: What|Removed |Added Status|NEW |ON_QA -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1691610] Review Request: python-elpy - Backend for the elpy Emacs mode
https://bugzilla.redhat.com/show_bug.cgi?id=1691610 --- Comment #2 from Chris Caron --- Another minor comment, the %changelog is just missing a proper close > tag after your email. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1540335] Review Request: primesieve - Fast C/C++ prime number generator
https://bugzilla.redhat.com/show_bug.cgi?id=1540335
--- Comment #14 from Petr Menšík ---
Ok, made mistake with offered Provides in 2):
Correct version would be:
Provides: lib%{name}%{?_isa} = %{version}-%{release}
Just a question, it has also working GUI with a different license (GPLv3). Is
there some reason why gui variant is not packaged? Subpackage with gui variant
would not hurt I think, would you need help with that? License tag is correct
for command line version, gui subpackage would require License: BSD and GPLv3+
I checked it is not packaged also on Debian [2]. Compiled just fine to me, some
Qt dependencies would be required, should not be difficult. Not mandatory for
review, would be nice to have it.
2. https://packages.debian.org/source/sid/primesieve
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1691610] Review Request: python-elpy - Backend for the elpy Emacs mode
https://bugzilla.redhat.com/show_bug.cgi?id=1691610 Chris Caron changed: What|Removed |Added Assignee|nob...@fedoraproject.org|lead2g...@gmail.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 1691610] Review Request: python-elpy - Backend for the elpy Emacs mode
https://bugzilla.redhat.com/show_bug.cgi?id=1691610
Chris Caron changed:
What|Removed |Added
CC||lead2g...@gmail.com
--- Comment #1 from Chris Caron ---
Package Build
=
Builds Okay:
- https://koji.fedoraproject.org/koji/taskinfo?taskID=34038420
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- Sources used to build the package do not match the upstream source, as
provided
in the spec URL. Checksum differences are provided at the bottom of this
review.
- Your .spec file identifies a GPL license, yet your GitHub page and tar.gz
file
includes a GPLv3 license. I imagine this is just a typo in your spec?
- You should have a peek at
https://fedoraproject.org/wiki/User:Bkabrda/EPEL7_Python3.
From here you'll see some things you're doing nicely and some spots you just
need
a few tweeks such as:
- drop reference to %{?python_enable_dependency_generator}
- BuildRequires: %{py3_dist coverage} should read:
BuildRequires: python%{python3_pkgversion}-coverage
- You need to apply this to all of the BuildRequrires.
- There are a few rpmlint warnings below which are not show-stoppers by any
means, but
useful to show as you can optionally fix them up too! :)
- %check is disabled; but i can see that recently python-yapf (bz#1691609) has
been pushed
upstream now. So you can turn that back on and try it out to see how it works
for you.
= 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.
[!]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses
found: "Unknown or generated", "GPL (v3 or later)". 307 files have
unknown license. Detailed output of licensecheck in
/home/l2g/Development/reviews/review-python-elpy/licensecheck.txt
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 10240 bytes in 1 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %license.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package must not depend on deprecated() packages.
[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]: Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
Python:
[x]: Python eggs must not download any dependencies during the build
process.
[x]: A package which is used by another package via an egg interface should
provide egg info.
[!]: Package meets the Packaging
[Bug 1540335] Review Request: primesieve - Fast C/C++ prime number generator
https://bugzilla.redhat.com/show_bug.cgi?id=1540335 --- Comment #13 from Petr Menšík --- (In reply to Jani Juhani Sinervo from comment #11) > (In reply to Kim Walisch from comment #10) > > Thanks for your review. > > > The Rpmlint spelling-error warnings are false positives. > > Oh, good to know for future reference! I was actually thinking if that would > actually affect the review outcome significantly in an actual review. I > included it as a "problem" in that preliminary review just in case. > > -- > > But yeah, with the fix to the ldconfig problem sorted, and updating to the > new version, I'd see no problems with this being included. But I'll leave > the decision for someone less amateurish than I. But while we wait for an > official review, here's an updated preliminary review for this new version. > I think your review was a good one, there is no need for package to be perfect. It met basic requirements, you did review it well. Please, do not be afraid giving it approval next time. It would not provide sponsor, but mark is as acceptable. We can always submit bugs to fix not important issues. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1695139] Review Request: xonsh - A general purpose, Python-ish shell
https://bugzilla.redhat.com/show_bug.cgi?id=1695139
--- Comment #14 from Carmen Bianca Bakker ---
I found the problem(s):
- The tests very specifically need to be run within a xonsh shell using the
`py.test-3` command. Calling `py.test-3` from bash or calling `python3 -m
pytest` from xonsh results in failing tests.
- There is an error in `%{_bindir}/xon.sh`.
- The tests in the tarball are incomplete. i.e., files are missing.
I have submitted merge requests upstream for the last two bullet points:
- https://github.com/xonsh/xonsh/pull/3068
- https://github.com/xonsh/xonsh/pull/3069
This is a WIP specfile that doesn't currently build because of failing tests,
but would run if upstream released those fixes (I think):
https://gitlab.com/snippets/1844781
Should we wait for upstream to release the fixes?
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1540335] Review Request: primesieve - Fast C/C++ prime number generator
https://bugzilla.redhat.com/show_bug.cgi?id=1540335
Petr Menšík changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||pemen...@redhat.com
Assignee|nob...@fedoraproject.org|pemen...@redhat.com
--- Comment #12 from Petr Menšík ---
1) Primary package primesieve does not depend on library. Requires is currently
part of %description, where it does not have effect. Move it before description
please.
2) In Fedora, lib is not common for libraries. Instead, -libs is
usually used. If there is no strong objection, I would suggest using just
%package libs, %files libs.
If you want to keep compatibility with Debian naming, use Provides:
libprimesieve%{?_isa} = libprimesieve-%{version}. It would still be accepted in
yum install libprimesieve or other package dependencies. Some tool write
warnings because it lacks common prefix, fedora-review tool is one of them.
Note -n parameter is not required in this case
Similar issue is with %package devel, %files devel.
3) Unless there is expected usage of static libraries, that offers significant
speed advantage, I would omit static subpackage. It requires explicit exception
and I doubt it is required. Use rm to remove *.la in %build or %install phase.
[1]
4) %ldconfig_scriptlets libs should be used, only for library package it should
be used. In current setup, %ldconfig_scriptlets -n libprimesieve would have to
be used instead.
5) New version should be used now, since more recent release is available. I
know, it was up to date when started. Sorry :)
Other than these few issues, package is ready to be packaged in Fedora. Good
job for first package!
1.
https://docs.fedoraproject.org/en-US/packaging-guidelines/#packaging-static-libraries
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1692070] Review Request: python-pystemd - A thin Cython-based wrapper on top of libsystemd
https://bugzilla.redhat.com/show_bug.cgi?id=1692070 Chris Caron changed: What|Removed |Added Assignee|nob...@fedoraproject.org|lead2g...@gmail.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 1696874] Review Request: butt - Broadcast using this tool
https://bugzilla.redhat.com/show_bug.cgi?id=1696874 --- Comment #5 from Robert-André Mauchin --- (In reply to Carl George from comment #4) > While verifying the pkgconfig provides, I re-read the guidelines for this > [0], and noticed that it only applies when the software is using pkg-config > to locate the library. In this case most of the libraries are detected by > AC_CHECK_LIB [1], with the exception of dbus, which is detected with > PKG_CHECK_MODULES. As such the only change I made was: > > -BuildRequires: dbus-devel > +BuildRequires: pkgconfig(dbus-1) > > > [0]: > https://docs.fedoraproject.org/en-US/packaging-guidelines/ > PkgConfigBuildRequires/ > [1]: https://www.gnu.org/software/autoconf/manual/autoconf.html#Libraries I kind of disagree with this. This is true that the guidelines say: "Fedora packages which use pkg-config to build against a library" but the rationale expressed below: "Where package names change, and/or a required pkgconfig module is later provided by a different package, these hard-coded dependencies break." is true even if the package doesn't use it internally. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1695139] Review Request: xonsh - A general purpose, Python-ish shell
https://bugzilla.redhat.com/show_bug.cgi?id=1695139 --- Comment #13 from Miro Hrončok --- > The tests only fail inside of the mock environment. Not really sure how to > address this. Try exporting custom PATH and MANPATH form some of the failing tests to actually see the executables and manpages. For tests that fail due to missing internet connection, feel free to keep them disabled. If you really want to get fancy, do a %bcond_with internet (see python-sphinx as an example). I appreciate the ignored breakdown in comment. I see you ignore entire files. If you only want to skip some of the tests, examine -k option of pytest. It can do bool operations, e.g. -k "not (remote or web or (slow and tedious))". -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696874] Review Request: butt - Broadcast using this tool
https://bugzilla.redhat.com/show_bug.cgi?id=1696874 --- Comment #4 from Carl George --- While verifying the pkgconfig provides, I re-read the guidelines for this [0], and noticed that it only applies when the software is using pkg-config to locate the library. In this case most of the libraries are detected by AC_CHECK_LIB [1], with the exception of dbus, which is detected with PKG_CHECK_MODULES. As such the only change I made was: -BuildRequires: dbus-devel +BuildRequires: pkgconfig(dbus-1) [0]: https://docs.fedoraproject.org/en-US/packaging-guidelines/PkgConfigBuildRequires/ [1]: https://www.gnu.org/software/autoconf/manual/autoconf.html#Libraries -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696924] Review Request: python-prettyprinter - Syntax-highlighting, declarative and composable pretty printer
https://bugzilla.redhat.com/show_bug.cgi?id=1696924 Carl George changed: What|Removed |Added Status|POST|CLOSED Resolution|--- |RAWHIDE Last Closed||2019-04-07 17:07:25 --- Comment #3 from Carl George --- https://koji.fedoraproject.org/koji/buildinfo?buildID=1246384 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1684956] Review Request: golang-github-anacrolix-dms - A UPnP DLNA Digital Media Server that includes basic video transcoding
https://bugzilla.redhat.com/show_bug.cgi?id=1684956 --- Comment #4 from Robert-André Mauchin --- And thanks for reviewing! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1684956] Review Request: golang-github-anacrolix-dms - A UPnP DLNA Digital Media Server that includes basic video transcoding
https://bugzilla.redhat.com/show_bug.cgi?id=1684956 --- Comment #3 from Robert-André Mauchin --- > + Requires: golang(github.com/mattn/go-gtk) No, the Requires are automatically computed in Golang packages What you should do is add the missing Requires to your fedora-review process with the -L parameters Grab the missing dependency from my COPR: https://copr-be.cloud.fedoraproject.org/results/eclipseo/rclone1.45/fedora-rawhide-x86_64/00873461-golang-github-mattn-gtk/ Copy it in a folder. fedora-review --mock-config fedora-rawhide-x86_64 -n golang-github-anacrolix-dms -L path/to/the/folder/containing/the/dependency -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696692] Review Request: ocaml-ocp-indent - A simple tool to indent OCaml programs
https://bugzilla.redhat.com/show_bug.cgi?id=1696692 --- Comment #7 from Fedora Update System --- ocaml-ocp-indent-1.7.0-2.fc29 has been submitted as an update to Fedora 29. https://bodhi.fedoraproject.org/updates/FEDORA-2019-0aa4b65b9e -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696692] Review Request: ocaml-ocp-indent - A simple tool to indent OCaml programs
https://bugzilla.redhat.com/show_bug.cgi?id=1696692 --- Comment #6 from Fedora Update System --- ocaml-ocp-indent-1.7.0-2.fc28 has been submitted as an update to Fedora 28. https://bodhi.fedoraproject.org/updates/FEDORA-2019-85ad7afba8 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696692] Review Request: ocaml-ocp-indent - A simple tool to indent OCaml programs
https://bugzilla.redhat.com/show_bug.cgi?id=1696692 Fedora Update System changed: What|Removed |Added Status|POST|MODIFIED --- Comment #5 from Fedora Update System --- ocaml-ocp-indent-1.7.0-2.fc30 has been submitted as an update to Fedora 30. https://bodhi.fedoraproject.org/updates/FEDORA-2019-777ccd6a43 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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
https://bugzilla.redhat.com/show_bug.cgi?id=1658153 --- Comment #80 from J. Scheurich --- Sorry, wrong URL 8-( Spec URL: ftp://ftp.ourproject.org/pub/wdune/wdune.spec SRPM URL: ftp://ftp.ourproject.org/pub/wdune/wdune-0.99-3pl1515.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
https://bugzilla.redhat.com/show_bug.cgi?id=1658153
--- Comment #79 from J. Scheurich ---
Spec URL: ftp://ftp.ourproject.org/pub/wdune/wdune-0.99.spec
SRPM URL: ftp://ftp.ourproject.org/pub/wdune/wdune-0.99-3pl1515.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
Added human animation to opengl example
> Requires: %{name}%{?_isa} = %{name}-%{version}
I failed to include this. The thing would require two "-" characters to work,
which is not accepted.
Instead i added
Recommends: wdune
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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 1662572] Review Request: python-aiohttp-socks - SOCKS proxy connector for aiohttp
https://bugzilla.redhat.com/show_bug.cgi?id=1662572 Artem S. Tashkinov changed: What|Removed |Added Blocks||1697063 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1697063 [Bug 1697063] Electrum can't be installed in Fedora 30 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1693014] Review Request: python-pyrpm - Python module for parsing RPM spec filesPython module for parsing RPM spec files
https://bugzilla.redhat.com/show_bug.cgi?id=1693014 --- Comment #9 from Gwyn Ciesla --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/python-pyrpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1693014] Review Request: python-pyrpm - Python module for parsing RPM spec filesPython module for parsing RPM spec files
https://bugzilla.redhat.com/show_bug.cgi?id=1693014 Robert-André Mauchin changed: What|Removed |Added Assignee|nob...@fedoraproject.org|fed...@georg.so -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1649059] Review Request: golang-contrib-opencensus-exporter-ocagent - OpenCensus Go exporters for OpenCensus Agent
https://bugzilla.redhat.com/show_bug.cgi?id=1649059 --- Comment #26 from Fedora Update System --- golang-contrib-opencensus-exporter-ocagent-0.4.11-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-7e97e745c4 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1662572] Review Request: python-aiohttp-socks - SOCKS proxy connector for aiohttp
https://bugzilla.redhat.com/show_bug.cgi?id=1662572 --- Comment #9 from Jonny Heggheim --- (In reply to Igor Gnatenko from comment #7) > I will update this review request tonight. Thanks, I will approve it when you have fixed the feedback in comment 3 and comment 5. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1662572] Review Request: python-aiohttp-socks - SOCKS proxy connector for aiohttp
https://bugzilla.redhat.com/show_bug.cgi?id=1662572 Igor Gnatenko changed: What|Removed |Added Flags|needinfo?(i.gnatenko.brain@ | |gmail.com) | --- Comment #7 from Igor Gnatenko --- Sorry, I didn't have time lately. I will update this review request tonight. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1695746] Review Request: python-aiohttp-socks - SOCKS proxy connector for aiohttp
https://bugzilla.redhat.com/show_bug.cgi?id=1695746 Igor Gnatenko changed: What|Removed |Added Status|NEW |CLOSED CC||i.gnatenko.br...@gmail.com Resolution|--- |DUPLICATE Last Closed||2019-04-07 10:09:58 --- Comment #3 from Igor Gnatenko --- *** This bug has been marked as a duplicate of bug 1662572 *** -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1662572] Review Request: python-aiohttp-socks - SOCKS proxy connector for aiohttp
https://bugzilla.redhat.com/show_bug.cgi?id=1662572 --- Comment #8 from Igor Gnatenko --- *** Bug 1695746 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 1696048] Review Request: udisks2-qt5 - Qt5 binding for udisks2
https://bugzilla.redhat.com/show_bug.cgi?id=1696048 Fedora Update System changed: What|Removed |Added Status|POST|MODIFIED --- Comment #3 from Fedora Update System --- udisks2-qt5-0.0.1-1.fc30 has been submitted as an update to Fedora 30. https://bodhi.fedoraproject.org/updates/FEDORA-2019-cf958470b9 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696692] Review Request: ocaml-ocp-indent - A simple tool to indent OCaml programs
https://bugzilla.redhat.com/show_bug.cgi?id=1696692 --- Comment #4 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/ocaml-ocp-indent -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696924] Review Request: python-prettyprinter - Syntax-highlighting, declarative and composable pretty printer
https://bugzilla.redhat.com/show_bug.cgi?id=1696924 --- Comment #2 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/python-prettyprinter -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1695362] Review Request: python-metaextract - get metadata for python modules
https://bugzilla.redhat.com/show_bug.cgi?id=1695362 --- Comment #10 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/python-metaextract -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696874] Review Request: butt - Broadcast using this tool
https://bugzilla.redhat.com/show_bug.cgi?id=1696874 --- Comment #3 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/butt -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696399] Review Request: python-jaraco-functools - Functools like those found in stdlib
https://bugzilla.redhat.com/show_bug.cgi?id=1696399 --- Comment #16 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/python-jaraco-functools -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1693013] Review Request: perl-Test-RunValgrind - Tests that an external program is valgrind-clean
https://bugzilla.redhat.com/show_bug.cgi?id=1693013 --- Comment #2 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/perl-Test-RunValgrind -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1695526] Review Request: python-inema - A Python interface to the Deutsche Post Internetmarke Online Franking
https://bugzilla.redhat.com/show_bug.cgi?id=1695526 --- Comment #3 from dan.cer...@cgc-instruments.com --- Thanks for your work, I've approved the package. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org 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 1695526] Review Request: python-inema - A Python interface to the Deutsche Post Internetmarke Online Franking
https://bugzilla.redhat.com/show_bug.cgi?id=1695526 dan.cer...@cgc-instruments.com 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 1695803] Review Request: python-jaraco-classes - Utility functions for Python class constructs
https://bugzilla.redhat.com/show_bug.cgi?id=1695803 --- Comment #6 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/python-jaraco-classes -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1672047] Review Request: smoldyn - A particle-based spatial stochastic simulator
https://bugzilla.redhat.com/show_bug.cgi?id=1672047 --- Comment #11 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/smoldyn -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1696250] Review Request: python-cheroot - Highly-optimized, pure-python HTTP server
https://bugzilla.redhat.com/show_bug.cgi?id=1696250 --- Comment #8 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/python-cheroot -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1695172] Review Request: python-requests-unixsocket - Use requests to talk HTTP via a UNIX domain socket
https://bugzilla.redhat.com/show_bug.cgi?id=1695172 --- Comment #7 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/python-requests-unixsocket -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1692344] Review Request: golang-github-mattn-pointer - Utility for cgo
https://bugzilla.redhat.com/show_bug.cgi?id=1692344 --- Comment #3 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-mattn-pointer -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1688881] Review Request: golang-github-armon-consul-api - Golang API client for Consul
https://bugzilla.redhat.com/show_bug.cgi?id=161 --- Comment #9 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-armon-consul-api -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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 1643996] Review Request: golang-github-eapache-xerial-snappy - Xerial-compatible Snappy framing support for golang
https://bugzilla.redhat.com/show_bug.cgi?id=1643996 --- Comment #2 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-eapache-xerial-snappy -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1688944] Review Request: golang-github-tmc-grpc-websocket-proxy - A proxy to transparently upgrade grpc-gateway streaming endpoints to use websockets
https://bugzilla.redhat.com/show_bug.cgi?id=1688944 --- Comment #3 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-tmc-grpc-websocket-proxy -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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 1689006] Review Request: golang-github-grpc-ecosystem-middleware - Golang gRPC Middlewares: interceptor chaining, auth, logging, retries and more
https://bugzilla.redhat.com/show_bug.cgi?id=1689006 --- Comment #2 from Igor Gnatenko --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-grpc-ecosystem-middleware -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this 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
