From: "Tobin C. Harding" <me@tobin.cc>
To: Jonathan Corbet <corbet@lwn.net>
Cc: Daniel Borkmann <daniel@iogearbox.net>,
Alexei Starovoitov <ast@kernel.org>,
"David S. Miller" <davem@davemloft.net>,
linux-doc@vger.kernel.org, netdev@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [RFC bpf-next v2 3/3] docs: Split filter.txt into separate documents.
Date: Wed, 8 Aug 2018 09:19:52 +1000 [thread overview]
Message-ID: <20180807231951.GD11191@eros> (raw)
In-Reply-To: <20180807071405.58d38277@lwn.net>
On Tue, Aug 07, 2018 at 07:14:05AM -0600, Jonathan Corbet wrote:
> On Tue, 7 Aug 2018 12:48:44 +1000
> "Tobin C. Harding" <me@tobin.cc> wrote:
>
> > How about these steps:
> >
> > 1. start with foo.txt
> > 2. do typo and grammar fixes (any number of patches).
> > 3. rename to foo.rst, do whitespace changes, code snippet
> > indentation, heading adornments, update references to this file.
> > (single patch).
> > 4. Fix up references in the file text to use RST (i.e :ref: blah)
> > 5. Fix up RST markers (backticks etc). (any number of patches)
>
> That can certainly work; just don't call it foo.rst until it actually is a
> valid RST file.
>
> And, of course, go easy with the later steps and try to avoid the
> temptation to mark up everything; we really want to preserve the
> readability of the plain-text files.
Yeah I get over zealous sometimes, I'll keep it in mind.
What is the current view on references embedded in the text versus a
local label with the full reference at the bottom of the file. I've
seen both and do not know which is _more_ readable?
1. some random text referencing
:ref:`Documentation/path/to/file.rst <label>` and more text
2. some random text referencing `file`_ and more text
...
.. Links
.. _file path/to/file.rst:
Although the later form is found in Documentation/ it does not seem work
for local build (since the path is relative) or am I doing something
wrong with my local build?
And for the first form I could not get the html link produced to work if
the <label> was omitted - this label seems superfluous if it is to the
top of the file (I've already added a couple of such labels).
These are minor issues, answer only if and when you have time.
thanks,
Tobin.
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: "Tobin C. Harding" <me@tobin.cc>
To: Jonathan Corbet <corbet@lwn.net>
Cc: Daniel Borkmann <daniel@iogearbox.net>,
Alexei Starovoitov <ast@kernel.org>,
"David S. Miller" <davem@davemloft.net>,
linux-doc@vger.kernel.org, netdev@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [RFC bpf-next v2 3/3] docs: Split filter.txt into separate documents.
Date: Wed, 8 Aug 2018 09:19:52 +1000 [thread overview]
Message-ID: <20180807231951.GD11191@eros> (raw)
In-Reply-To: <20180807071405.58d38277@lwn.net>
On Tue, Aug 07, 2018 at 07:14:05AM -0600, Jonathan Corbet wrote:
> On Tue, 7 Aug 2018 12:48:44 +1000
> "Tobin C. Harding" <me@tobin.cc> wrote:
>
> > How about these steps:
> >
> > 1. start with foo.txt
> > 2. do typo and grammar fixes (any number of patches).
> > 3. rename to foo.rst, do whitespace changes, code snippet
> > indentation, heading adornments, update references to this file.
> > (single patch).
> > 4. Fix up references in the file text to use RST (i.e :ref: blah)
> > 5. Fix up RST markers (backticks etc). (any number of patches)
>
> That can certainly work; just don't call it foo.rst until it actually is a
> valid RST file.
>
> And, of course, go easy with the later steps and try to avoid the
> temptation to mark up everything; we really want to preserve the
> readability of the plain-text files.
Yeah I get over zealous sometimes, I'll keep it in mind.
What is the current view on references embedded in the text versus a
local label with the full reference at the bottom of the file. I've
seen both and do not know which is _more_ readable?
1. some random text referencing
:ref:`Documentation/path/to/file.rst <label>` and more text
2. some random text referencing `file`_ and more text
...
.. Links
.. _file path/to/file.rst:
Although the later form is found in Documentation/ it does not seem work
for local build (since the path is relative) or am I doing something
wrong with my local build?
And for the first form I could not get the html link produced to work if
the <label> was omitted - this label seems superfluous if it is to the
top of the file (I've already added a couple of such labels).
These are minor issues, answer only if and when you have time.
thanks,
Tobin.
next prev parent reply other threads:[~2018-08-07 23:20 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-02 22:30 [RFC bpf-next v2 0/3] docs: Convert filter.txt to RST Tobin C. Harding
2018-08-02 22:30 ` Tobin C. Harding
2018-08-02 22:30 ` [RFC bpf-next v2 1/3] docs: net: Fix various minor typos Tobin C. Harding
2018-08-02 22:30 ` Tobin C. Harding
2018-08-02 22:30 ` [RFC bpf-next v2 2/3] docs: net: Use lowercase 'k' for kernel Tobin C. Harding
2018-08-02 22:30 ` Tobin C. Harding
2018-08-02 22:31 ` [RFC bpf-next v2 3/3] docs: Split filter.txt into separate documents Tobin C. Harding
2018-08-02 22:31 ` Tobin C. Harding
2018-08-03 13:08 ` Jonathan Corbet
2018-08-03 13:08 ` Jonathan Corbet
2018-08-07 2:48 ` Tobin C. Harding
2018-08-07 2:48 ` Tobin C. Harding
2018-08-07 13:14 ` Jonathan Corbet
2018-08-07 13:14 ` Jonathan Corbet
2018-08-07 23:19 ` Tobin C. Harding [this message]
2018-08-07 23:19 ` Tobin C. Harding
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=20180807231951.GD11191@eros \
--to=me@tobin.cc \
--cc=ast@kernel.org \
--cc=corbet@lwn.net \
--cc=daniel@iogearbox.net \
--cc=davem@davemloft.net \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.