Bug 2122024 - Review Request: golang-github-cloudflare-circl - CIRCL: Cloudflare Interoperable Reusable Cryptographic Library
Summary: Review Request: golang-github-cloudflare-circl - CIRCL: Cloudflare Interopera...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Maxwell G
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 2122023
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-08-28 14:52 UTC by Mikel Olasagasti Uranga
Modified: 2022-09-08 21:42 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2022-09-08 21:42:35 UTC
Type: ---
Embargoed:
maxwell: fedora-review+


Attachments (Terms of Use)
Install missing license file (540 bytes, patch)
2022-09-05 18:26 UTC, Maxwell G
no flags Details | Diff

Description Mikel Olasagasti Uranga 2022-08-28 14:52:29 UTC
Spec URL: https://mikel.olasagasti.info/tmp/fedora/golang-github-cloudflare-circl.spec
SRPM URL: https://mikel.olasagasti.info/tmp/fedora/golang-github-cloudflare-circl-1.2.0-1.fc36.src.rpm
Description: CIRCL: Cloudflare Interoperable Reusable Cryptographic Library.
Fedora Account System Username: mikelo2

Comment 2 Maxwell G 2022-09-05 18:26:40 UTC
Created attachment 1909682 [details]
Install missing license file

There's a license file in the source tree that you didn't install. Please see the attached patch.

Comment 4 Maxwell G 2022-09-05 21:12:52 UTC
It looks like you forgot to upload the new specfile.

Comment 5 Mikel Olasagasti Uranga 2022-09-05 21:33:35 UTC
It's a cache problem, can you retry? I need to check my webserver to avoid caching specs.

Comment 6 Maxwell G 2022-09-05 21:57:45 UTC
(In reply to Mikel Olasagasti Uranga from comment #5)
> It's a cache problem, can you retry? I need to check my webserver to avoid
> caching specs.

Ah. Ctrl+Shift+R works.

Comment 7 Maxwell G 2022-09-05 22:10:22 UTC
I am not going to go through the whole fedora-review template, as this package uses go2rpm.

- [x] The specfile is sane.
- [x] License is correct
- [x] Builds successfully in mock
- [x] Package is installable (checked by fedora-review)
- [x] No relevant rpmlint errors
- [x] %check section passes
- [x] The latest version is packaged
- [x] `%goipath` is set correctly
- [-] Binaries don't conflict with binaries already in the distribution
- [x] The package complies with the Packaging Guidelines.

Package approved! On import, don't forget to do the following:

- [ ] Add package to release-monitoring.org
- [ ] Give go-sig privileges on package
- [ ] Close the review bug by referencing it in the rpm changelog and the Bodhi ticket.

If you have time to look at any of my review requests, that would be nice ;). See https://bugzilla.redhat.com/buglist.cgi?component=Package Review&email1=gotmax%40e.email&emailreporter1=1&emailtype1=substring&query_format=advanced&status=NEW

Thanks!

Comment 9 Gwyn Ciesla 2022-09-06 16:06:58 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-cloudflare-circl

Comment 10 Fedora Update System 2022-09-08 21:40:14 UTC
FEDORA-2022-a68b00e6fb has been submitted as an update to Fedora 38. https://bodhi.fedoraproject.org/updates/FEDORA-2022-a68b00e6fb

Comment 11 Fedora Update System 2022-09-08 21:42:35 UTC
FEDORA-2022-a68b00e6fb has been pushed to the Fedora 38 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.