Re: Ping (gcc/configure.ac, docs): [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias
On Tue, 21 Mar 2023, Arsen Arsenović wrote: > Done! > > Gerald, please update the scripts when you get a chance (but back the > old ones up just in case!) Done. Minus the backup, since everything is in Git anyways, isn't it? :-) The script should run in about 1 hour and 45 minutes. > If makeinfo is updated as I've asked in one of the other emails, will > the script eventually automatically regenerate docs with the newer > makeinfo? Only what is covered by update_web_docs_git and then whenever our release managers create docs for a new release. Which makes sense since we cannot guarantee older *.texi sources actually building, or at least properly, with newer makeinfo releases. And there may be other factors, such as names of file changing,... Which makes me realize we may have an issue building releases: Joseph, you are release manager - do you and your peers create releases on a local system or gcc.gnu.org? If the former, installing newer texinfo on gcc.gnu.org is not going to be sufficient. Gerald
Re: Ping (gcc/configure.ac, docs): [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias
Arsen Arsenović via Gcc-patches writes: > Thank you! I'll do a final rebase and push in the morning. Done! Gerald, please update the scripts when you get a chance (but back the old ones up just in case!) If makeinfo is updated as I've asked in one of the other emails, will the script eventually automatically regenerate docs with the newer makeinfo? Thanks, have a lovely day. -- Arsen Arsenović signature.asc Description: PGP signature
Re: Ping (gcc/configure.ac, docs): [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias
Sandra Loosemore writes: > On 3/18/23 14:14, Arsen Arsenović via Gcc-patches wrote: >> Evening, >> Ping on this patch series. >> I believe we're close to being able to apply it, with contents of the >> documentation changes being approved, and only a small build system >> change and a revised patch series "shape" left to review. > > Yup. I think you can go ahead and commit everything but the part with the > configure/makefile hackery, which still needs somebody other than me to > approve > it. > >> The former change of these two is in: >> https://inbox.sourceware.org/gcc-patches/20230311203234.2257423-1-ar...@aarsen.me/ >> The patch adds a test for the presence of CONTENTS_OUTPUT_LOCATION into >> the build system in order to avoid shortcontents coming after contents >> in HTML output. We've also updated Texinfo to output shortcontents >> before contents by default as a result of the discussion here, though >> that will only happen in newer versions. >> The latter "shape" is a git log documented at: >> https://inbox.sourceware.org/gcc-patches/87ttysppsc@aarsen.me/ >> As always, I've freshly rebased the patch series at >>https://git.sr.ht/~arsen/gcc texinfo_improvements >> ... or, in (git)web form, at: >>https://git.sr.ht/~arsen/gcc/log/texinfo_improvements >> ... with the render at: >>https://www.aarsen.me/~arsen/final/ >> Thank you all in advance, have a lovely day. > > Joseph, could you maybe review the last piece? A direct pointer to it in > Arsen's git is > > https://git.sr.ht/~arsen/gcc/commit/bc734311cbca1085a1728f79b7eebef8cc7aeac3 > > -Sandra Thank you! I'll do a final rebase and push in the morning. -- Arsen Arsenović signature.asc Description: PGP signature
Re: Ping (gcc/configure.ac, docs): [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias
Joseph Myers writes: > On Mon, 20 Mar 2023, Sandra Loosemore wrote: > >> Joseph, could you maybe review the last piece? A direct pointer to it in >> Arsen's git is >> >> https://git.sr.ht/~arsen/gcc/commit/bc734311cbca1085a1728f79b7eebef8cc7aeac3 > > That's OK, assuming I understand correctly that makeinfo will still > succeed with a warning when it's an older version (gcc.gnu.org, where > update_web_docs_git runs, has version 6.5). It should, yes, but I'd like to ask for that server to be updated to Texinfo straight from the press (the press is currently at commit e2d30b1270deacf5a1eab1d383733a5a088827d6). Mark told me that this needs approval from the GCC admins to do. The reason for this is that the in-dev versions of Texinfo produce more accessible HTML documentation due to a few changes that I've incorporated into them, as well as a few made by the Texinfo maintainers. Do you think that would be OK? Thanks, have a lovely night. -- Arsen Arsenović signature.asc Description: PGP signature
Re: Ping (gcc/configure.ac, docs): [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias
On Mon, 20 Mar 2023, Sandra Loosemore wrote: > Joseph, could you maybe review the last piece? A direct pointer to it in > Arsen's git is > > https://git.sr.ht/~arsen/gcc/commit/bc734311cbca1085a1728f79b7eebef8cc7aeac3 That's OK, assuming I understand correctly that makeinfo will still succeed with a warning when it's an older version (gcc.gnu.org, where update_web_docs_git runs, has version 6.5). -- Joseph S. Myers jos...@codesourcery.com
Re: Ping (gcc/configure.ac, docs): [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias
On 3/18/23 14:14, Arsen Arsenović via Gcc-patches wrote: Evening, Ping on this patch series. I believe we're close to being able to apply it, with contents of the documentation changes being approved, and only a small build system change and a revised patch series "shape" left to review. Yup. I think you can go ahead and commit everything but the part with the configure/makefile hackery, which still needs somebody other than me to approve it. The former change of these two is in: https://inbox.sourceware.org/gcc-patches/20230311203234.2257423-1-ar...@aarsen.me/ The patch adds a test for the presence of CONTENTS_OUTPUT_LOCATION into the build system in order to avoid shortcontents coming after contents in HTML output. We've also updated Texinfo to output shortcontents before contents by default as a result of the discussion here, though that will only happen in newer versions. The latter "shape" is a git log documented at: https://inbox.sourceware.org/gcc-patches/87ttysppsc@aarsen.me/ As always, I've freshly rebased the patch series at https://git.sr.ht/~arsen/gcc texinfo_improvements ... or, in (git)web form, at: https://git.sr.ht/~arsen/gcc/log/texinfo_improvements ... with the render at: https://www.aarsen.me/~arsen/final/ Thank you all in advance, have a lovely day. Joseph, could you maybe review the last piece? A direct pointer to it in Arsen's git is https://git.sr.ht/~arsen/gcc/commit/bc734311cbca1085a1728f79b7eebef8cc7aeac3 -Sandra
Ping (gcc/configure.ac, docs): [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias
Evening, Ping on this patch series. I believe we're close to being able to apply it, with contents of the documentation changes being approved, and only a small build system change and a revised patch series "shape" left to review. The former change of these two is in: https://inbox.sourceware.org/gcc-patches/20230311203234.2257423-1-ar...@aarsen.me/ The patch adds a test for the presence of CONTENTS_OUTPUT_LOCATION into the build system in order to avoid shortcontents coming after contents in HTML output. We've also updated Texinfo to output shortcontents before contents by default as a result of the discussion here, though that will only happen in newer versions. The latter "shape" is a git log documented at: https://inbox.sourceware.org/gcc-patches/87ttysppsc@aarsen.me/ As always, I've freshly rebased the patch series at https://git.sr.ht/~arsen/gcc texinfo_improvements ... or, in (git)web form, at: https://git.sr.ht/~arsen/gcc/log/texinfo_improvements ... with the render at: https://www.aarsen.me/~arsen/final/ Thank you all in advance, have a lovely day. -- Arsen Arsenović signature.asc Description: PGP signature
