Bug 2245299 - Review Request: gpgme1.22 - GnuPG Made Easy - high level crypto API - version 1.22
Summary: Review Request: gpgme1.22 - GnuPG Made Easy - high level crypto API - version...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Neal Gompa
QA Contact: Fedora Extras Quality Assurance
URL: https://gnupg.org/related_software/gp...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-10-20 16:07 UTC by Troy Dawson
Modified: 2023-10-24 01:02 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2023-10-24 01:02:40 UTC
Type: ---
Embargoed:
ngompa13: fedora-review+


Attachments (Terms of Use)

Description Troy Dawson 2023-10-20 16:07:50 UTC
Spec URL: https://tdawson.fedorapeople.org/review/gpgme1.22.spec
SRPM URL: https://tdawson.fedorapeople.org/review/gpgme1.22-1.22.0-1.el9.src.rpm
Description: Fedoras gpgme version 1.22 for EPEL 9
Fedora Account System Username:tdawson

Comment 1 Troy Dawson 2023-10-20 17:11:11 UTC
Note:
This is for EPEL9 only.
This package is from the current Fedora Rawhide (F40) gpgme.  The changes are minimal. There was no change to the build. Only to the naming and what goes into packages.
There are no runtime conflicts with any other packages.
There are -devel conflicts with what is in RHEL 9.  The policy of conflicting -devel packages was discussed and approved by the EPEL Steering Committee.
I picked that version because it should be in RHEL 10.  Thus it should have a longer lifetime of support.

Comment 2 Fedora Review Service 2023-10-20 21:30:36 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6552057
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2245299-gpgme1.22/fedora-rawhide-x86_64/06552057-gpgme1.22/fedora-review/review.txt

Please take a look if any issues were found.

---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 3 Neal Gompa 2023-10-21 16:36:22 UTC
Review notes:

* Package follows Fedora Packaging Guidelines
* Package builds and installs
* Package licensing is correctly handled
* No serious issues from rpmlint

Nit: I would prefer to see the subpackage definitions for everything not being shipped to be fully erased. That makes it more obvious that some subpackages aren't being shipped (e.g. python3-gpg).

PACKAGE APPROVED.

Comment 4 Maxwell G 2023-10-21 21:07:52 UTC
> Nit: I would prefer to see the subpackage definitions for everything not being shipped to be fully erased. That makes it more obvious that some subpackages aren't being shipped (e.g. python3-gpg).

Can we build gpgme out of this specfile for python3.11 (the subpackage would be named python3.11-gpg instead)? This way, it wouldn't conflict with RHEL, but packages that need a new version could use that one and build for python3.11.

Comment 5 Neal Gompa 2023-10-21 21:29:43 UTC
I would be okay with that too.

Comment 6 Fedora Admin user for bugzilla script actions 2023-10-23 15:07:56 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/gpgme1.22

Comment 7 Troy Dawson 2023-10-23 15:09:58 UTC
I will update the spec to do the python3.11 mentioned, double check the build, and if it builds ok and doesn't conflict (which it shouldn't) I'll use that as the first build.

Comment 8 Fedora Update System 2023-10-23 16:53:44 UTC
FEDORA-EPEL-2023-5a7d70d01b has been submitted as an update to Fedora EPEL 9. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2023-5a7d70d01b

Comment 9 Troy Dawson 2023-10-23 16:55:54 UTC
It looks really good.  Thanks for the review.
I also noticed that I had turned off testing (so my test builds went faster).  I have turned testing back on, along with the python3.11 stuff.

Comment 10 Fedora Update System 2023-10-24 01:02:40 UTC
FEDORA-EPEL-2023-5a7d70d01b has been pushed to the Fedora EPEL 9 stable repository.
If problem still persists, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.