Spec URL: https://people.redhat.com/pgier/reviews/golang-github-mdlayher-netlink/golang-github-mdlayher-netlink.spec SRPM URL: https://people.redhat.com/pgier/reviews/golang-github-mdlayher-netlink/golang-github-mdlayher-netlink-0-1.git82077d5.fc27.src.rpm Description: Package netlink provides low-level access to Linux netlink sockets Fedora Account System Username: pgier Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=24161025
- In case of git snapshot, you need to add the commit date to the Release field: %global commit 82077d593cd599c082317ceb5cfb22ee3fbc75ea %global shortcommit %(c=%{commit}; echo ${c:0:7}) %global commitdate 20170915 Name: golang-%{provider}-%{project}-%{repo} Version: 0 Release: 1.%{commitdate}git%{shortcommit}%{?dist} - Packaging the latest git commit would be nice. - Again, why did you comment out all BR and R?
If you're ok with only packaging for F28 and later, you could switch to the new Go packaging like your other package.
Updated spec file to follow new packaging recommendations. New spec sources and srpm are here: https://pgier.fedorapeople.org/golang/golang-github-mdlayher-netlink/ Koji scratch build here: https://koji.fedoraproject.org/koji/taskinfo?taskID=26843056
New Koji scratch build to fix diff between arches: https://koji.fedoraproject.org/koji/taskinfo?taskID=26857433
Not ok: # force these to be installed to avoid diffs between arches GOSRCPATH=/usr/share/gocode/src/github.com/mdlayher/netlink install sockopt_linux.go %{buildroot}${GOSRCPATH}/sockopt_linux.go install sockopt_linux_386.go %{buildroot}${GOSRCPATH}/sockopt_linux_386.go install sockopt_linux_386.s %{buildroot}${GOSRCPATH}/sockopt_linux_386.s The package should not be noarch ih this case: It may be tempting to make the header library package noarch, since the header files themselves are simply text. However, a library should have tests which should be run on all architectures. Also, the install process may modify the installed headers depending on the build architecture. For these reasons, header-only packages must not be marked noarch. https://fedoraproject.org/wiki/Packaging:Guidelines#Do_not_use_noarch
- Release should start at 0.1 for snapshot: Release: 0.1%{?dist} Also fix it in the %changelog entry.
Is there any reasons to disable the checks?
Thanks! I have made the requested changes, and a new build is here: https://koji.fedoraproject.org/koji/taskinfo?taskID=26861566 I had disabled the tests because one of the was failing locally using upstream sources. I created a patch to skip the failing test and re-enabled the checks in the specfile. Is it correct to specify the arches using this? ExclusiveArch: %{go_arches} I also had to exclude the s390x arch due to failing tests.
(In reply to Paul Gier from comment #8) > Thanks! I have made the requested changes, and a new build is here: > https://koji.fedoraproject.org/koji/taskinfo?taskID=26861566 > > I had disabled the tests because one of the was failing locally using > upstream sources. I created a patch to skip the failing test and re-enabled > the checks in the specfile. > > Is it correct to specify the arches using this? > ExclusiveArch: %{go_arches} > Not needed, %gometa already takes care of that. > I also had to exclude the s390x arch due to failing tests. Report the failing tests upstream. Also per the guidelines, you'll need to create a bugreport for this, see https://fedoraproject.org/wiki/Packaging:Guidelines#Architecture_Build_Failures - Reduce the Summary size and remove the dot at the end: golang-github-mdlayher-netlink-devel.x86_64: W: summary-ended-with-dot C Package netlink provides low-level access to Linux netlink sockets. MIT Licensed. golang-github-mdlayher-netlink-devel.x86_64: E: summary-too-long C Package netlink provides low-level access to Linux netlink sockets. MIT Licensed. - Split the description to stay below 80 characters per line: golang-github-mdlayher-netlink-devel.x86_64: E: description-line-too-long C Package netlink provides low-level access to Linux netlink sockets. MIT Licensed. - Latest version packaged - License ok - Builds in Mock - Conforms to the Packaging Guidelines Package approved.
I made the requested changed, reported the failing tests upstream, and was able to fix one of the tests so it no longer needs to be patched out. Also re-enabled the build (but skipped tests) on s390x. https://koji.fedoraproject.org/koji/taskinfo?taskID=26883450
(fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-mdlayher-netlink
Pagure repo has been created: https://pagure.io/releng/fedora-scm-requests/issue/6509 Build is complete: https://koji.fedoraproject.org/koji/taskinfo?taskID=26895873
Please close your review requests after you have imported and built the package.