Bug 1736012 (golang-github-zmap-zcrypto)

Summary: Review Request: golang-github-zmap-zcrypto - Liberal Go TLS + X.509 Library for Research
Product: [Fedora] Fedora Reporter: Robert-André Mauchin 🐧 <eclipseo>
Component: Package ReviewAssignee: Hirotaka Wakabayashi <hiwkby>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: hiwkby, package-review
Target Milestone: ---Flags: hiwkby: 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: 2019-08-12 14:42:47 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 Robert-André Mauchin 🐧 2019-08-01 14:01:46 UTC
Spec URL: https://eclipseo.fedorapeople.org/for-review/golang-github-zmap-zcrypto.spec
SRPM URL: https://eclipseo.fedorapeople.org/for-review/golang-github-zmap-zcrypto-0-0.1.20190801git9051775.fc31.src.rpm

Description:
 Liberal Go TLS + X.509 Library for Research.

Fedora Account System Username: eclipseo

Comment 1 Hirotaka Wakabayashi 2019-08-11 05:52:22 UTC
Here is a successful Koji scratch build.
https://koji.fedoraproject.org/koji/taskinfo?taskID=36925497

Comment 2 Hirotaka Wakabayashi 2019-08-11 07:50:39 UTC
Hello Robert-André,

The LICENSE: field should be "ISC and MIT and ASL 2.0" because some files are
licensed under BSD-style license and MIT although some files are under ASL 2.0.
Please check the upstream LICENSE file. https://github.com/zmap/zcrypto/blob/master/LICENSE

Here are logs for reference.

1. rpmlint
```
$ rpmlint golang-github-zmap-zcrypto-0-0.1.20190801git9051775.fc31.src.rpm
golang-github-zmap-zcrypto.src: W: no-%build-section
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

$ rpmlint ~/rpmbuild/RPMS/noarch/golang-github-zmap-zcrypto-devel-0-0.1.20190801git9051775.fc31~bootstrap.noarch.rpm
golang-github-zmap-zcrypto-devel.noarch: W: hidden-file-or-dir /usr/share/gocode/src/github.com/zmap/zcrypto/.goipath
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
```

2. fedora-review
fedora-review fails because 'golang(github.com/zmap/rc2)' package still
doesn't exist. I manually installed that package and tested golang-github-zmap-zcrypto
package.
```
ERROR: Command failed:
 # /usr/bin/dnf builddep --installroot /var/lib/mock/fedora-rawhide-x86_64/root/ --releasever 31 --setopt=deltarpm=False --disableplugin=local --disableplug\
in=spacewalk /var/lib/mock/fedora-rawhide-x86_64/root//builddir/build/SRPMS/golang-github-zmap-zcrypto-0-0.1.20190801git9051775.fc31~bootstrap.src.rpm
No matches found for the following disable plugin patterns: local, spacewalk
fedora                                          4.9 kB/s | 6.2 kB     00:01
No matching package to install: 'golang(github.com/zmap/rc2)'
```

Thanks in advance,
Hirotaka Wakabayashi

Comment 3 Robert-André Mauchin 🐧 2019-08-11 15:55:31 UTC
Thanks, I hadn't checked that.

Comment 5 Robert-André Mauchin 🐧 2019-08-11 16:08:27 UTC
golang(github.com/zmap/rc2) should be in Rawhide by the next compose. I built it yesterday.

Comment 6 Hirotaka Wakabayashi 2019-08-11 22:59:32 UTC
(In reply to Robert-André Mauchin from comment #5)

Hello Robert-André,

Thanks for your fix. I confirmed it. This package meets requirements for
quality.

> golang(github.com/zmap/rc2) should be in Rawhide by the next compose. I
> built it yesterday.

Thank you for letting me know. I couldn't find that package in the nightly
Fedora-Rawhide-20190811.n.1. But I think it is not a problem because I have
manually reviewed golang-github-zmap-zcrypto.


The followings are logs for reference.

1. rpmlint
```
$ rpmlint golang-github-zmap-zcrypto-0-0.1.20190801git9051775.fc31.src.rpm
golang-github-zmap-zcrypto.src: W: no-%build-section
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
$ rpmlint ~/rpmbuild/RPMS/noarch/golang-github-zmap-zcrypto-devel-0-0.1.20190801git9051775.fc31~bootstrap.noarch.rpm
golang-github-zmap-zcrypto-devel.noarch: W: hidden-file-or-dir /usr/share/gocode/src/github.com/zmap/zcrypto/.goipath
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
```

2. a successful koji scratch build
https://koji.fedoraproject.org/koji/taskinfo?taskID=36942354

Best Regards,
Hirotaka Wakabayashi

Comment 7 Gwyn Ciesla 2019-08-12 13:24:15 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-zmap-zcrypto