Skip to contentSkip to conversation

insane.bbclass: package_qa_check_license -> do_qa_populate_lic

PATCH Superseded Robert Yang 5 participants 17 messages Apr 25, 2016 – Apr 27, 2016

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore
#0 lore raw

cover insane.bbclass: package_qa_check_license -> do_qa_populate_lic

The package_qa_check_license() was in do_qa_configure which had a potential problem, when the recipe sets do_configure[noexec] = "1", then the license checking won't run, which may hidden license issues. Rename it to do_qa_populate_lic() and run in do_populate_lic[postfuncs] which is more reasonable (when no populate license, no license checking).

// Robert

The following changes since commit e2dbe5eb869b8336b91023b83d7ca866197efa73:

license.bbclass: make sure that image manifest dir exists (2016-04-22 16:28:47 +0100)

are available in the git repository at:

git://git.openembedded.org/openembedded-core-contrib rbt/lic
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=rbt/lic

Robert Yang (6):
insane.bbclass: remove workdir from package_qa_check_license() insane.bbclass: package_qa_check_license -> do_qa_populate_lic packagegroup.bbclass: set LICENSE and LIC_FILES_CHKSUM packagegroup-foo.bb: remove LICENSE = "MIT"
meta-world-pkgdata: add LIC_FILES_CHKSUM
libxsettings-client: fix LICENSE

 meta/classes/insane.bbclass                        | 15 ++++++---------
 meta/classes/packagegroup.bbclass                  |  5 +++++
 meta/conf/bitbake.conf                             |  2 +-
 meta/recipes-core/meta/meta-world-pkgdata.bb       |  2 ++
 .../packagegroups/packagegroup-base.bb             |  1 -
 .../packagegroups/packagegroup-core-boot.bb        |  1 -
 .../packagegroup-core-buildessential.bb            |  1 -
 .../packagegroup-core-eclipse-debug.bb             |  1 -
 .../packagegroups/packagegroup-core-nfs.bb         |  1 -
 .../packagegroups/packagegroup-core-sdk.bb         |  1 -
 .../packagegroup-core-ssh-dropbear.bb              |  1 -
 .../packagegroups/packagegroup-core-ssh-openssh.bb |  1 -
 .../packagegroup-core-standalone-sdk-target.bb     |  1 -
 .../packagegroups/packagegroup-core-tools-debug.bb |  1 -
 .../packagegroup-core-tools-profile.bb             |  1 -
 .../packagegroup-core-tools-testapps.bb            |  1 -
 .../packagegroups/packagegroup-cross-canadian.bb   |  1 -
 .../packagegroups/packagegroup-self-hosted.bb      |  1 -
 .../packagegroup-core-device-devel.bb              |  1 -
 .../packagegroup-core-full-cmdline.bb              |  1 -
 .../packagegroups/packagegroup-core-lsb.bb         |  1 -
 .../libxsettings-client/MIT-style-license          | 22 ----------------------
 .../libxsettings-client_0.10.bb                    | 13 ++-----------
 .../packagegroups/packagegroup-core-clutter.bb     |  1 -
 .../packagegroups/packagegroup-core-directfb.bb    |  1 -
 .../packagegroups/packagegroup-core-x11-base.bb    |  1 -
 .../packagegroups/packagegroup-core-x11-xserver.bb |  1 -
 .../packagegroups/packagegroup-core-x11.bb         |  1 -
 .../packagegroups/packagegroup-core-x11-sato.bb    |  1 -
 29 files changed, 16 insertions(+), 66 deletions(-)
 delete mode 100644 meta/recipes-graphics/libxsettings-client/libxsettings-client/MIT-style-license
Signature · 1 line
2.7.4
#1 lore raw

PATCH 1/6 insane.bbclass: remove workdir from package_qa_check_license()

The parameter workdir is not used in package_qa_check_license()

---
 meta/classes/insane.bbclass | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
meta/classes/insane.bbclass +2 -2
index c57b217..a1d23d0 100644
@@ -672,7 +672,7 @@ def package_qa_check_symlink_to_sysroot(path, name, d, elf, messages):
                 trimmed = path.replace(os.path.join (d.getVar("PKGDEST", True), name), "")
                 package_qa_add_message(messages, "symlink-to-sysroot", "Symlink %s in %s points to TMPDIR" % (trimmed, name))
 
-def package_qa_check_license(workdir, d):
+def package_qa_check_license(d):
     """
     Check for changes in the license files 
     """
@@ -1211,7 +1211,7 @@ Missing inherit gettext?""" % (gt, config))
     # Check license variables
     ###########################################################################
 
-    package_qa_check_license(workdir, d)
+    package_qa_check_license(d)
 
     ###########################################################################
     # Check unrecognised configure options (with a white list)
Signature · 1 line
2.7.4
#2 lore raw

PATCH 2/6 insane.bbclass: package_qa_check_license -> do_qa_populate_lic

In reply to Robert Yang

The package_qa_check_license() was in do_qa_configure which had a potential problem, when the recipe sets do_configure[noexec] = "1", then the license checking won't run, which may hidden license issues. Rename it to do_qa_populate_lic() and run in do_populate_lic[postfuncs] which is more reasonable (when no populate license, no license checking).

---
 meta/classes/insane.bbclass | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)
meta/classes/insane.bbclass +6 -9
index a1d23d0..df4816c 100644
@@ -672,7 +672,9 @@ def package_qa_check_symlink_to_sysroot(path, name, d, elf, messages):
                 trimmed = path.replace(os.path.join (d.getVar("PKGDEST", True), name), "")
                 package_qa_add_message(messages, "symlink-to-sysroot", "Symlink %s in %s points to TMPDIR" % (trimmed, name))
 
-def package_qa_check_license(d):
+# Check license variables
+do_populate_lic[postfuncs] += "do_qa_populate_lic"
+python do_qa_populate_lic() {
     """
     Check for changes in the license files 
     """
@@ -753,6 +755,7 @@ def package_qa_check_license(d):
                 msg = pn + ": LIC_FILES_CHKSUM is not specified for " +  url
                 msg = msg + "\n" + pn + ": The md5 checksum is " + md5chksum
             package_qa_handle_error("license-checksum", msg, d)
+}
 
 def package_qa_check_staged(path,d):
     """
@@ -1208,12 +1211,6 @@ Rerun configure task after fixing this.""")
 Missing inherit gettext?""" % (gt, config))
 
     ###########################################################################
-    # Check license variables
-    ###########################################################################
-
-    package_qa_check_license(d)
-
-    ###########################################################################
     # Check unrecognised configure options (with a white list)
     ###########################################################################
     if bb.data.inherits_class("autotools", d):
@@ -1261,8 +1258,8 @@ python do_qa_unpack() {
 #addtask qa_staging after do_populate_sysroot before do_build
 do_populate_sysroot[postfuncs] += "do_qa_staging "
 
-# Check broken config.log files, for packages requiring Gettext which don't
-# have it in DEPENDS and for correct LIC_FILES_CHKSUM
+# Check broken config.log files, for packages requiring Gettext which
+# don't have it in DEPENDS.
 #addtask qa_configure after do_configure before do_compile
 do_configure[postfuncs] += "do_qa_configure "
 
Signature · 1 line
2.7.4
#3 lore raw
Quoted 9 lines · Robert Yang

The package_qa_check_license() was in do_qa_configure which had a potential problem, when the recipe sets do_configure[noexec] = "1", then
the license checking won't run, which may hidden license issues. Rename
it to do_qa_populate_lic() and run in do_populate_lic[postfuncs] which
is more reasonable (when no populate license, no license checking).

Quoted 4 lines
---
 meta/classes/insane.bbclass | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

There is a convention that "do_" as a function prefix means its a task. In this case its not a task but a function, so can you use a different prefix please?

Cheers,

Richard

#4 lore raw
Quoted 10 lines · Richard Purdie
Quoted 9 lines · Robert Yang

The package_qa_check_license() was in do_qa_configure which had a potential problem, when the recipe sets do_configure[noexec] = "1", then
the license checking won't run, which may hidden license issues. Rename
it to do_qa_populate_lic() and run in do_populate_lic[postfuncs] which
is more reasonable (when no populate license, no license checking).

Quoted 8 lines
Quoted 4 lines
---
  meta/classes/insane.bbclass | 15 ++++++---------
  1 file changed, 6 insertions(+), 9 deletions(-)

There is a convention that "do_" as a function prefix means its a task. In this case its not a task but a function, so can you use a different prefix please?

Ah, yes, thanks, I will update it after Ross replies.

// Robert

Quoted 5 lines

Cheers,

Richard

#5 lore raw
In reply to Richard Purdie
Quoted 10 lines · Richard Purdie
Quoted 9 lines · Robert Yang

The package_qa_check_license() was in do_qa_configure which had a potential problem, when the recipe sets do_configure[noexec] = "1", then
the license checking won't run, which may hidden license issues. Rename
it to do_qa_populate_lic() and run in do_populate_lic[postfuncs] which
is more reasonable (when no populate license, no license checking).

Quoted 8 lines
Quoted 4 lines
---
  meta/classes/insane.bbclass | 15 ++++++---------
  1 file changed, 6 insertions(+), 9 deletions(-)

There is a convention that "do_" as a function prefix means its a task. In this case its not a task but a function, so can you use a different prefix please?

I had updated the code in V2, but there are others 3 use do_<foo> in insane.bbclass which are funtions, not tasks:

do_populate_sysroot[postfuncs] += "do_qa_staging " do_configure[postfuncs] += "do_qa_configure " do_unpack[postfuncs] += "do_qa_unpack"

What shall we do on them, please ? Keep them or update them?

// Robert

Quoted 5 lines

Cheers,

Richard

#6 lore raw

PATCH 3/6 packagegroup.bbclass: set LICENSE and LIC_FILES_CHKSUM

In reply to Robert Yang

* Use "??=" in bitbake.conf to set LICENSE, so that it can overrided by packagegroup.bbclass and recipes.

* Use "?=" to set LICENSE and LIC_FILES_CHKSUM to MIT by default in packagegroup.bbclass, this won't impact any packagegroup recipes which use non-MIT license, since they can be overrided by the recipe.

---
 meta/classes/packagegroup.bbclass | 5 +++++
 meta/conf/bitbake.conf            | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)
meta/classes/packagegroup.bbclass +5 -0
index 38bdbd3..3674894 100644
@@ -14,6 +14,11 @@ PACKAGE_ARCH ?= "all"
 # Fully expanded - so it applies the overrides as well
 PACKAGE_ARCH_EXPANDED := "${PACKAGE_ARCH}"
 
+LICENSE ?= "MIT"
+LIC_FILES_CHKSUM ?= "${@oe.utils.ifelse(d.getVar('LICENSE', True) == 'MIT', \
+                     'file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420', \
+                    '')}"
+
 inherit ${@oe.utils.ifelse(d.getVar('PACKAGE_ARCH_EXPANDED', True) == 'all', 'allarch', '')}
 
 # This automatically adds -dbg and -dev flavours of all PACKAGES
meta/conf/bitbake.conf +1 -1
index 40796aa..66646c8 100644
@@ -247,7 +247,7 @@ SUMMARY_${PN}-doc ?= "${SUMMARY} - Documentation files"
 DESCRIPTION_${PN}-doc ?= "${DESCRIPTION}  \
 This package contains documentation."
 
-LICENSE = "INVALID"
+LICENSE ??= "INVALID"
 MAINTAINER = "OE-Core Developers <[email protected]>"
 HOMEPAGE = ""
 
Signature · 1 line
2.7.4
#7 lore raw

PATCH 4/6 packagegroup-foo.bb: remove LICENSE = "MIT"

In reply to Robert Yang

It has been set in packagegroup.bbclass.

---
 meta/recipes-core/packagegroups/packagegroup-base.bb                     | 1 -
 meta/recipes-core/packagegroups/packagegroup-core-boot.bb                | 1 -
 meta/recipes-core/packagegroups/packagegroup-core-buildessential.bb      | 1 -
 meta/recipes-core/packagegroups/packagegroup-core-eclipse-debug.bb       | 1 -
 meta/recipes-core/packagegroups/packagegroup-core-nfs.bb                 | 1 -
 meta/recipes-core/packagegroups/packagegroup-core-sdk.bb                 | 1 -
 meta/recipes-core/packagegroups/packagegroup-core-ssh-dropbear.bb        | 1 -
 meta/recipes-core/packagegroups/packagegroup-core-ssh-openssh.bb         | 1 -
 .../packagegroups/packagegroup-core-standalone-sdk-target.bb             | 1 -
 meta/recipes-core/packagegroups/packagegroup-core-tools-debug.bb         | 1 -
 meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb       | 1 -
 meta/recipes-core/packagegroups/packagegroup-core-tools-testapps.bb      | 1 -
 meta/recipes-core/packagegroups/packagegroup-cross-canadian.bb           | 1 -
 meta/recipes-core/packagegroups/packagegroup-self-hosted.bb              | 1 -
 meta/recipes-devtools/packagegroups/packagegroup-core-device-devel.bb    | 1 -
 meta/recipes-extended/packagegroups/packagegroup-core-full-cmdline.bb    | 1 -
 meta/recipes-extended/packagegroups/packagegroup-core-lsb.bb             | 1 -
 meta/recipes-graphics/packagegroups/packagegroup-core-clutter.bb         | 1 -
 meta/recipes-graphics/packagegroups/packagegroup-core-directfb.bb        | 1 -
 meta/recipes-graphics/packagegroups/packagegroup-core-x11-base.bb        | 1 -
 meta/recipes-graphics/packagegroups/packagegroup-core-x11-xserver.bb     | 1 -
 meta/recipes-graphics/packagegroups/packagegroup-core-x11.bb             | 1 -
 meta/recipes-sato/packagegroups/packagegroup-core-x11-sato.bb            | 1 -
 23 files changed, 23 deletions(-)
meta/recipes-core/packagegroups/packagegroup-base.bb +0 -1
index 9e40b28..2e94fde 100644
@@ -1,5 +1,4 @@
 SUMMARY = "Merge machine and distro options to create a basic machine task/package"
-LICENSE = "MIT"
 PR = "r83"
 
 #
meta/recipes-core/packagegroups/packagegroup-core-boot.bb +0 -1
index bac93b8..04bc0f2 100644
@@ -4,7 +4,6 @@
 
 SUMMARY = "Minimal boot requirements"
 DESCRIPTION = "The minimal set of packages required to boot the system"
-LICENSE = "MIT"
 PR = "r17"
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
meta/recipes-core/packagegroups/packagegroup-core-buildessential.bb +0 -1
index 74ed247..78cc65d 100644
@@ -4,7 +4,6 @@
 #
 
 SUMMARY = "Essential build dependencies"
-LICENSE = "MIT"
 
 inherit packagegroup
 
meta/recipes-core/packagegroups/packagegroup-core-eclipse-debug.bb +0 -1
index e7b013d..c45463f 100644
@@ -1,5 +1,4 @@
 SUMMARY = "Remote debugging tools for Eclipse integration"
-LICENSE = "MIT"
 
 inherit packagegroup
 
meta/recipes-core/packagegroups/packagegroup-core-nfs.bb +0 -1
index 1882d3a..b345e31 100644
@@ -3,7 +3,6 @@
 #
 
 SUMMARY = "NFS package groups"
