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

[Simon Glass]

Hi Philippe,

On Thu, 27 Jan 2022 at 08:15, Philippe REYNES
<philippe.reynes at softathome.com> wrote:
>
> Hi Simon,
>
> Le 27/01/2022 à 16:06, Simon Glass a écrit :
> > 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?
>
>
> I have added a test that use binman in vboot (for global image signature,
> a header added before the image). This path should be added for the CI,
> otherwise tests "sandbox test.pyt " and "sandbox with clan test.py" fail.

But the tests should run in the board directory provided to pytest.

I will have to look at your test.

>
>
> >> 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
> Regards,
> Philippe
>
>
> -- This message and any attachments herein are confidential, intended solely for the addressees and are SoftAtHome’s ownership. Any unauthorized use or dissemination is prohibited. If you are not the intended addressee of this message, please cancel it immediately and inform the sender.

Is this supposed to be sent on the mailing list?

Regards,
Simon