public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
From: Christopher Larson <kergoth@gmail.com>
To: openembedded-core@lists.openembedded.org
Cc: Christopher Larson <chris_larson@mentor.com>
Subject: [PATCH 0/6] Fixes for INCOMPATIBLE_LICENSE handling
Date: Tue, 15 Jan 2013 13:22:45 -0700	[thread overview]
Message-ID: <cover.1358281118.git.chris_larson@mentor.com> (raw)

From: Christopher Larson <chris_larson@mentor.com>

This fixes a number of bugs spotted while trying to use INCOMPATIBLE_LICENSE
= "*GPLv3", and also while experimenting with a recipe whose LICENSE includes
GPLv3, but whose packages do not. In addition, it reduces the message spam by
switching the license incompatibility messages to debug.

Note: there's still one remaining bug I noticed, the wildcard support which
was added is not very well supported in concert with the host tools exclusion
variables. It asks for an spdx version of "*GPLv3", gets nothing but itself,
and tries to look up whitelists for that string. This is not addressed by this
patch series, but I mention it here for awareness, and also a reminder to
myself to look in to it.

The following changes since commit 6080d1dc719c38fd97c2abd38e0ad938fbf6f452:

  connman: fixed init script so connman can runs over nfs (2013-01-15 08:09:30 +0000)

are available in the git repository at:

  https://github.com/kergoth/oe-core incompatible-license

for you to fetch changes up to 5cb12f38b2ac74ea118b58888a0c37f63917fbfa:

  base.bbclass: improve the incompatible license logic a bit (2013-01-15 12:48:51 -0700)

----------------------------------------------------------------
Christopher Larson (6):
      license.bbclass: check LICENSE_pkg, not LICENSE_pn-pkg
      license: correct re.search/fnmatch indentation
      license.bbclass: cleanly handle invalid licenses in incompatible_license
      linux-firmware: use LICENSEs which can actually be parsed
      base.bbclass: use debug messages for INCOMPATIBLE_LICENSE
      base.bbclass: improve the incompatible license logic a bit

 meta/classes/base.bbclass                                |  9 ++++-----
 meta/classes/license.bbclass                             | 16 +++++++++-------
 meta/recipes-kernel/linux-firmware/linux-firmware_git.bb | 13 ++++++-------
 3 files changed, 19 insertions(+), 19 deletions(-)

Christopher Larson (6):
  license.bbclass: check LICENSE_pkg, not LICENSE_pn-pkg
  license: correct re.search/fnmatch indentation
  license.bbclass: cleanly handle invalid licenses in
    incompatible_license
  linux-firmware: use LICENSEs which can actually be parsed
  base.bbclass: use debug messages for INCOMPATIBLE_LICENSE
  base.bbclass: improve the incompatible license logic a bit

 meta/classes/base.bbclass                                |  9 ++++-----
 meta/classes/license.bbclass                             | 16 +++++++++-------
 meta/recipes-kernel/linux-firmware/linux-firmware_git.bb | 13 ++++++-------
 3 files changed, 19 insertions(+), 19 deletions(-)

-- 
1.8.0.3




             reply	other threads:[~2013-01-15 20:38 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-15 20:22 Christopher Larson [this message]
2013-01-15 20:22 ` [PATCH 1/6] license.bbclass: check LICENSE_pkg, not LICENSE_pn-pkg Christopher Larson
2013-01-15 20:22 ` [PATCH 2/6] license: correct re.search/fnmatch indentation Christopher Larson
2013-01-15 20:22 ` [PATCH 3/6] license.bbclass: cleanly handle invalid licenses in incompatible_license Christopher Larson
2013-01-15 20:22 ` [PATCH 4/6] linux-firmware: use LICENSEs which can actually be parsed Christopher Larson
2013-01-15 20:22 ` [PATCH 5/6] base.bbclass: use debug messages for INCOMPATIBLE_LICENSE Christopher Larson
2013-01-15 20:22 ` [PATCH 6/6] base.bbclass: improve the incompatible license logic a bit Christopher Larson
2013-01-19  5:02   ` Saul Wold
2013-01-20  2:12     ` Chris Larson
2013-01-17 19:34 ` [PATCH 0/6] Fixes for INCOMPATIBLE_LICENSE handling Flanagan, Elizabeth

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1358281118.git.chris_larson@mentor.com \
    --to=kergoth@gmail.com \
    --cc=chris_larson@mentor.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox