]> code.ossystems Code Review - openembedded-core.git/commit
recipetool: create: separate LICENSE items with & by default
authorPaul Eggleton <paul.eggleton@linux.intel.com>
Mon, 31 Oct 2016 03:59:47 +0000 (16:59 +1300)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Mon, 7 Nov 2016 11:04:17 +0000 (11:04 +0000)
commitecac6aee8cf3313350b58c21012bcd67cfb915e4
tree3601cb1badd08e795ee3947c0dcee174469f1020
parent4da8a58e2997db4f24ae0cac0ba27259d7857a05
recipetool: create: separate LICENSE items with & by default

recipetool sets the LICENSE value based on licenses detected from the
source tree. If there are multiple licenses then they were being
separated by spaces, but this isn't actually legal formatting and if
you're using "devtool add" you get a warning printed when devtool
parses the recipe internally.

Earlier I had made a conscious decision to do it this way since it's up
to the user to figure out whether the multiple licenses should all apply
(in which case they'd be separated with &) or if there is a choice of
license (in which case | is the correct separator). However, I've come
to the conclusion that we can just default to & and then the ugly
warning goes away, and it's the safest alternative of the two (and most
likely to be correct, since it's more common to have a codebase which is
made up of code with different licenses, i.e. all of them apply to the
combined work).

I've tweaked the comment that we add to the recipe to explicitly state
that we've used & and that the user needs to change that if that's not
accurate.

Fixes [YOCTO #10413].

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
meta/lib/oeqa/selftest/devtool.py
scripts/lib/recipetool/create.py