Bug 1601261 (rust-hyper-native-tls)

Summary:	Review Request: rust-hyper-native-tls - native-tls support for Hyper
Product:	[Fedora] Fedora	Reporter:	Igor Raits <igor.raits>
Component:	Package Review	Assignee:	Robert-André Mauchin 🐧 <eclipseo>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	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:	2018-07-20 16:24:24 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:

Description Igor Raits 2018-07-15 17:55:33 UTC

Spec URL: https://ignatenkobrain.fedorapeople.org/for-review/rust-hyper-native-tls.spec
SRPM URL: https://ignatenkobrain.fedorapeople.org/for-review/rust-hyper-native-tls-0.2.4-1.fc29.src.rpm
Description:
native-tls support for Hyper.
Fedora Account System Username: ignatenkobrain

Comment 1 Robert-André Mauchin 🐧 2018-07-17 22:40:37 UTC

DEBUG util.py:491:  Last metadata expiration check: 0:00:56 ago on mer. 18 juil. 2018 00:36:42 CEST.
DEBUG util.py:489:  BUILDSTDERR: No matching package to install: '(crate(native-tls) >= 0.1.2 with crate(native-tls) < 0.2.0)'
DEBUG util.py:489:  BUILDSTDERR: Not all dependencies satisfied
DEBUG util.py:489:  BUILDSTDERR: Error: Some packages could not be found.

rust-native-tls is 0.2.0-1 on f29. Shouldn't you patch the metadata?

Comment 2 Igor Raits 2018-07-18 06:03:53 UTC

(In reply to Robert-André Mauchin from comment #1)
> DEBUG util.py:491:  Last metadata expiration check: 0:00:56 ago on mer. 18
> juil. 2018 00:36:42 CEST.
> DEBUG util.py:489:  BUILDSTDERR: No matching package to install:
> '(crate(native-tls) >= 0.1.2 with crate(native-tls) < 0.2.0)'
> DEBUG util.py:489:  BUILDSTDERR: Not all dependencies satisfied
> DEBUG util.py:489:  BUILDSTDERR: Error: Some packages could not be found.
> 
> rust-native-tls is 0.2.0-1 on f29. Shouldn't you patch the metadata?

no, there is native-tls0.1 in f29-pending..

Comment 3 Robert-André Mauchin 🐧 2018-07-19 13:16:36 UTC

It doesn't build, some errors with the tests: see https://koji.fedoraproject.org/koji/taskinfo?taskID=28441536

Comment 4 Igor Raits 2018-07-19 13:22:09 UTC

(In reply to Robert-André Mauchin from comment #3)
> It doesn't build, some errors with the tests: see
> https://koji.fedoraproject.org/koji/taskinfo?taskID=28441536

I thought that I disabled tests because they require network.

Comment 5 Robert-André Mauchin 🐧 2018-07-19 15:00:24 UTC

With disabled tests:

 - Capitalize the summary.

rust-hyper-native-tls-devel.noarch: W: summary-not-capitalized C native-tls support for Hyper

 - Latest version packaged
 - License ok
 - Builds in Mock
 - Conforms to the Packaging Guidelines

Package approved.

Comment 6 Gwyn Ciesla 2018-07-20 13:34:34 UTC

(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-hyper-native-tls