From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from esa3.mentor.iphmx.com (esa3.mentor.iphmx.com [68.232.137.180]) by sourceware.org (Postfix) with ESMTPS id BD0813858D33 for ; Wed, 8 Mar 2023 18:39:03 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org BD0813858D33 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=codesourcery.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=mentor.com X-IronPort-AV: E=Sophos;i="5.98,244,1673942400"; d="scan'208";a="99313120" Received: from orw-gwy-01-in.mentorg.com ([192.94.38.165]) by esa3.mentor.iphmx.com with ESMTP; 08 Mar 2023 10:39:01 -0800 IronPort-SDR: zRwjmvASNvFk632dQ9NOQU4TUBDXb24Bjxmqm+fK3bpmFK4YYgBdDF2egUHcEPsVP6xl18G/nh j6gOHByX310YAvep91eblP5Co0Gij5HajFQ27cwVXet87ERfNA7/NpNDdSucQ1nO7sNPQaMdsd Q8VnkSFjolXRtlloDXHeaM5QagDJsDsz999vo4YorN7Rec32CY6JaApAYc3EgrCESa1osUQvcS ArZpR+6vfTnFzUiZz8vVjFGUpfu4neuSaxcLvtU29iENS+L3Spt7HztOCCZEXdmDlcMA7ApKXh RjQ= Message-ID: Date: Wed, 8 Mar 2023 11:38:57 -0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.7.1 Subject: Re: [PATCH v2 0/5] A small Texinfo refinement Content-Language: en-US To: =?UTF-8?Q?Arsen_Arsenovi=c4=87?= CC: , Gerald Pfeifer , "Joseph Myers" References: <20230223102714.3606058-1-arsen@aarsen.me> <76f92bd2-7d4b-15ba-12ca-8de44e91b886@codesourcery.com> <86lek7txey.fsf@aarsen.me> From: Sandra Loosemore In-Reply-To: <86lek7txey.fsf@aarsen.me> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit X-ClientProxiedBy: svr-orw-mbx-13.mgc.mentorg.com (147.34.90.213) To svr-orw-mbx-13.mgc.mentorg.com (147.34.90.213) X-Spam-Status: No, score=-3.5 required=5.0 tests=BAYES_00,HEADER_FROM_DIFFERENT_DOMAINS,KAM_DMARC_STATUS,KAM_INFOUSMEBIZ,NICE_REPLY_A,SPF_HELO_PASS,SPF_PASS,TXREP autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: On 3/8/23 02:11, Arsen Arsenović wrote: > > Sandra Loosemore 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