Fedora Account System
Red Hat Associate
Red Hat Customer
Spec URL: https://zdohnal.fedorapeople.org/ipp-usb/golang-github-openprinting-ipp-usb.spec SRPM URL: https://zdohnal.fedorapeople.org/ipp-usb/golang-github-openprinting-ipp-usb-0.9.13-1.fc32.src.rpm Description: HTTP reverse proxy, backed by IPP-over-USB connection to device Fedora Account System Username: zdohnal
- Please do not gzip the manpages yourself, the compression is handled by rpm # for compressing man page BuildRequires: gzip […] %{_bindir}/gzip <ipp-usb.8 > %{buildroot}%{_mandir}/man8/ipp-usb.8.gz - Please glob the man page extension as the compression may change in the future: %{_mandir}/man8/ipp-usb.8.* - You install a systemd service, please add the required scriptlets: BuildRequires: systemd-rpm-macros […] %post %systemd_post ipp-usb.service %preun %systemd_preun ipp-usb.service %postun %systemd_postun_with_restart ipp-usb.service - You also install the library files (%gopkg/%gopkginstall/%gopkgfiles). If this library is used by another project to build stuff, would they also need pkgconfig(libusb-1.0) >= 1.0 and pkgconfig(avahi-client) >= 0.7 to compile? If yes, please add a %godevelheader with those Requires: %global godevelheader %{expand: Requires: pkgconfig(avahi-client) >= 0.7 Requires: pkgconfig(libusb-1.0) >= 1.0 } - Shouldn't this config file be marked as %config(noreplace) %config(noreplace) %{_sysconfdir}/ipp-usb/ipp-usb.conf - Own %{_sysconfdir}/ipp-usb/ %dir %{_sysconfdir}/ipp-usb/ - Your summary is too long (over 80 characters). Consider switch it with the the description which is too short. Don't repeat the package name in the Summary. - Summary of the issues detected by fedora-review: Issues: ======= - systemd_post is invoked in %post, systemd_preun in %preun, and systemd_postun in %postun for Systemd service files. Note: Systemd service file(s) in ipp-usb See: https://docs.fedoraproject.org/en-US/packaging- guidelines/Scriptlets/#_scriptlets [!]: Package requires other packages for directories it uses. Note: No known owner of /etc/ipp-usb Rpmlint ------- Checking: ipp-usb-0.9.13-1.fc34.x86_64.rpm golang-github-openprinting-ipp-usb-devel-0.9.13-1.fc34.noarch.rpm golang-github-openprinting-ipp-usb-debugsource-0.9.13-1.fc34.x86_64.rpm golang-github-openprinting-ipp-usb-0.9.13-1.fc34.src.rpm ipp-usb.x86_64: E: summary-too-long C Ipp-usb is an application implementing IPP protocol for USB printers, which are capable of IPP-over-USB connection ipp-usb.x86_64: W: name-repeated-in-summary C Ipp-usb ipp-usb.x86_64: W: description-shorter-than-summary ipp-usb.x86_64: W: non-conffile-in-etc /etc/ipp-usb/ipp-usb.conf golang-github-openprinting-ipp-usb-devel.noarch: E: summary-too-long C Ipp-usb is an application implementing IPP protocol for USB printers, which are capable of IPP-over-USB connection golang-github-openprinting-ipp-usb-devel.noarch: W: hidden-file-or-dir /usr/share/gocode/src/github.com/OpenPrinting/ipp-usb/.goipath golang-github-openprinting-ipp-usb-debugsource.x86_64: E: description-line-too-long C This package provides debug sources for package golang-github-openprinting-ipp-usb. 4 packages and 0 specfiles checked; 3 errors, 4 warnings. - License ok - Latest version packaged - Builds in mock - Conforms to Packaging Guidelines
Hi Robert-André, thank you for the review and notes! I would like to ask how should I fix the following warning and error: golang-github-openprinting-ipp-usb-devel.noarch: W: hidden-file-or-dir /usr/share/gocode/src/github.com/OpenPrinting/ipp-usb/.goipath golang-github-openprinting-ipp-usb-debugsource.x86_64: E: description-line-too-long C This package provides debug sources for package golang-github-openprinting-ipp-usb. Both of them seem to be generated during building the package.
And the fixed spec and srpm (without two issue mentioned above) are uploaded and ready for review.
> golang-github-openprinting-ipp-usb-devel.noarch: W: hidden-file-or-dir /usr/share/gocode/src/github.com/OpenPrinting/ipp-usb/.goipath Not a problem, it is a sideeffect of Go packaging macros. > golang-github-openprinting-ipp-usb-debugsource.x86_64: E: description-line-too-long C This package provides debug sources for package golang-github-openprinting-ipp-usb. You can't technically fix it cause these debug packages are auto generated. LGTM, package approved.
(fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-openprinting-ipp-usb
Built for rawhide for now, updates for F33 and F32 is coming after golang-github-openprinting-goipp will get into repos.
Built in rawhide for now https://koji.fedoraproject.org/koji/buildinfo?buildID=1608039 .