Re: [PATCH] nativesdk-buildtools-perl-dummy.bb: disable ICECC

[Alex Franco <alejandro.franco@linux.intel.com>]

[-- Attachment #1.1.1: Type: text/plain, Size: 1619 bytes --]

Ross, I suppose you mean " icc_is_allarch()" ?

Ok , so this recipe does inherit allarch but it is sort of nativesdk,
and when the test icc_is_allarch() checks for the value of
'PACKAGE_ARCH', it gets "buildtools-dummy-${SDKPKGSUFFIX}" (which should
expand to 'buildtools-dummy-nativesdk', but doesn't) instead of the
expected "all".

I am now modifying this test to check for an inherit of allarch, and
playing a bit with this. I'll keep you posted.

Other stuff to consider:

- Expand the value of PACKAGE_ARCH if necessary? I am doing it now as
part of my tests but not sure I will be committing this:

     package_arch = d.getVar("PACKAGE_ARCH", False)
     if '${' in package_arch:
         package_arch = d.expand(package_arch)
    ...

- All of these functions in the icecc.bbclass that incude "icc" in their
names, should probably be modified to say "icecc" instead. As it stands
now, it can cause confusion as some people might think we're talking
about the intel c compiler rather than the icecream compiler - I wonder
if there's a bug for this already

Alex Franco

On 03/22/2016 10:45 AM, Burton, Ross wrote:
>
> On 22 March 2016 at 16:41, Burton, Ross <ross.burton@intel.com
> <mailto:ross.burton@intel.com>> wrote:
>
>     Do we understand why this recipe breaks the icecc class?
>
>
> So I'm guessing it's because the architecture of the recipe is
> "special" - it's sort-of allarch and sort of nativesdk.
>
> I wonder if changing the icecc_is_allarch() test to check for inherit
> of allarch would fix this without working around the problem.
>
> Ross 
>


[-- Attachment #1.1.2: Type: text/html, Size: 2995 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 484 bytes --]