* [PATCH 1/3] license.bbclass: Variable standardization
2012-09-26 21:50 [PATCH 0/3] license.bbclass fixes Elizabeth Flanagan
@ 2012-09-26 21:50 ` Elizabeth Flanagan
2012-09-27 16:55 ` Saul Wold
2012-09-26 21:50 ` [PATCH 2/3] bitbake.conf: parse license config Elizabeth Flanagan
` (3 subsequent siblings)
4 siblings, 1 reply; 9+ messages in thread
From: Elizabeth Flanagan @ 2012-09-26 21:50 UTC (permalink / raw)
To: openembedded-core
The variable mentioned in license.conf is LICENSE_PATH. The variable
used in license.bbclass is LICENSE_DIR. Conforming to what is in
license.conf
Signed-off-by: Elizabeth Flanagan <elizabeth.flanagan@intel.com>
---
meta/classes/license.bbclass | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/meta/classes/license.bbclass b/meta/classes/license.bbclass
index ffcaf93..ad543f6 100644
--- a/meta/classes/license.bbclass
+++ b/meta/classes/license.bbclass
@@ -186,7 +186,7 @@ python do_populate_lic() {
license_source_dirs = []
license_source_dirs.append(generic_directory)
try:
- additional_lic_dirs = d.getVar('LICENSE_DIR', True).split()
+ additional_lic_dirs = d.getVar('LICENSE_PATH', True).split()
for lic_dir in additional_lic_dirs:
license_source_dirs.append(lic_dir)
except:
--
1.7.5.4
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 1/3] license.bbclass: Variable standardization
2012-09-26 21:50 ` [PATCH 1/3] license.bbclass: Variable standardization Elizabeth Flanagan
@ 2012-09-27 16:55 ` Saul Wold
0 siblings, 0 replies; 9+ messages in thread
From: Saul Wold @ 2012-09-27 16:55 UTC (permalink / raw)
To: Elizabeth Flanagan; +Cc: openembedded-core
On 09/26/2012 02:50 PM, Elizabeth Flanagan wrote:
> The variable mentioned in license.conf is LICENSE_PATH. The variable
> used in license.bbclass is LICENSE_DIR. Conforming to what is in
> license.conf
>
> Signed-off-by: Elizabeth Flanagan <elizabeth.flanagan@intel.com>
> ---
> meta/classes/license.bbclass | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/meta/classes/license.bbclass b/meta/classes/license.bbclass
> index ffcaf93..ad543f6 100644
> --- a/meta/classes/license.bbclass
> +++ b/meta/classes/license.bbclass
> @@ -186,7 +186,7 @@ python do_populate_lic() {
> license_source_dirs = []
> license_source_dirs.append(generic_directory)
> try:
> - additional_lic_dirs = d.getVar('LICENSE_DIR', True).split()
> + additional_lic_dirs = d.getVar('LICENSE_PATH', True).split()
Beth, I found that we also have LICENSE_DIRECTORY, so the intent here is
to be a multiple directory (such as a search PATH) for licenses?
There is also some documentation that needs to be changed, should not
prevent this patch from being pulled.
Sau!
> for lic_dir in additional_lic_dirs:
> license_source_dirs.append(lic_dir)
> except:
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 2/3] bitbake.conf: parse license config
2012-09-26 21:50 [PATCH 0/3] license.bbclass fixes Elizabeth Flanagan
2012-09-26 21:50 ` [PATCH 1/3] license.bbclass: Variable standardization Elizabeth Flanagan
@ 2012-09-26 21:50 ` Elizabeth Flanagan
2012-09-26 21:50 ` [PATCH 3/3] license.conf/bbclass: Move globals to license.conf Elizabeth Flanagan
` (2 subsequent siblings)
4 siblings, 0 replies; 9+ messages in thread
From: Elizabeth Flanagan @ 2012-09-26 21:50 UTC (permalink / raw)
To: openembedded-core
license.conf hasn't been being parsed. It probably should be.
Signed-off-by: Elizabeth Flanagan <elizabeth.flanagan@intel.com>
---
meta/conf/bitbake.conf | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 3496d2b..e168ef1 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -683,6 +683,7 @@ include conf/machine-sdk/${SDKMACHINE}.conf
include conf/distro/${DISTRO}.conf
include conf/distro/defaultsetup.conf
include conf/documentation.conf
+include conf/licenses.conf
require conf/sanity.conf
##################################################################
--
1.7.5.4
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH 3/3] license.conf/bbclass: Move globals to license.conf
2012-09-26 21:50 [PATCH 0/3] license.bbclass fixes Elizabeth Flanagan
2012-09-26 21:50 ` [PATCH 1/3] license.bbclass: Variable standardization Elizabeth Flanagan
2012-09-26 21:50 ` [PATCH 2/3] bitbake.conf: parse license config Elizabeth Flanagan
@ 2012-09-26 21:50 ` Elizabeth Flanagan
2012-09-27 17:08 ` [PATCH 0/3] license.bbclass fixes Saul Wold
2012-09-28 10:20 ` Richard Purdie
4 siblings, 0 replies; 9+ messages in thread
From: Elizabeth Flanagan @ 2012-09-26 21:50 UTC (permalink / raw)
To: openembedded-core
This requires the changes to bitbake.conf that allow parsing of
license.conf.
As we should now be parsing license.conf, we can move some globals
out of license.bblcass and into the conf file.
Signed-off-by: Elizabeth Flanagan <elizabeth.flanagan@intel.com>
---
meta/classes/license.bbclass | 65 ------------------------------------------
meta/conf/licenses.conf | 64 +++++++++++++++++++++++++++++++++++++++++
2 files changed, 64 insertions(+), 65 deletions(-)
diff --git a/meta/classes/license.bbclass b/meta/classes/license.bbclass
index ad543f6..a66933f 100644
--- a/meta/classes/license.bbclass
+++ b/meta/classes/license.bbclass
@@ -10,71 +10,6 @@ addtask populate_lic after do_patch before do_package
do_populate_lic[dirs] = "${LICSSTATEDIR}/${PN}"
do_populate_lic[cleandirs] = "${LICSSTATEDIR}"
-# Standards are great! Everyone has their own. In an effort to standardize licensing
-# names, common-licenses will use the SPDX standard license names. In order to not
-# break the non-standardized license names that we find in LICENSE, we'll set
-# up a bunch of VarFlags to accomodate non-SPDX license names.
-#
-# We should really discuss standardizing this field, but that's a longer term goal.
-# For now, we can do this and it should grab the most common LICENSE naming variations.
-#
-# We should NEVER have a GPL/LGPL without a version!!!!
-# Any mapping to MPL/LGPL/GPL should be fixed
-# see: https://wiki.yoctoproject.org/wiki/License_Audit
-
-# GPL variations
-SPDXLICENSEMAP[GPL-1] = "GPL-1.0"
-SPDXLICENSEMAP[GPLv1] = "GPL-1.0"
-SPDXLICENSEMAP[GPLv1.0] = "GPL-1.0"
-SPDXLICENSEMAP[GPL-2] = "GPL-2.0"
-SPDXLICENSEMAP[GPLv2] = "GPL-2.0"
-SPDXLICENSEMAP[GPLv2.0] = "GPL-2.0"
-SPDXLICENSEMAP[GPL-3] = "GPL-3.0"
-SPDXLICENSEMAP[GPLv3] = "GPL-3.0"
-SPDXLICENSEMAP[GPLv3.0] = "GPL-3.0"
-
-#LGPL variations
-SPDXLICENSEMAP[LGPLv2] = "LGPL-2.0"
-SPDXLICENSEMAP[LGPLv2.0] = "LGPL-2.0"
-SPDXLICENSEMAP[LGPL2.1] = "LGPL-2.1"
-SPDXLICENSEMAP[LGPLv2.1] = "LGPL-2.1"
-SPDXLICENSEMAP[LGPLv3] = "LGPL-3.0"
-
-#MPL variations
-SPDXLICENSEMAP[MPL-1] = "MPL-1.0"
-SPDXLICENSEMAP[MPLv1] = "MPL-1.0"
-SPDXLICENSEMAP[MPLv1.1] = "MPL-1.1"
-SPDXLICENSEMAP[MPLv2] = "MPL-2.0"
-
-#MIT variations
-SPDXLICENSEMAP[MIT-X] = "MIT"
-SPDXLICENSEMAP[MIT-style] = "MIT"
-
-#Openssl variations
-SPDXLICENSEMAP[openssl] = "OpenSSL"
-
-#Python variations
-SPDXLICENSEMAP[PSF] = "Python-2.0"
-SPDXLICENSEMAP[PSFv2] = "Python-2.0"
-SPDXLICENSEMAP[Python-2] = "Python-2.0"
-
-#Apache variations
-SPDXLICENSEMAP[Apachev2] = "Apache-2.0"
-SPDXLICENSEMAP[Apache-2] = "Apache-2.0"
-
-#Artistic variations
-SPDXLICENSEMAP[Artisticv1] = "Artistic-1.0"
-SPDXLICENSEMAP[Artistic-1] = "Artistic-1.0"
-
-#Academic variations
-SPDXLICENSEMAP[AFL-2] = "AFL-2.0"
-SPDXLICENSEMAP[AFL-1] = "AFL-1.2"
-SPDXLICENSEMAP[AFLv2] = "AFL-2.0"
-SPDXLICENSEMAP[AFLv1] = "AFL-1.2"
-
-#Other variations
-SPDXLICENSEMAP[EPLv1.0] = "EPL-1.0"
-
license_create_manifest() {
mkdir -p ${LICENSE_DIRECTORY}/${IMAGE_NAME}
# Get list of installed packages
diff --git a/meta/conf/licenses.conf b/meta/conf/licenses.conf
index 6ed4dfe..3cb143c 100644
--- a/meta/conf/licenses.conf
+++ b/meta/conf/licenses.conf
@@ -39,6 +39,70 @@ SRC_DISTRIBUTE_LICENSES += "SPL-1.0 SugarCRM-1 SugarCRM-1.1.3 UCB VSL-1.0 W3C"
SRC_DISTRIBUTE_LICENSES += "Watcom-1.0 WXwindows XFree86-1.1 Xnet YPL-1.1"
SRC_DISTRIBUTE_LICENSES += "Zimbra-1.3 Zlib ZPL-1.1 ZPL-2.0 ZPL-2.1"
+# Standards are great! Everyone has their own. In an effort to standardize licensing
+# names, common-licenses will use the SPDX standard license names. In order to not
+# break the non-standardized license names that we find in LICENSE, we'll set
+# up a bunch of VarFlags to accomodate non-SPDX license names.
+#
+# We should really discuss standardizing this field, but that's a longer term goal.
+# For now, we can do this and it should grab the most common LICENSE naming variations.
+#
+# We should NEVER have a GPL/LGPL without a version!!!!
+# Any mapping to MPL/LGPL/GPL should be fixed
+# see: https://wiki.yoctoproject.org/wiki/License_Audit
+
+# GPL variations
+SPDXLICENSEMAP[GPL-1] = "GPL-1.0"
+SPDXLICENSEMAP[GPLv1] = "GPL-1.0"
+SPDXLICENSEMAP[GPLv1.0] = "GPL-1.0"
+SPDXLICENSEMAP[GPL-2] = "GPL-2.0"
+SPDXLICENSEMAP[GPLv2] = "GPL-2.0"
+SPDXLICENSEMAP[GPLv2.0] = "GPL-2.0"
+SPDXLICENSEMAP[GPL-3] = "GPL-3.0"
+SPDXLICENSEMAP[GPLv3] = "GPL-3.0"
+SPDXLICENSEMAP[GPLv3.0] = "GPL-3.0"
+
+#LGPL variations
+SPDXLICENSEMAP[LGPLv2] = "LGPL-2.0"
+SPDXLICENSEMAP[LGPLv2.0] = "LGPL-2.0"
+SPDXLICENSEMAP[LGPL2.1] = "LGPL-2.1"
+SPDXLICENSEMAP[LGPLv2.1] = "LGPL-2.1"
+SPDXLICENSEMAP[LGPLv3] = "LGPL-3.0"
+
+#MPL variations
+SPDXLICENSEMAP[MPL-1] = "MPL-1.0"
+SPDXLICENSEMAP[MPLv1] = "MPL-1.0"
+SPDXLICENSEMAP[MPLv1.1] = "MPL-1.1"
+SPDXLICENSEMAP[MPLv2] = "MPL-2.0"
+
+#MIT variations
+SPDXLICENSEMAP[MIT-X] = "MIT"
+SPDXLICENSEMAP[MIT-style] = "MIT"
+
+#Openssl variations
+SPDXLICENSEMAP[openssl] = "OpenSSL"
+
+#Python variations
+SPDXLICENSEMAP[PSF] = "Python-2.0"
+SPDXLICENSEMAP[PSFv2] = "Python-2.0"
+SPDXLICENSEMAP[Python-2] = "Python-2.0"
+
+#Apache variations
+SPDXLICENSEMAP[Apachev2] = "Apache-2.0"
+SPDXLICENSEMAP[Apache-2] = "Apache-2.0"
+
+#Artistic variations
+SPDXLICENSEMAP[Artisticv1] = "Artistic-1.0"
+SPDXLICENSEMAP[Artistic-1] = "Artistic-1.0"
+
+#Academic variations
+SPDXLICENSEMAP[AFL-2] = "AFL-2.0"
+SPDXLICENSEMAP[AFL-1] = "AFL-1.2"
+SPDXLICENSEMAP[AFLv2] = "AFL-2.0"
+SPDXLICENSEMAP[AFLv1] = "AFL-1.2"
+
+#Other variations
+SPDXLICENSEMAP[EPLv1.0] = "EPL-1.0"
# Additional license directories. Add your custom licenses directories this path.
# LICENSE_PATH += "${COREBASE}/custom-licenses"
--
1.7.5.4
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 0/3] license.bbclass fixes
2012-09-26 21:50 [PATCH 0/3] license.bbclass fixes Elizabeth Flanagan
` (2 preceding siblings ...)
2012-09-26 21:50 ` [PATCH 3/3] license.conf/bbclass: Move globals to license.conf Elizabeth Flanagan
@ 2012-09-27 17:08 ` Saul Wold
2012-09-27 23:50 ` Flanagan, Elizabeth
2012-09-28 10:20 ` Richard Purdie
4 siblings, 1 reply; 9+ messages in thread
From: Saul Wold @ 2012-09-27 17:08 UTC (permalink / raw)
To: Elizabeth Flanagan; +Cc: openembedded-core
On 09/26/2012 02:50 PM, Elizabeth Flanagan wrote:
> [YOCTO #3118]
>
> license.conf hasn't been being parsed. This commit series ensures that
> license.conf is parsed, that it contains useful variables that should be
> in a global conf file and that those variables are not kept within
> license.bbclass.
>
So where is the inherit license bbclass?
Sau!
> This also fixes an discreprancy between the documentation of LICENSE_PATH (which
> is what we tell people to use) and what we were using "LICENSE_DIR". This ensures
> that when the end user needs to add additional licenses, that they use
> LICENSE_PATH += "path/to/additional/licenses" and that it works.
>
> The following changes since commit 0b09e50810162a07ef0aecee91ee32b4a36334a3:
>
> busybox: Fix misplaced quote (2012-09-26 18:28:36 +0100)
>
> are available in the git repository at:
> git://git.yoctoproject.org/poky-contrib eflanagan/LICENSE_PATH
> http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=eflanagan/LICENSE_PATH
>
> Elizabeth Flanagan (3):
> license.bbclass: Variable standardization
> bitbake.conf: parse license config
> license.conf/bbclass: Move globals to license.conf
>
> meta/classes/license.bbclass | 67 +-----------------------------------------
> meta/conf/bitbake.conf | 1 +
> meta/conf/licenses.conf | 64 ++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 66 insertions(+), 66 deletions(-)
>
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH 0/3] license.bbclass fixes
2012-09-27 17:08 ` [PATCH 0/3] license.bbclass fixes Saul Wold
@ 2012-09-27 23:50 ` Flanagan, Elizabeth
2012-09-27 23:56 ` Saul Wold
0 siblings, 1 reply; 9+ messages in thread
From: Flanagan, Elizabeth @ 2012-09-27 23:50 UTC (permalink / raw)
To: Saul Wold; +Cc: openembedded-core
On Thu, Sep 27, 2012 at 10:08 AM, Saul Wold <sgw@linux.intel.com> wrote:
> On 09/26/2012 02:50 PM, Elizabeth Flanagan wrote:
>>
>> [YOCTO #3118]
>>
>> license.conf hasn't been being parsed. This commit series ensures that
>> license.conf is parsed, that it contains useful variables that should be
>> in a global conf file and that those variables are not kept within
>> license.bbclass.
>>
> So where is the inherit license bbclass?
For the yocto project it's an INHERIT += within poky.conf.
meta-angstrom does not inherit it within
conf/distro/include/angstrom.inc.
-b
>
> Sau!
>
>
>> This also fixes an discreprancy between the documentation of LICENSE_PATH
>> (which
>> is what we tell people to use) and what we were using "LICENSE_DIR". This
>> ensures
>> that when the end user needs to add additional licenses, that they use
>> LICENSE_PATH += "path/to/additional/licenses" and that it works.
>>
>
>> The following changes since commit
>> 0b09e50810162a07ef0aecee91ee32b4a36334a3:
>>
>> busybox: Fix misplaced quote (2012-09-26 18:28:36 +0100)
>>
>> are available in the git repository at:
>> git://git.yoctoproject.org/poky-contrib eflanagan/LICENSE_PATH
>>
>> http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=eflanagan/LICENSE_PATH
>>
>> Elizabeth Flanagan (3):
>> license.bbclass: Variable standardization
>> bitbake.conf: parse license config
>> license.conf/bbclass: Move globals to license.conf
>>
>> meta/classes/license.bbclass | 67
>> +-----------------------------------------
>> meta/conf/bitbake.conf | 1 +
>> meta/conf/licenses.conf | 64
>> ++++++++++++++++++++++++++++++++++++++++
>> 3 files changed, 66 insertions(+), 66 deletions(-)
>>
>
--
Elizabeth Flanagan
Yocto Project
Build and Release
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 0/3] license.bbclass fixes
2012-09-27 23:50 ` Flanagan, Elizabeth
@ 2012-09-27 23:56 ` Saul Wold
0 siblings, 0 replies; 9+ messages in thread
From: Saul Wold @ 2012-09-27 23:56 UTC (permalink / raw)
To: Flanagan, Elizabeth; +Cc: openembedded-core
On 09/27/2012 04:50 PM, Flanagan, Elizabeth wrote:
> On Thu, Sep 27, 2012 at 10:08 AM, Saul Wold <sgw@linux.intel.com> wrote:
>> On 09/26/2012 02:50 PM, Elizabeth Flanagan wrote:
>>>
>>> [YOCTO #3118]
>>>
>>> license.conf hasn't been being parsed. This commit series ensures that
>>> license.conf is parsed, that it contains useful variables that should be
>>> in a global conf file and that those variables are not kept within
>>> license.bbclass.
>>>
>> So where is the inherit license bbclass?
>
> For the yocto project it's an INHERIT += within poky.conf.
> meta-angstrom does not inherit it within
> conf/distro/include/angstrom.inc.
>
> -b
>
Acked-by: Saul Wold <sgw@linux.intel.com?>
>>
>> Sau!
>>
>>
>>> This also fixes an discreprancy between the documentation of LICENSE_PATH
>>> (which
>>> is what we tell people to use) and what we were using "LICENSE_DIR". This
>>> ensures
>>> that when the end user needs to add additional licenses, that they use
>>> LICENSE_PATH += "path/to/additional/licenses" and that it works.
>>>
>>
>>> The following changes since commit
>>> 0b09e50810162a07ef0aecee91ee32b4a36334a3:
>>>
>>> busybox: Fix misplaced quote (2012-09-26 18:28:36 +0100)
>>>
>>> are available in the git repository at:
>>> git://git.yoctoproject.org/poky-contrib eflanagan/LICENSE_PATH
>>>
>>> http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=eflanagan/LICENSE_PATH
>>>
>>> Elizabeth Flanagan (3):
>>> license.bbclass: Variable standardization
>>> bitbake.conf: parse license config
>>> license.conf/bbclass: Move globals to license.conf
>>>
>>> meta/classes/license.bbclass | 67
>>> +-----------------------------------------
>>> meta/conf/bitbake.conf | 1 +
>>> meta/conf/licenses.conf | 64
>>> ++++++++++++++++++++++++++++++++++++++++
>>> 3 files changed, 66 insertions(+), 66 deletions(-)
>>>
>>
>
>
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 0/3] license.bbclass fixes
2012-09-26 21:50 [PATCH 0/3] license.bbclass fixes Elizabeth Flanagan
` (3 preceding siblings ...)
2012-09-27 17:08 ` [PATCH 0/3] license.bbclass fixes Saul Wold
@ 2012-09-28 10:20 ` Richard Purdie
4 siblings, 0 replies; 9+ messages in thread
From: Richard Purdie @ 2012-09-28 10:20 UTC (permalink / raw)
To: Elizabeth Flanagan; +Cc: openembedded-core
On Wed, 2012-09-26 at 14:50 -0700, Elizabeth Flanagan wrote:
> [YOCTO #3118]
>
> license.conf hasn't been being parsed. This commit series ensures that
> license.conf is parsed, that it contains useful variables that should be
> in a global conf file and that those variables are not kept within
> license.bbclass.
>
> This also fixes an discreprancy between the documentation of LICENSE_PATH (which
> is what we tell people to use) and what we were using "LICENSE_DIR". This ensures
> that when the end user needs to add additional licenses, that they use
> LICENSE_PATH += "path/to/additional/licenses" and that it works.
>
> The following changes since commit 0b09e50810162a07ef0aecee91ee32b4a36334a3:
>
> busybox: Fix misplaced quote (2012-09-26 18:28:36 +0100)
>
> are available in the git repository at:
> git://git.yoctoproject.org/poky-contrib eflanagan/LICENSE_PATH
> http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=eflanagan/LICENSE_PATH
>
> Elizabeth Flanagan (3):
> license.bbclass: Variable standardization
> bitbake.conf: parse license config
> license.conf/bbclass: Move globals to license.conf
Merged to master, thanks.
Richard
^ permalink raw reply [flat|nested] 9+ messages in thread