From: Qu Wenruo <quwenruo@cn.fujitsu.com>
To: Hugo Mills <hugo@carfax.org.uk>, <dsterba@suse.cz>,
<linux-btrfs@vger.kernel.org>, <clm@fb.com>
Subject: Re: [PATCH 00/27] Replace the old man page with asciidoc and man page for each btrfs subcommand.
Date: Mon, 19 May 2014 08:35:00 +0800 [thread overview]
Message-ID: <537951B4.2060407@cn.fujitsu.com> (raw)
In-Reply-To: <20140518120514.GB5783@carfax.org.uk>
-------- Original Message --------
Subject: Re: [PATCH 00/27] Replace the old man page with asciidoc and
man page for each btrfs subcommand.
From: Hugo Mills <hugo@carfax.org.uk>
To: Qu Wenruo <quwenruo@cn.fujitsu.com>
Date: 2014年05月18日 20:05
> On Sun, May 18, 2014 at 03:04:33PM +0800, Qu Wenruo wrote:
>> -------- Original Message --------
>> Subject: Re: [PATCH 00/27] Replace the old man page with asciidoc and man
>> page for each btrfs subcommand.
>> From: Hugo Mills <hugo@carfax.org.uk>
>> To: dsterba@suse.cz, Qu Wenruo <quwenruo@cn.fujitsu.com>,
>> linux-btrfs@vger.kernel.org, clm@fb.com
>> Date: 2014年05月18日 02:22
>>> On Sat, May 17, 2014 at 06:43:15PM +0100, Hugo Mills wrote:
>>>> On Wed, Apr 16, 2014 at 07:12:19PM +0200, David Sterba wrote:
>>>>> On Wed, Apr 02, 2014 at 04:29:11PM +0800, Qu Wenruo wrote:
>>>>>> Convert the old btrfs man pages to new asciidoc and split the huge
>>>>>> btrfs man page into subcommand man page.
>>>>> I'm merging this patchset into the base series of integration because
>>>>> several patches need to update the docs and it's no longer feasible to
>>>>> keep it in a separate branch from the patches.
>>>> I've just been poking around in the docs for a completely different
>>>> reason, and I think there's a fairly serious problem (well, as serious
>>>> as problems get with documentation).
>>>>
>>>> Take, for example, the format for btrfs fi resize:
>>>>
>>>> 'resize' [devid:][+/-]<size>[gkm]|[devid:]max <path>::
>>>>
>>>> Now, this has just thrown away all of the useful markup which
>>>> indicates the semantics of the command. The asciidoc renders all of
>>>> that text literally and unformatted, making alphasymbolic(*) soup of
>>>> the docs. Compare this to the old roff man page:
>>>>
>>>> \fBbtrfs\fP \fBfilesystem resize\fP [\fIdevid\fP:][+/\-]\fI<size>\fP[gkm]|[\fIdevid\fP:]\fImax <path>\fP
>>>>
>>>> This isn't perfect -- we're missing a \fB around the "max" -- but
>>>> it has text in bold(⁑) and italics(⁂) and neither(☃). I've just looked
>>>> at some of the other pages, and they've also got similar typographical
>>>> problems. This is a lot of fiddly tedious work to get it right, and if
>>>> it doesn't get done now in the initial commit, then we're going to end
>>>> up with poor examples copied for every new feature or docs update,
>>>> making the problem worse before anyone does the work to make it
>>>> better.
>>> Oh, and asciidoc appears to be the most horrible capricious
>>> inconsistent parser in existence. I've just spent 5 minutes getting
>>> this one line of text to do what I want it to:
>>>
>>> =====
>>> __N__**c**[\[_Mmin_**-**]__Mmax__**s**[_P_**p**]]
>>> =====
>>>
>>> I had to run through the list of block quote operators one at a
>>> time in order to find the one I needed for this (the =====); it's
>>> still not indenting it correctly on the resulting man page.
>>>
>>> Note also the fun things like the fact that [[]] is special, so you
>>> have to quote the opening part of it -- but if you try quoting the
>>> first [ with a \ you get a literal \[ in the output. You get the right
>>> output from quoting the *second* [ only.
>> I have already encountered problems like that, especially when convert
>> 'btrfs-resize' related things.
>>
>> But wait for a minute, do we really need the *fascinating* highlight things
>> in a user documentation?
> Yes, absolutely. The formatting is a part of the _meaning_ of the
> documentation. Otherwise you're left guessing as to which pieces of
> the string of characters are meant to be there literally, and which
> pieces have to be replaced by suitable text, and which pieces are
> optional.
>
>> The most important thing is the content not the format.
> My point is that in this case the formatting _is_ a part of the
> content.
>
>> I choose asciidoc to make developers take less effort on the format things,
>> not the opposite.
>> In this point of view, I think asciidoc does things considerately well.
>>
>> Although the problem you mentioned is true, but it only affects a small part
>> of the documentation,
>> compared to the overall benefits, I still consider converting to asciidoc is
>> worthy.
> I'm finding it almost impossible to make it do what I want. I think
> in some cases it actually _is_ impossible. This is a truly frustrating
> tool that is really not making things simpler, and I can see is going
> to lead to even more badly marked up documentation -- simply because
> it's too difficult and frustrating to get it right.
>
>>> The Nc can only be italicised and emboldened properly with __ and
>>> ** because _ and * require whitespace around them in order to work
>>> (seriously, WTF?). However, we can't be consistent with that in the
>>> _Mmin_**-** because the quoted \[ appears to count as whitespace, so
>>> using __Mmin__ gives us a leading literal _. The closing __ appears to
>>> close the single opening _ correctly in that case, though.
>>>
>>> Seriously, this is meant to be _easy_ to use? I think I'd rather
>>> type docbook by hand that have to struggle with this. Even the troff
>>> macros for man pages are simpler to get right.
>> From the purpose of documentation and the above explaination, I think the
>> answer is *Yes*.
> Only if you don't care about the typography of the resulting document.
Since you seems really unhappy with the git documentation style, I have
nothing else to say.
I recommended you to send your opinion to git mail list and see what
they respond.
I think it is now upon David to make the decision.
Thanks,
Qu.
>
>> If you really think there is a better choice, I will be very happy
>> to listen, but please consider what I mentioned above and the
>> privious mail first.
> I don't have any real suggestions for alternatives coming from my
> experience, other than "not this". I've used docbook for man pages
> briefly, many years ago. Looking around on the web, reStructuredText
> might be a good option. Personally, I'd like to write docs in LaTeX,
> but I'm not sure how easy it is to convert that to man pages.
>
> Hugo.
>
next prev parent reply other threads:[~2014-05-19 1:33 UTC|newest]
Thread overview: 124+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-02 8:29 [PATCH 00/27] Replace the old man page with asciidoc and man page for each btrfs subcommand Qu Wenruo
2014-04-02 8:29 ` [PATCH 01/27] btrfs-progs: Introduce asciidoc based man page and btrfs man page Qu Wenruo
2014-04-02 8:29 ` [PATCH 02/27] btrfs-progs: Convert man page for btrfs-subvolume Qu Wenruo
2014-04-02 8:29 ` [PATCH 03/27] btrfs-progs: Convert man page for filesystem subcommand Qu Wenruo
2014-04-02 8:29 ` [PATCH 04/27] btrfs-progs: Convert man page for btrfs-balance Qu Wenruo
2014-04-02 8:29 ` [PATCH 05/27] btrfs-progs: Convert man page for btrfs-device subcommand Qu Wenruo
2014-04-02 8:29 ` [PATCH 06/27] btrfs-progs: Convert man page for btrfs-scrub Qu Wenruo
2014-04-02 8:29 ` [PATCH 07/27] btrfs-progs: Convert man page for btrfs-check Qu Wenruo
2014-04-02 8:29 ` [PATCH 08/27] btrfs-progs: Convert man page for btrfs-rescue Qu Wenruo
2014-04-02 8:29 ` [PATCH 09/27] btrfs-progs: Convert man page for btrfs-inspect-internal Qu Wenruo
2014-04-02 8:29 ` [PATCH 10/27] btrfs-progs: Convert man page for btrfs-send Qu Wenruo
2014-04-02 8:29 ` [PATCH 11/27] btrfs-progs: Convert man page for btrfs-receive Qu Wenruo
2014-04-02 8:29 ` [PATCH 12/27] btrfs-progs: Convert man page for btrfs-quota Qu Wenruo
2014-04-02 8:29 ` [PATCH 13/27] btrfs-progs: Convert and enhance the man page of btrfs-qgroup Qu Wenruo
2014-04-02 8:29 ` [PATCH 14/27] btrfs-progs: Convert man page for btrfs-replace Qu Wenruo
2014-04-04 20:29 ` Marc MERLIN
2014-04-08 1:20 ` Qu Wenruo
2014-04-02 8:29 ` [PATCH 15/27] btrfs-progs: Convert man page for btrfs-dedup Qu Wenruo
2014-04-02 8:29 ` [PATCH 16/27] btrfs-progs: Convert man page for btrfsck Qu Wenruo
2014-04-02 8:29 ` [PATCH 17/27] btrfs-progs: Convert man page for btrfs-convert Qu Wenruo
2014-04-02 8:29 ` [PATCH 18/27] btrfs-progs: Convert man page for btrfs-debug-tree Qu Wenruo
2014-04-02 8:29 ` [PATCH 19/27] btrfs-progs: Convert man page for btrfs-find-root Qu Wenruo
2014-04-02 8:29 ` [PATCH 20/27] btrfs-progs: Convert man page for btrfs-image Qu Wenruo
2014-04-02 8:29 ` [PATCH 21/27] btrfs-progs: Convert man page for btrfs-map-logical Qu Wenruo
2014-04-02 8:29 ` [PATCH 22/27] btrfs-progs: Convert man page for btrfs-show-super Qu Wenruo
2014-04-02 8:29 ` [PATCH 23/27] btrfs-progs: Convert man page for btrfstune Qu Wenruo
2014-04-02 8:29 ` [PATCH 24/27] btrfs-progs: Convert man page for btrfs-zero-log Qu Wenruo
2014-04-04 18:46 ` Marc MERLIN
2014-04-05 22:00 ` cwillu
2014-04-05 22:02 ` Marc MERLIN
2014-04-05 22:03 ` Hugo Mills
2014-04-05 22:21 ` Marc MERLIN
2014-04-05 22:05 ` Marc MERLIN
2014-04-05 22:02 ` Hugo Mills
2014-04-08 1:42 ` Qu Wenruo
2014-04-11 5:54 ` Marc MERLIN
2014-04-02 8:29 ` [PATCH 25/27] btrfs-progs: Convert man page for fsck.btrfs Qu Wenruo
2014-04-02 8:29 ` [PATCH 26/27] btrfs-progs: Convert man page for mkfs.btrfs Qu Wenruo
2014-04-02 8:29 ` [PATCH 27/27] btrfs-progs: Switch to the new asciidoc Documentation Qu Wenruo
2014-04-02 13:24 ` [PATCH 00/27] Replace the old man page with asciidoc and man page for each btrfs subcommand Chris Mason
2014-04-02 14:47 ` Marc MERLIN
2014-04-03 20:33 ` Zach Brown
2014-04-02 17:29 ` David Sterba
2014-04-16 17:12 ` David Sterba
2014-04-16 17:16 ` [PATCH] btrfs-progs: doc: link btrfsck to btrfs-check David Sterba
2014-04-17 0:47 ` Qu Wenruo
2014-04-18 14:48 ` David Sterba
2014-04-30 12:14 ` WorMzy Tykashi
2014-05-05 14:57 ` David Sterba
2014-05-08 1:40 ` Qu Wenruo
2014-05-12 14:09 ` David Sterba
2014-06-03 9:38 ` WorMzy Tykashi
2014-06-03 12:19 ` David Sterba
2014-05-17 17:43 ` [PATCH 00/27] Replace the old man page with asciidoc and man page for each btrfs subcommand Hugo Mills
2014-05-17 18:22 ` Hugo Mills
2014-05-18 7:04 ` Qu Wenruo
2014-05-18 12:05 ` Hugo Mills
2014-05-18 16:02 ` Brendan Hide
2014-05-19 0:35 ` Qu Wenruo [this message]
2014-05-18 6:51 ` Qu Wenruo
2014-05-18 10:10 ` Hugo Mills
2014-05-19 13:02 ` Chris Mason
2014-05-19 14:01 ` David Sterba
2014-05-19 14:33 ` David Sterba
2014-05-20 0:34 ` Qu Wenruo
2014-05-20 11:08 ` David Sterba
-- strict thread matches above, loose matches on Subject: below --
2014-04-07 16:05 btrfs on 3.14rc5 stuck on "btrfs_tree_read_lock sync" Marc MERLIN
2014-04-07 16:10 ` Josef Bacik
2014-04-07 18:51 ` Marc MERLIN
2014-04-07 19:32 ` Chris Mason
2014-04-07 20:00 ` Marc MERLIN
2014-04-09 17:38 ` Marc MERLIN
2014-03-25 1:49 ` How to debug very very slow file delete? Marc MERLIN
2014-03-25 12:13 ` How to debug very very slow file delete? (btrfs on md-raid5) Martin
2014-03-25 13:57 ` Xavier Nicollet
2014-03-25 16:41 ` Marc MERLIN
2014-04-10 17:07 ` How to debug very very slow file delete? (btrfs on md-raid5 with many files, 70GB metadata) Marc MERLIN
2014-04-11 14:15 ` How to debug very very slow file delete? (btrfs on md-raid5) Chris Samuel
2014-04-11 17:23 ` Marc MERLIN
2014-04-11 18:00 ` Duncan
2014-04-11 19:15 ` Roman Mamedov
2014-04-12 20:25 ` very slow btrfs filesystem: any data needed before I wipe it? Marc MERLIN
2014-04-13 4:02 ` Duncan
2014-04-14 1:43 ` Marc MERLIN
2014-04-14 10:28 ` Duncan
2014-04-16 22:35 ` Marc MERLIN
2014-04-13 14:57 ` Marc MERLIN
2014-04-13 16:59 ` what does your btrfsck look like? Marc MERLIN
2014-04-14 2:15 ` How to debug very very slow file delete? Liu Bo
2014-04-14 2:21 ` Liu Bo
2014-06-09 23:40 ` btrfs balance crash BUG ON fs/btrfs/relocation.c:1062 or RIP build_backref_tree+0x9fc/0xcc4 Marc MERLIN
2014-06-10 0:32 ` Russell Coker
2014-06-10 4:58 ` Marc MERLIN
2014-06-14 16:21 ` Marc MERLIN
2014-06-17 18:29 ` Josef Bacik
2014-06-17 18:55 ` Marc MERLIN
2014-06-18 15:26 ` Josef Bacik
2014-06-18 20:21 ` Marc MERLIN
2014-06-19 16:12 ` Josef Bacik
2014-06-19 22:25 ` Marc MERLIN
2014-06-19 22:50 ` Josef Bacik
2014-06-20 0:53 ` Marc MERLIN
2014-06-20 15:40 ` Josef Bacik
2014-06-25 19:40 ` Marc MERLIN
2014-06-25 21:05 ` Josef Bacik
2015-05-05 21:02 ` 3.19.6: __btrfs_free_extent:5987: errno=-2 No such entry, did btrfs check --repair break it? Marc MERLIN
2015-05-06 11:04 ` Duncan
2015-05-06 17:25 ` Chris Murphy
2015-05-07 3:15 ` Duncan
2015-05-06 17:49 ` Marc MERLIN
2014-09-03 17:42 kernel BUG at fs/btrfs/extent-tree.c:7727! with 3.17-rc3 Tomasz Chmielewski
2014-09-03 12:04 ` kernel BUG at fs/btrfs/relocation.c:1065 in 3.14.16 to 3.17-rc3 Olivier Bonvalet
2014-09-29 14:13 ` Liu Bo
[not found] ` <20140824000720.GN3875@merlins.org>
[not found] ` <20140926214821.GX13219@merlins.org>
[not found] ` <20150502141102.GB1809@merlins.org>
[not found] ` <20150501210013.GH13624@merlins.org>
2015-04-29 23:21 ` 3.19.3, btrfs send/receive error: failed to clone extents Marc MERLIN
2015-05-02 16:30 ` 3.19.3: check tree block failed + WARNING: device 0 not present on scrub Marc MERLIN
2015-05-02 16:50 ` Christian Dysthe
2015-05-02 17:05 ` Marc MERLIN
2015-05-02 17:20 ` Christian Dysthe
2015-05-02 17:29 ` Marc MERLIN
2015-05-02 18:56 ` Christian Dysthe
2015-05-05 6:32 ` Marc MERLIN
2015-05-05 19:56 ` 3.19.6: __btrfs_free_extent:5987: errno=-2 No such entry Marc MERLIN
2014-09-08 18:04 ` kernel BUG at fs/btrfs/extent-tree.c:7727! with 3.17-rc3 Tomasz Chmielewski
2014-10-04 1:19 ` Tomasz Chmielewski
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=537951B4.2060407@cn.fujitsu.com \
--to=quwenruo@cn.fujitsu.com \
--cc=clm@fb.com \
--cc=dsterba@suse.cz \
--cc=hugo@carfax.org.uk \
--cc=linux-btrfs@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 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 NNTP newsgroup(s).