Re: [oe] [PATCH 1/1] lvm2: disable thin provisioning support if GPLv3 incompatible

2017-09-22 Thread Kang Kai 

On 2017年09月22日 03:44, Andre McCurdy wrote:

On Thu, Sep 21, 2017 at 7:16 AM, Khem Raj  wrote:

On Wed, Sep 20, 2017 at 11:47 PM,   wrote:

From: Kai Kang 

Disable thin provisioning support for lvm2 if it is set GPLv3
incompatible.

Signed-off-by: Kai Kang 
---
  meta-oe/recipes-support/lvm2/lvm2.inc | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-oe/recipes-support/lvm2/lvm2.inc 
b/meta-oe/recipes-support/lvm2/lvm2.inc
index ab8db9cba..9791554a9 100644
--- a/meta-oe/recipes-support/lvm2/lvm2.inc
+++ b/meta-oe/recipes-support/lvm2/lvm2.inc
@@ -23,7 +23,7 @@ inherit autotools-brokensep pkgconfig systemd
  LVM2_PACKAGECONFIG = "dmeventd lvmetad"
  LVM2_PACKAGECONFIG_append_class-target = " \
  ${@bb.utils.filter('DISTRO_FEATURES', 'selinux', d)} \
-thin-provisioning-tools \
+${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '', 
'thin-provisioning-tools', d)} \
  udev \
  "

there are other strings like GPL-3.0, as well as other types of GPL 3
derivative licenses, I wonder if this will work for all

Perhaps something like:

   ${@incompatible_license_contains('GPLv3', '', 'thin-provisioning-tools', d)}

would work more reliably?


OK. Thanks. I'll send V2.


--Kai




___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel



--
Regards,
Neil | Kai Kang

--
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Re: [oe] [PATCH 1/1] lvm2: disable thin provisioning support if GPLv3 incompatible

2017-09-21 Thread Andre McCurdy 
On Thu, Sep 21, 2017 at 7:16 AM, Khem Raj  wrote:
> On Wed, Sep 20, 2017 at 11:47 PM,   wrote:
>> From: Kai Kang 
>>
>> Disable thin provisioning support for lvm2 if it is set GPLv3
>> incompatible.
>>
>> Signed-off-by: Kai Kang 
>> ---
>>  meta-oe/recipes-support/lvm2/lvm2.inc | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta-oe/recipes-support/lvm2/lvm2.inc 
>> b/meta-oe/recipes-support/lvm2/lvm2.inc
>> index ab8db9cba..9791554a9 100644
>> --- a/meta-oe/recipes-support/lvm2/lvm2.inc
>> +++ b/meta-oe/recipes-support/lvm2/lvm2.inc
>> @@ -23,7 +23,7 @@ inherit autotools-brokensep pkgconfig systemd
>>  LVM2_PACKAGECONFIG = "dmeventd lvmetad"
>>  LVM2_PACKAGECONFIG_append_class-target = " \
>>  ${@bb.utils.filter('DISTRO_FEATURES', 'selinux', d)} \
>> -thin-provisioning-tools \
>> +${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '', 
>> 'thin-provisioning-tools', d)} \
>>  udev \
>>  "
> there are other strings like GPL-3.0, as well as other types of GPL 3
> derivative licenses, I wonder if this will work for all

Perhaps something like:

  ${@incompatible_license_contains('GPLv3', '', 'thin-provisioning-tools', d)}

would work more reliably?

> ___
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Re: [oe] [PATCH 1/1] lvm2: disable thin provisioning support if GPLv3 incompatible

2017-09-21 Thread Khem Raj 
On Wed, Sep 20, 2017 at 11:47 PM,   wrote:
> From: Kai Kang 
>
> Disable thin provisioning support for lvm2 if it is set GPLv3
> incompatible.
>
> Signed-off-by: Kai Kang 
> ---
>  meta-oe/recipes-support/lvm2/lvm2.inc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta-oe/recipes-support/lvm2/lvm2.inc 
> b/meta-oe/recipes-support/lvm2/lvm2.inc
> index ab8db9cba..9791554a9 100644
> --- a/meta-oe/recipes-support/lvm2/lvm2.inc
> +++ b/meta-oe/recipes-support/lvm2/lvm2.inc
> @@ -23,7 +23,7 @@ inherit autotools-brokensep pkgconfig systemd
>  LVM2_PACKAGECONFIG = "dmeventd lvmetad"
>  LVM2_PACKAGECONFIG_append_class-target = " \
>  ${@bb.utils.filter('DISTRO_FEATURES', 'selinux', d)} \
> -thin-provisioning-tools \
> +${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '', 
> 'thin-provisioning-tools', d)} \
>  udev \
>  "
there are other strings like GPL-3.0, as well as other types of GPL 3
derivative licenses, I wonder if this will work for all
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

[oe] [PATCH 1/1] lvm2: disable thin provisioning support if GPLv3 incompatible

2017-09-21 Thread kai.kang 
From: Kai Kang 

Disable thin provisioning support for lvm2 if it is set GPLv3
incompatible.

Signed-off-by: Kai Kang 
---
 meta-oe/recipes-support/lvm2/lvm2.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-oe/recipes-support/lvm2/lvm2.inc 
b/meta-oe/recipes-support/lvm2/lvm2.inc
index ab8db9cba..9791554a9 100644
--- a/meta-oe/recipes-support/lvm2/lvm2.inc
+++ b/meta-oe/recipes-support/lvm2/lvm2.inc
@@ -23,7 +23,7 @@ inherit autotools-brokensep pkgconfig systemd
 LVM2_PACKAGECONFIG = "dmeventd lvmetad"
 LVM2_PACKAGECONFIG_append_class-target = " \
 ${@bb.utils.filter('DISTRO_FEATURES', 'selinux', d)} \
-thin-provisioning-tools \
+${@bb.utils.contains('INCOMPATIBLE_LICENSE', 'GPLv3', '', 
'thin-provisioning-tools', d)} \
 udev \
 "
 
-- 
2.14.1

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel