On Wed, Mar 12, 2025 at 12:09:44PM +0100, Mark Wielaard wrote:
> Hi David,
>
> On Mon, Mar 10, 2025 at 05:40:18PM -0400, David Malcolm wrote:
> > FWIW gcc/cobol/lang.opt.urls has some D-specific things that look like
> > copy-and-paste cruft, but hopefully it won't cause problems.
>
> And some Fortran stuff. The autoregen bot currently flags it with the
> following diff:
>
> diff --git a/gcc/cobol/lang.opt.urls b/gcc/cobol/lang.opt.urls
> index a0e1f1944fe..6a5dc1c0f58 100644
> --- a/gcc/cobol/lang.opt.urls
> +++ b/gcc/cobol/lang.opt.urls
> @@ -10,20 +10,27 @@ UrlSuffix(gcc/Preprocessor-Options.html#index-D-1)
> I
> UrlSuffix(gcc/Directory-Options.html#index-I)
> LangUrlSuffix_D(gdc/Directory-Options.html#index-I)
>
> +ffixed-form
> +LangUrlSuffix_Fortran(gfortran/Fortran-Dialect-Options.html#index-ffixed-form)
> +
> fsyntax-only
> -UrlSuffix(gcc/Warning-Options.html#index-fsyntax-only)
> LangUrlSuffix_D(gdc/Warnings.html#index-fno-syntax-only)
> +UrlSuffix(gcc/Warning-Options.html#index-fsyntax-only)
> LangUrlSuffix_D(gdc/Warnings.html#index-fno-syntax-only)
> LangUrlSuffix_Fortran(gfortran/Error-and-Warning-Options.html#index-fsyntax-only)
> +
> +ffree-form
> +LangUrlSuffix_Fortran(gfortran/Fortran-Dialect-Options.html#index-ffree-form)
>
> fmax-errors
> UrlSuffix(gcc/Warning-Options.html#index-fmax-errors)
> LangUrlSuffix_D(gdc/Warnings.html#index-fmax-errors)
>
> iprefix
> -UrlSuffix(gcc/Directory-Options.html#index-iprefix)
> LangUrlSuffix_D(gdc/Directory-Options.html#index-iprefix)
> +UrlSuffix(gcc/Directory-Options.html#index-iprefix)
> LangUrlSuffix_D(gdc/Directory-Options.html#index-iprefix)
> LangUrlSuffix_Fortran(gfortran/Preprocessing-Options.html#index-iprefix)
>
> include
> UrlSuffix(gcc/Preprocessor-Options.html#index-include)
>
> isysroot
> -UrlSuffix(gcc/Directory-Options.html#index-isysroot)
> +UrlSuffix(gcc/Directory-Options.html#index-isysroot)
> LangUrlSuffix_Fortran(gfortran/Preprocessing-Options.html#index-isysroot)
>
> isystem
> -UrlSuffix(gcc/Directory-Options.html#index-isystem)
> +UrlSuffix(gcc/Directory-Options.html#index-isystem)
> LangUrlSuffix_Fortran(gfortran/Preprocessing-Options.html#index-isystem)
> +
>
> I don't really understand what these D and Fortran references are
> for. Are these expected? Should we simply regenerate them with the
> above diff? Or does the make regenerate-opt-urls target need an
> update?
I'd commit this patch as is if it makes the CI bot happy.
Longer term, the COBOL docs need to be sorted out (see e.g. PR119227)
and then perhaps regenerate-opt-urls.py adjusted so that it can
deal with the COBOL docs.
Jakub
