[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aMILGtrAHb3dKVq7@archie.me>
Date: Thu, 11 Sep 2025 06:34:50 +0700
From: Bagas Sanjaya <bagasdotme@...il.com>
To: Jonathan Corbet <corbet@....net>,
Michal Koutný <mkoutny@...e.com>,
Mauro Carvalho Chehab <mchehab@...nel.org>
Cc: Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Linux Documentation <linux-doc@...r.kernel.org>,
Linux cgroups <cgroups@...r.kernel.org>, Tejun Heo <tj@...nel.org>,
Johannes Weiner <hannes@...xchg.org>,
Andrea Righi <arighi@...dia.com>,
Johannes Bechberger <me@...tlynerdless.de>,
Changwoo Min <changwoo@...lia.com>,
Shashank Balaji <shashank.mahadasyam@...y.com>,
Ingo Molnar <mingo@...nel.org>, Jake Rice <jake@...erice.dev>,
Cengiz Can <cengiz@...nel.wtf>
Subject: Re: [PATCH 2/2] Documentation: cgroup-v2: Replace manual table of
contents with contents:: directive
On Wed, Sep 10, 2025 at 07:24:45AM -0600, Jonathan Corbet wrote:
> Michal Koutný <mkoutny@...e.com> writes:
>
> > On Wed, Sep 10, 2025 at 02:23:34PM +0700, Bagas Sanjaya <bagasdotme@...il.com> wrote:
> >> manually-arranged table of contents (as reST comments) gets out-of-sync
> >> with actual toctree as not all of these are added to it.
> >
> > Is this true? I generated HTML with this patch and the resulting ToC
> > matches what's in the comment.
> >
> >> Replace it with automatically-generated table of contents via contents::
> >> directive.
> >
> > Mauro, what's the best practice wrt consistent ToC and having it in
> > plaintext form?
>
> I fairly routinely get patches fixing manual TOCs that are not updated
> to match changes elsewhere. We have a nice system that can manage the
> TOC automatically for us, it seems best to me to use it.
>
> That said, if having the TOC in the plain-text version of the document
> is deemed to be important, then it needs to be kept and manually
> maintained.
So for the v2, should I fix up the manual toctree to match autogenerated
one?
Thanks.
--
An old man doll... just what I always wanted! - Clara
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists