Product: Fedora
Version: rawhide
Component: Package Review

Dan Čermák <[email protected]> has canceled Package Review
<[email protected]>'s request for Dan Čermák
<[email protected]>'s needinfo:
Bug 2050552: Review Request: python-elasticsearch7 - Client for Elasticsearch
https://bugzilla.redhat.com/show_bug.cgi?id=2050552



--- Comment #10 from Dan Čermák <[email protected]> ---
(In reply to Steve Traylen from comment #7)
> Spec URL:
> https://cern.ch/straylen/rpms/python-elasticsearch7/python-elasticsearch7.
> spec
> SRPM URL:
> https://cern.ch/straylen/rpms/python-elasticsearch7/python-elasticsearch7-7.
> 17.9-1.el9.src.rpm
> 
> Switches to the %py3_build and _install macros - thanks.
> 
> As for doing elasticsearch8 this would be a seperate review. elasticsearch7
> and elasticsearch8 are distinct pypi names and
> can co-exist on EPEL9 for instance.

Makes sense!

Looks good now, thanks for the fixes. I'd suggest that you switch the license
identifier to SPDX though (`Apache-2.0`) instead of the Callway identifier.
Also upstream published two new minor releases in the mean time, feel free to
add them now too or upgrade it later.

Package is approved

-- 
_______________________________________________
package-review mailing list -- [email protected]
To unsubscribe send an email to [email protected]
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/[email protected]
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to