[Bug 1868850] Review Request: fcitx5-chinese-addons - Chinese related addon for fcitx5

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1868850



--- Comment #6 from Qiyu Yan  ---
Also, I will investigate into if this test error can be safely ignored (not
easy, since I don't have a s390x testing environment.) And this problem occurs
in debian builds to, maybe they can provide some help.

Similar to plans before, if we can't ignore this failure, I will keep the
ExcludeArch and ask upstream for help, if otherwise, I will ask FPC for an
exception.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1857584] Review Request: python-serpent - Serialization based on ast.literal_eval

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1857584

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #4 from Fedora Update System  ---
FEDORA-2020-2387058a2d has been pushed to the Fedora 32 testing repository.
In short time you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing
--advisory=FEDORA-2020-2387058a2d \*`
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2020-2387058a2d

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information
on how to test updates.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1868850] Review Request: fcitx5-chinese-addons - Chinese related addon for fcitx5

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1868850



--- Comment #5 from Qiyu Yan  ---
This is harder to fix, I am contacting upstream for help.

And I added a ExcludeArch, and would file a build-fail ticket 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1868850] Review Request: fcitx5-chinese-addons - Chinese related addon for fcitx5

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1868850

Qiyu Yan  changed:

   What|Removed |Added

Comment|0   |updated



--- Comment #0 has been edited ---

Spec URL:
https://download.copr.fedorainfracloud.org/results/yanqiyu/fcitx5/fedora-33-x86_64/01628036-fcitx5-chinese-addons/fcitx5-chinese-addons.spec
SRPM URL:
https://download.copr.fedorainfracloud.org/results/yanqiyu/fcitx5/fedora-33-x86_64/01628036-fcitx5-chinese-addons/fcitx5-chinese-addons-0-0.2.20200812gitef9beb7.fc33.src.rpm
Description: Chinese related addon for fcitx5
Fedora Account System Username: yanqiyu

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1858639] Review Request: qt-avif-image-plugin - Qt plug-in to read/write AVIF images

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1858639



--- Comment #6 from Michel Alexandre Salim  ---
Ack, re-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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1868992] Review Request: jakarta-server-pages - Jakarta Server Pages (JSP)

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1868992



--- Comment #2 from Mat Booth  ---
Issues:

1)

Hmm, just like jakarta-servlet package, half the source files have "ASL 2.0"
headers instead of the "EPL-2.0 or GPLv2", so I think the license field would
be more accurate as this:

ASL 2.0 and (EPL-2.0 or GPLv2 with exceptions)

I read the upstream bug you filed for servlet-api (
https://github.com/eclipse-ee4j/servlet-api/issues/347 ) and it seemed to take
them by surprise! So probably worth filing a bug upstream for this one too.


2)

For some boring technical reasons, Eclipse requires the JDTJavaCompiler adapter
implementation, which in the old package was enabled by this patch:

https://src.fedoraproject.org/rpms/glassfish-jsp/blob/84ff3bc904d196045a9ec2d5140391847dd3/f/0001-Enable-Eclipse-JDT-compiler-support-for-runtime-JSP-.patch

This does not appear to have made the transition. The new package instead
deletes JDTJavaCompiler and removes the dep on ecj. Any chance this can be
restored?

I know it's a deviation from the upstream's distribution available from maven
central, but the Eclipse Foundation ships it's own build with this addition to
allow JSP to use Eclipse's built-in ECJ Java compiler for compiling JSP pages
at runtime.


3)

The "compat namespace" (cp -pr && sed s/javax/jakarta/) causes compilation
errors in Eclipse. The problem is due to mixed usage of javax.* and jakarta.*
classes when referencing classes directly from the implementation jar.

Eclipse does stuff like this example:

import javax.servlet.Servlet; // An API class
import org.apache.jasper.servlet.JspServlet; // An impl class

Servlet s = new JspServlet;

HOWEVER, JspServlet extends "jakarta.servlet.Servlet" and NOT
"javax.servlet.Servlet" which makes JspServlet unassignable to Servlet in this
context (type mismatch, compilation error)

So although this compat "hack" works fine when only the API classes are used
(javax.* classes) it fails when we try to use classes from the implementation
jar without fully migrating to jakarta.* APIs.

I need a bit of time to think about how best to solve this. It looks like we
only reference JSP implementation classes a couple times in Eclipse code, so
perhaps I will be able to come up with a concise patch there. Or do something
cleverer in this package and jakarta-servlet package. Hmmm




For completion, the full review tool output follows below. FTR,
obsoletes/provides look fine to me, I don't know why the review tool does not
enumerate the obsoletes as it does with the provides...


Package Review
==

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


= 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", "*No copyright* Eclipse Public License
 2.0", "Eclipse Public License 2.0", "Apache License 2.0". 45 files
 have unknown license. Detailed output of licensecheck in
 /home/mbooth/fedora/1868992-jakarta-server-pages/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[!]: If the package is under multiple licenses, the licensing breakdown
 must be documented in the spec.
[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.
[x]: 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 

[Bug 1858639] Review Request: qt-avif-image-plugin - Qt plug-in to read/write AVIF images

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1858639



--- Comment #5 from Robert-André Mauchin   ---
New Spec URL:
https://eclipseo.fedorapeople.org/for-review/qt-avif-image-plugin.spec
New SRPM URL:
https://eclipseo.fedorapeople.org/for-review/qt-avif-image-plugin-0.3.1-1.fc34.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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871908] Review Request: python-aiohomekit - Python HomeKit client

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871908



--- Comment #3 from Fabian Affolter  ---
%changelog
* Mon Aug 24 2020 Fabian Affolter  - 0.2.50-1
- LICENSE file is now present (rhbz#1871908)


Updated files:
Spec URL: https://fab.fedorapeople.org/packages/SRPMS/python-aiohomekit.spec
SRPM URL:
https://fab.fedorapeople.org/packages/SRPMS/python-aiohomekit-0.2.50-1.fc32.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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871908] Review Request: python-aiohomekit - Python HomeKit client

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871908



--- Comment #2 from Fabian Affolter  ---
PR was merged and upstream released 0.2.50 with the LICENSE file that has
content.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871997] Review Request: (unretiring) menulibre - MenuLibre is an advanced FreeDesktop.org compliant menu editor

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871997



Spec URL: https://lyessaadi.fedorapeople.org/menulibre/menulibre.spec

SOURCE1: https://lyessaadi.fedorapeople.org/menulibre/menulibre.appdata.xml
Upstream Bug: https://github.com/bluesabre/menulibre/issues/61

SRPM URL:
https://lyessaadi.fedorapeople.org/menulibre/menulibre-2.2.1-3.fc32.src.rpm

Koji Build: https://koji.fedoraproject.org/koji/taskinfo?taskID=50086730

Description:
MenuLibre is an advanced FreeDesktop.org compliant menu editor.

All fields specified in the FreeDesktop.org Desktop Entry and Menu
specifications are available to quickly update. Additionally, MenuLibre 
provides an editor for the launcher actions used by applications such as Unity 
and Plank.

Features:

- A beautiful interface powered by the latest version of GTK+.
- Create new launchers, or modify existing ones with complete control over
  common settings and access to advanced settings.
- Add, remove, and adjust desktop actions: powerful shortcuts available used by
  Unity, Xfce, and Pantheon.
- Easily rearrange menu items to suit your needs.

Fedora Account System Username: lyessaadi

Note: I am unretiring this package (I made some changes to the original spec
file).


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


[Bug 1842225] Review Request: python-authlib - Build OAuth and OpenID Connect servers

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1842225

Robert-André Mauchin   changed:

   What|Removed |Added

 Blocks|177841 (FE-NEEDSPONSOR) |
  Flags|fedora-review+  |fedora-review?





Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1842225] Review Request: python-authlib - Build OAuth and OpenID Connect servers

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1842225

Robert-André Mauchin   changed:

   What|Removed |Added

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



--- Comment #2 from Robert-André Mauchin   ---
Refreshed flag + sponsored.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871997] New: Review Request: (unretiring) menulibre - MenuLibre is an advanced FreeDesktop.org compliant menu editor

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871997

Bug ID: 1871997
   Summary: Review Request: (unretiring) menulibre - MenuLibre is
an advanced FreeDesktop.org compliant menu editor
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fed...@lyes.eu
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora




-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871908] Review Request: python-aiohomekit - Python HomeKit client

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871908



--- Comment #1 from Andy Mender  ---
All is good. We just need that license file.

Package Review
==

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


Issues:
===
- Package installs properly.
  Note: Installation errors (see attachment)
  See: https://docs.fedoraproject.org/en-US/packaging-guidelines/
  Review: Problems with mock. Ignore this warning.


= MUST items =

Generic:
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
 Note: Using prebuilt packages
 Review: Tested in Koji by the submitter.
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[ ]: 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.
 Review: Leaving this one empty, since a ticket with upstream was opened.
[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", "Apache License 2.0". 8 files have
 unknown license. Detailed output of licensecheck in
 /home/amender/rpmbuild/SPECS/python-aiohomekit/python-
 aiohomekit/licensecheck.txt
 Review: Yes, but see above point.
[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
[?]: 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.
 Review: It's a Python noarch package.
[x]: 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]: 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 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]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

Python:
[x]: Python eggs must not download any dependencies during the build
 process.
[x]: A package which is used by another package via an egg interface should
 provide egg info.
[x]: Package meets the Packaging Guidelines::Python
[x]: Package contains BR: python2-devel or python3-devel
[x]: Packages MUST NOT have dependencies (either build-time or runtime) on
 packages named with the unversioned python- prefix unless no properly
 versioned package exists. Dependencies on Python packages instead MUST
 use names beginning with python2- or python3- as appropriate.
[x]: Python packages must not contain %{pythonX_site(lib|arch)}/* in %files
[x]: Binary eggs must be removed in %prep

= SHOULD items =

Generic:
[x]: Reviewer should test that the package builds in mock.
[x]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
 Review: ticket with upstream https://github.com/Jc2k/aiohomekit/pull/19
[x]: Final provides and requires are sane 

[Bug 1871908] Review Request: python-aiohomekit - Python HomeKit client

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871908

Andy Mender  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||andymenderu...@gmail.com
   Assignee|nob...@fedoraproject.org|andymenderu...@gmail.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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1868850] Review Request: fcitx5-chinese-addons - Chinese related addon for fcitx5

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1868850



--- Comment #4 from Andy Mender  ---
> - removed ExcludeArch
> - switched to pkgconfig(curl) 
> - added -data subpackage
> - more specific on /usr/share/*

Nice!

Koji build based on latest changes:
https://koji.fedoraproject.org/koji/taskinfo?taskID=50083236

I triggered it again just in case, but unfortunately 1 test now fails on the
s390x architecture:
https://koji.fedoraproject.org/koji/taskinfo?taskID=50083448

Here's the error from the tail of the log:
F2020-08-24 18:50:41.880776 testpinyin.cpp:23] pinyin failed
67% tests passed, 1 tests failed out of 3
Total Test time (real) =  53.79 sec
The following tests FAILED:
  3 - testpinyin (Child aborted)
Errors while running CTest
error: Bad exit status from /var/tmp/rpm-tmp.o9qgTj (%check)
RPM build errors:
Bad exit status from /var/tmp/rpm-tmp.o9qgTj (%check)
Child return code was: 1
EXCEPTION: [Error()]


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1868853] Review Request: fcitx5-configtool - Configuration tools used by fcitx5

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1868853

Andy Mender  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #5 from Andy Mender  ---
Koji build based on latest changes:
https://koji.fedoraproject.org/koji/taskinfo?taskID=50082338

> they doesn't provide pkgconfig(foo) but have 
> cmake(foo), while cmake(foo) was introduced recently, some package didn't 
> build with that. On f33+, mostly yes, on f32 some don't have tags like this.

We can do without it in this case.

> Just in case upstream add content to readme, I won't forget to include that.

Fair point. Let's keep it then.

> added 
> - Requires:   kf5-kcmutils

Verified:
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.

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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1861839



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


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871908] New: Review Request: python-aiohomekit - Python HomeKit client

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871908

Bug ID: 1871908
   Summary: Review Request: python-aiohomekit - Python HomeKit
client
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: m...@fabian-affolter.ch
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL: https://fab.fedorapeople.org/packages/SRPMS/python-aiohomekit.spec
SRPM URL:
https://fab.fedorapeople.org/packages/SRPMS/python-aiohomekit-0.2.49-1.fc32.src.rpm

Project URL: https://github.com/Jc2k/aiohomekit

Description:
This library implements the HomeKit protocol for controlling Homekit
accessories.

Koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=50070619

rpmlint output:
$ rpmlint python-aiohomekit-0.2.49-1.fc32.src.rpm 
python-aiohomekit.src:39: W: macro-in-comment %license
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

$ rpmlint python3-aiohomekit-0.2.49-1.fc32.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

Fedora Account System Username: fab


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871908] Review Request: python-aiohomekit - Python HomeKit client

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871908

Fabian Affolter  changed:

   What|Removed |Added

 Blocks||1269538 (IoT)
  Alias||python-aiohomekit
   Doc Type|--- |If docs needed, set a value





Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1269538
[Bug 1269538] Tracker for IoT on Fedora
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1867999] Review Request: perl-GeoIP2 - Perl API for MaxMind's GeoIP2 web services and databases

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1867999

Petr Pisar  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-GeoIP2-2.006002-1.fc34
 Resolution|--- |RAWHIDE
Last Closed||2020-08-24 14:10:55



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

I fixed both the issues. Although I'm curious how your build end up with
/usr/bin/env shebang if rpmbuild rewrites on build for some time.


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


[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1861839



--- Comment #9 from Remi Collet  ---
(In reply to Artur Iwicki from comment #8)
> Thanks for the review. Please assign the ticket to yourself (required per
> "fedpkg request-repo" procedures).

Sorry about this, done.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1861839

Remi Collet  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|fed...@famillecollet.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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1861839



--- Comment #8 from Artur Iwicki  ---
Thanks for the review. Please assign the ticket to yourself (required per
"fedpkg request-repo" procedures).


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871825] Review Request: python-ratelimiter - Python module providing rate limiting

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871825

Nasir Hussain  changed:

   What|Removed |Added

 CC||nasirhussain...@gmail.com



--- Comment #1 from Nasir Hussain  ---
Looks like a simple Py3 library, LGTM.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1867999] Review Request: perl-GeoIP2 - Perl API for MaxMind's GeoIP2 web services and databases

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1867999



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


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1867999] Review Request: perl-GeoIP2 - Perl API for MaxMind's GeoIP2 web services and databases

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1867999



--- Comment #2 from Petr Pisar  ---
https://pagure.io/releng/fedora-scm-requests/issue/27854


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871837] Review Request: golang-github-ubccr-kerby - Go wrapper for Kerberos GSSAPI

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871837

obu...@redhat.com changed:

   What|Removed |Added

Comment|0   |updated



--- Comment #0 has been edited ---

Spec URL:
https://download.copr.fedorainfracloud.org/results/obudai/kerby/fedora-rawhide-x86_64/01629888-golang-github-ubccr-kerby/golang-github-ubccr-kerby.spec
SRPM URL: golang-github-ubccr-kerby-0-0.1.20200824git201a958.fc34.src.rpm
Description: Go wrapper for Kerberos GSSAPI
Fedora Account System Username: obudai


Koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=50042441

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1867999] Review Request: perl-GeoIP2 - Perl API for MaxMind's GeoIP2 web services and databases

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1867999

Jitka Plesnikova  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



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

BuildRequires are almost ok
FIX: Please add perl(lib) - GeoIP2-2.006002/t/GeoIP2/Model/City.t:4

$ rpm -qp --requires perl-GeoIP2-2.006002-1.fc34.noarch.rpm | sort | uniq -c |
grep -v rpmlib
  1 perl(B)
  1 perl(Data::Dumper)
  1 perl(Data::Validate::IP) >= 0.25
  1 perl(Exporter)
  1 perl(GeoIP2::Error::Generic)
  1 perl(GeoIP2::Error::HTTP)
  1 perl(GeoIP2::Error::IPAddressNotFound)
  1 perl(GeoIP2::Error::Type)
  1 perl(GeoIP2::Error::WebService)
  1 perl(GeoIP2::Model::AnonymousIP)
  1 perl(GeoIP2::Model::ASN)
  1 perl(GeoIP2::Model::City)
  1 perl(GeoIP2::Model::ConnectionType)
  1 perl(GeoIP2::Model::Country)
  1 perl(GeoIP2::Model::Domain)
  1 perl(GeoIP2::Model::Enterprise)
  1 perl(GeoIP2::Model::Insights)
  1 perl(GeoIP2::Model::ISP)
  1 perl(GeoIP2::Record::City)
  1 perl(GeoIP2::Record::Continent)
  1 perl(GeoIP2::Record::Country)
  1 perl(GeoIP2::Record::Location)
  1 perl(GeoIP2::Record::MaxMind)
  1 perl(GeoIP2::Record::Postal)
  1 perl(GeoIP2::Record::RepresentedCountry)
  1 perl(GeoIP2::Record::Subdivision)
  1 perl(GeoIP2::Record::Traits)
  1 perl(GeoIP2::Types)
  1 perl(GeoIP2::WebService::Client)
  1 perl(Getopt::Long)
  1 perl(HTTP::Headers)
  1 perl(HTTP::Request)
  1 perl(JSON::MaybeXS)
  1 perl(List::SomeUtils)
  1 perl(List::Util)
  1 perl(LWP::Protocol::https)
  1 perl(LWP::UserAgent)
  1 perl(MaxMind::DB::Reader) >= 1.00
  1 perl(MIME::Base64)
  1 perl(:MODULE_COMPAT_5.32.0)
  1 perl(Moo)
  1 perl(Moo::Role)
  1 perl(namespace::clean)
  1 perl(Params::Validate)
  1 perl(Scalar::Util)
  1 perl(strict)
  1 perl(Sub::Quote)
  1 perl(Throwable::Error)
  1 perl(Try::Tiny)
  1 perl(URI)
  1 perl(:VERSION) >= 5.8.0
  1 perl(warnings)
  1 /usr/bin/env
Binary requires are Ok.

$ rpm -qp --provides perl-GeoIP2-2.006002-1.fc34.noarch.rpm | sort | uniq -c
  1 perl(GeoIP2) = 2.006002
  1 perl-GeoIP2 = 2.006002-1.fc34
  1 perl(GeoIP2::Database::Reader) = 2.006002
  1 perl(GeoIP2::Error::Generic) = 2.006002
  1 perl(GeoIP2::Error::HTTP) = 2.006002
  1 perl(GeoIP2::Error::IPAddressNotFound) = 2.006002
  1 perl(GeoIP2::Error::Type) = 2.006002
  1 perl(GeoIP2::Error::WebService) = 2.006002
  1 perl(GeoIP2::Model::AnonymousIP) = 2.006002
  1 perl(GeoIP2::Model::ASN) = 2.006002
  1 perl(GeoIP2::Model::City) = 2.006002
  1 perl(GeoIP2::Model::ConnectionType) = 2.006002
  1 perl(GeoIP2::Model::Country) = 2.006002
  1 perl(GeoIP2::Model::Domain) = 2.006002
  1 perl(GeoIP2::Model::Enterprise) = 2.006002
  1 perl(GeoIP2::Model::Insights) = 2.006002
  1 perl(GeoIP2::Model::ISP) = 2.006002
  1 perl(GeoIP2::Record::City) = 2.006002
  1 perl(GeoIP2::Record::Continent) = 2.006002
  1 perl(GeoIP2::Record::Country) = 2.006002
  1 perl(GeoIP2::Record::Location) = 2.006002
  1 perl(GeoIP2::Record::MaxMind) = 2.006002
  1 perl(GeoIP2::Record::Postal) = 2.006002
  1 perl(GeoIP2::Record::RepresentedCountry) = 2.006002
  1 perl(GeoIP2::Record::Subdivision) = 2.006002
  1 perl(GeoIP2::Record::Traits) = 2.006002
  1 perl(GeoIP2::Role::Error::HTTP) = 2.006002
  1 perl(GeoIP2::Role::HasIPAddress) = 2.006002
  1 perl(GeoIP2::Role::HasLocales) = 2.006002
  1 perl(GeoIP2::Role::Model) = 2.006002
  1 perl(GeoIP2::Role::Model::Flat) = 2.006002
  1 perl(GeoIP2::Role::Model::HasSubdivisions) = 2.006002
  1 perl(GeoIP2::Role::Model::Location) = 2.006002
  1 perl(GeoIP2::Role::Record::Country) = 2.006002
  1 perl(GeoIP2::Role::Record::HasNames) = 2.006002
  1 perl(GeoIP2::Types) = 2.006002
Binary provides are Ok.

$ rpmlint ./perl-GeoIP2*
perl-GeoIP2.noarch: E: env-script-interpreter
/usr/bin/geoip2-web-service-request /usr/bin/env perl
perl-GeoIP2.noarch: W: no-manual-page-for-binary geoip2-web-service-request
2 packages and 1 specfiles checked; 1 errors, 1 warnings.
FIX: Please update the script interpreter

Please correct all 'FIX' issues.
Otherwise the package looks good.

Resolution:
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: 

[Bug 1871837] New: Review Request: golang-github-ubccr-kerby - Go wrapper for Kerberos GSSAPI

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871837

Bug ID: 1871837
   Summary: Review Request: golang-github-ubccr-kerby - Go wrapper
for Kerberos GSSAPI
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: obu...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://download.copr.fedorainfracloud.org/results/obudai/kerby/fedora-rawhide-x86_64/01629888-golang-github-ubccr-kerby/golang-github-ubccr-kerby.spec
SRPM URL: golang-github-ubccr-kerby-0-0.1.20200824git201a958.fc34.src.rpm
Description: Go wrapper for Kerberos GSSAPI
Fedora Account System Username: obudai


Koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=50042441


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1869719] Review Request: rubygem-image_processing - High-level wrapper for processing images for the web with ImageMagick or libvips

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1869719



--- Comment #8 from Vít Ondruch  ---
(In reply to Pavel Valena from comment #6)
> > * `skip` vs `return`
> 
> To me, the benefit of `skip` is that I see "S" in the test output,
> so I know the tests are skipped actually. `return` would just hide the
> issue, right?

That is nice of course, but if the test case contains more asserts, the `skip`
won't execute the following asserts, while the `return` does. So what are
pros/cons? :)


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1821189] Package Review: Snakemake - Workflow management system to create reproducible and scalable data analyses

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1821189

Aniket Pradhan  changed:

   What|Removed |Added

 Depends On||1871824, 1871825





Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1871824
[Bug 1871824] Review Request: python-toposort - Implements a topological sort
algorithm
https://bugzilla.redhat.com/show_bug.cgi?id=1871825
[Bug 1871825] Review Request: python-ratelimiter - Python module providing rate
limiting
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871824] Review Request: python-toposort - Implements a topological sort algorithm

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871824

Aniket Pradhan  changed:

   What|Removed |Added

 Blocks||1821189





Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1821189
[Bug 1821189] Package Review: Snakemake - Workflow management system to create
reproducible and scalable data analyses
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871825] Review Request: python-ratelimiter - Python module providing rate limiting

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871825

Aniket Pradhan  changed:

   What|Removed |Added

 Blocks||1821189





Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1821189
[Bug 1821189] Package Review: Snakemake - Workflow management system to create
reproducible and scalable data analyses
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871824] Review Request: python-toposort - Implements a topological sort algorithm

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871824

Aniket Pradhan  changed:

   What|Removed |Added

 Blocks||1276941 (fedora-neuro)
   Doc Type|--- |If docs needed, set a value





Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1276941
[Bug 1276941] Fedora NeuroImaging and NeuroScience tracking 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871825] Review Request: python-ratelimiter - Python module providing rate limiting

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871825

Aniket Pradhan  changed:

   What|Removed |Added

 Blocks||1276941 (fedora-neuro)
   Doc Type|--- |If docs needed, set a value





Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1276941
[Bug 1276941] Fedora NeuroImaging and NeuroScience tracking 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871825] New: Review Request: python-ratelimiter - Python module providing rate limiting

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871825

Bug ID: 1871825
   Summary: Review Request: python-ratelimiter - Python module
providing rate limiting
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: aniketpradhan1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://major.fedorapeople.org/python-ratelimiter/python-ratelimiter.spec
SRPM URL:
https://major.fedorapeople.org/python-ratelimiter/python3-ratelimiter-1.2.0.post0-1.fc32.src.rpm

Description: This package provides the ratelimiter module, which ensures that
an operation will not be executed more than a given number of times on a given
period. This can prove useful when working with third parties APIs which
require for example a maximum of 10 requests per second.

Fedora Account System Username: major

Koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=50051239


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871824] New: Review Request: python-toposort - Implements a topological sort algorithm

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871824

Bug ID: 1871824
   Summary: Review Request: python-toposort - Implements a
topological sort algorithm
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: aniketpradhan1...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL: https://major.fedorapeople.org/python-toposort/python-toposort.spec
SRPM URL:
https://major.fedorapeople.org/python-toposort/python3-toposort-1.5-1.fc32.src.rpm

Description: In computer science, a topological sort (sometimes abbreviated
topsort or toposort) or topological ordering of a directed graph is a linear
ordering of its vertices such that for every directed edge uv from vertex u to
vertex v, u comes before v in the ordering.

Fedora Account System Username: major

Koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=50051232


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1824156] Review Request: exfatprogs - Userspace utilities for exFAT filesystems

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1824156



--- Comment #35 from Simone Caronni  ---
(In reply to Peter Robinson from comment #34)
> See https://pagure.io/Fedora-Council/tickets/issue/311 for details.
> Everything is still in motion, handled currently by Matthew and Ben

I get a 404 on that URL, but good to hear. 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871813] New: Review Request: python-aioesphomeapi - Library to interact with devices flashed with esphome

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871813

Bug ID: 1871813
   Summary: Review Request: python-aioesphomeapi - Library to
interact with devices flashed with esphome
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: m...@fabian-affolter.ch
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL: https://fab.fedorapeople.org/packages/SRPMS/python-aioesphomeapi.spec
SRPM URL:
https://fab.fedorapeople.org/packages/SRPMS/python-aioesphomeapi-2.6.2-1.fc32.src.rpm

Project URL: https://esphome.io/

Description:
aioesphomeapi allows you to interact with devices flashed with esphome.

Koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=50049193

rpmlint output:
$ rpmlint python-aioesphomeapi-2.6.2-1.fc32.src.rpm 
python-aioesphomeapi.src: W: spelling-error Summary(en_US) esphome -> esp home,
esp-home, homespun
python-aioesphomeapi.src: W: spelling-error %description -l en_US esphome ->
esp home, esp-home, homespun
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

$ rpmlint python3-aioesphomeapi-2.6.2-1.fc32.noarch.rpm
python3-aioesphomeapi.noarch: W: spelling-error Summary(en_US) esphome -> esp
home, esp-home, homespun
python3-aioesphomeapi.noarch: W: spelling-error %description -l en_US esphome
-> esp home, esp-home, homespun
1 packages and 0 specfiles checked; 0 errors, 2 warnings.


Fedora Account System Username: fab


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1824156] Review Request: exfatprogs - Userspace utilities for exFAT filesystems

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1824156



--- Comment #34 from Peter Robinson  ---
(In reply to Simone Caronni from comment #33)
> Just saw that Tom is no longer working for Red Hat.. does this change
> anything regarding the legal approval? He has been removed from CC in this
> ticket.
> 
> I would set a needinfo to some address but I wouldn't know who to...

See https://pagure.io/Fedora-Council/tickets/issue/311 for details. Everything
is still in motion, handled currently by Matthew and Ben


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1809405] Review Request: python-sumatra - Tool for managing and tracking projects based on numerical simulation and/or analysis, with the aim of supporting reproducible research

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1809405



--- Comment #24 from Ankur Sinha (FranciscoD)  ---
Thanks! I'll have a look this week. On my list now.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1856005] Review Request: dmtcp - Checkpoint/Restart functionality for Linux processes

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1856005



--- Comment #14 from Dave Love  ---
I don't know about Orion, wouldn't worry about a build time of half an hour
with tests.  I'm more worried about not being able to get the tests to pass if
I make sure they're not skipped (which they still are in the build).  One or
more fail in mock on el7 and f33, as well as with a straight build from the
tarball on centos7 and debian 10.  I don't have time to try to debug them.  Can
you say what's going wrong and how significant it is?

Koji examples:
https://koji.fedoraproject.org/koji/taskinfo?taskID=49885693 
https://koji.fedoraproject.org/koji/taskinfo?taskID=49893986


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871345] Review Request: jakarta-saaj - SOAP with Attachments API for Java

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871345

Fabio Valentini  changed:

   What|Removed |Added

 Status|POST|CLOSED
   Fixed In Version||jakarta-saaj-1.4.2-1.fc33
 Resolution|--- |NEXTRELEASE
Last Closed||2020-08-24 08:41:37



--- Comment #5 from Fabio Valentini  ---
Built for rawhide:
https://koji.fedoraproject.org/koji/buildinfo?buildID=1599021
And for fedora 33:
https://koji.fedoraproject.org/koji/buildinfo?buildID=1599022


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871350] Review Request: jakarta-xml-rpc - Jakarta XML RPC API

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871350
Bug 1871350 depends on bug 1871345, which changed state.

Bug 1871345 Summary: Review Request: jakarta-saaj - SOAP with Attachments API 
for Java
https://bugzilla.redhat.com/show_bug.cgi?id=1871345

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |NEXTRELEASE




-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871765] Review Request: thoth-toolbox - This is a toolbox container image for Thoth tools

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871765

Christoph Görn  changed:

   What|Removed |Added

Comment|0   |updated



--- Comment #0 has been edited ---

Spec URL: n/a, its just a container image
SRPM URL: n/a, its just a container image
Description: This is a toolbox container image aggregating some of Thoth tools,
including micropipenv and thamos
Fedora Account System Username: goern

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871765] New: Review Request: thoth-toolbox - This is a toolbox container image for Thoth tools

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871765

Bug ID: 1871765
   Summary: Review Request: thoth-toolbox - This is a toolbox
container image for Thoth tools
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: cgo...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL: n/a, its just a container image
SRPM URL: n/a, its just a container image
Description: This is a toolbox container image aggregating some of Thoth tools,
including micropipenv and thamos
Fedora Account System Username: goern


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1824156] Review Request: exfatprogs - Userspace utilities for exFAT filesystems

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1824156



--- Comment #33 from Simone Caronni  ---
Just saw that Tom is no longer working for Red Hat.. does this change anything
regarding the legal approval? He has been removed from CC in this ticket.

I would set a needinfo to some address but I wouldn't know who to...


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


[Bug 1867617] Review Request: perl-MaxMind-DB-Reader - Read MaxMind database files and look up IP addresses

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1867617
Bug 1867617 depends on bug 1867162, which changed state.

Bug 1867162 Summary: Review Request: perl-Math-Int128 - Manipulate 128 bits 
integers in Perl
https://bugzilla.redhat.com/show_bug.cgi?id=1867162

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |RAWHIDE




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


[Bug 1867967] Review Request: perl-MaxMind-DB-Reader-XS - Fast XS implementation of MaxMind DB reader

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1867967
Bug 1867967 depends on bug 1867162, which changed state.

Bug 1867162 Summary: Review Request: perl-Math-Int128 - Manipulate 128 bits 
integers in Perl
https://bugzilla.redhat.com/show_bug.cgi?id=1867162

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |RAWHIDE




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


[Bug 1867611] Review Request: perl-Net-Works - API for IP addresses and networks

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1867611
Bug 1867611 depends on bug 1867162, which changed state.

Bug 1867162 Summary: Review Request: perl-Math-Int128 - Manipulate 128 bits 
integers in Perl
https://bugzilla.redhat.com/show_bug.cgi?id=1867162

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |RAWHIDE




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


[Bug 1867162] Review Request: perl-Math-Int128 - Manipulate 128 bits integers in Perl

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1867162

Petr Pisar  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2020-08-24 07:26:26




-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1867611] Review Request: perl-Net-Works - API for IP addresses and networks

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1867611

Petr Pisar  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2020-08-24 07:26:07




-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1867617] Review Request: perl-MaxMind-DB-Reader - Read MaxMind database files and look up IP addresses

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1867617
Bug 1867617 depends on bug 1867611, which changed state.

Bug 1867611 Summary: Review Request: perl-Net-Works - API for IP addresses and 
networks
https://bugzilla.redhat.com/show_bug.cgi?id=1867611

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |RAWHIDE




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


[Bug 1857584] Review Request: python-serpent - Serialization based on ast.literal_eval

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1857584

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #3 from Fedora Update System  ---
FEDORA-2020-2387058a2d has been submitted as an update to Fedora 32.
https://bodhi.fedoraproject.org/updates/FEDORA-2020-2387058a2d


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1867967] Review Request: perl-MaxMind-DB-Reader-XS - Fast XS implementation of MaxMind DB reader

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1867967

Petr Pisar  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-MaxMind-DB-Reader-XS-1
   ||.08-1.fc34
 Resolution|--- |RAWHIDE
Last Closed||2020-08-24 07:09:43



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

The /usr/lib64/perl5/vendor_perl/auto/MaxMind/DB/Reader/XS/XS.so permissions
should be corrected with %{_fixperms} call. Moreover it's a fresh file
generated with a compiler. It should not inherit any bad mode. Also I cannot
reproduce it. I suspect you build the package with Mock tool that recently
suffered from a glibc bug meddling with mode checks everywhere. This
transitional bug should be fixed now.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1861839

Remi Collet  changed:

   What|Removed |Added

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



--- Comment #7 from Remi Collet  ---
[x]: Package complies to the Packaging Guidelines


See minor TIPS above, but no blocker


** 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1861839



--- Comment #6 from Remi Collet  ---
Created attachment 1712302
  --> https://bugzilla.redhat.com/attachment.cgi?id=1712302=edit
review.txt


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1861839



--- Comment #5 from Remi Collet  ---
> # Upstream repo does not contain a LICENSE file

It have been recently added
https://github.com/flack/UniversalFeedCreator/blob/master/LICENSE

Find to add it later, on next version upgrade, ad LGPL doesn't require it to be
present.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1861839



--- Comment #4 from Remi Collet  ---
> sed -e 's|/lib/|/|g' -i autoload.php

TIPS: generate the autoolader directly in lib (--output=lib/autoload.php) or
use --basedir option to avoid having this sed


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1081434] Review Request: ip2location - IP2location library

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1081434



--- Comment #44 from Peter Bieringer  ---
sorry: forgot to mention option c)

c) IP2Location

-> I will vote for c) instead of b) as this is the current definition also in
spec file


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


[Bug 1871717] New: Review Request: php-league-mime-type-detection - Mime-type detection for Flysystem

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871717

Bug ID: 1871717
   Summary: Review Request: php-league-mime-type-detection -
Mime-type detection for Flysystem
   Product: Fedora
   Version: rawhide
  Hardware: All
OS: Linux
Status: NEW
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: fed...@famillecollet.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
  Target Milestone: ---
Classification: Fedora



Spec URL:
https://git.remirepo.net/cgit/rpms/php/league/php-league-mime-type-detection.git/plain/php-league-mime-type-detection.spec?id=29d48958a8bbb33fe1fa334abadbcdb57346ad44
SRPM URL:
https://rpms.remirepo.net/SRPMS/php-league-mime-type-detection-1.4.0-1.remi.src.rpm
Description: 
This package supplies a generic mime-type detection interface with a finfo
based implementation.

Autoloader: /usr/share/php/League/MimeTypeDetection/autoload.php


Fedora Account System Username: remi


--

New dependency of league/flysystem 1.1


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


[Bug 1081434] Review Request: ip2location - IP2location library

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1081434



--- Comment #43 from Peter Bieringer  ---
If no maintainer will be found, I can act as (ipv6calc is supporting that
library anyhow)

btw: RPM build from current source works fine (beside some minor compiler
warnings, which I have already fixed in my fork):

wget https://github.com/chrislim2888/IP2Location-C-Library/archive/8.0.9.tar.gz
rpmbuild -ts 8.0.9.tar.gz
rpmbuild --rebuild IP2Location-8.0.9-5.el8.src.rpm

> in source included spec file is at least working


=> have I to apply for maintainer or can one trigger the workflow?

Beside that, the RPM name should be proper defined in advance:

a) suggestion from 2014: ip2location-c

b) current github project: IP2Location-C-Library

I would vote for b) in alignment to GeoIP-*


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1867999] Review Request: perl-GeoIP2 - Perl API for MaxMind's GeoIP2 web services and databases

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1867999
Bug 1867999 depends on bug 1867617, which changed state.

Bug 1867617 Summary: Review Request: perl-MaxMind-DB-Reader - Read MaxMind 
database files and look up IP addresses
https://bugzilla.redhat.com/show_bug.cgi?id=1867617

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE




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


[Bug 1867967] Review Request: perl-MaxMind-DB-Reader-XS - Fast XS implementation of MaxMind DB reader

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1867967
Bug 1867967 depends on bug 1867617, which changed state.

Bug 1867617 Summary: Review Request: perl-MaxMind-DB-Reader - Read MaxMind 
database files and look up IP addresses
https://bugzilla.redhat.com/show_bug.cgi?id=1867617

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE




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


[Bug 1867617] Review Request: perl-MaxMind-DB-Reader - Read MaxMind database files and look up IP addresses

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1867617

Petr Pisar  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-MaxMind-DB-Reader-1.00
   ||0014-1.fc34
 Resolution|--- |RAWHIDE
Last Closed||2020-08-24 06:11:24



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

I realized that excluding 32-bit architectures is not needed for this package
and reworked the conditions to be buildable everywhere.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1871345] Review Request: jakarta-saaj - SOAP with Attachments API for Java

2020-08-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1871345



--- Comment #4 from Igor Raits  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/jakarta-saaj


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this 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://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org