On Fri, 2023-12-15 at 01:59 +0100, Mark Wielaard wrote:
> Hi David,
> 
> On Thu, Dec 14, 2023 at 10:01:39AM -0500, David Malcolm wrote:
> > > Once your patch is in please feel free to sent an email to
> > > buildbot@sourceware.org
> > > https://sourceware.org/mailman/listinfo/buildbot
> > > And we'll add the above build steps and update the autotools
> > > Containerfile to include the fortran (gfortran?) and d (gdc?)
> > > build
> > > dependencies.
> > 
> > Joseph: it seems that we have a way to add CI for this.
> > 
> > I refreshed the patches and successfully bootstrapped & regrtested
> > them
> > on x86_64-pc-linux-gnu; here's the v3 version of them.
> > 
> > Are these OK for trunk, assuming I followup with adding CI for
> > this?
> > (that said, I disappear for the rest of 2023 at the end of this
> > week, so
> > I'd work on the CI in early January)
> 
> I will be around next week to fixup any CI issues.
> But once you commit this we can immediate activate the check.
> 
> I have attached a patch for the gcc-autoregen builder to also do
> regenerate-opt-urls. Since it is a --disable-bootstrap build and uses
> ccache it should take just a few minutes. So can be done on every
> commit.

Thanks - this looks great.

> 
> Note that with you patch applied to master it does flag and generate
> the attached diff (I assume that is expected).

As it happens, it wasn't - the test detected that I messed up when I
regenerated the .opt.urls.  Mea culpa, and I guess that's a win for CI.

I now see that the jit html documentation is failing to build on my
machine [1].  So what I think happened is that I ran "make html"
(without -k), it failed on the jit docs, and I didn't notice, and
regenerated the .opt.urls with an old copy of the texinfo-generated
html for gcc, d, and fortran.

I'm attaching a regenerated version of patch 2 of the kit (regenerated
using 863df360fb6549ae35af9f052343d2578d4d2b63); I checked it by hand
and it includes all the entries your CI script indicated were missing

Dave

[1]v fwiw with ImportError: cannot import name 'environmentfilter' from
'jinja2' (/usr/local/lib64/python3.8/site-packages/jinja2/__init__.py)