All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michael Opdenacker" <michael.opdenacker@bootlin.com>
To: Ross Burton <ross@burtonini.com>, docs@lists.yoctoproject.org
Subject: Re: [docs] [PATCH] kickstart: update busybox note regarding PARTUUID
Date: Wed, 21 Jul 2021 17:31:22 +0200	[thread overview]
Message-ID: <fb906e49-40dc-bad7-eac4-9b01c0d195dc@bootlin.com> (raw)
In-Reply-To: <1693D42FF18C1F82.16442@lists.yoctoproject.org>

Hi again,

On 7/21/21 4:19 PM, Michael Opdenacker wrote:
> Hi Ross,
>
> Thank you for this patch!
> See my comments and questions below...
>
> On 7/21/21 2:50 PM, Ross Burton wrote:
>> As of oe-core c996cabf the default BusyBox configuration supports
>> PARTUUID, so update the note to reflect this.
>>
>> Signed-off-by: Ross Burton <ross.burton@arm.com>
>> ---
>>  documentation/ref-manual/kickstart.rst | 5 +++--
>>  1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/documentation/ref-manual/kickstart.rst b/documentation/ref-manual/kickstart.rst
>> index fc723ccbe..9de3f763d 100644
>> --- a/documentation/ref-manual/kickstart.rst
>> +++ b/documentation/ref-manual/kickstart.rst
>> @@ -54,8 +54,9 @@ must also provide one of the ``--ondrive``, ``--ondisk``, or
>>  .. note::
>>  
>>     The mount program must understand the PARTUUID syntax you use with
>> -   ``--use-uuid`` and non-root *mountpoint*, including swap. The BusyBox
>> -   versions of these application are currently excluded.
>> +   ``--use-uuid`` and non-root *mountpoint*, including swap. The default
>> +   configuration of BusyBox in OpenEmbedded support this, but this may
>> +   be disabled in custom configurations.
> s/support/supports/ ... but I can update this by myself.
>
> Written this way (not your fault, unless you wrote the initial text
> too!), this suggests that "mount" has a "--use-uuid" option, but I can't
> seem to use it.
>
> Here's what works for me:
> mount UUID=675e2a50-fd3b-43b7-aede-6714e2404070 /mnt/mount/
>
> Reading further, I know understand that this corresponds to Wic's
> option. Could we say "... you use with Wic's ``--use-uuid`` option for
> non-root mountpoints" instead?
>
> I'm proposing "for" instead of "and" here because I first believed the
> text was talking about the "mountpoints" BusyBox command run as a
> non-root user. I may be a little twisted, but I think that would be less
> confusing.


Actually, the existing text is worse than this, as I checked that
"--use-uuid" doesn't take any argument. Instead "--fsuuid" and "--uuid" do.

After further digging, I realize that most of the text in this document
comes from Wic's help usage in oe-core's scripts/lib/wic/help.py .
That's where the text should be fixed (and may already be fixed in a
pending patch).

That's not very satisfactory to maintain a duplicate copy in the
documentation that will always tend to be out of date. I already saw
that too with the run-qemu command options.

I see three possible solutions...

 1. Go on using a copy, but use a verbatim copy of "wic help kickstart",
    being more explicit about where the text comes from. A good example
    is devtool's documentation in the reference manual:
    https://docs.yoctoproject.org/ref-manual/devtool-reference.html
 2. In the reference manual, link to the help text output of the commands
 3. Automatically include the help text from the sources (or from the
    output of commands) in the reference documentation.

1. Seems the easiest way to go for the moment, but if someone has a good
idea going further that would avoid duplicating command output, this
would be great! Anyway, I'd rather wait for scripts/lib/wic/help.py to
be updated (with correct contents) before adding a different change to
the reference manual.

What do you think?

Cheers,

Michael.

-- 
Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


      parent reply	other threads:[~2021-07-21 15:31 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-21 12:50 Ross Burton
2021-07-21 14:19 ` [docs] " Michael Opdenacker
     [not found] ` <1693D42FF18C1F82.16442@lists.yoctoproject.org>
2021-07-21 15:31   ` Michael Opdenacker [this message]

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=fb906e49-40dc-bad7-eac4-9b01c0d195dc@bootlin.com \
    --to=michael.opdenacker@bootlin.com \
    --cc=docs@lists.yoctoproject.org \
    --cc=ross@burtonini.com \
    --subject='Re: [docs] [PATCH] kickstart: update busybox note regarding PARTUUID' \
    /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

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.