Re: [PATCH 0/4 v5] Isar apt deployment

[Alexander Smirnov <asmirnov@ilbers.de>]

On 10/19/2017 07:16 PM, Henning Schild wrote:
> On Wed, 18 Oct 2017 20:14:18 +0300
> Alexander Smirnov <asmirnov@ilbers.de> wrote:
> 
>> On 10/18/2017 08:10 PM, Alexander Smirnov wrote:
>>> On 10/18/2017 04:44 PM, Henning Schild wrote:
>>>> On Wed, 18 Oct 2017 14:06:53 +0300
>>>> Alexander Smirnov <asmirnov@ilbers.de> wrote:
>>>>   
>>>>> Hi,
>>>>>   
>>>>>> i just tried it and the task do_cache_config was not executed
>>>>>> before the first do_populate.
>>>>>
>>>>> Could you please specify more details:
>>>>>
>>>>>     - Build command
>>>>
>>>> Well i am using layering, and so is Claudius. So the command wont
>>>> help you much. It should basically boil down to
>>>>
>>>> bitbake multiconfig:qemuamd64-stretch:isar-image-base
>>>>   
>>>>>     - Commit ID
>>>>
>>>> f33e48bd07039032e3
>>>>
>>>> I am guessing you can not reproduce the issue, we should find out
>>>> why. But since Claudius confirmend that he can not use these
>>>> patches either, and there are still some open points, i think you
>>>> should remove those patches from next.
>>>>   
>>>
>>> At first, as I understand you use private layering. So if something
>>> doesn't work in your custom environment, without details and
>>> analysis it's not the case to reset 'next' branch.
>>>
>>> Before perform any merge, apart from my local builds, the reference
>>> build on CI machine is also passed. So the upstream Isar does
>>> support the claimed features.
>>>
>>> If I'm not able to reproduce your issue and there is no information
>>> about problem, then I'm not able to fix it. So if you need
>>> assistance in resolving the issue, please provide more details. If
>>> the problem will be found, the fix should be sent.
>>
>> Please try the patch attached.
> 
> That solved the problem but introduced the new problem that do_rootfs
> could start before do_fetch/unpack.
> 
> See my fix here:
> https://github.com/henning-schild-work/isar/commit/1e1092be974dbffbf87ab943031c44481ac0908c
> 

Merged to v5, thanks.

Alex