Bug 2265338 - Review Request: goarrange - Automatic arrangement of Go source code
Summary: Review Request: goarrange - Automatic arrangement of Go source code
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mikel Olasagasti Uranga
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2024-02-21 15:13 UTC by Arthur Bols
Modified: 2024-03-26 02:01 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2024-03-17 11:17:41 UTC
Type: ---
Embargoed:
mikel: fedora-review+


Attachments (Terms of Use)
The .spec file difference from Copr build 7046001 to 7131963 (713 bytes, patch)
2024-03-10 17:21 UTC, Fedora Review Service
no flags Details | Diff
The .spec file difference from Copr build 7131963 to 7132078 (240 bytes, patch)
2024-03-10 20:46 UTC, Fedora Review Service
no flags Details | Diff

Description Arthur Bols 2024-02-21 15:13:20 UTC
Spec URL: https://principis.fedorapeople.org/goarrange.spec
SRPM URL: https://principis.fedorapeople.org/goarrange-1.0.0-1.fc39.src.rpm
Description: Automatic arrangement of Go source code 
Fedora Account System Username: principis

Comment 1 Fedora Review Service 2024-02-21 15:17:01 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/7046001
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2265338-goarrange/fedora-rawhide-x86_64/07046001-goarrange/fedora-review/review.txt

Please take a look if any issues were found.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 2 Mikel Olasagasti Uranga 2024-03-09 15:43:35 UTC
Can you update go2rpm to 1.11.0? There a couple of changes in the template.

> %global goname goarrange

This is not required anymore, newer go2rpm does it correctly.

> %gopkginstall
> (...)
> %gopkgfiles

These are not required if you don't need the `golang-github-jdeflander-goarrange-devel` package. Remove these to create only the binary package.

Comment 3 Arthur Bols 2024-03-10 17:17:05 UTC
Thanks, recreated the spec with go2rpm 1.11.0.

Spec URL: https://principis.fedorapeople.org/goarrange.spec
SRPM URL: https://principis.fedorapeople.org/goarrange-1.0.0-1.fc39.src.rpm

Comment 4 Fedora Review Service 2024-03-10 17:21:47 UTC
Created attachment 2020993 [details]
The .spec file difference from Copr build 7046001 to 7131963

Comment 5 Fedora Review Service 2024-03-10 17:21:50 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/7131963
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2265338-goarrange/fedora-rawhide-x86_64/07131963-goarrange/fedora-review/review.txt

Please take a look if any issues were found.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 6 Arthur Bols 2024-03-10 20:42:07 UTC
- Removed glob under shared directory

Spec URL: https://principis.fedorapeople.org/goarrange.spec
SRPM URL: https://principis.fedorapeople.org/goarrange-1.0.0-1.fc39.src.rpm

Comment 7 Fedora Review Service 2024-03-10 20:46:58 UTC
Created attachment 2021045 [details]
The .spec file difference from Copr build 7131963 to 7132078

Comment 8 Fedora Review Service 2024-03-10 20:47:00 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/7132078
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2265338-goarrange/fedora-rawhide-x86_64/07132078-goarrange/fedora-review/review.txt

Please take a look if any issues were found.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 9 Mikel Olasagasti Uranga 2024-03-16 16:43:35 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
- [x] 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 10 Fedora Admin user for bugzilla script actions 2024-03-17 10:59:49 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/goarrange

Comment 11 Arthur Bols 2024-03-17 11:00:39 UTC
Thank you for the review!

Comment 12 Fedora Update System 2024-03-17 11:14:39 UTC
FEDORA-2024-51a86268c6 (goarrange-1.0.0-1.fc41) has been submitted as an update to Fedora 41.
https://bodhi.fedoraproject.org/updates/FEDORA-2024-51a86268c6

Comment 13 Fedora Update System 2024-03-17 11:17:41 UTC
FEDORA-2024-51a86268c6 (goarrange-1.0.0-1.fc41) has been pushed to the Fedora 41 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 14 Fedora Update System 2024-03-17 20:40:01 UTC
FEDORA-2024-89d69f6080 (goarrange-1.0.0-1.fc40) has been submitted as an update to Fedora 40.
https://bodhi.fedoraproject.org/updates/FEDORA-2024-89d69f6080

Comment 15 Fedora Update System 2024-03-17 20:40:22 UTC
FEDORA-2024-fc83f16f2d (goarrange-1.0.0-1.fc39) has been submitted as an update to Fedora 39.
https://bodhi.fedoraproject.org/updates/FEDORA-2024-fc83f16f2d

Comment 16 Fedora Update System 2024-03-18 01:15:55 UTC
FEDORA-2024-fc83f16f2d has been pushed to the Fedora 39 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2024-fc83f16f2d \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2024-fc83f16f2d

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

Comment 17 Fedora Update System 2024-03-18 01:24:43 UTC
FEDORA-2024-89d69f6080 has been pushed to the Fedora 40 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2024-89d69f6080 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2024-89d69f6080

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

Comment 18 Fedora Update System 2024-03-23 00:39:44 UTC
FEDORA-2024-89d69f6080 (goarrange-1.0.0-1.fc40) has been pushed to the Fedora 40 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 19 Fedora Update System 2024-03-26 02:01:33 UTC
FEDORA-2024-fc83f16f2d (goarrange-1.0.0-1.fc39) 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.