[OE-core] [PATCH][fido] classes/license.bbclass: Improve generic license copy validation.

Joshua Lock joshua.lock at collabora.co.uk
Fri Jun 26 14:30:08 UTC 2015


On Wed, 2015-06-24 at 11:49 -0500, Aníbal Limón wrote:
> Remove + char in any position for cover cases when license has
> the form like GPL-2.0+-with-OpenSSL-exception.
> 
> [YOCTO #7584]

Thanks Aníbal,

This is queued in my fido-next branch[1].

Regards,

Joshua

1. http://cgit.openembedded.org/openembedded-core
-contrib/log/?h=joshuagl/fido-next

> 
> Signed-off-by: Aníbal Limón <anibal.limon at linux.intel.com>
> ---
>  meta/classes/license.bbclass | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/classes/license.bbclass 
> b/meta/classes/license.bbclass
> index 73a0e97..e00a695 100644
> --- a/meta/classes/license.bbclass
> +++ b/meta/classes/license.bbclass
> @@ -71,8 +71,12 @@ license_create_manifest() {
>  
>  		lics="$(echo ${pkged_lic} | sed "s/[|&()*]/ /g" | 
> sed "s/  */ /g" )"
>  		for lic in ${lics}; do
> +			# remove + chars in any position this cover 
> cases like
> +			# GPL-2.0+-with-OpenSSL-exception -> GPL-2.0
> -with-OpenSSL-exception
> +			lic="$(echo ${lic} | sed "s/\+//g")"
> +
>  			# to reference a license file trim trailing 
> + symbol
> -			if ! [ -e 
> "${LICENSE_DIRECTORY}/${pkged_pn}/generic_${lic%+}" ]; then
> +			if ! [ -e 
> "${LICENSE_DIRECTORY}/${pkged_pn}/generic_${lic}" ]; then
>  				bbwarn "The license listed ${lic} 
> was not in the licenses collected for ${pkged_pn}"
>  			fi
>  		done
> -- 
> 1.9.1
> 



More information about the Openembedded-core mailing list