-LICENSE = "MIT"
 PR = "r2"
 
 inherit packagegroup
meta/recipes-core/packagegroups/packagegroup-core-sdk.bb +0 -1
index bee1665..ea07e98 100644
@@ -3,7 +3,6 @@
 #
 
 SUMMARY = "Software development tools"
-LICENSE = "MIT"
 PR = "r9"
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
meta/recipes-core/packagegroups/packagegroup-core-ssh-dropbear.bb +0 -1
index e99946f..5ec3f6c 100644
@@ -1,5 +1,4 @@
 SUMMARY = "Dropbear SSH client/server"
-LICENSE = "MIT"
 PR = "r1"
 
 inherit packagegroup
meta/recipes-core/packagegroups/packagegroup-core-ssh-openssh.bb +0 -1
index 32d20e6..703f956 100644
@@ -1,5 +1,4 @@
 SUMMARY = "OpenSSH SSH client/server"
-LICENSE = "MIT"
 PR = "r1"
 
 inherit packagegroup
meta/recipes-core/packagegroups/packagegroup-core-standalone-sdk-target.bb +0 -1
index 6997f39..274e11a 100644
@@ -1,6 +1,5 @@
 SUMMARY = "Target packages for the standalone SDK"
 PR = "r8"
-LICENSE = "MIT"
 
 inherit packagegroup
 
meta/recipes-core/packagegroups/packagegroup-core-tools-debug.bb +0 -1
index 82347b9..9fc2b0e 100644
@@ -3,7 +3,6 @@
 #
 
 SUMMARY = "Debugging tools"
-LICENSE = "MIT"
 
 inherit packagegroup
 
meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb +0 -1
index ff2b577..71b3651 100644
@@ -3,7 +3,6 @@
 #
 
 SUMMARY = "Profiling tools"
-LICENSE = "MIT"
 
 PR = "r3"
 
meta/recipes-core/packagegroups/packagegroup-core-tools-testapps.bb +0 -1
index e8d585e..5938d7e 100644
@@ -3,7 +3,6 @@
 #
 
 SUMMARY = "Testing tools/applications"
-LICENSE = "MIT"
 
 PR = "r2"
 
meta/recipes-core/packagegroups/packagegroup-cross-canadian.bb +0 -1
index 180dd96..d551147 100644
@@ -1,6 +1,5 @@
 SUMMARY = "Host SDK package for cross canadian toolchain"
 PN = "packagegroup-cross-canadian-${MACHINE}"
-LICENSE = "MIT"
 
 inherit cross-canadian packagegroup
 
meta/recipes-core/packagegroups/packagegroup-self-hosted.bb +0 -1
index 225204d..aa88cf9 100644
@@ -5,7 +5,6 @@
 SUMMARY = "Self-hosting"
 DESCRIPTION = "Packages required to run the build system"
 PR = "r13"
-LICENSE = "MIT"
 
 inherit packagegroup  distro_features_check
 # rdepends on libx11-dev
meta/recipes-devtools/packagegroups/packagegroup-core-device-devel.bb +0 -1
index e831860..b037057 100644
@@ -1,5 +1,4 @@
 SUMMARY = "Provides a small set of tools for development on the device"
-LICENSE = "MIT"
 
 PR = "r1"
 
meta/recipes-extended/packagegroups/packagegroup-core-full-cmdline.bb +0 -1
index 1d7e608..f33b43e 100644
@@ -5,7 +5,6 @@
 SUMMARY = "Standard full-featured Linux system"
 DESCRIPTION = "Package group bringing in packages needed for a more traditional full-featured Linux system"
 PR = "r6"
-LICENSE = "MIT"
 
 inherit packagegroup
 
