https://bugzilla.redhat.com/show_bug.cgi?id=2421450
Maxwell G <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|[email protected] |[email protected] Flags| |fedora-review+ Status|ASSIGNED |POST --- Comment #1 from Maxwell G <[email protected]> --- This package is APPROVED, but please make sure to read the special notes at the end of this review before proceeding with importing this package. Thank you! Vendored Go Package Review ============== Source: <https://git.sr.ht/~gotmax23/fedora-scripts/tree/main/item/go-sig/vendored_review_template.md> This package was generated using go2rpm and Go Vendor Tools, which simplifies the review. Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated - [x] The specfile is legible. - [x] The SRPM contains a valid `go-vendor-tools.toml`. - The default `detector = "askalono"` is included unless there is justification. - Manual license overrides are correct. - [x] The latest version is packaged or packaging an earlier version is justified. - [x] The License tag reflects the package contents and uses the correct identifiers. - [x] The package builds successfully in mock. - [x] Package is installable (checked by fedora-review). - [x] There are no relevant rpmlint errors. - [x] The package runs tests in %check. - [x] `%goipath` is set correctly. - [-] The package's binaries don't conflict with binaries already in the distribution. (Some Go projects include utility binaries with very generic names) (This is a package rename so n/a) - [x] There are no `%{_bindir}/*` wildcards in %files. - [x] The package does not use `%gometa -f` if it has dependents that still build for %ix86. - [x] The package complies with the rest of the Golang and general Packaging Guidelines. - [x] The Obsoletes and Provides are correct Package approved! On import, don't forget to do the following: - [ ] Add the package to release-monitoring.org. - [ ] Give go-sig privileges (at least commit) on the package. - [ ] Close the review bug by referencing its ID in the commit message and the Bodhi ticket. SPECIAL NOTES: - [ ] When importing this package, you should preserve the old history from golang-github-katalix-l2tp. When requesting the repo with `fedpkg request-repo`, make sure to pass `--no-initial-commit`. This way, you can just push the exact contents of the old repository to the new repository and then create a new commit on top for the package rename. Let me know if you have any questions about this. This step is important here, as the rpmautospec changelog and release number generation and the obsoletes that rely on the release numbering will get messed up without the history. - [ ] You should only need to build the new package in rawhide. The old golang libraries will not be removed from stable branches, so you can keep the old golang-github-katalix-l2tp there. - [ ] Retire the original golang-github-katalix-l2tp package with `fedpkg retire` on rawhide after this new package is imported, built, and pushed by Bodhi. Thanks! -- 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=2421450 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202421450%23c1 -- _______________________________________________ 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
