Bug 2107888 - Review Request: vale - A syntax-aware, command-line linter for prose
Summary: Review Request: vale - A syntax-aware, command-line linter for prose
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
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: 2022-07-17 08:02 UTC by Marek Czernek
Modified: 2024-10-24 08:12 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2024-10-24 08:12:09 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Marek Czernek 2022-07-17 08:02:10 UTC
Spec URL: https://github.com/m-czernek/vale-spec/releases/download/v2.20.0/vale.spec
SRPM URL: https://github.com/m-czernek/vale-spec/releases/download/v2.20.0/vale-2.20.0-1.fc36.src.rpm

Description: I want to package the errata-ai/vale package (https://github.com/errata-ai/vale). Currently, Fedora users can download the RPM from my COPR repository (see official Vale documentation at https://vale.sh/docs/vale-cli/installation/) but I would love to provide this package as part of the standard Fedora repositories

Fedora Account System Username: mczernek

Comment 1 Mikel Olasagasti Uranga 2022-07-17 08:42:25 UTC
Hi Marek,

Fedora builds don't have access to Internet, so all the required libraries need to be part of Fedora.

For golang packages this means that the package is unbundled:

"At the moment golang projects packaged in Fedora SHOULD be unbundled by default. It means projects are built from dependencies packaged in Fedora."

https://docs.fedoraproject.org/en-US/packaging-guidelines/Golang/#_bundled_or_unbundled

I'll recommend you to use `go2rpm` to create a package and then work on top of that.

$ go2rpm -r github.com/errata-ai/vale

or in case you want to have static BuildRequires:

$ go2rpm -r --no-dynamic-buildrequires github.com/errata-ai/vale

Trying to build the spec generated by go2rpm, the build process reports the following dependencies missing:

No matching package to install: 'golang(github.com/errata-ai/ini)'
No matching package to install: 'golang(github.com/errata-ai/regexp2)'
No matching package to install: 'golang(github.com/jdkato/regexp)'
No matching package to install: 'golang(github.com/jdkato/titlecase)'
No matching package to install: 'golang(github.com/pterm/pterm)'

Those should be imported first to Fedora and then go with vale. 

The spec generated by go2rpm will need some tweaking, but it's a good start to detect missing dependencies.

Comment 2 Marek Czernek 2022-07-18 07:15:33 UTC
Thank you, for the detailed review,

I didn't realize that the COPR and the official Fedora build process are different (makes sense in retrospect).

I'll see what I can do with the missing dependencies. 

M.

Comment 3 Package Review 2023-09-24 00:45:29 UTC
This is an automatic check from review-stats script.

This review request ticket hasn't been updated for some time, but it seems
that the review is still being working out by you. If this is right, please
respond to this comment clearing the NEEDINFO flag and try to reach out the
submitter to proceed with the review.

If you're not interested in reviewing this ticket anymore, please clear the
fedora-review flag and reset the assignee, so that a new reviewer can take
this ticket.

Without any reply, this request will shortly be resetted.

Comment 4 Package Review 2024-09-24 00:45:22 UTC
This is an automatic check from review-stats script.

This review request ticket hasn't been updated for some time, but it seems
that the review is still being working out by you. If this is right, please
respond to this comment clearing the NEEDINFO flag and try to reach out the
submitter to proceed with the review.

If you're not interested in reviewing this ticket anymore, please clear the
fedora-review flag and reset the assignee, so that a new reviewer can take
this ticket.

Without any reply, this request will shortly be resetted.

Comment 5 Package Review 2024-10-24 00:45:29 UTC
This is an automatic action taken by review-stats script.

The ticket reviewer failed to clear the NEEDINFO flag in a month.
As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews
we reset the status and the assignee of this ticket.

Comment 6 Petr Pisar 2024-10-24 08:12:09 UTC
An account of the review submitter was close.


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