Re: [PATCH] sfdisk: (man) do not use the common 'include' for --help and --version

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Sep 09, 2025 at 10:56:09AM +0200, Benno Schulenberg wrote:
> Anomalously, `sfdisk` uses -v for --version instead of the standard
> uppercase -V.  In `sfdisk` -V means --verify.
> 
> This reverts commit 82d0bfe59a from last week.
> 
> CC: Mario Blättermann <mario.blaettermann@xxxxxxxxx>
> Signed-off-by: Benno Schulenberg <bensberg@xxxxxxxxxx>
> ---
>  disk-utils/sfdisk.8.adoc | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/disk-utils/sfdisk.8.adoc b/disk-utils/sfdisk.8.adoc
> index 01bcfbfff..99849a638 100644
> --- a/disk-utils/sfdisk.8.adoc
> +++ b/disk-utils/sfdisk.8.adoc
> @@ -239,7 +239,11 @@ Wipe filesystem, RAID and partition-table signatures from the device, in order t
>  *-W*, *--wipe-partitions* _when_::
>  Wipe filesystem, RAID and partition-table signatures from a newly created partition, in order to avoid possible collisions. The argument _when_ can be *auto*, *never* or *always*. When this option is not given, the default is *auto*, in which case signatures are wiped only when in interactive mode and after confirmation by user. In all cases detected signatures are reported by warning messages after a new partition is created. See also *wipefs*(8) command.
>  
> -include::man-common/help-version.adoc[]

It would be helpful to add a comment there to prevent this mistake in
the future, such as:

// Do not use man-common/help-version.adoc; -V is incompatible.

> +*-h*, *--help*::
> +Display help text and exit.
> +
> +*-v*, *--version*::
> +Display version and exit.

Benno, can you update the patch? Thanks.

    Karel

-- 
 Karel Zak  <kzak@xxxxxxxxxx>
 http://karelzak.blogspot.com





[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux