public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Sandra Loosemore <sandra@codesourcery.com>
To: "Arsen Arsenović" <arsen@aarsen.me>
Cc: <gcc-patches@gcc.gnu.org>, Gerald Pfeifer <gerald@pfeifer.com>,
	"Joseph Myers" <joseph@codesourcery.com>
Subject: Re: [PATCH v2 0/5] A small Texinfo refinement
Date: Wed, 8 Mar 2023 11:38:57 -0700	[thread overview]
Message-ID: <c4f2f18d-07fa-868b-d1ff-59b05188f57f@codesourcery.com> (raw)
In-Reply-To: <86lek7txey.fsf@aarsen.me>

On 3/8/23 02:11, Arsen Arsenović wrote:
> 
> Sandra Loosemore <sandra@codesourcery.com> writes:
> 
>> On 2/23/23 03:27, Arsen Arsenović via Gcc-patches wrote:
>>> I've rerendered the updated documentation with latest development
>>> Texinfo (as some of the changes I made for the purposes of the GCC
>>> manual still aren't in releases) at:
>>>     https://www.aarsen.me/~arsen/final/
>>
>> Ummm.  I don't think GCC's documentation should depend on an unreleased version
>> of Texinfo.  Currently install.texi documents that version 4.7 or later is
>> required, 4.8 for "make pdf"; did I miss something in your patch set that bumps
>> this requirement?  Exactly what features do you depend on that are not yet
>> supported by an official Texinfo release?
> 
> This patch should still build with older Texinfo versions (albeit, I
> hadn't tested 4.7, I missed that requirement).  The unreleased version
> should be installed on the server building HTML documentation as it
> produces better results w.r.t clickable anchors and index-in-table
> handling.  It should not be a hard dependency, and should only degrade
> to its current state should in-dev Texinfo be missing.

Hmmm, OK.  We presently have Texinfo version 6.7 installed here, so I'll 
give that a try.  I'm not sure I'd be able to detect problems with 
incorrect HTML anchors or whatever, though.

Most people building GCC from source probably use whatever versions of 
build dependencies are provided by their OS distribution.  In our group 
we need reproducible builds for long-term support so we maintain our own 
list of dependencies and normally update to the latest stable versions 
only once every few years unless there is a hard requirement to upgrade 
some particular tool meanwhile.  I personally do not know how the 
manuals for the GCC web site are built, but it seems kind of important 
to make sure that works as intended since it's the main online resource 
for ordinary GCC users.

> It might be worth bumping the minimum, 4.7 is a version from 2004; in
> the meanwhile, I'll try a few older versions too.

I agree that it's unlikely anyone is building current GCC with a Texinfo 
version as old as 4.7 any more, and it may be that the manual doesn't 
even build properly with such an old release due to existing 
unintentional dependencies on newer features, independently of your 
patch.  If we do update the version, there's a version check in 
configure.ac and some hack for "makeinfo 4.7 brokenness" in 
doc/install.texi2html that need to be changed, as well as install.texi.

-Sandra

  reply	other threads:[~2023-03-08 18:39 UTC|newest]

Thread overview: 59+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-23 10:27 Arsen Arsenović
2023-02-23 10:27 ` [PATCH v2 1/5] docs: Create Indices appendix Arsen Arsenović
2023-03-09  3:16   ` Sandra Loosemore
2023-03-09 10:12     ` Arsen Arsenović
2023-03-09 20:38     ` Arsen Arsenović
2023-03-11  1:12       ` Sandra Loosemore
2023-03-11  1:34         ` Arsen Arsenović
2023-03-11 12:22         ` Arsen Arsenović
2023-03-11 19:22           ` Gerald Pfeifer
2023-03-11 20:22             ` Arsen Arsenović
2023-03-11 19:34           ` Sandra Loosemore
2023-03-11 20:14             ` Arsen Arsenović
2023-03-11 20:32               ` [PATCH v2] html: Set CONTENTS_OUTPUT_LOCATION=inline if makeinfo supports it Arsen Arsenović
2023-03-12 10:25               ` [PATCH v2 1/5] docs: Create Indices appendix Gerald Pfeifer
2023-03-12 12:23                 ` Arsen Arsenović
2023-02-23 10:27 ` [PATCH v2 2/5] **/*.texi: Reorder index entries Arsen Arsenović
2023-02-23 23:50   ` Gerald Pfeifer
2023-02-23 10:27 ` [PATCH v2 3/5] doc: Add @defbuiltin family of helpers, set documentlanguage Arsen Arsenović
2023-03-09 23:50   ` Sandra Loosemore
2023-02-23 10:27 ` [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias Arsen Arsenović
2023-03-09 23:55   ` Sandra Loosemore
2023-03-10  0:17     ` Gerald Pfeifer
2023-03-10 10:33       ` Thomas Schwinge
2023-03-10 12:49         ` Arsen Arsenović
2023-03-10 15:51           ` Gerald Pfeifer
2023-03-10 15:58             ` Arsen Arsenović
2023-03-10 12:48     ` Arsen Arsenović
2023-03-10 15:49     ` Arsen Arsenović
2023-03-18 20:14       ` Ping (gcc/configure.ac, docs): " Arsen Arsenović
2023-03-20 22:01         ` Sandra Loosemore
2023-03-20 23:06           ` Joseph Myers
2023-03-20 23:27             ` Arsen Arsenović
2023-03-20 23:35           ` Arsen Arsenović
2023-03-21  9:52             ` Arsen Arsenović
2023-03-21 23:11               ` Gerald Pfeifer
2023-02-23 10:27 ` [PATCH v2 5/5] update_web_docs_git: Update CSS reference to new manual CSS Arsen Arsenović
2023-03-10  0:07   ` Sandra Loosemore
2023-03-10 13:00     ` Arsen Arsenović
2023-02-25 22:04 ` [PATCH v2 0/5] A small Texinfo refinement Thomas Schwinge
2023-02-26 14:54   ` Arsen Arsenović
2023-03-08  1:56 ` Sandra Loosemore
2023-03-08  9:11   ` Arsen Arsenović
2023-03-08 18:38     ` Sandra Loosemore [this message]
2023-03-08 21:22       ` Arsen Arsenović
2023-03-09  1:09         ` Sandra Loosemore
2023-03-09  1:19           ` Andrew Pinski
2023-03-09  8:26             ` Richard Biener
2023-03-09 23:35               ` Sandra Loosemore
2023-03-10  8:50                 ` Iain Sandoe
2023-03-10 17:51                   ` Sandra Loosemore
2023-03-10 18:09                     ` Sandra Loosemore
2023-03-10 19:00                     ` Gerald Pfeifer
2023-03-13  9:05                       ` Richard Biener
2023-03-13 11:01                         ` Arsen Arsenović
2023-05-29 15:06                       ` NightStrike
2023-05-29 17:26                         ` Arsen Arsenović
2023-05-29 19:27                           ` Sandra Loosemore
2023-03-09 10:22           ` Arsen Arsenović
2023-03-09  8:28       ` Gerald Pfeifer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=c4f2f18d-07fa-868b-d1ff-59b05188f57f@codesourcery.com \
    --to=sandra@codesourcery.com \
    --cc=arsen@aarsen.me \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=gerald@pfeifer.com \
    --cc=joseph@codesourcery.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).