https://bugzilla.redhat.com/show_bug.cgi?id=1178940
Bug ID: 1178940 Summary: Review Request: libqhyccd - Library to control QHY CCD devices Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: lupinix@mailbox.org QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org
Spec URL: https://lupinix.fedorapeople.org/review/libqhyccd/libqhyccd.spec SRPM URL: https://lupinix.fedorapeople.org/review/libqhyccd/libqhyccd-0.1-0.1.20150105... Description: Library to control QHY CCD devices Fedora Account System Username: lupinix
Known issues: * Firmware uses a nonfree license, I will add the review to legal tracker. License looks similar to linux-firmware licenses. We have to wait for legal approvement. * Most source files use wrong FSF address, reported upstream * No copy of GPLv2 in upstream repo, reported upstream
Thanks for review in advance!
https://bugzilla.redhat.com/show_bug.cgi?id=1178940
Christian Dersch lupinix@mailbox.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |182235 (FE-Legal)
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=182235 [Bug 182235] Fedora Legal Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1178940
Christian Dersch lupinix@mailbox.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |1159999 (Astronomy-SIG)
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1159999 [Bug 1159999] Astronomy SIG - review tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1178940
Tom "spot" Callaway tcallawa@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |tcallawa@redhat.com Blocks|182235 (FE-Legal) |
--- Comment #1 from Tom "spot" Callaway tcallawa@redhat.com --- Firmware license is okay. Lifting FE-Legal.
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=182235 [Bug 182235] Fedora Legal Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1178940
--- Comment #2 from Christian Dersch lupinix@mailbox.org --- Thank you for checking the license Tom!
Koji builds: Rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=8543609 F20: http://koji.fedoraproject.org/koji/taskinfo?taskID=8543622 F21: http://koji.fedoraproject.org/koji/taskinfo?taskID=8543616
Patch for library path submitted upstream: http://sourceforge.net/p/indi/bugs/58/
Missing GPLv2 copy and wrong FSF address: http://sourceforge.net/p/indi/bugs/59/
https://bugzilla.redhat.com/show_bug.cgi?id=1178940
Wolfgang Ulbrich chat-to-me@raveit.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |chat-to-me@raveit.de Assignee|nobody@fedoraproject.org |chat-to-me@raveit.de
https://bugzilla.redhat.com/show_bug.cgi?id=1178940
--- Comment #3 from Wolfgang Ulbrich chat-to-me@raveit.de --- there are some rpmlint warning which can be fixed.
libqhyccd.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libqhyccd.so.0 /lib64/libopencv_videostab.so.2.4 libqhyccd.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libqhyccd.so.0 /lib64/libopencv_ts.so.2.4 libqhyccd.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libqhyccd.so.0 /lib64/libopencv_superres.so.2.4 libqhyccd.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libqhyccd.so.0 /lib64/libopencv_stitching.so.2.4 libqhyccd.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libqhyccd.so.0 /lib64/libopencv_photo.so.2.4 libqhyccd.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libqhyccd.so.0 /lib64/libopencv_ocl.so.2.4 libqhyccd.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libqhyccd.so.0 /lib64/libopencv_legacy.so.2.4 libqhyccd.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libqhyccd.so.0 /lib64/libopencv_contrib.so.2.4 libqhyccd.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libqhyccd.so.0 /lib64/libopencv_calib3d.so.2.4 libqhyccd.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libqhyccd.so.0 /lib64/libopencv_video.so.2.4 libqhyccd.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libqhyccd.so.0 /lib64/libopencv_objdetect.so.2.4 libqhyccd.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libqhyccd.so.0 /lib64/libopencv_ml.so.2.4 libqhyccd.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libqhyccd.so.0 /lib64/libopencv_features2d.so.2.4
https://fedoraproject.org/wiki/Common_Rpmlint_issues#unused-direct-shlib-dep...
source macro doesn't match. Source0: %{name}-%{version}.svn%{svn}.tar.xz
libqhyccd.src: W: invalid-url Source0: libqhyccd-0.1.svn1920.tar.xz
If there isn't a valid link to svn source at sourceforge, pls upload it to your fedora people account and mentioned it in spec file.
I'm not shure how we handle the missing COPYING file. The rest looks good.
https://bugzilla.redhat.com/show_bug.cgi?id=1178940
--- Comment #4 from Wolfgang Ulbrich chat-to-me@raveit.de --- another one.
qhy-firmware.noarch: W: only-non-binary-in-usr-lib
can you move this to /usr/share ?
https://bugzilla.redhat.com/show_bug.cgi?id=1178940
--- Comment #5 from Wolfgang Ulbrich chat-to-me@raveit.de --- Forget about my last comment. Firmware and udev folders are under /usr/share :)
https://bugzilla.redhat.com/show_bug.cgi?id=1178940
--- Comment #6 from Wolfgang Ulbrich chat-to-me@raveit.de --- (In reply to Wolfgang Ulbrich from comment #5)
Forget about my last comment. Firmware and udev folders are under /usr/share :)
typo, i meant Firmware and udev folders are under /usr/lib
https://bugzilla.redhat.com/show_bug.cgi?id=1178940
Sergio Pascual sergio.pasra@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |sergio.pasra@gmail.com Flags| |fedora-review?
https://bugzilla.redhat.com/show_bug.cgi?id=1178940
Sergio Pascual sergio.pasra@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |
https://bugzilla.redhat.com/show_bug.cgi?id=1178940
--- Comment #7 from Sergio Pascual sergio.pasra@gmail.com --- Ups, sorry for the noise. Wolfgang, could you put fedora-review to "?" ?
https://bugzilla.redhat.com/show_bug.cgi?id=1178940
Wolfgang Ulbrich chat-to-me@raveit.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review?
https://bugzilla.redhat.com/show_bug.cgi?id=1178940
Christian Dersch lupinix@mailbox.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution|--- |WONTFIX Last Closed| |2015-02-04 08:37:28
--- Comment #8 from Christian Dersch lupinix@mailbox.org --- Closed for now, I think it is wise to wait some more time due to big changes upstream.
package-review@lists.fedoraproject.org