Re: [PATCH v2 2/2] cross-compilation: Fixes build broken with host-distro != target-distro

[Jan Kiszka <jan.kiszka@siemens.com>]

On 2018-09-05 10:29, [ext] Jan Kiszka wrote:
> On 2018-09-05 10:16, Maxim Yu. Osipov wrote:
>> From: Jan Kiszka <jan.kiszka@siemens.com>
>>
>> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
> 
> It's good the see that my workaround is apparently also a solution. But 
> this still lacks the explanation, and I'm still unable to provide that. 
> So please help, specifically by answering the questions I raised in the 
> original bug report.
> 

[...]

>> --- a/meta/recipes-devtools/buildchroot/buildchroot-host.bb
>> +++ b/meta/recipes-devtools/buildchroot/buildchroot-host.bb
>> @@ -30,5 +30,5 @@ BUILDCHROOT_PREINSTALL_append_arm64 += 
>> "binutils-aarch64-linux-gnu \
>>                                           crossbuild-essential-arm64"
>> -PARAMS = "--host-arch"
>> +PARAMS = "--host-arch --host-distro"
>>   do_build[depends] = "isar-apt:do_cache_config 
>> isar-bootstrap-host:do_bootstrap"
>>

For this part of the commit, I originally had the following written in 
my queue:

"Because we were not passing --host-distro to setup_root_file_system,
that helper defaulted to the target distro. And that broke the
crossbuild if the target distro differed from the host one."

If you can provide a reasoning for the first half, either in this patch 
or in form of a separate one (I suspect that this will look nicer), I 
would be happy.

Jan

-- 
Siemens AG, Corporate Technology, CT RDA IOT SES-DE
Corporate Competence Center Embedded Linux