Re: [PATCH 3/4] ci: Include de0-nano-soc target into fast build

[Claudius Heine <claudius.heine.ext@siemens.com>]

Hi Henning,

On 18/02/2019 11.18, [ext] Henning Schild wrote:
> Am Fri, 15 Feb 2019 08:20:19 +0100
> schrieb "[ext] Jan Kiszka" <jan.kiszka@siemens.com>:
> 
>> On 14.02.19 17:08, Maxim Yu. Osipov wrote:
>>> Note: So far de0-nano-soc build is launched separately as
>>> parallel build with the same target arch (qemuarm) fails.
>>
>> jailhouse-images builds multiple images out of the same arch for
>> quite a while now. You may want to cross-check and adopt the patterns
>> there.
>>
>> Even if unrelated to this issue: A proper multiconfig file only
>> contains two assignments: MACHINE and DISTRO. This should be
>> eventually cleaned up in meta-isar so that user do not copy the wrong
>> patterns.
> 
> It is most likely false-sharing on multiconfig. Now if we look for
> "[stamp-extra-info]" in the source-tree we will find several
> combinations.
> 
> "${DISTRO}-${DISTRO_ARCH}"
> "${DISTRO}-${MACHINE}"
> "${MACHINE}-${DISTRO}"
> "${DISTRO}"
> 
> And possibly several tasks that should have an extra-stamp but do not.

Good point. That is also something that should be looked at before a 1.0 
release ;)

> Avoiding multiconfig is a very good idea, not just for CI ;).

I agree. Another way would be to set the tmp directory in the 
multiconfig like the bitbake manual documents:

https://www.yoctoproject.org/docs/latest/bitbake-user-manual/bitbake-user-manual.html#executing-a-multiple-configuration-build

regards,
Claudius

> 
> Henning
> 
>> Jan
>>
>>> While the problem with parallel build is being investigated
>>> de0-nano-soc build has to be checked by CI due to high
>>> importance.
>>>
>>> Signed-off-by: Maxim Yu. Osipov <mosipov@ilbers.de>
>>> ---
>>>    scripts/ci_build.sh | 7 +++++++
>>>    1 file changed, 7 insertions(+)
>>>
>>> diff --git a/scripts/ci_build.sh b/scripts/ci_build.sh
>>> index acce152..9c2322f 100755
>>> --- a/scripts/ci_build.sh
>>> +++ b/scripts/ci_build.sh
>>> @@ -126,6 +126,13 @@ fi
>>>    # Start build for the defined set of configurations
>>>    bitbake $BB_ARGS $TARGETS_SET
>>>    
>>> +# Note: de0-nano-soc build is launched separately as
>>> +# parallel build with the same target arch (armhf) fails.
>>> +# The problem is being investigated
>>> +if [ -n "$FAST_BUILD" ]; then
>>> +    bitbake $BB_ARGS
>>> multiconfig:de0-nano-soc-stretch:isar-image-base +fi
>>> +
>>>    cp -a "${ISARROOT}/meta/classes/dpkg-base.bbclass"
>>> "${ISARROOT}/meta/classes/dpkg-base.bbclass.ci-backup" echo -e
>>> "do_fetch_append() {\n\n}" >>
>>> "${ISARROOT}/meta/classes/dpkg-base.bbclass"
>>
> 

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-54 Fax: (+49)-8142-66989-80 Email: ch@denx.de