Bug 1968641

Summary: Review Request: rust-hyper-rustls - Rustls+hyper integration for pure rust HTTPS
Product: [Fedora] Fedora Reporter: Fabio Valentini <decathorpe>
Component: Package ReviewAssignee: Robert-André Mauchin 🐧 <eclipseo>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: eclipseo, package-review
Target Milestone: ---Flags: eclipseo: fedora-review+
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2021-06-11 19:28:43 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1966648, 1968630, 1968633    
Bug Blocks: 1948875    

Description Fabio Valentini 2021-06-07 18:06:52 UTC
Spec URL: https://decathorpe.fedorapeople.org/rust-hyper-rustls.spec
SRPM URL: https://decathorpe.fedorapeople.org/rust-hyper-rustls-0.22.1-1.fc34.src.rpm

Description:
Rustls+hyper integration for pure rust HTTPS.

Fedora Account System Username: decathorpe

Comment 1 Robert-André Mauchin 🐧 2021-06-11 06:26:57 UTC
 -

# ring is not available on ppc64le and s390x
ExcludeArch:    ppc64le s390x

I wonder if it wouldn't be more practical to define a %{ring_arches} macros with its supported arches in that ring package and use it in all subsequent packages as an ExclusiveArch


 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved.

Comment 2 Fabio Valentini 2021-06-11 13:56:21 UTC
Thanks for the review!

> I wonder if it wouldn't be more practical to define a %{ring_arches} macros with its
> supported arches in that ring package and use it in all subsequent packages as an ExclusiveArch

I've been wondering that as well ... I can try to add that macro with the next rust-ring update.

https://pagure.io/releng/fedora-scm-requests/issue/34561
https://pagure.io/releng/fedora-scm-requests/issue/34562

Comment 3 Gwyn Ciesla 2021-06-11 14:03:03 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-hyper-rustls

Comment 4 Fedora Update System 2021-06-11 19:28:43 UTC
FEDORA-2021-f8f523b9f3 has been pushed to the Fedora 35 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 5 Fedora Update System 2021-06-11 19:29:20 UTC
FEDORA-2021-154cc48752 has been submitted as an update to Fedora 34. https://bodhi.fedoraproject.org/updates/FEDORA-2021-154cc48752

Comment 6 Fedora Update System 2021-06-12 01:57:21 UTC
FEDORA-2021-154cc48752 has been pushed to the Fedora 34 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2021-154cc48752`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2021-154cc48752

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

Comment 7 Fedora Update System 2021-06-20 01:07:20 UTC
FEDORA-2021-154cc48752 has been pushed to the Fedora 34 stable repository.
If problem still persists, please make note of it in this bug report.