meta/recipes-extended/packagegroups/packagegroup-core-lsb.bb +0 -1
index a29c3d7..2b6b7a5 100644
@@ -5,7 +5,6 @@
 SUMMARY = "Linux Standard Base (LSB)"
 DESCRIPTION = "Packages required to satisfy the Linux Standard Base (LSB) specification"
 PR = "r10"
-LICENSE = "MIT"
 
 inherit packagegroup distro_features_check
 
meta/recipes-graphics/packagegroups/packagegroup-core-clutter.bb +0 -1
index 5734b27..87c700e 100644
@@ -3,7 +3,6 @@
 #
 
 SUMMARY = "Clutter package groups"
-LICENSE = "MIT"
 
 PR = "r6"
 
meta/recipes-graphics/packagegroups/packagegroup-core-directfb.bb +0 -1
index a657a1a..5ead60d 100644
@@ -1,5 +1,4 @@
 SUMMARY = "DirectFB without X11"
-LICENSE = "MIT"
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
 
meta/recipes-graphics/packagegroups/packagegroup-core-x11-base.bb +0 -1
index 17301a0..db3dfe3 100644
@@ -1,6 +1,5 @@
 SUMMARY = "Basic X11 session"
 DESCRIPTION = "Packages required to set up a basic working X11 session"
-LICENSE = "MIT"
 PR = "r1"
 
 inherit packagegroup distro_features_check
meta/recipes-graphics/packagegroups/packagegroup-core-x11-xserver.bb +0 -1
index c53f1b7..6b50028 100644
@@ -3,7 +3,6 @@
 #
 
 SUMMARY = "X11 display server"
-LICENSE = "MIT"
 PR = "r40"
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
meta/recipes-graphics/packagegroups/packagegroup-core-x11.bb +0 -1
index 3537d8c..4291424 100644
@@ -2,7 +2,6 @@
 # Copyright (C) 2011 Intel Corporation
 #
 
-LICENSE = "MIT"
 PR = "r40"
 
 inherit packagegroup distro_features_check
meta/recipes-sato/packagegroups/packagegroup-core-x11-sato.bb +0 -1
index 8ba4923..913e8cd 100644
@@ -3,7 +3,6 @@
 #
 
 SUMMARY = "Sato desktop"
-LICENSE = "MIT"
 PR = "r33"
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
Signature · 1 line
2.7.4
#8 lore raw

PATCH 5/6 meta-world-pkgdata: add LIC_FILES_CHKSUM

In reply to Robert Yang

Fixed:
ERROR: meta-world-pkgdata-1.0-r0 do_populate_lic: QA Issue: meta-world-pkgdata: Recipe file does not have license file information (LIC_FILES_CHKSUM) [license-checksum]

---
 meta/recipes-core/meta/meta-world-pkgdata.bb | 2 ++
 1 file changed, 2 insertions(+)
meta/recipes-core/meta/meta-world-pkgdata.bb +2 -0
index 3c5db50..b3989cd 100644
@@ -1,5 +1,7 @@
 SUMMARY = "Pulls in pkgdata for world"
 LICENSE = "MIT"
+LIC_FILES_CHKSUM ?= "file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
+
 INHIBIT_DEFAULT_DEPS = "1"
 
 addtask do_allpackagedata before do_build
Signature · 1 line
2.7.4
#9 lore raw

PATCH 6/6 libxsettings-client: fix LICENSE

In reply to Robert Yang

The tarball has a COPYING file now, and is GPLv2+.

---
 .../libxsettings-client/MIT-style-license          | 22 ----------------------
 .../libxsettings-client_0.10.bb                    | 13 ++-----------
 2 files changed, 2 insertions(+), 33 deletions(-)
 delete mode 100644 meta/recipes-graphics/libxsettings-client/libxsettings-client/MIT-style-license
meta/recipes-graphics/libxsettings-client/libxsettings-client/MIT-style-license deleted +0 -22
deleted file mode 100644
index 628de98..0000000
@@ -1,22 +0,0 @@
-/*
- * Copyright © 2001 Red Hat, Inc.
- *
- * Permission to use, copy, modify, distribute, and sell this software and its
- * documentation for any purpose is hereby granted without fee, provided that
- * the above copyright notice appear in all copies and that both that
- * copyright notice and this permission notice appear in supporting
- * documentation, and that the name of Red Hat not be used in advertising or
- * publicity pertaining to distribution of the software without specific,
- * written prior permission.  Red Hat makes no representations about the
- * suitability of this software for any purpose.  It is provided "as is"
- * without express or implied warranty.
- *
- * RED HAT DISCLAIMS ALL WARRANTIES WITH REGARD TO THIS SOFTWARE, INCLUDING ALL
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS, IN NO EVENT SHALL RED HAT
- * BE LIABLE FOR ANY SPECIAL, INDIRECT OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
- * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION
- * OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN 
- * CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
- *
- * Author:  Owen Taylor, Red Hat, Inc.
- */
meta/recipes-graphics/libxsettings-client/libxsettings-client_0.10.bb +2 -11
index 67c5e27..2999653 100644
@@ -6,8 +6,8 @@ desktop."
 HOMEPAGE = "http://matchbox-project.org/sources/optional-dependencies/"
 BUGTRACKER = "http://bugzilla.yoctoproject.org/"
 SECTION = "x/libs"
-LICENSE = "MIT-style"
-LIC_FILES_CHKSUM = "file://COPYING;md5=7cfac9d2d4dc3694cc7eb605cf32a69b \
+LICENSE = "GPLv2+"
+LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f \
                     file://xsettings-client.h;endline=22;md5=7cfac9d2d4dc3694cc7eb605cf32a69b \
                     file://xsettings-common.h;endline=22;md5=7cfac9d2d4dc3694cc7eb605cf32a69b"
 DEPENDS = "virtual/libx11"
@@ -17,7 +17,6 @@ PR = "r5"
 headers = "xsettings-common.h xsettings-client.h"
 
 SRC_URI = "http://downloads.yoctoproject.org/releases/matchbox/optional-dependencies/Xsettings-client-0.10.tar.gz \
-        file://MIT-style-license \
         file://link-x11.patch;apply=yes \
         file://obsolete_automake_macros.patch \
 "
@@ -30,11 +29,3 @@ S = "${WORKDIR}/Xsettings-client-0.10"
 inherit autotools gettext distro_features_check
 # depends on virtual/libx11
 REQUIRED_DISTRO_FEATURES = "x11"
-
-do_configure_prepend() {
-    # This package doesn't ship with its own COPYING file and
-    # autotools will install a GPLv2 one instead of the actual MIT-style license here.
-    # Add the correct license here to avoid confusion.
-    cp -f ${WORKDIR}/MIT-style-license ${S}/COPYING
-}
-
Signature · 1 line
2.7.4
#10 lore raw

Re: [PATCH 6/6] libxsettings-client: fix LICENSE

Quoted 2 lines · Robert Yang

The tarball has a COPYING file now, and is GPLv2+.

Just had a proper look at this. The tarball has a COPYING that says GPL, but the source files are all clearly marked MIT.

Ross

#11 lore raw

Re: [PATCH 6/6] libxsettings-client: fix LICENSE

Quoted 9 lines · Ross

On 25 April 2016 at 14:51, Robert Yang <[email protected] <mailto:[email protected]>> wrote:

The tarball has a COPYING file now, and is GPLv2+.

Just had a proper look at this. The tarball has a COPYING that says GPL, but the source files are all clearly marked MIT.

Thanks, did you mean the following text ? I can't find MIT license there.

