https://bugzilla.redhat.com/show_bug.cgi?id=2183400 Maxwell G <maxwell@xxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |maxwell@xxxxxxx Doc Type|--- |If docs needed, set a value Assignee|nobody@xxxxxxxxxxxxxxxxx |maxwell@xxxxxxx Flags| |fedora-review? --- Comment #1 from Maxwell G <maxwell@xxxxxxx> --- I'll take this review :). Here are some initial comments: > %gometa -f Remove the `-f` flag and just use plain `%gometa`. The `-f` flag disables building/testing the package on i386 which we don't want in this case. We're only using this flag for new packages that don't have dependents. containerd is still built for i386. The package only contains a btrfs-test binary which isn't useful for end users, so I wouldn't ship it. You'll have to add ``` %global debug_package %{nil} ``` to the beginning of the specfile (right above %bcond_without check is a standard place, but it doesn't particularly matter) and make the changes listed below. > %build > for cmd in cmd/* ; do > %gobuild -o %{gobuilddir}/bin/$(basename $cmd) %{goipath}/$cmd > done Remove the entire %build section. We don't want to ship the btrfs-test binary. > > %install > %gopkginstall > install -m 0755 -vd %{buildroot}%{_bindir} > install -m 0755 -vp %{gobuilddir}/bin/* %{buildroot}%{_bindir}/ Remove the two install invocations. We're not installing any binaries. > %files > %license LICENSE > %doc README.md > %{_bindir}/* Remove this %files section. -- 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=2183400 _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx 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/package-review@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue