https://bugzilla.redhat.com/show_bug.cgi?id=2143804
Fabio Alessandro Locati <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?([email protected]) | Doc Type|--- |If docs needed, set a value Assignee|[email protected] |[email protected] Status|NEW |ASSIGNED --- Comment #3 from Fabio Alessandro Locati <[email protected]> --- Hi James, Glad to see that you managed to package OliveTin. There are a couple of things to cleanup on the spec file, to start with: 1. Please rename the package to OliveTin (and also this Review Request) so that it's aligned with the upstream project name as for the known packages exception in go's Fedora policy 2. Drop the %patch since it is not used 3. Replace the `cp` and `chown` commands in the install section with `install` 4. Use the https://docs.fedoraproject.org/en-US/packaging-guidelines/Systemd/ policy for the systemd unit Best, Fale -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component https://bugzilla.redhat.com/show_bug.cgi?id=2143804 _______________________________________________ package-review mailing list -- [email protected] To unsubscribe send an email to [email protected] Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/[email protected] Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