/*
* Copyright © 2001 Red Hat, Inc
*
* Permission to use, copy, modify, distribute, and sell this software and its
* documentation for any purpose is hereby granted without fee, provided that
* the above copyright notice appear in all copies and that both that
* copyright notice and this permission notice appear in supporting
* documentation, and that the name of Red Hat not be used in advertising or
* publicity pertaining to distribution of the software without specific,
* written prior permission. Red Hat makes no representations about the
* suitability of this software for any purpose. It is provided "as is"
* without express or implied warranty.
*
* RED HAT DISCLAIMS ALL WARRANTIES WITH REGARD TO THIS SOFTWARE, INCLUDING ALL
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS, IN NO EVENT SHALL RED HAT
* BE LIABLE FOR ANY SPECIAL, INDIRECT OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
* WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION
* OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN
* CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
*
* Author: Owen Taylor, Red Hat, Inc.
*/

// Robert

Quoted 2 lines

Ross

#12 lore raw

Re: [PATCH 6/6] libxsettings-client: fix LICENSE

Quoted 6 lines · Robert Yang

Thanks, did you mean the following text ? I can't find MIT license there.

/*
* Copyright © 2001 Red Hat, Inc
*

Yes, this one. I've just had a chat with Beth and we both agree that it's practically MIT, even if it's not worded traditionally. My personal belief is that this license is Red Hat's legal department's wording of MIT. Pointing it at the MIT license is acceptable, but preferable would be adding a new "RedHat-MIT" license to common-licenses and using that.

Ross

#13 lore raw

Re: [PATCH 6/6] libxsettings-client: fix LICENSE

Quoted 10 lines · Ross
Quoted 6 lines · Robert Yang

Thanks, did you mean the following text ? I can't find MIT license there.

/*
* Copyright © 2001 Red Hat, Inc
*

Quoted 6 lines

Yes, this one. I've just had a chat with Beth and we both agree that it's practically MIT, even if it's not worded traditionally. My personal belief is that this license is Red Hat's legal department's wording of MIT. Pointing it at the MIT license is acceptable, but preferable would be adding a new "RedHat-MIT" license to common-licenses and using that.

Yes, this is MIT enough that, in that, it doesn't really add any restrictions to MIT that looking at it again I think leaving it MIT is enough.

Quoted 8 lines

Ross

--
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Signature · 3 lines
Elizabeth Flanagan
Yocto Project
Build and Release
#14 lore raw

Re: [PATCH 6/6] libxsettings-client: fix LICENSE

Quoted 21 lines · Elizabeth
Quoted 10 lines · Ross
Quoted 6 lines · Robert Yang

Thanks, did you mean the following text ? I can't find MIT license there.

/*
* Copyright © 2001 Red Hat, Inc
*

Quoted 6 lines

Yes, this one. I've just had a chat with Beth and we both agree that it's practically MIT, even if it's not worded traditionally. My personal belief is that this license is Red Hat's legal department's wording of MIT. Pointing it at the MIT license is acceptable, but preferable would be adding a new "RedHat-MIT" license to common-licenses and using that.

Quoted 3 lines

Yes, this is MIT enough that, in that, it doesn't really add any restrictions to MIT that looking at it again I think leaving it MIT is enough.

It may be good to send a query to concerned folks at redhat and clarify so we don’t have to make assumptions.

Quoted 21 lines
Quoted 9 lines

Ross

--
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

--
Elizabeth Flanagan
Yocto Project
Build and Release
--
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

#15 lore raw

Re: [PATCH 6/6] libxsettings-client: fix LICENSE

Quoted 3 lines · Khem Raj

It may be good to send a query to concerned folks at redhat and clarify so we don’t have to make assumptions.

Owen just confirmed on IRC that this code is MIT.

Ross

#16 lore raw

Re: [PATCH 6/6] libxsettings-client: fix LICENSE

Quoted 9 lines · Ross

On 26 April 2016 at 17:43, Khem Raj <[email protected] <mailto:[email protected]>> wrote:

It may be good to send a query to concerned folks at redhat and clarify so we don’t have to make assumptions.

Owen just confirmed on IRC that this code is MIT.

Thanks, I will let it use MIT as before.

// Robert

Quoted 4 lines

Ross