[PATCH 2/2] tools: binman: main.py: add /tmp/sandbox sys.path

Simon Glass sjg at chromium.org
Thu Jan 27 16:06:06 CET 2022


Hi Philippe,

On Thu, 27 Jan 2022 at 07:03, Philippe Reynes
<philippe.reynes at 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.reynes at 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


More information about the U-Boot mailing list