Re: [PATCH] Add binutils-gdb-autoregen builder

[Mark Wielaard <mark@klomp.org>]

Hi Frank,

On Sun, Nov 12, 2023 at 11:49:11AM -0500, Frank Ch. Eigler wrote:
> > Generalize the autoregen.py script by searching for 'configure.ac'
> > instead of 'configure'. Skip "foreign" readline and minizip dirs.
> > Make autoregen_step and git_diff_step generic, define early.
> > Add binutils-gdb-autoregen builder to binutils_gdb_scheduler.
> 
> > +autoregen_step = steps.ShellCommand(
> > +        command="autoregen.py",
> > +        name="autoregen.py",
> > +        haltOnFailure=True)
> 
> Note that this stuff literally can only add work for the developers.
> If the tooling knows what the output purportedly should be, it is in a
> position to correct the mismatches, not merely complain about it.
> That would actually save human effort.

The build step does generate a diff that when applies makes sure the
files are regenerated with the correct version of the tools. The
buildbots however don't have write access, so cannot apply and push
the patch itself.

The container can also be used to generate the same locally, see
https://inbox.sourceware.org/20231112171138.1553448-1-mark@klomp.org/T/

So it could also be done through some other mechanism which has direct
write access to the git repro. But you'll have to negotiate with the
projects to see if they want that.

Cheers,

Mark