On Tue, 2023-11-21 at 02:09 +0100, Hans-Peter Nilsson wrote: > > From: David Malcolm <dmalc...@redhat.com> > > Date: Thu, 16 Nov 2023 09:28:54 -0500 > > > How is this looking for trunk? > > > > Thanks > > Dave > > > > > > David Malcolm (4): > > options: add gcc/regenerate-opt-urls.py > > Add generated .opt.urls files > > opts: add logic to generate options-urls.cc > > options: wire up options-urls.cc into gcc_urlifier > > > > gcc/Makefile.in | 29 +- > > gcc/ada/gcc-interface/lang.opt.urls | 30 + > > gcc/analyzer/analyzer.opt.urls | 206 ++ > > gcc/c-family/c.opt.urls | 1409 ++++++++++++++ > > gcc/common.opt.urls | 1832 > > ++++++++++++++++++ > > gcc/config/aarch64/aarch64.opt.urls | 84 + > > gcc/config/alpha/alpha.opt.urls | 76 + > > gcc/config/alpha/elf.opt.urls | 2 + > > gcc/config/arc/arc-tables.opt.urls | 2 + > > [... etc .opt.urls particularly in gcc/config/* > autogenerated for each *.opt ...] > > Sorry for barging in though I did try finding the relevant > discussion, but is committing this generated stuff necessary? > Is it because we don't want to depend on Python being > present at build time?
Partly, yes, but also because this information is generated from the generated HTML for the user manuals: gcc, gdc, and gfortran, and it gets used to generate "optionlist", which is a dependency for all of the rest of the "gcc" subdirectory. Doing it automatically would make the generation of the HTML docs for gcc, d and gfortran be a hard dependency early on in the build, which seems to me to be a bad idea - better to have this rarely-changing and non-critical information be regenerated when it's needed, and not impose the requirement to build the HTML docs for all these langs on all builds of gcc. > > If nothing else, can you add a few lines to the commit > message why this can't be/is preferably not done at build > time? The wording here: https://gcc.gnu.org/pipermail/gcc-patches/2023-November/636060.html may be what you're looking for. Maybe that text (or something like it) should go in as a big comment at the top of regenerate-opt-urls.py ? Hope this clarifies things Dave