Bug 719324 (libkdcraw) - Review Request: libkdcraw - A C++ interface around LibRaw library
Summary: Review Request: libkdcraw - A C++ interface around LibRaw library
Keywords:
Status: CLOSED RAWHIDE
Alias: libkdcraw
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: nucleo
QA Contact: nucleo
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-07-06 13:50 UTC by Rex Dieter
Modified: 2011-12-08 18:19 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-07-12 17:07:54 UTC
Type: ---
alekcejk: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Rex Dieter 2011-07-06 13:50:31 UTC
Spec URL: http://rdieter.fedorapeople.org/rpms/kdegraphics/libkdcraw.spec
SRPM URL: http://rdieter.fedorapeople.org/rpms/kdegraphics/libkdcraw-4.6.90-1.fc15.src.rpm
Description: A C++ interface around LibRaw library

Comment 1 Rex Dieter 2011-07-10 14:48:50 UTC
Spec URL: http://rdieter.fedorapeople.org/rpms/kdegraphics/libkdcraw.spec
SRPM URL:
http://rdieter.fedorapeople.org/rpms/kdegraphics/libkdcraw-4.6.90-3.fc15.src.rpm

%changelog
* Sun Jul 10 2011 Rex Dieter <rdieter> 4.6.90-3
- License: GPLv2+
- %%doc: +ChangeLog NEWS
- add %%check section

* Wed Jul 06 2011 Rex Dieter <rdieter> 4.6.90-2
- fix URL
- Conflicts: kdegraphics < 7:4.6.90-10

Comment 2 nucleo 2011-07-10 19:08:08 UTC
rpmlint output for libkdcraw-4.6.90-3 rpms:

libkdcraw.i686: E: description-line-too-long C picture files. More information about LibRaw can be found at http://www.libraw.org.
libkdcraw.src: E: description-line-too-long C picture files. More information about LibRaw can be found at http://www.libraw.org.
libkdcraw.x86_64: E: description-line-too-long C picture files. More information about LibRaw can be found at http://www.libraw.org.
libkdcraw-devel.i686: W: no-documentation
libkdcraw-devel.x86_64: W: no-documentation
7 packages and 0 specfiles checked; 3 errors, 2 warnings.

So description-line-too-long should be fixed.

update-desktop-database in %posttrans not needed because no desktop files installed.

libkdcraw license is GPLv2+ but it built using libRaw sources which have different licenses.
libRaw have license LGPLv2 or CDDL. It uses demosaic-pack-GPL2 and demosaic-pack-GPL3 sources.
There is notes about demosaic-packs license in README.demosaic-packs:

Unfortunately, some of these methods are distributed under the terms of 
different versions of GNU General Public License (GPL). So it is not possible
to include implementation of these demosaics into the LibRaw distribution 
because LibRaw is distributed under more liberal licenses (LGPL and CDDL).

Of course, it is possible to distribute these demosaic methods in separate 
packages and use within LibRaw under following conditions:

 * LibRaw is used under the terms of LGPL license which is GPL-compatible
 * Resulting product (which uses LibRaw AND LibRaw-demosaic-pack) is licensed
   under GPL2+ (for GPL2 demosaic-pack) or GPL3+ (if demosaic-pack-GPL3 is used).



So I guess that resulting License tag for libkdkraw should be GPLv2+ and GPLv3+?

This README also can be added in %doc.

Comment 3 nucleo 2011-07-10 19:26:25 UTC
Or maybe should be License: GPLv3+?

Comment 4 Rex Dieter 2011-07-10 22:49:36 UTC
A bit of a mess, wrt licening and bundled code, I'll look into using system-libraw.

Comment 5 nucleo 2011-07-10 23:01:28 UTC
There is no system-wide shared libRaw library if you mean this package
http://koji.fedoraproject.org/koji/packageinfo?packageID=10605

Only static lib available there libraw.a which version older than in libkdcraw tarball.
So using libRaw-devel not makes sense because even if libRaw-devel will be updated libkdcraw will use previous version.

Comment 6 nucleo 2011-07-10 23:41:20 UTC
Here explanation why libraw included in libkdcraw
http://mail.kde.org/pipermail/digikam-devel/2010-March/041682.html

> So, due the experimental stage of libraw for the moment, and until the
> implementation and the API will be stabilized, i think that including
> libraw as well is better.

Comment 7 Rex Dieter 2011-07-11 00:49:11 UTC
Ah thanks! 

ok, I'll rework this assuming bundling then.

Comment 8 nucleo 2011-07-11 01:17:34 UTC
It is will be better to change URL to https://projects.kde.org/projects/kde/kdegraphics/libs/libkdcraw

Comment 9 Rex Dieter 2011-07-11 13:48:43 UTC
Spec URL: http://rdieter.fedorapeople.org/rpms/kdegraphics/libkdcraw.spec
SRPM URL:
http://rdieter.fedorapeople.org/rpms/kdegraphics/libkdcraw-4.6.90-5.fc15.src.rpm

%changelog
* Mon Jul 11 2011 Rex Dieter <rdieter> 4.6.90-5
- License: GPLv2+ and LGPLv2 and GPLv3+
- Provides: bundled(LibRaw)

* Sun Jul 10 2011 Rex Dieter <rdieter> 4.6.90-4
- fix URL
- fix scriptlets

Comment 10 nucleo 2011-07-11 14:29:40 UTC
rpmlint output for ibkdcraw-4.6.90-5 rpms:

libkdcraw.i686: W: spurious-executable-perm /usr/share/doc/libkdcraw-4.6.90/LICENSE.LGPL
libkdcraw.i686: E: incorrect-fsf-address /usr/share/doc/libkdcraw-4.6.90/LICENSE.LGPL
libkdcraw.src:25: W: unversioned-explicit-provides bundled(LibRaw)
libkdcraw.src:27: W: unversioned-explicit-provides bundled(LibRaw-demosaic-pack-GPL2)
libkdcraw.src:28: W: unversioned-explicit-provides bundled(LibRaw-demosaic-pack-GPL3)
libkdcraw.x86_64: W: spurious-executable-perm /usr/share/doc/libkdcraw-4.6.90/LICENSE.LGPL
libkdcraw.x86_64: E: incorrect-fsf-address /usr/share/doc/libkdcraw-4.6.90/LICENSE.LGPL
libkdcraw-devel.i686: W: no-documentation
libkdcraw-devel.x86_64: W: no-documentation
7 packages and 0 specfiles checked; 2 errors, 7 warnings.

Comment 11 nucleo 2011-07-11 14:35:25 UTC
rpmlint errors not serious and can be fixed later.

Package and spec names are fine, 
description is fine now,
license GPLv2+ and LGPLv2 and GPLv3+ is actual license used in libkdcraw and bundled libraw sources, 
Requires in devel package fine,
scriptlets are fine,
package builds in mock http://koji.fedoraproject.org/koji/taskinfo?taskID=3190903

Package APPROVED

Comment 12 Rex Dieter 2011-07-11 14:45:21 UTC
Thanks.  What's the purpose of marking fedora_requires_release_note flag?

New Package SCM Request
=======================
Package Name: libkdcraw
Short Description: A C++ interface around LibRaw library
Owners: than rdieter jreznik kkofler ltinkl rnovacek rrix
Branches: f15

Comment 13 Gwyn Ciesla 2011-07-11 15:15:34 UTC
pkgdb says it already exists.

Comment 14 Rex Dieter 2011-07-11 16:56:25 UTC
doh, forgot about that, let's try change request instead then.  fyi, this needs also to be marked not deprecated.


Package Change Request
======================
Package Name: libkdcraw
Short Description: A C++ interface around LibRaw library
Owners: than rdieter jreznik kkofler ltinkl rnovacek rrix
Branches: f15

Comment 15 Gwyn Ciesla 2011-07-11 17:13:44 UTC
Git done (by process-git-requests).

Comment 16 Gwyn Ciesla 2011-07-11 17:14:50 UTC
Did f15 branch, can't do 1 or 3, you'll need to file a rel-eng trac.

Comment 17 Rex Dieter 2011-07-12 17:07:54 UTC
imported.


Note You need to log in before you can comment on or make changes to this bug.