[U-Boot] [PATCH v2 03/15] buildman: Convert to Python 3

[Simon Glass]

Hi Tom,

On Wed, 30 Oct 2019 at 07:15, Tom Rini <trini at konsulko.com> wrote:
>
> On Tue, Oct 29, 2019 at 07:47:55PM -0600, Simon Glass wrote:
> > Hi Tom,
> >
> > On Thu, 24 Oct 2019 at 19:04, Simon Glass <sjg at chromium.org> wrote:
> > >
> > > Convert buildman to Python 3 and make it use that, to meet the 2020
> > > deadline.
> > >
> > > Signed-off-by: Simon Glass <sjg at chromium.org>
> > > ---
> > >
> > > Changes in v2: None
> > >
> > >  tools/buildman/board.py         |  7 +--
> > >  tools/buildman/bsettings.py     | 20 +++----
> > >  tools/buildman/builder.py       | 45 ++++++++--------
> > >  tools/buildman/builderthread.py | 24 ++++-----
> > >  tools/buildman/buildman.py      | 10 ++--
> > >  tools/buildman/control.py       | 40 +++++++-------
> > >  tools/buildman/func_test.py     | 16 +++---
> > >  tools/buildman/test.py          | 22 ++++----
> > >  tools/buildman/toolchain.py     | 93 +++++++++++++++++----------------
> > >  9 files changed, 139 insertions(+), 138 deletions(-)
> >
> > I am a bit mystified by this:
> >
> > https://gitlab.denx.de/u-boot/custodians/u-boot-dm/-/jobs/25186
> >
> > It runs locally and I cannot see why it is not using unicode. Are
> > there instructions on how to get to the same Python version as gitlab?
> > I fear a steep learning curve coming.
>
> First, I think the problem is this series will also need the test.py
> python3 series as that does a few more things to get and provide a
> python3 environment to use.  I'll try and throw something together to
> test that shortly.
>
> Second, to have the same environment locally:
> $ sudo docker pull trini/u-boot-gitlab-ci-runner:bionic-20191010-20Oct2019
>
> And change the tag to whatever .gitlab-ci.yml says we use.  That one is
> what the test.py series brings us up to, and is newer than your failing
> job.
>
> $ sudo docker run --rm -it -v /your/local/u-boot/clone/..:/build/u-boot \
>     trini/u-boot-gitlab-ci-runner:bionic-20191010-20Oct2019 /bin/bash
>
> And don't pass --rm if you don't want the image deleted once you exit.
> That will put you in a bash prompt as the user we run everything as.
> Follow the steps in the job as described in .gitlab-ci.yml to do a build
> by hand.
>
> And yes, you want to pass in one level before your clone so that in the
> container you have /build/u-boot/u-boot

Thank you, that is very helpful!

It seems like the default file encoding for open() is different. I'll
try again and hopefully send a v2 series soon.

Regards,
Simon