Bug 2010119 - Review Request: amfora - terminal browser for the Gemini protocol
Summary: Review Request: amfora - terminal browser for the Gemini protocol
Keywords:
Status: CLOSED DEFERRED
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-10-03 21:17 UTC by Adam Thiede
Modified: 2022-11-17 12:37 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-11-04 11:45:08 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Adam Thiede 2021-10-03 21:17:54 UTC
Spec URL: https://download.copr.fedorainfracloud.org/results/adamthiede/gemini/fedora-34-x86_64/02872416-amfora/amfora.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/adamthiede/gemini/fedora-34-x86_64/02872416-amfora/amfora-1.8.0-1.fc34.src.rpm
Description: Fancy terminal browser for Gemini (new Internet protocol, like gopher)
Fedora Account System Username: adamthiede

This is my first package and I will require a sponsor. I am not the upstream maintainer but have permission/endorsement to package this for Fedora. I welcome and and all corrections or suggestions.
A successful copr build lives here: https://copr.fedorainfracloud.org/coprs/adamthiede/gemini/build/2872416/

(minor edit, updated links to a slightly better build that had rawhide included.)

Comment 1 Artur Frenszek-Iwicki 2021-10-04 15:11:53 UTC
I know next to nothing about go packaging, but - this looks like a handwritten spec. Looking at other golang packages, most specs are based on the output of "go2rpm". Maybe try using that?

> %global debug_package %{nil}
This won't fly. We want debuginfo.
https://docs.fedoraproject.org/en-US/packaging-guidelines/Debuginfo/

> %if 0%{?suse_version}
> BuildRequires: go
> %endif
Specs must be Fedora/RHEL only. No conditionals for other distros are allowed.
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_spec_legibility

Comment 2 Adam Thiede 2021-10-05 00:35:31 UTC
Thank you for the advice and for taking a look at this. I will attempt to correct these.
I was also not aware of go2rpm. Is that required if this is just a piece of software built with go, and not a library to be imported by other go packages? There are several other pieces of go software that amfora imports that are not yet packaged. I assume I'd have to get those packaged first?
No matching package to install: 'golang(github.com/makeworld-the-better-one/go-gemini)'
No matching package to install: 'golang(github.com/makeworld-the-better-one/go-isemoji)'
No matching package to install: 'golang(github.com/mmcdole/gofeed)'
No matching package to install: 'golang(github.com/rkoesters/xdg/basedir)'
No matching package to install: 'golang(github.com/rkoesters/xdg/userdirs)'
No matching package to install: 'golang(gitlab.com/tslocum/cview)'

Comment 3 Artur Frenszek-Iwicki 2021-10-05 07:03:12 UTC
> I was also not aware of go2rpm. Is that required if this is just a piece of software built with go, and not a library to be imported by other go packages? 
You're free to write a spec by hand. It's just that looking at other package reviews, almost all golang packages are based on go2rpm's output.

> There are several other pieces of go software that amfora imports that are not yet packaged. I assume I'd have to get those packaged first?
Yes. All the dependencies need to be packaged and submitted for review as well.

Also, take a look at the Golang packaging guidelines: https://docs.fedoraproject.org/en-US/packaging-guidelines/Golang/

Comment 4 Adam Thiede 2021-10-05 13:27:20 UTC
Based on your suggestion and how other go packages look I'm going to try and do this properly. I've begun to build some of the dependencies.
The spec files are located at this repo https://github.com/elagost/golang-specfiles
They are mostly building in this copr https://copr.fedorainfracloud.org/coprs/adamthiede/golang-specfiles/packages/
golang-code-rocketnine-tslocum-cview and golang-code-rocketnine-tslocum-cbind are giving me some trouble with paths - their generated spec files don't build and the output of go2rpm says
> The forge provided is not known by go-rpm-macros. You will have to provide the source and archive parameters manually.
but I think I'll have it figured out soon.

Comment 5 Adam Thiede 2021-10-09 13:09:08 UTC
I got amfora successfully building.
SRPM: https://download.copr.fedorainfracloud.org/results/adamthiede/golang-specfiles/fedora-34-x86_64/02882552-golang-github-makeworld-the-better-one-amfora/golang-github-makeworld-the-better-one-amfora-1.8.0-1.fc34.src.rpm
Spec: https://download.copr.fedorainfracloud.org/results/adamthiede/golang-specfiles/fedora-34-x86_64/02882552-golang-github-makeworld-the-better-one-amfora/golang-github-makeworld-the-better-one-amfora.spec

The other dependencies are also building here. https://copr.fedorainfracloud.org/coprs/adamthiede/golang-specfiles/builds/
I had to self-build golang-github-schollz-progressbar since the version in F34 and F35 isn't up to date enough (the version in Rawhide is).

Still have yet to figure out the code-rocketnine dependencies, but since moving to the new git host the author hasn't made any updates, and I'd assume most packages still reference them at their former gitlab url, so I've built them from that as well.

Comment 6 Package Review 2022-10-10 00:45:17 UTC
This is an automatic check from review-stats script.

This review request ticket hasn't been updated for some time. We're sorry
it is taking so long. If you're still interested in packaging this software
into Fedora repositories, please respond to this comment clearing the
NEEDINFO flag.

You may want to update the specfile and the src.rpm to the latest version
available and to propose a review swap on Fedora devel mailing list to increase
chances to have your package reviewed. If this is your first package and you
need a sponsor, you may want to post some informal reviews. Read more at
https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group.

Without any reply, this request will shortly be considered abandoned
and will be closed.
Thank you for your patience.


Note You need to log in before you can comment on or make changes to this bug.