Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 2074463 - Review Request: golang-gopkg-op-logging-1 - Golang logging library
Summary: Review Request: golang-gopkg-op-logging-1 - Golang logging library
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:
Blocks: 2074467
TreeView+ depends on / blocked
 
Reported: 2022-04-12 09:29 UTC by Julien Rische
Modified: 2022-07-27 02:34 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-07-13 16:47:50 UTC
Type: ---
Embargoed:
maxwell: fedora-review+


Attachments (Terms of Use)

Description Julien Rische 2022-04-12 09:29:59 UTC
Spec URL: https://pagure.io/rpm-golang-gopkg-op-logging-1/raw/rawhide/f/golang-gopkg-op-logging.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/jrische/golang-gopkg-op-logging-1/fedora-rawhide-x86_64/04176893-golang-gopkg-op-logging-1/golang-gopkg-op-logging-1-1-1.fc37.src.rpm

Description:
Package logging implements a logging infrastructure for Go. Its output 
format is customizable and supports different logging backends like 
syslog, file and memory. Multiple backends can be utilized with 
different log levels per backend and logger.

Fedora Account System Username: jrische

Source RPM repo: https://pagure.io/rpm-golang-gopkg-op-logging-1

Copr: https://copr.fedorainfracloud.org/coprs/jrische/golang-gopkg-op-logging-1/

The source RPM includes 1 patch in order to fix a change of behavior of the runtime.FuncForPC() function.

Comment 2 Julien Rische 2022-06-29 14:59:43 UTC
Note that the version 2 of op-logging is already packaged[1], but is not compatible[2] with version 1. Version 1 is required by yq (bug 2074467).

[1] https://src.fedoraproject.org/rpms/golang-github-op-logging
[2] https://pkg.go.dev/github.com/op/go-logging#section-readme

Comment 3 Maxwell G 2022-07-04 22:33:51 UTC
Please regenerate this specfile with `go2rpm -f https://github.com/op/go-logging -v 1 gopkg.in/op/go-logging.v1`. You can look at `go2rpm --help` to see what those options actually do :).

Comment 5 Maxwell G 2022-07-11 01:10:24 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
- [-] The latest version is packaged
This is a compat package.
- [x] `%goipath` is set correctly
- [-] Binaries don't conflict with binaries already in the distribution
- [x] The package complies with the Packaging Guidelines.

Small nitpick:

If you'd like, you can also remove the numbers from Source0 and Patch0 (sed -e 's|Source0:|Source: |' -e 's|Patch0:|Patch: |' -i golang-gopkg-op-logging-1.spec) [1]. RPM has supported automatic source/patch file numbering since 4.15.x. The source numbers are not referenced anywhere in this specfile, so I don't see any good reason to keep them. I also submitted a PR to go2rpm to fix this.

[1]: The extra spaces are to keep the standard specfile indentation.

---

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

(I did the first three steps for you for your first two packages)

- [-] Add package to release-monitoring.org by doing.
Normally, you'd want to do this, but we can skip this step for this specific package, as it's a compat package which deliberately does not include the latest version.

Here are the steps for when you actually have to:

* Go to https://release-monitoring.org/ and login.
* Click add project in the navbar
* Fill in the project name. You can use the upstream name or the Fedora package name here.
* Fill in the homepage. This can be the project's website or just the Github page.
* Select Github as the backend (or whatever is appropriate).
* Select semantic as the Verison scheme.
* Fill in owner/project
* If it's a go project or another project that prefixes its tags with `v`, add that to Version prefix.
* Select Check latest release on submit
* Set the Distro to Fedora and put the name of the source package in Package.
* Click the Test check button and make sure everything looks correct before submitting.
- [ ] Add package to Koschei.
1. Go to https://koschei.fedoraproject.org and log in
2. Click on your username in the top right corner and click the `Add packages` button in the dropdown
3. Put the name of the source package (golang-gopkg-op-logging-1) in the names box and click `Add packages`.

The previous two tasks can be done any time, but the next two can only be done after the repo request goes through.
- [ ] Give go-sig privileges on package
1. Go to https://src.fedoraproject.org/rpms/golang-gopkg-op-logging-1/addgroup
2. Put `go-sig` in the group name field and give it `commit` access. You can also add me as an admin if you'd like, but you don't have to for this package.
- [ ] Close the review bug by referencing it in the rpm changelog and the Bodhi ticket.
This is the same thing you did for your other two packages.

Thanks!

Comment 6 Gwyn Ciesla 2022-07-13 13:43:13 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-gopkg-op-logging-1

Comment 7 Fedora Update System 2022-07-13 16:47:21 UTC
FEDORA-2022-f0cd65f7e6 has been submitted as an update to Fedora 37. https://bodhi.fedoraproject.org/updates/FEDORA-2022-f0cd65f7e6

Comment 8 Fedora Update System 2022-07-13 16:47:50 UTC
FEDORA-2022-f0cd65f7e6 has been pushed to the Fedora 37 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 9 Fedora Update System 2022-07-18 09:22:56 UTC
FEDORA-2022-be00d52700 has been submitted as an update to Fedora 36. https://bodhi.fedoraproject.org/updates/FEDORA-2022-be00d52700

Comment 10 Fedora Update System 2022-07-18 09:22:56 UTC
FEDORA-2022-deeebc98af has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2022-deeebc98af

Comment 11 Fedora Update System 2022-07-19 01:11:12 UTC
FEDORA-2022-be00d52700 has been pushed to the Fedora 36 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2022-be00d52700 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-be00d52700

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

Comment 12 Fedora Update System 2022-07-19 01:25:27 UTC
FEDORA-2022-deeebc98af has been pushed to the Fedora 35 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2022-deeebc98af \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-deeebc98af

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

Comment 13 Fedora Update System 2022-07-27 02:21:59 UTC
FEDORA-2022-be00d52700 has been pushed to the Fedora 36 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 14 Fedora Update System 2022-07-27 02:34:31 UTC
FEDORA-2022-deeebc98af has been pushed to the Fedora 35 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.