Hi Philippe,

On Thu, 27 Jan 2022 at 07:03, Philippe Reynes
<philippe.rey...@softathome.com> wrote:
>
> Adds /tmp/sandbox to sys.path to look for libfdt,
> otherwise ci is broken if a py_test uses binman.
>
> Signed-off-by: Philippe Reynes <philippe.rey...@softathome.com>
> ---
>  tools/binman/main.py | 1 +
>  1 file changed, 1 insertion(+)

Which test uses /tmp/sandbox to build U-Boot? We might need to use
--toolpath for this, or set PYTHONPATH in the caller?

>
> diff --git a/tools/binman/main.py b/tools/binman/main.py
> index f62394043d..7d52482c0c 100755
> --- a/tools/binman/main.py
> +++ b/tools/binman/main.py
> @@ -42,6 +42,7 @@ sys.path.insert(2, 'scripts/dtc/pylibfdt')
>  sys.path.insert(2, os.path.join(srctree, 'scripts/dtc/pylibfdt'))
>  sys.path.insert(2, os.path.join(srctree, 
> 'build-sandbox/scripts/dtc/pylibfdt'))
>  sys.path.insert(2, os.path.join(srctree, 
> 'build-sandbox_spl/scripts/dtc/pylibfdt'))
> +sys.path.insert(2, os.path.join(srctree, 
> '/tmp/sandbox/scripts/dtc/pylibfdt'))
>
>  # When running under python-coverage on Ubuntu 16.04, the dist-packages
>  # directories are dropped from the python path. Add them in so that we can 
> find
> --
> 2.17.1
>

Regards,
Simon

Reply via email to