Bug 2187844 - Review Request: golang-github-hashicorp-lru-2 - Golang LRU cache
Summary: Review Request: golang-github-hashicorp-lru-2 - Golang LRU cache
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Mikel Olasagasti Uranga
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1930952 2187845
TreeView+ depends on / blocked
 
Reported: 2023-04-18 21:35 UTC by W. Michael Petullo
Modified: 2023-04-20 19:58 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2023-04-20 19:58:06 UTC
Type: ---
Embargoed:
mikel: fedora-review+


Attachments (Terms of Use)
The .spec file difference from Copr build 5803423 to 5807051 (1.05 KB, patch)
2023-04-19 22:45 UTC, Fedora Review Service
no flags Details | Diff

Description W. Michael Petullo 2023-04-18 21:35:21 UTC
User-Agent:       Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/112.0
Build Identifier: 

Spec URL: https://www.flyn.org/SRPMS/golang-github-hashicorp-lru-2.spec
SRPM URL: https://www.flyn.org/SRPMS/golang-github-hashicorp-lru-2-2.0.2-1.fc38.src.rpm

Description:
Golang LRU cache.

Fedora Account System Username: mikep

Reproducible: Always

Comment 1 Mikel Olasagasti Uranga 2023-04-19 21:57:28 UTC
Package structure is correct, but I'm not sure about if License is correct.

`simplelru/LICENSE_list` file suggest part of the code is BSD-3, for example simplelru/list.go.

That file is part of `%global golicenses`, but then `License: MPL-2.0` won't match those BSD files.

Can you manually add the other License that was not capture by go2rpm?

Comment 2 W. Michael Petullo 2023-04-19 22:40:06 UTC
Spec URL: https://www.flyn.org/SRPMS/golang-github-hashicorp-lru-2.spec
SRPM URL: https://www.flyn.org/SRPMS/golang-github-hashicorp-lru-2-2.0.2-1.fc38.src.rpm

I added this:

License: MPL-2.0 and BSD-3-Clause

Comment 3 Fedora Review Service 2023-04-19 22:45:34 UTC
Created attachment 1958429 [details]
The .spec file difference from Copr build 5803423 to 5807051

Comment 4 Mikel Olasagasti Uranga 2023-04-20 09:32:59 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.

Comment 5 Mikel Olasagasti Uranga 2023-04-20 09:35:13 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.

Comment 6 Fedora Admin user for bugzilla script actions 2023-04-20 12:59:30 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-hashicorp-lru-2

Comment 7 W. Michael Petullo 2023-04-20 13:01:55 UTC
https://pagure.io/releng/fedora-scm-requests/issue/52875

Comment 8 Fedora Update System 2023-04-20 19:57:14 UTC
FEDORA-2023-55344d729c has been submitted as an update to Fedora 39. https://bodhi.fedoraproject.org/updates/FEDORA-2023-55344d729c

Comment 9 Fedora Update System 2023-04-20 19:58:06 UTC
FEDORA-2023-55344d729c has been pushed to the Fedora 39 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.