[oe] [meta-java][PATCH v3] openjdk-8: ignore wrong GCCVERSION-flag

[Maxin B. John]

Hi Sven,

On Mon, Aug 01, 2016 at 06:19:05AM +0000, Ebenfeld, Sven wrote:
> What is about this patch?
> I see it's archived but not merged. My builds of the fido branch are failing
> without this patch.

Pushed to master-next. Sorry for the delay.

> Best Regards,
> 
> Sven Ebenfeld
> IRE-S CoC Connectivity & Controls - Technology Advisor Network Technologies
> & Security
> Vaillant

Best Regards,
Maxin

> 
> -----Original Message-----
> From: Sven Ebenfeld [mailto:sven.ebenfeld at vaillant-group.com] 
> Sent: Monday, July 25, 2016 8:17 AM
> To: openembedded-devel at lists.openembedded.org
> Cc: Ebenfeld, Sven <sven.ebenfeld at vaillant-group.com>
> Subject: [meta-java][PATCH v3] openjdk-8: ignore wrong GCCVERSION-flag
> 
> During my recipe parsing, GCCVERSION in the cross case seems to be 'l' this
> leads to a ValueError during recipe parsing. Therefore check type before
> trying to parse the version flag.
> 
> Signed-off-by: Sven Ebenfeld <sven.ebenfeld at vaillant-group.com>
> ---
>  recipes-core/openjdk/openjdk-8-common.inc | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/recipes-core/openjdk/openjdk-8-common.inc
> b/recipes-core/openjdk/openjdk-8-common.inc
> index 7ad802a..c2f9eb4 100644
> --- a/recipes-core/openjdk/openjdk-8-common.inc
> +++ b/recipes-core/openjdk/openjdk-8-common.inc
> @@ -271,8 +271,10 @@ def version_specific_cflags(d):
>          # doesn't work anyway.
>          version = d.getVar('GCCVERSION', expand=True)[0]
>  
> -    extraflags = d.getVar('FLAGS_GCC%d' % int(version), True) or ''
> -    return extraflags
> +    if version.isdigit():
> +        extraflags = d.getVar('FLAGS_GCC%d' % int(version), True) or ''
> +        return extraflags
> +    return ''
>  
>  CFLAGS_append = " ${@version_specific_cflags(d)}"
>  CXXFLAGS_append = " ${@version_specific_cflags(d)}"
> --
> 1.9.1
> 



> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel