[PATCH 1/2] tools: binman: main.py: add build-sandbox in sys.path

[Simon Glass]

Hi Philippe

On Thu, 27 Jan 2022 at 07:03, Philippe Reynes
<philippe.reynes at softathome.com> wrote:
>
> Adds build-sandbox in sys.path to look for libfdt,
> otherwise py_test can't use binman.
>
> Signed-off-by: Philippe Reynes <philippe.reynes at softathome.com>
> ---
>  tools/binman/main.py | 1 +
>  1 file changed, 1 insertion(+)

Reviewed-by: Simon Glass <sjg at chromium.org>

>
> diff --git a/tools/binman/main.py b/tools/binman/main.py
> index 35944f314a..f62394043d 100755
> --- a/tools/binman/main.py
> +++ b/tools/binman/main.py
> @@ -40,6 +40,7 @@ from patman import test_util
>  # Bring in the libfdt module
>  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'))
>
>  # When running under python-coverage on Ubuntu 16.04, the dist-packages
> --
> 2.17.1
>

The build-sandbox_spl path should be enough though, shouldn't it? Have
you not built that?

Regards,
Simon