[U-Boot] [PATCH v1 1/2] buildman: Extract environment as part of each build

[Alex Kiernan]

On Fri, May 25, 2018 at 9:05 AM Lukasz Majewski <lukma at denx.de> wrote:

> Hi Alex,

> > As we're building the boards, extract the default U-Boot environment
> > to uboot.env so we can interrogate it later.

> If applicable, you may also consider using:
> scripts/get_default_envs.sh

> script to extract default envs (to txt file).


I started with that, but then tripped over boards which have embedded
newlines in their environment variables, which then made parsing the
resulting text file much harder, which is why I swapped back to just using
the NUL terminated blob. Of course I can't now find the one I tripped over
to point at :(

> >
> > Signed-off-by: Alex Kiernan <alex.kiernan at gmail.com>
> > ---
> >
> >  tools/buildman/builderthread.py | 10 ++++++++++
> >  1 file changed, 10 insertions(+)
> >
> > diff --git a/tools/buildman/builderthread.py
> > b/tools/buildman/builderthread.py index 0efe80d945..688322ced2 100644
> > --- a/tools/buildman/builderthread.py
> > +++ b/tools/buildman/builderthread.py
> > @@ -351,6 +351,16 @@ class BuilderThread(threading.Thread):
> >                      lines.append(size_result.stdout.splitlines()[1]
> > + ' ' + rodata_size)
> >
> > +            # Extract the environment from U-Boot and dump it out
> > +            cmd = ['%sobjcopy' % self.toolchain.cross, '-O',
> > 'binary',
> > +                   '-j', '.rodata.default_environment',
> > +                   'env/built-in.o', 'uboot.env']
> > +            command.RunPipe([cmd], capture=False,
> > +                            capture_stderr=False, cwd=result.out_dir,
> > +                            raise_on_error=False, env=env)
> > +            ubootenv = os.path.join(result.out_dir, 'uboot.env')
> > +            self.CopyFiles(result.out_dir, build_dir, '',
> > ['uboot.env']) +
> >              # Write out the image sizes file. This is similar to the
> > output # of binutil's 'size' utility, but it omits the header line and
> >              # adds an additional hex value at the end of each line
> > for the




> Best regards,

> Lukasz Majewski

> --

> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de



-- 
Alex Kiernan