Re: [PATCH] bitbake: Allow setting DL_DIR from environment

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

On 06.11.18 12:06, Harald Seiler wrote:
> On Wed, 2018-10-31 at 07:39 +0100, Jan Kiszka wrote:
>> On 30.10.18 17:44, Harald Seiler wrote:
>>> This patch allows adding DL_DIR to BB_ENV_EXTRAWHITE
>>> and then specifying a custom DL_DIR in the environment.
>>>
>>> This behavior mirrors the one in poky.
>>>
>>> Signed-off-by: Harald Seiler <hws@denx.de>
>>> ---
>>>    bitbake/conf/bitbake.conf   | 2 +-
>>
>> We do not patch bitbake so far. Is this fix in upstream already? Then let's
>> fast-forward bitbake to that revision. If not, send it upstream first. Local,
>> isar-specific changes go to isar-bitbake.conf.
> 
> Upstream (bitbake) does not have this fix.  The reason it works in poky is that
> they have removed the `bitbake/conf/bitbake.conf` file completely and instead
> use their own config in `meta/conf/bitbake.conf`.
> 
> I am not sure why this is done this way, though.
> 

Then I would say - in order to keep bitbake/ clean - we should promote 
isar-bitbake.conf to the only bitbake.conf we use, migrating current 
bitbake.conf content over.

But maybe it is still worth try to convince upstream bitbake of this tiny change.

Jan

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