Re: [PATCH] Add LZO1X compression support to the kernel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Richard Purdie wrote:
On Sat, 2007-05-12 at 09:47 -0700, Andrew Morton wrote:
On Sat, 12 May 2007 16:17:35 +0100 Richard Purdie <[email protected]> wrote:

On Sat, 2007-05-12 at 13:17 +0200, Adrian Bunk wrote:
On Wed, May 02, 2007 at 09:56:23AM +0100, Richard Purdie wrote:
...
I've asked the LZO author about the comments on lzo_copyright function
but the code is GPLv2 licensed so is suitable for inclusion in the
kernel.
This sounds as if LZO is GPL incompatible similar to code under
the 4 clause BSD licence.

...
+/* If you use the LZO library in a product, you *must* keep this
+ * copyright string in the executable of your product.
+ */
I've had a reply from the copyright holder/author saying he's going to
change this wording in the next LZO release and that the code *is* GPLv2
licensed as per the headers on the files.

It would be ideal if the author of that code could send us a
signed-off-by: as per secton 11 of Documentation/SubmittingPatches.

Section 11b says I can submit it but I've asked the author about a
signed off by line (he's cc'd) for the alleviation of any doubt.

I agree that the comment does conflict with the GPL. Just to avoid any doubts, LZO _is_ GPL v2, and I will fix that comment for the next version. Please accept my signed-off-by for Richard's kernel patch in the meantime.

  Signed-off-by: Markus F.X.J. Oberhumer <[email protected]>

~Markus



Cheers,

Richard



--
Markus Oberhumer, <[email protected]>, http://www.oberhumer.com/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux