You're not reviewing the full diff of
request 1164398
, but the diff to the superseded
request 1164374
(Show full diff)
Overview
Loading...
author
source maintainer
I wonder if it wouldn't even be better to just package goipp itself.
target maintainer
Richard Rahl, thank you for your efforts!
In general it is better to Keep Separated Items Separated - "KSIS" :-) cf. item (5) in RFC 1925 https://www.rfc-editor.org/rfc/rfc1925
In general it is better to have only one upstream source tarball in each package.
So when possible it is better to package goipp as a separated package.
goipp also belongs to the Printing project because it is about the IPP protocol.
Login required, please
login
in order to comment
Richard Rahl, FYI the ipp-usb source code security AUDIT review (cf. https://build.opensuse.org/request/show/1157901) is done. The overall result is that things are OK in general. Next week I will make an issue at its upstream GitHub https://github.com/OpenPrinting/ipp-usb about this review so that all are publicly informed and further things can be handled at upstream also together with other Linux distributors.
I forwarded the findings in our security audit https://bugzilla.opensuse.org/show_bug.cgi?id=1221671#c6 to upstream via https://github.com/OpenPrinting/ipp-usb/issues/78
@WernerFink, @dmolkentin, @jsmeix, @mwilck: review reminder
ipp-usb cannot build until I set goipp to be used for build in "Printing" which I did hereby
ipp-usb fails to build for SLE12 with
so I disabled building ipp-usb for SLE12 because goipp cannot be built for SLE12 see https://build.opensuse.org/request/show/1164399#comment-1928859 and in general I think it is hopeless there when "nothing provides golang-packaging" and SLE12 is no longer really important for new software - in contrast to SLE15 where some users might be interested.
Richard Rahl, could you test that ipp-usb works for you with your printer?
I do not have a printer that supports IPP over USB.
I already did that, when I packaged it. But will try again
yes, working on my end