commands: new method for defining commands

. Define a prototype for every lvm command.
. Match every user command with one definition.
. Generate help text and man pages from them.

The new file command-lines.in defines a prototype for every
unique lvm command.  A unique lvm command is a unique
combination of: command name + required option args +
required positional args.  Each of these prototypes also
includes the optional option args and optional positional
args that the command will accept, a description, and a
unique string ID for the definition.  Any valid command
will match one of the prototypes.

Here's an example of the lvresize command definitions from
command-lines.in, there are three unique lvresize commands:

lvresize --size SizeMB LV
OO: --alloc Alloc, --autobackup Bool, --force,
--nofsck, --nosync, --noudevsync, --reportformat String, --resizefs,
--stripes Number, --stripesize SizeKB, --test, --poolmetadatasize SizeMB
OP: PV ...
ID: lvresize_by_size
DESC: Resize an LV by a specified size.

lvresize LV PV ...
OO: --alloc Alloc, --autobackup Bool, --force,
--nofsck, --nosync, --noudevsync,
--reportformat String, --resizefs, --stripes Number, --stripesize SizeKB,
--test
ID: lvresize_by_pv
DESC: Resize an LV by a specified PV.

lvresize --poolmetadatasize SizeMB LV_thinpool
OO: --alloc Alloc, --autobackup Bool, --force,
--nofsck, --nosync, --noudevsync,
--reportformat String, --stripes Number, --stripesize SizeKB,
--test
OP: PV ...
ID: lvresize_pool_metadata_by_size
DESC: Resize a pool metadata SubLV by a specified size.

The three commands have separate definitions because they have
different required parameters.  Required parameters are specified
on the first line of the definition.  Optional options are
listed after OO, and optional positional args are listed after OP.

This data is used to generate corresponding command definition
structures for lvm in command-lines.h.  usage/help output is also
auto generated, so it is always in sync with the definitions.

Example of the corresponding generated structure in
command-lines.h for the first lvresize prototype
(these structures are never edited directly):

commands[85].name = "lvresize";
commands[85].command_line_id = "lvresize_by_size";
commands[85].command_line_enum = lvresize_by_size_CMD;
commands[85].fn = lvresize;
commands[85].ro_count = 1;
commands[85].rp_count = 1;
commands[85].oo_count = 22;
commands[85].op_count = 1;
commands[85].desc = "DESC: Resize an LV by a specified size.";
commands[85].usage = "lvresize --size Number[m|unit] LV"
" [ --resizefs, --poolmetadatasize Number[m|unit], COMMON_OPTIONS ]"
" [ PV ... ]";
commands[85].usage_common =
" [ --alloc contiguous|cling|cling_by_tags|normal|anywhere|inherit, --commandprofile String, --config String, --driverloaded y|n, --nosync, --noudevsync, --profile String, --reportformat String, --version, --autobackup y|n, --debug, --force, --help, --stripes Number, --stripesize Number[k|unit], --nofsck, --quiet, --test, --verbose, --yes, --commandprofile String, --config String, --debug, --driverloaded y|n, --help, --profile String, --quiet, --verbose, --version, --yes, --force, --test, --noudevsync ]";
commands[85].required_opt_args[0].opt = size_ARG;
commands[85].required_opt_args[0].def.val_bits = val_enum_to_bit(sizemb_VAL);
commands[85].required_pos_args[0].pos = 1;
commands[85].required_pos_args[0].def.val_bits = val_enum_to_bit(lv_VAL);
commands[85].optional_opt_args[0].opt = commandprofile_ARG;
commands[85].optional_opt_args[0].def.val_bits = val_enum_to_bit(string_VAL);
commands[85].optional_opt_args[1].opt = config_ARG;
commands[85].optional_opt_args[1].def.val_bits = val_enum_to_bit(string_VAL);
commands[85].optional_opt_args[2].opt = debug_ARG;
commands[85].optional_opt_args[3].opt = driverloaded_ARG;
commands[85].optional_opt_args[3].def.val_bits = val_enum_to_bit(bool_VAL);
commands[85].optional_opt_args[4].opt = help_ARG;
commands[85].optional_opt_args[5].opt = profile_ARG;
commands[85].optional_opt_args[5].def.val_bits = val_enum_to_bit(string_VAL);
commands[85].optional_opt_args[6].opt = quiet_ARG;
commands[85].optional_opt_args[7].opt = verbose_ARG;
commands[85].optional_opt_args[8].opt = version_ARG;
commands[85].optional_opt_args[9].opt = yes_ARG;
commands[85].optional_opt_args[10].opt = alloc_ARG;
commands[85].optional_opt_args[10].def.val_bits = val_enum_to_bit(alloc_VAL);
commands[85].optional_opt_args[11].opt = autobackup_ARG;
commands[85].optional_opt_args[11].def.val_bits = val_enum_to_bit(bool_VAL);
commands[85].optional_opt_args[12].opt = force_ARG;
commands[85].optional_opt_args[13].opt = nofsck_ARG;
commands[85].optional_opt_args[14].opt = nosync_ARG;
commands[85].optional_opt_args[15].opt = noudevsync_ARG;
commands[85].optional_opt_args[16].opt = reportformat_ARG;
commands[85].optional_opt_args[16].def.val_bits = val_enum_to_bit(string_VAL);
commands[85].optional_opt_args[17].opt = resizefs_ARG;
commands[85].optional_opt_args[18].opt = stripes_ARG;
commands[85].optional_opt_args[18].def.val_bits = val_enum_to_bit(number_VAL);
commands[85].optional_opt_args[19].opt = stripesize_ARG;
commands[85].optional_opt_args[19].def.val_bits = val_enum_to_bit(sizekb_VAL);
commands[85].optional_opt_args[20].opt = test_ARG;
commands[85].optional_opt_args[21].opt = poolmetadatasize_ARG;
commands[85].optional_opt_args[21].def.val_bits = val_enum_to_bit(sizemb_VAL);
commands[85].optional_pos_args[0].pos = 2;
commands[85].optional_pos_args[0].def.val_bits = val_enum_to_bit(pv_VAL);
commands[85].optional_pos_args[0].def.flags = ARG_DEF_FLAG_MAY_REPEAT;

Every user-entered command is compared against the set of
command structures, and matched with one.  An error is
reported if an entered command does not have the required
parameters for any definition.  The closest match is printed
as a suggestion, and running lvresize --help will display
the usage for each possible lvresize command.

The prototype syntax used for help/man output includes
required --option and positional args on the first line,
and optional --option and positional args enclosed in [ ]
on subsequent lines.

  command_name <required_opt_args> <required_pos_args>
          [ <optional_opt_args> ]
          [ <optional_pos_args> ]

$ lvresize --help
  lvresize - Resize a logical volume

  Resize an LV by a specified size.
  lvresize --size Number[m|unit] LV
        [ --resizefs,
          --poolmetadatasize Number[m|unit],
          COMMON_OPTIONS ]
        [ PV ... ]

  Resize an LV by a specified PV.
  lvresize LV PV ...
        [ --resizefs,
          COMMON_OPTIONS ]

  Resize a pool metadata SubLV by a specified size.
  lvresize --poolmetadatasize Number[m|unit] LV_thinpool
        [ COMMON_OPTIONS ]
        [ PV ... ]

  Common options:
        [ --alloc contiguous|cling|cling_by_tags|normal|anywhere|inherit,
          --commandprofile String,
          --config String,
          --driverloaded y|n,
          --nosync,
          --noudevsync,
          --profile String,
          --reportformat String,
          --version,
          --autobackup y|n,
          --debug,
          --force,
          --help,
          --stripes Number,
          --stripesize Number[k|unit],
          --nofsck,
          --quiet,
          --test,
          --verbose,
          --yes,
          --commandprofile String,
          --config String,
          --debug,
          --driverloaded y|n,
          --help,
          --profile String,
          --quiet,
          --verbose,
          --version,
          --yes,
          --force,
          --test,
          --noudevsync ]

  (Use --help --help for usage notes.)

$ lvresize --poolmetadatasize 4
  Failed to find a matching command definition.
  Closest command usage is:
  lvresize --poolmetadatasize Number[m|unit] LV_thinpool

Man page prototypes are also generated from the same original
command definitions, and are always in sync with the code
and help text.

Very early in command execution, a matching command definition
is found.  lvm then knows the operation being done, and that
the provided args conform to the definition.  This will allow
lots of ad hoc checking/validation to be removed throughout
the code.

Each command definition can also be routed to a specific
function to implement it.  The function is associated with
an enum value for the command definition (generated from
the ID string.)  These per-command-definition implementation
functions have not yet been created, so all commands
currently fall back to the existing per-command-name
implementation functions.

Using per-command-definition functions will allow lots of
code to be removed which tries to figure out what the
command is meant to do.  This is currently based on ad hoc
and complicated option analysis.  When using the new
functions, what the command is doing is already known
from the associated command definition.

So, this first phase validates every user-entered command
against the set of command prototypes, then calls the existing
implementation.  The second phase can associate an implementation
function with each definition, and take further advantage of the
known operation to avoid the complicated option analysis.

scripts/command-lines.in

@@ -113,16 +113,8 @@
 # could use differnet IDs, in which case the implementation would
 # know where to look for each parameter.
 #
-# DESC: A description of the command.  The "DESC:" tags will be
-# included in the text as indicators of new lines when printing
-# the descriptions for help/man output.
+# DESC: A description of the command.
 #
-# Note that one the most difficult aspect of these definitions is
-# the variants of --thin / --type thin / --type thin-pool,
-# --cache / --type cache / --type cache-pool.
-# There are no consistent rules to follow and the behaviors are
-# unpredictable; each possible variation and combination needs
-# to be tested individually to see what it means.
 
 #
 # For efficiency, sets of options can be defined and reused
@@ -145,10 +137,12 @@
 
 #
 # OO_ALL is included in every command automatically.
+# FIXME: add --force and --test to OO_ALL so that all commands will
+# accept them even if they are not used?
 #
 OO_ALL: --commandprofile String, --config String, --debug,
 --driverloaded Bool, --help, --profile String, --quiet,
---verbose, --version, --yes, --test
+--verbose, --version, --yes
 
 #
 # This list only applies to printing the usage text.
@@ -162,7 +156,7 @@ OO_ALL: --commandprofile String, --config String, --debug,
 # not relate to which optional opts are accepted by commands,
 # which is defined by the OO line.
 #
-OO_USAGE_COMMON: OO_ALL, --force, --noudevsync
+OO_USAGE_COMMON: OO_ALL, --force, --test, --noudevsync
 
 #
 # options for pvs, lvs, vgs, fullreport
@@ -188,7 +182,7 @@ OO_CONFIG: --atversion String, --typeconfig String, --file String, --ignoreadvan
 
 OO_LVCHANGE: --autobackup Bool, --force, --ignorelockingfailure,
 --ignoremonitoring, --ignoreskippedcluster, --noudevsync,
---reportformat String, --sysinit, --select String
+--reportformat String, --sysinit, --test, --select String
 
 # Any of these can function as a required option for lvchange.
 # profile is also part of OO_ALL, but is repeated in OO_LVCHANGE_META
@@ -257,7 +251,7 @@ OO_LVCONVERT_POOL: --poolmetadata LV, --poolmetadatasize SizeMB,
 --poolmetadataspare Bool, --readahead Readahead, --chunksize SizeKB
 
 OO_LVCONVERT: --alloc Alloc, --background, --force, --noudevsync,
---usepolicies
+--test, --usepolicies
 
 ---
 
@@ -299,17 +293,17 @@ DESC: Merge COW snapshot LV into its origin.
 # Before calling process_each, it steals the first positional arg
 # and adjusts argv/argc so it's not seen by process_each.
 
-# alternate form of lvconvert --snapshot
 lvconvert --type snapshot LV_linear_striped_raid LV_snapshot
-OO: --snapshot, --chunksize SizeKB, --zero Bool, OO_LVCONVERT
+OO: --chunksize SizeKB, --zero Bool, OO_LVCONVERT
 ID: lvconvert_combine_split_snapshot
 DESC: Combine LV with a previously split snapshot LV.
-FLAGS: SECONDARY_SYNTAX
 
+# alternate form of lvconvert --type snapshot
 lvconvert --snapshot LV_linear_striped_raid LV_snapshot
-OO: --type snapshot, --chunksize SizeKB, --zero Bool, OO_LVCONVERT
+OO: --chunksize SizeKB, --zero Bool, OO_LVCONVERT
 ID: lvconvert_combine_split_snapshot
-DESC: Combine LV with a previously split snapshot LV.
+DESC: Combine LV with a previously split snapshot LV
+DESC: (variant, infers --type snapshot).
 
 ---
 
@@ -324,7 +318,6 @@ OO: --type thin, --originname LV_new, --zero Bool, OO_LVCONVERT_POOL, OO_LVCONVE
 ID: lvconvert_to_thin_with_external
 DESC: Convert LV to type thin with an external origin
 DESC: (variant, infers --type thin).
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
@@ -344,7 +337,6 @@ OO: --type cache, --cachemode CacheMode, --cachepolicy String,
 --cachesettings String, --zero Bool, OO_LVCONVERT_POOL, OO_LVCONVERT
 ID: lvconvert_to_cache_vol
 DESC: Convert LV to type cache (variant, infers --type cache).
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
@@ -363,11 +355,10 @@ DESC: Convert LV to type thin-pool.
 # alternate form of lvconvert --type thin-pool
 # deprecated because of non-standard syntax (missing positional arg)
 lvconvert --thinpool LV_linear_striped_raid_cache
-OO: --type thin-pool, --stripes_long Number, --stripesize SizeKB,
+OO: --stripes_long Number, --stripesize SizeKB,
 --discards Discards, --zero Bool, OO_LVCONVERT_POOL, OO_LVCONVERT
 ID: lvconvert_to_thinpool
 DESC: Convert LV to type thin-pool (variant, use --type thin-pool).
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
@@ -388,7 +379,6 @@ OO: --type cache-pool, OO_LVCONVERT_POOL, OO_LVCONVERT,
 --cachemode CacheMode, --cachepolicy String, --cachesettings String
 ID: lvconvert_to_cachepool
 DESC: Convert LV to type cache-pool (variant, use --type cache-pool).
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
@@ -493,7 +483,6 @@ lvconvert --mirrors 0 LV_raid_mirror
 OO: --type linear, --type mirror, OO_LVCONVERT
 ID: lvconvert_raid_or_mirror_to_linear
 DESC: Convert LV to type linear (variant, infers --type linear).
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
@@ -577,7 +566,6 @@ lvconvert LV_mirror
 OO: OO_LVCONVERT
 ID: lvconvert_poll_mirror
 DESC: Poll mirror LV to collapse resync layers.
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
@@ -587,7 +575,6 @@ lvconvert --poolmetadata LV LV_thinpool_cachepool
 OO: OO_LVCONVERT
 ID: lvconvert_swap_pool_metadata
 DESC: Swap metadata LV in a thin pool or cache pool (temporary command).
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
@@ -597,7 +584,7 @@ OO_LVCREATE: --addtag Tag, --alloc Alloc, --autobackup Bool, --activate Active,
 --contiguous Bool, --ignoreactivationskip, --ignoremonitoring, --major Number,
 --metadataprofile String, --minor Number, --monitor Bool, --name String, --nosync,
 --noudevsync, --permission Permission, --persistent Bool, --readahead Readahead,
---reportformat String, --setactivationskip Bool, --wipesignatures Bool,
+--reportformat String, --setactivationskip Bool, --test, --wipesignatures Bool,
 --zero Bool
 
 OO_LVCREATE_CACHE: --cachemode CacheMode, --cachepolicy String, --cachesettings String
@@ -620,7 +607,6 @@ lvcreate --type error --size SizeMB VG
 OO: OO_LVCREATE
 ID: lvcreate_error_vol
 DESC: Create an LV that returns errors when used.
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
@@ -628,7 +614,6 @@ lvcreate --type zero --size SizeMB VG
 OO: OO_LVCREATE
 ID: lvcreate_zero_vol
 DESC: Create an LV that returns zeros when read.
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
@@ -643,7 +628,6 @@ OO: --mirrors 0, --stripes 1, OO_LVCREATE
 OP: PV ...
 ID: lvcreate_linear
 DESC: Create a linear LV.
-FLAGS: SECONDARY_SYNTAX
 
 # This is the one place we mention the optional --name
 # because it's the most common case and may be confusing
@@ -710,37 +694,37 @@ DESC: Create a raid LV (a specific raid level must be used, e.g. raid1.)
 # all the other cases of using --type, where the resulting
 # LV does have the specified type.)
 
-# alternate form of lvcreate --snapshot
 lvcreate --type snapshot --size SizeMB LV
 OO: --snapshot, --stripes Number, --stripesize SizeKB,
 --chunksize SizeKB, OO_LVCREATE
 OP: PV ...
 ID: lvcreate_cow_snapshot
 DESC: Create a COW snapshot LV from an origin LV.
-FLAGS: SECONDARY_SYNTAX
 
+# alternate form of lvcreate --type snapshot
 lvcreate --snapshot --size SizeMB LV
 OO: --type snapshot, --stripes Number, --stripesize SizeKB,
 --chunksize SizeKB, OO_LVCREATE
 OP: PV ...
 ID: lvcreate_cow_snapshot
-DESC: Create a COW snapshot LV from an origin LV.
+DESC: Create a COW snapshot LV from an origin LV
+DESC: (infers --type snapshot).
 
 ---
 
-# alternate form of lvcreate --snapshot
 lvcreate --type snapshot --size SizeMB --virtualsize SizeMB VG
 OO: --snapshot, --chunksize SizeKB, OO_LVCREATE
 OP: PV ...
 ID: lvcreate_cow_snapshot_with_virtual_origin
 DESC: Create a sparse COW snapshot LV of a virtual origin LV.
-FLAGS: SECONDARY_SYNTAX
 
+# alternate form of lvcreate --type snapshot
 lvcreate --snapshot --size SizeMB --virtualsize SizeMB VG
 OO: --type snapshot, --chunksize SizeKB, OO_LVCREATE
 OP: PV ...
 ID: lvcreate_cow_snapshot_with_virtual_origin
-DESC: Create a sparse COW snapshot LV of a virtual origin LV.
+DESC: Create a sparse COW snapshot LV of a virtual origin LV
+DESC: (infers --type snapshot).
 
 ---
 
@@ -758,7 +742,6 @@ OO: --type thin-pool, OO_LVCREATE_POOL, OO_LVCREATE_THIN, OO_LVCREATE,
 OP: PV ...
 ID: lvcreate_thinpool
 DESC: Create a thin pool (variant, infers --type thin-pool).
-FLAGS: SECONDARY_SYNTAX
 
 # alternate form of lvcreate --type thin-pool
 lvcreate --size SizeMB --thinpool LV_new VG
@@ -768,29 +751,29 @@ OP: PV ...
 ID: lvcreate_thinpool
 DESC: Create a thin pool named by the --thinpool arg
 DESC: (variant, infers --type thin-pool).
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
-# NB. there are no alternate forms of these commands that
-# use --cache in place of --type cache-pool, but --cache
-# still needs to be listed as an optional addition to
-# --type cache-pool.
-
 lvcreate --type cache-pool --size SizeMB VG
-OO: --cache, OO_LVCREATE_POOL, OO_LVCREATE_CACHE, OO_LVCREATE
+OO: OO_LVCREATE_POOL, OO_LVCREATE_CACHE, OO_LVCREATE
 OP: PV ...
 ID: lvcreate_cachepool
 DESC: Create a cache pool.
 
+# alternate form of lvcreate --type cache-pool
+lvcreate --cache --size SizeMB VG
+OO: --type cache-pool, OO_LVCREATE_POOL, OO_LVCREATE_CACHE, OO_LVCREATE
+OP: PV ...
+ID: lvcreate_cachepool
+DESC: Create a cache pool (variant, infers --type cache-pool).
+
 # alternate form of lvcreate --type cache-pool
 lvcreate --type cache-pool --size SizeMB --cachepool LV_new VG
-OO: --cache, OO_LVCREATE_POOL, OO_LVCREATE_CACHE, OO_LVCREATE
+OO: OO_LVCREATE_POOL, OO_LVCREATE_CACHE, OO_LVCREATE
 OP: PV ...
 ID: lvcreate_cachepool
 DESC: Create a cache pool named by the --cachepool arg
 DESC: (variant, uses --cachepool in place of --name).
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
@@ -805,17 +788,13 @@ OO: --thin, OO_LVCREATE_THIN, OO_LVCREATE
 ID: lvcreate_thin_vol
 DESC: Create a thin LV in a thin pool named in the first arg
 DESC: (variant, also see --thinpool for naming pool).
-FLAGS: SECONDARY_SYNTAX
-
-# NB. this is the variant which can substitute
-# --thin for --type thin, even though --thin is in OO.
 
 # alternate form of lvcreate --type thin
-lvcreate --virtualsize SizeMB --thinpool LV_thinpool VG
+lvcreate --virtualsize SizeMB --thinpool LV_thinpool
 OO: --type thin, --thin, OO_LVCREATE_THIN, OO_LVCREATE
+OP: VG
 ID: lvcreate_thin_vol
 DESC: Create a thin LV in a thin pool (variant, infers --type thin).
-FLAGS: SECONDARY_SYNTAX
 
 # alternate form of lvcreate --type thin
 lvcreate --virtualsize SizeMB LV_thinpool
@@ -824,7 +803,6 @@ ID: lvcreate_thin_vol
 DESC: Create a thin LV in the thin pool named in the first arg
 DESC: (variant, infers --type thin, also see --thinpool for
 DESC: naming pool.)
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
@@ -839,7 +817,6 @@ OO: --type thin, OO_LVCREATE_THIN, OO_LVCREATE
 ID: lvcreate_thin_snapshot
 DESC: Create a thin LV that is a snapshot of an existing thin LV
 DESC: (infers --type thin).
-FLAGS: SECONDARY_SYNTAX
 
 # alternate form of lvcreate --type thin
 lvcreate --snapshot LV_thin
@@ -849,7 +826,7 @@ DESC: Create a thin LV that is a snapshot of an existing thin LV
 DESC: (infers --type thin).
 
 lvcreate --type thin --thinpool LV_thinpool LV
-OO: --thin, OO_LVCREATE_POOL, OO_LVCREATE_THIN, OO_LVCREATE
+OO: OO_LVCREATE_POOL, OO_LVCREATE_THIN, OO_LVCREATE
 ID: lvcreate_thin_snapshot_of_external
 DESC: Create a thin LV that is a snapshot of an external origin LV.
 
@@ -866,20 +843,8 @@ DESC: (infers --type thin).
 # but here it applies to creating the new thin pool that
 # is used for the thin LV
 
-# FIXME: there are commands here that differ only in that
-# one takes LV_new in arg pos 1, and the other takes a VG name
-# in arg pos 1.  The commands that take LV_new use that
-# name as the new name of the pool, but the commands that
-# take a VG automatically generate the LV name.  The problem
-# is that currently the command matching function cannot
-# distinguish between an LV name and a VG name being used
-# in arg pos 1, so a user-entered command would just match
-# the first it finds and not necessarily the correct
-# definition.  Note that when LV_new is used in arg pos 1,
-# it needs to include a VG name, i.e. VG/LV_new
-
 lvcreate --type thin --virtualsize SizeMB --size SizeMB --thinpool LV_new
-OO: --thin, OO_LVCREATE_POOL, OO_LVCREATE_THIN, OO_LVCREATE,
+OO: OO_LVCREATE_POOL, OO_LVCREATE_THIN, OO_LVCREATE,
 --stripes Number, --stripesize SizeKB
 OP: PV ...
 ID: lvcreate_thin_vol_and_thinpool
@@ -895,31 +860,31 @@ ID: lvcreate_thin_vol_and_thinpool
 DESC: Create a thin LV, first creating a thin pool for it,
 DESC: where the new thin pool is named by the --thinpool arg
 DESC: (variant, infers --type thin).
-FLAGS: SECONDARY_SYNTAX
 
-# alternate form of lvcreate --type thin
-lvcreate --type thin --virtualsize SizeMB --size SizeMB LV_new|VG
-OO: --thin, OO_LVCREATE_POOL, OO_LVCREATE_THIN, OO_LVCREATE,
+lvcreate --type thin --virtualsize SizeMB --size SizeMB LV_new
+OO: OO_LVCREATE_POOL, OO_LVCREATE_THIN, OO_LVCREATE,
 --stripes Number, --stripesize SizeKB
 OP: PV ...
 ID: lvcreate_thin_vol_and_thinpool
 DESC: Create a thin LV, first creating a thin pool for it,
-DESC: where the new thin pool is named in the first arg,
-DESC: or the new thin pool name is generated when the first
-DESC: arg is a VG name.
-FLAGS: SECONDARY_SYNTAX
+DESC: where the new thin pool is named in the first arg.
 
-# alternate form of lvcreate --type thin
-lvcreate --thin --virtualsize SizeMB --size SizeMB LV_new|VG
+lvcreate --thin --virtualsize SizeMB --size SizeMB LV_new
 OO: --type thin, OO_LVCREATE_POOL, OO_LVCREATE_THIN, OO_LVCREATE,
 --stripes Number, --stripesize SizeKB
 OP: PV ...
 ID: lvcreate_thin_vol_and_thinpool
 DESC: Create a thin LV, first creating a thin pool for it,
-DESC: where the new thin pool is named in the first arg,
-DESC: or the new thin pool name is generated when the first
-DESC: arg is a VG name (variant, infers --type thin).
-FLAGS: SECONDARY_SYNTAX
+DESC: where the new thin pool is named in the first arg
+DESC: (variant, infers --type thin).
+
+lvcreate --type thin --virtualsize SizeMB --size SizeMB VG
+OO: OO_LVCREATE_POOL, OO_LVCREATE_THIN, OO_LVCREATE,
+--stripes Number, --stripesize SizeKB
+OP: PV ...
+ID: lvcreate_thin_vol_and_thinpool
+DESC: Create a thin LV, first creating a thin pool for it,
+DESC: where the new thin pool name is generated automatically.
 
 ---
 
@@ -934,7 +899,6 @@ DESC: Create a sparse snapshot of a virtual origin LV
 DESC: (infers --type snapshot).
 DESC: Chooses --type thin or --type snapshot according to
 DESC: config setting sparse_segtype_default.
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
@@ -960,7 +924,6 @@ ID: lvcreate_cache_vol_with_new_origin
 DESC: Create a cache LV, first creating a new origin LV,
 DESC: then combining it with the existing cache pool named
 DESC: by the --cachepool arg (variant, infers --type cache).
-FLAGS: SECONDARY_SYNTAX
 
 # alternate form of lvcreate --type cache
 lvcreate --type cache --size SizeMB LV_cachepool
@@ -971,7 +934,6 @@ ID: lvcreate_cache_vol_with_new_origin
 DESC: Create a cache LV, first creating a new origin LV,
 DESC: then combining it with the existing cache pool named
 DESC: in the first arg (variant, also use --cachepool).
-FLAGS: SECONDARY_SYNTAX
 
 # This is a ridiculously crazy command which nobody could
 # understand.  It should be be eliminated.  It does two different
@@ -1012,8 +974,7 @@ FLAGS: SECONDARY_SYNTAX
 # def1: alternate form of lvcreate --type cache, or
 # def2: it should be done by lvconvert.
 lvcreate --cache --size SizeMB LV
-OO: OO_LVCREATE_CACHE, OO_LVCREATE_POOL, OO_LVCREATE,
---stripes Number, --stripesize SizeKB
+OO: OO_LVCREATE_CACHE, OO_LVCREATE, --stripes Number, --stripesize SizeKB
 OP: PV ...
 ID: lvcreate_cache_vol_with_new_origin_or_convert_to_cache_vol_with_cachepool
 DESC: When LV is a cache pool, create a cache LV,
@@ -1023,7 +984,6 @@ DESC: (variant, infers --type cache, also use --cachepool).
 DESC: When LV is not a cache pool, convert the specified LV
 DESC: to type cache after creating a new cache pool LV to use
 DESC: (use lvconvert).
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
@@ -1044,7 +1004,7 @@ ID: lvdisplay_general
 lvextend --size SizeMB LV
 OO: --alloc Alloc, --autobackup Bool, --force, --mirrors SNumber,
 --nofsck, --nosync, --noudevsync, --reportformat String, --resizefs,
---stripes Number, --stripesize SizeKB, --poolmetadatasize SizeMB
+--stripes Number, --stripesize SizeKB, --test, --poolmetadatasize SizeMB
 OP: PV ...
 ID: lvextend_by_size
 DESC: Extend an LV by a specified size.
@@ -1052,15 +1012,16 @@ DESC: Extend an LV by a specified size.
 lvextend LV PV ...
 OO: --alloc Alloc, --autobackup Bool, --force, --mirrors SNumber,
 --nofsck, --nosync, --noudevsync,
---reportformat String, --resizefs, --stripes Number, --stripesize SizeKB
+--reportformat String, --resizefs, --stripes Number, --stripesize SizeKB,
+--test
 ID: lvextend_by_pv
-DESC: Extend an LV by specified PV extents.
-FLAGS: SECONDARY_SYNTAX
+DESC: Extend an LV by a specified PV.
 
 lvextend --poolmetadatasize SizeMB LV_thinpool
 OO: --alloc Alloc, --autobackup Bool, --force, --mirrors SNumber,
 --nofsck, --nosync, --noudevsync,
---reportformat String, --stripes Number, --stripesize SizeKB
+--reportformat String, --stripes Number, --stripesize SizeKB,
+--test
 OP: PV ...
 ID: lvextend_pool_metadata_by_size
 DESC: Extend a pool metadata SubLV by a specified size.
@@ -1068,7 +1029,8 @@ DESC: Extend a pool metadata SubLV by a specified size.
 lvextend --usepolicies LV_thinpool_snapshot
 OO: --alloc Alloc, --autobackup Bool, --force, --mirrors SNumber,
 --nofsck, --nosync, --noudevsync,
---reportformat String, --resizefs
+--reportformat String, --resizefs,
+--test
 ID: lvextend_by_policy
 DESC: Extend an LV according to a predefined policy.
 
@@ -1082,24 +1044,24 @@ ID: lvmconfig_general
 
 lvreduce --size SizeMB LV
 OO: --autobackup Bool, --force, --nofsck, --noudevsync,
---reportformat String, --resizefs
+--reportformat String, --resizefs, --test
 ID: lvreduce_general
 
 ---
 
 lvremove VG|LV|Tag|Select ...
 OO: --autobackup Bool, --force, --nohistory, --noudevsync,
---reportformat String, --select String
+--reportformat String, --select String, --test
 ID: lvremove_general
 
 ---
 
 lvrename VG LV LV_new
-OO: --autobackup Bool, --noudevsync, --reportformat String
+OO: --autobackup Bool, --noudevsync, --reportformat String, --test
 ID: lvrename_vg_lv_lv
 
 lvrename LV LV_new
-OO: --autobackup Bool, --noudevsync, --reportformat String
+OO: --autobackup Bool, --noudevsync, --reportformat String, --test
 ID: lvrename_lv_lv
 
 ---
@@ -1107,7 +1069,7 @@ ID: lvrename_lv_lv
 lvresize --size SizeMB LV
 OO: --alloc Alloc, --autobackup Bool, --force,
 --nofsck, --nosync, --noudevsync, --reportformat String, --resizefs,
---stripes Number, --stripesize SizeKB, --poolmetadatasize SizeMB
+--stripes Number, --stripesize SizeKB, --test, --poolmetadatasize SizeMB
 OP: PV ...
 ID: lvresize_by_size
 DESC: Resize an LV by a specified size.
@@ -1115,15 +1077,16 @@ DESC: Resize an LV by a specified size.
 lvresize LV PV ...
 OO: --alloc Alloc, --autobackup Bool, --force,
 --nofsck, --nosync, --noudevsync,
---reportformat String, --resizefs, --stripes Number, --stripesize SizeKB
+--reportformat String, --resizefs, --stripes Number, --stripesize SizeKB,
+--test
 ID: lvresize_by_pv
-DESC: Resize an LV by specified PV extents.
-FLAGS: SECONDARY_SYNTAX
+DESC: Resize an LV by a specified PV.
 
 lvresize --poolmetadatasize SizeMB LV_thinpool
 OO: --alloc Alloc, --autobackup Bool, --force,
 --nofsck, --nosync, --noudevsync,
---reportformat String, --stripes Number, --stripesize SizeKB
+--reportformat String, --stripes Number, --stripesize SizeKB,
+--test
 OP: PV ...
 ID: lvresize_pool_metadata_by_size
 DESC: Resize a pool metadata SubLV by a specified size.
@@ -1146,7 +1109,7 @@ ID: lvscan_general
 
 # None of these can function as a required option for pvchange.
 OO_PVCHANGE: --autobackup Bool, --force, --ignoreskippedcluster,
---reportformat String, --uuid
+--reportformat String, --test, --uuid
 
 # Any of these can function as a required option for pvchange.
 OO_PVCHANGE_META: --allocatable Bool, --addtag Tag, --deltag Tag,
@@ -1165,7 +1128,7 @@ DESC: Change properties of specified PVs.
 ---
 
 pvresize PV ...
-OO: --setphysicalvolumesize SizeMB, --reportformat String
+OO: --setphysicalvolumesize SizeMB, --reportformat String, --test
 ID: pvresize_general
 
 ---
@@ -1181,7 +1144,7 @@ ID: pvck_general
 
 pvcreate PV ...
 OO: --dataalignment SizeKB, --dataalignmentoffset SizeKB, --bootloaderareasize SizeMB,
---force, --labelsector Number, --metadatatype MetadataType,
+--force, --test, --labelsector Number, --metadatatype MetadataType,
 --pvmetadatacopies Number, --metadatacopies MetadataCopies, --metadatasize SizeMB,
 --metadataignore Bool, --norestorefile, --setphysicalvolumesize SizeMB,
 --reportformat String, --restorefile String, --uuidstr String, --zero Bool
@@ -1202,20 +1165,20 @@ ID: pvdisplay_general
 
 pvmove PV
 OO: --abort, --alloc Alloc, --atomic, --autobackup Bool, --background,
---interval Number, --name LV, --noudevsync, --reportformat String
+--interval Number, --name LV, --noudevsync, --reportformat String, --test
 OP: PV ...
 ID: pvmove_one
 DESC: Move PV extents.
 
 pvmove
-OO: --abort, --background
+OO: --abort, --background, --test
 ID: pvmove_any
 DESC: Continue or abort existing pvmove operations.
 
 ---
 
 pvremove PV ...
-OO: --force, --reportformat String
+OO: --force, --reportformat String, --test
 ID: pvremove_general
 
 ---
@@ -1251,7 +1214,7 @@ ID: vgcfgbackup_general
 ---
 
 vgcfgrestore VG
-OO: --file String, --force_long, --list, --metadatatype MetadataType
+OO: --file String, --force_long, --list, --metadatatype MetadataType, --test
 ID: vgcfgrestore_by_vg
 
 vgcfgrestore --list --file String
@@ -1262,7 +1225,7 @@ ID: vgcfgrestore_by_file
 # None of these can function as a required option for vgchange.
 
 OO_VGCHANGE: --autobackup Bool, --ignoremonitoring, --ignoreskippedcluster,
---noudevsync, --reportformat String, --select String, --force
+--noudevsync, --reportformat String, --select String, --test, --force
 
 # Any of these can function as a required option for vgchange.
 # profile is also part of OO_ALL, but is repeated in OO_VGCHANGE_META
@@ -1276,44 +1239,44 @@ OO_VGCHANGE_META: --addtag Tag, --deltag Tag,
 
 vgchange OO_VGCHANGE_META
 OO: OO_VGCHANGE
-OP: VG|Tag|Select ...
+OP: VG|Tag ...
 ID: vgchange_properties
 DESC: Change a general VG property.
 
 vgchange --monitor Bool
-OO: --sysinit, --ignorelockingfailure, --poll Bool, OO_VGCHANGE
-OP: VG|Tag|Select ...
+OO: --sysinit, --ignorelockingfailure, --poll Bool, OO_VGCHANGE_META, OO_VGCHANGE
+OP: VG|Tag ...
 ID: vgchange_monitor
 DESC: Start or stop monitoring LVs from dmeventd.
 
 vgchange --poll Bool
-OO: --ignorelockingfailure, OO_VGCHANGE
-OP: VG|Tag|Select ...
+OO: --ignorelockingfailure, OO_VGCHANGE_META, OO_VGCHANGE
+OP: VG|Tag ...
 ID: vgchange_poll
 DESC: Start or stop processing LV conversions.
 
 vgchange --activate Active
 OO: --activationmode ActivationMode, --ignoreactivationskip, --partial, --sysinit,
---ignorelockingfailure, --monitor Bool, --poll Bool, OO_VGCHANGE
-OP: VG|Tag|Select ...
+--ignorelockingfailure, --monitor Bool, --poll Bool, OO_VGCHANGE_META, OO_VGCHANGE
+OP: VG|Tag ...
 ID: vgchange_activate
 DESC: Activate or deactivate LVs.
 
 vgchange --refresh
-OO: --sysinit, --ignorelockingfailure, --monitor Bool, --poll Bool, OO_VGCHANGE
-OP: VG|Tag|Select ...
+OO: --sysinit, --ignorelockingfailure, --monitor Bool, --poll Bool, OO_VGCHANGE_META, OO_VGCHANGE
+OP: VG|Tag ...
 ID: vgchange_refresh
 DESC: Reactivate LVs using the latest metadata.
 
 vgchange --lockstart
-OO: --lockopt String, OO_VGCHANGE
-OP: VG|Tag|Select ...
+OO: --lockopt String, OO_VGCHANGE_META, OO_VGCHANGE
+OP: VG|Tag ...
 ID: vgchange_lockstart
 DESC: Start the lockspace of a shared VG in lvmlockd.
 
 vgchange --lockstop
-OO: --lockopt String, OO_VGCHANGE
-OP: VG|Tag|Select ...
+OO: --lockopt String, OO_VGCHANGE_META, OO_VGCHANGE
+OP: VG|Tag ...
 ID: vgchange_lockstop
 DESC: Stop the lockspace of a shared VG in lvmlockd.
 
@@ -1327,7 +1290,7 @@ ID: vgck_general
 ---
 
 vgconvert VG ...
-OO: --force, --labelsector Number, --bootloaderareasize SizeMB,
+OO: --force, --test, --labelsector Number, --bootloaderareasize SizeMB,
 --metadatatype MetadataType, --pvmetadatacopies Number,
 --metadatasize SizeMB, --reportformat String
 ID: vgconvert_general
@@ -1337,7 +1300,7 @@ ID: vgconvert_general
 vgcreate VG_new PV ...
 OO: --addtag Tag, --alloc Alloc, --autobackup Bool, --clustered Bool, --maxlogicalvolumes Number,
 --maxphysicalvolumes Number, --metadataprofile String, --metadatatype MetadataType,
---physicalextentsize SizeMB, --force, --zero Bool, --labelsector Number,
+--physicalextentsize SizeMB, --test, --force, --zero Bool, --labelsector Number,
 --metadatasize SizeMB, --pvmetadatacopies Number, --reportformat String, --metadatacopies MetadataCopies,
 --vgmetadatacopies MetadataCopies, --dataalignment SizeKB, --dataalignmentoffset SizeKB,
 --shared, --systemid String, --locktype LockType, --lockopt String
@@ -1356,7 +1319,7 @@ ID: vgdisplay_general
 
 ---
 
-OO_VGEXPORT: --reportformat String
+OO_VGEXPORT: --reportformat String, --test
 
 vgexport VG|Tag|Select ...
 OO: --select String, OO_VGEXPORT
@@ -1367,12 +1330,11 @@ vgexport --all
 OO: OO_VGEXPORT
 ID: vgexport_all
 DESC: Export all VGs.
-FLAGS: SECONDARY_SYNTAX
 
 ---
 
 vgextend VG PV ...
-OO: --autobackup Bool,
+OO: --autobackup Bool, --test,
 --force, --zero Bool, --labelsector Number, --metadatatype MetadataType,
 --metadatasize SizeMB, --pvmetadatacopies Number,
 --metadataignore Bool, --dataalignment SizeKB, --dataalignmentoffset SizeKB,
@@ -1381,7 +1343,7 @@ ID: vgextend_general
 
 ---
 
-OO_VGIMPORT: --force, --reportformat String
+OO_VGIMPORT: --force, --reportformat String, --test
 
 vgimport VG|Tag|Select ...
 OO: --select String, OO_VGIMPORT
@@ -1396,13 +1358,13 @@ DESC: Import all VGs.
 ---
 
 vgimportclone PV ...
-OO: --basevgname VG, --import
+OO: --basevgname VG, --test, --import
 ID: vgimportclone_general
 
 ---
 
 vgmerge VG VG
-OO: --autobackup Bool, --list
+OO: --autobackup Bool, --list, --test
 ID: vgmerge_general
 
 ---
@@ -1414,7 +1376,7 @@ ID: vgmknodes_general
 
 ---
 
-OO_VGREDUCE: --autobackup Bool, --force, --reportformat String
+OO_VGREDUCE: --autobackup Bool, --force, --reportformat String, --test
 
 vgreduce VG PV ...
 OO: OO_VGREDUCE
@@ -1434,18 +1396,18 @@ DESC: Remove all missing PVs from a VG.
 ---
 
 vgremove VG|Tag|Select ...
-OO: --force, --noudevsync, --reportformat String, --select String
+OO: --force, --noudevsync, --reportformat String, --select String, --test
 ID: vgremove_general
 
 ---
 
 vgrename VG VG_new
-OO: --autobackup Bool, --force, --reportformat String
+OO: --autobackup Bool, --force, --reportformat String, --test
 ID: vgrename_by_name
 DESC: Rename a VG.
 
 vgrename String VG_new
-OO: --autobackup Bool, --force, --reportformat String
+OO: --autobackup Bool, --force, --reportformat String, --test
 ID: vgrename_by_uuid
 DESC: Rename a VG by specifying the VG UUID.
 
@@ -1465,7 +1427,7 @@ ID: vgscan_general
 
 ---
 
-OO_VGSPLIT: --autobackup Bool
+OO_VGSPLIT: --autobackup Bool, --test
 
 OO_VGSPLIT_NEW: --alloc Alloc, --clustered Bool,
 --maxlogicalvolumes Number, --maxphysicalvolumes Number,
@@ -1534,7 +1496,7 @@ OO: --reportformat String, --select String
 ID: lastlog_general
 
 lvpoll --polloperation String LV ...
-OO: --abort, --autobackup Bool, --handlemissingpvs, --interval Number
+OO: --abort, --autobackup Bool, --handlemissingpvs, --interval Number, --test
 ID: lvpoll_general
 
 formats

scripts/create-commands.c

@@ -577,13 +577,6 @@ static int is_desc_line(char *str)
 	return 0;
 }
 
-static int is_flags_line(char *str)
-{
-	if (!strncmp(str, "FLAGS:", 6))
-		return 1;
-	return 0;
-}
-
 static int is_id_line(char *str)
 {
 	if (!strncmp(str, "ID:", 3))
@@ -1154,46 +1147,14 @@ static char *flags_to_str(int flags)
 
 	memset(buf_flags, 0, sizeof(buf_flags));
 
-	if (flags & ARG_DEF_FLAG_MAY_REPEAT) {
-		if (buf_flags[0])
-			strcat(buf_flags, " | ");
+	if (flags & ARG_DEF_FLAG_MAY_REPEAT)
 		strcat(buf_flags, "ARG_DEF_FLAG_MAY_REPEAT");
-	}
-	if (flags & ARG_DEF_FLAG_NEW) {
-		if (buf_flags[0])
-			strcat(buf_flags, " | ");
+	if (flags & ARG_DEF_FLAG_NEW)
 		strcat(buf_flags, "ARG_DEF_FLAG_NEW");
-	}
 
 	return buf_flags;
 }
 
-static void add_flags(struct command *cmd, char *line)
-{
-	if (strstr(line, "SECONDARY_SYNTAX"))
-		cmd->cmd_flags |= CMD_FLAG_SECONDARY_SYNTAX;
-}
-
-static char *cmd_flags_to_str(uint32_t flags)
-{
-	static char buf_cmd_flags[32];
-
-	memset(buf_cmd_flags, 0, sizeof(buf_cmd_flags));
-
-	if (flags & CMD_FLAG_SECONDARY_SYNTAX) {
-		if (buf_cmd_flags[0])
-			strcat(buf_cmd_flags, " | ");
-		strcat(buf_cmd_flags, "CMD_FLAG_SECONDARY_SYNTAX");
-	}
-	if (flags & CMD_FLAG_ONE_REQUIRED_OPT) {
-		if (buf_cmd_flags[0])
-			strcat(buf_cmd_flags, " | ");
-		strcat(buf_cmd_flags, "CMD_FLAG_ONE_REQUIRED_OPT");
-	}
-
-	return buf_cmd_flags;
-}
-
 void print_command_count(void)
 {
 	struct command *cmd;
@@ -2109,10 +2070,8 @@ void print_command_struct(int only_usage)
 		printf("commands[%d].oo_count = %d;\n", i, cmd->oo_count);
 		printf("commands[%d].op_count = %d;\n", i, cmd->op_count);
 
-		if (cmd->cmd_flags)
-			printf("commands[%d].cmd_flags = %s;\n", i, cmd_flags_to_str(cmd->cmd_flags));
-		else
-			printf("commands[%d].cmd_flags = 0;\n", i, cmd_flags_to_str(cmd->cmd_flags));
+		if (cmd->cmd_flags & CMD_FLAG_ONE_REQUIRED_OPT)
+			printf("commands[%d].cmd_flags = CMD_FLAG_ONE_REQUIRED_OPT;\n", i);
 
 		printf("commands[%d].desc = \"%s\";\n", i, cmd->desc ?: "");
 		printf("commands[%d].usage = ", i);
@@ -2439,22 +2398,15 @@ int main(int argc, char *argv[])
 		if (is_desc_line(line_argv[0])) {
 			char *desc = strdup(line_orig);
 			if (cmd->desc) {
-				int newlen = strlen(cmd->desc) + strlen(desc) + 2;
-				char *newdesc = malloc(newlen);
-				memset(newdesc, 0, newlen);
-				snprintf(newdesc, newlen, "%s %s", cmd->desc, desc);
-				cmd->desc = newdesc;
+				cmd->desc = realloc((char *)cmd->desc, strlen(cmd->desc) + strlen(desc) + 2);
+				strcat((char *)cmd->desc, "  ");
+				strcat((char *)cmd->desc, desc);
 				free(desc);
 			} else
 				cmd->desc = desc;
 			continue;
 		}
 
-		if (is_flags_line(line_argv[0])) {
-			add_flags(cmd, line_orig);
-			continue;
-		}
-
 		if (is_id_line(line_argv[0])) {
 			cmd->command_line_id = strdup(line_argv[1]);
 			continue;
@@ -2527,7 +2479,5 @@ int main(int argc, char *argv[])
 		print_man_command();
 	else
 		print_help(argc, argv);
-
-	return 0;
 }
 

tools/Makefile.in

@@ -76,7 +76,6 @@ SOURCES2 =\
 
 TARGETS =\
 	.commands \
-	command-lines.h \
 	liblvm2cmd.a \
 	lvm
 
@@ -100,8 +99,7 @@ LIB_VERSION = $(LIB_VERSION_LVM)
 CLEAN_TARGETS = liblvm2cmd.$(LIB_SUFFIX) $(TARGETS_DM) \
 	liblvm2cmd.$(LIB_SUFFIX).$(LIB_VERSION) lvm-static.o \
 	liblvm2cmd-static.a dmsetup.static lvm.static \
-	$(LDDEPS) .exported_symbols_generated \
-	ccmd command-lines.h command-lines-count.h
+	$(LDDEPS) .exported_symbols_generated
 
 ifeq ("@CMDLIB@", "yes")
 	TARGETS += liblvm2cmd.$(LIB_SUFFIX).$(LIB_VERSION)
@@ -173,13 +171,6 @@ liblvm2cmd.$(LIB_SUFFIX).$(LIB_VERSION): liblvm2cmd.$(LIB_SUFFIX)
 	$(CC) -E -P $(srcdir)/cmdnames.h 2> /dev/null | \
 		egrep -v '^ *(|#.*|config|devtypes|dumpconfig|formats|fullreport|help|lastlog|lvpoll|pvdata|segtypes|systemid|tags|version) *$$' > .commands
 
-ccmd: create-commands.c
-	$(CC) create-commands.c -o ccmd
-
-command-lines.h: ccmd
-	./ccmd --output struct command-lines.in > command-lines.h
-	./ccmd --output count command-lines.in > command-lines-count.h
-
 ifneq ("$(CFLOW_CMD)", "")
 CFLOW_SOURCES = $(addprefix $(srcdir)/, $(SOURCES))
 -include $(top_builddir)/libdm/libdevmapper.cflow

tools/command-lines-count.h

@@ -0,0 +1,129 @@
+/* Do not edit. This file is generated by scripts/create-commands */
+/* using command definitions from scripts/command-lines.in */
+#define COMMAND_COUNT 151
+enum {
+	no_CMD,
+	lvchange_properties_CMD,
+	lvchange_resync_CMD,
+	lvchange_syncaction_CMD,
+	lvchange_rebuild_CMD,
+	lvchange_activate_CMD,
+	lvchange_refresh_CMD,
+	lvchange_monitor_CMD,
+	lvchange_poll_CMD,
+	lvchange_persistent_CMD,
+	lvconvert_merge_CMD,
+	lvconvert_combine_split_snapshot_CMD,
+	lvconvert_to_thin_with_external_CMD,
+	lvconvert_to_cache_vol_CMD,
+	lvconvert_to_thinpool_CMD,
+	lvconvert_to_cachepool_CMD,
+	lvconvert_to_mirror_CMD,
+	lvconvert_general_to_raid_CMD,
+	lvconvert_to_mirrored_or_change_image_count_CMD,
+	lvconvert_raid_to_striped_CMD,
+	lvconvert_raid_or_mirror_to_linear_CMD,
+	lvconvert_split_mirror_images_to_new_CMD,
+	lvconvert_split_mirror_images_and_track_CMD,
+	lvconvert_repair_pvs_or_thinpool_CMD,
+	lvconvert_replace_pv_CMD,
+	lvconvert_change_mirrorlog_CMD,
+	lvconvert_split_and_keep_cachepool_CMD,
+	lvconvert_split_and_delete_cachepool_CMD,
+	lvconvert_split_cow_snapshot_CMD,
+	lvconvert_poll_mirror_CMD,
+	lvconvert_swap_pool_metadata_CMD,
+	lvcreate_error_vol_CMD,
+	lvcreate_zero_vol_CMD,
+	lvcreate_linear_CMD,
+	lvcreate_striped_CMD,
+	lvcreate_mirror_CMD,
+	lvcreate_mirror_or_raid1_CMD,
+	lvcreate_raid_any_CMD,
+	lvcreate_cow_snapshot_CMD,
+	lvcreate_cow_snapshot_with_virtual_origin_CMD,
+	lvcreate_thinpool_CMD,
+	lvcreate_cachepool_CMD,
+	lvcreate_thin_vol_CMD,
+	lvcreate_thin_snapshot_CMD,
+	lvcreate_thin_snapshot_of_external_CMD,
+	lvcreate_thin_vol_and_thinpool_CMD,
+	lvcreate_thin_vol_with_thinpool_or_sparse_snapshot_CMD,
+	lvcreate_cache_vol_with_new_origin_CMD,
+	lvcreate_cache_vol_with_new_origin_or_convert_to_cache_vol_with_cachepool_CMD,
+	lvdisplay_general_CMD,
+	lvextend_by_size_CMD,
+	lvextend_by_pv_CMD,
+	lvextend_pool_metadata_by_size_CMD,
+	lvextend_by_policy_CMD,
+	lvmconfig_general_CMD,
+	lvreduce_general_CMD,
+	lvremove_general_CMD,
+	lvrename_vg_lv_lv_CMD,
+	lvrename_lv_lv_CMD,
+	lvresize_by_size_CMD,
+	lvresize_by_pv_CMD,
+	lvresize_pool_metadata_by_size_CMD,
+	lvs_general_CMD,
+	lvscan_general_CMD,
+	pvchange_properties_all_CMD,
+	pvchange_properties_some_CMD,
+	pvresize_general_CMD,
+	pvck_general_CMD,
+	pvcreate_general_CMD,
+	pvdisplay_general_CMD,
+	pvmove_one_CMD,
+	pvmove_any_CMD,
+	pvremove_general_CMD,
+	pvs_general_CMD,
+	pvscan_show_CMD,
+	pvscan_cache_CMD,
+	vgcfgbackup_general_CMD,
+	vgcfgrestore_by_vg_CMD,
+	vgcfgrestore_by_file_CMD,
+	vgchange_properties_CMD,
+	vgchange_monitor_CMD,
+	vgchange_poll_CMD,
+	vgchange_activate_CMD,
+	vgchange_refresh_CMD,
+	vgchange_lockstart_CMD,
+	vgchange_lockstop_CMD,
+	vgck_general_CMD,
+	vgconvert_general_CMD,
+	vgcreate_general_CMD,
+	vgdisplay_general_CMD,
+	vgexport_some_CMD,
+	vgexport_all_CMD,
+	vgextend_general_CMD,
+	vgimport_some_CMD,
+	vgimport_all_CMD,
+	vgimportclone_general_CMD,
+	vgmerge_general_CMD,
+	vgmknodes_general_CMD,
+	vgreduce_by_pv_CMD,
+	vgreduce_all_CMD,
+	vgreduce_missing_CMD,
+	vgremove_general_CMD,
+	vgrename_by_name_CMD,
+	vgrename_by_uuid_CMD,
+	vgs_general_CMD,
+	vgscan_general_CMD,
+	vgsplit_by_pv_CMD,
+	vgsplit_by_lv_CMD,
+	devtypes_general_CMD,
+	fullreport_general_CMD,
+	lastlog_general_CMD,
+	lvpoll_general_CMD,
+	formats_general_CMD,
+	help_general_CMD,
+	version_general_CMD,
+	pvdata_general_CMD,
+	segtypes_general_CMD,
+	systemid_general_CMD,
+	tags_general_CMD,
+	lvmchange_general_CMD,
+	lvmdiskscan_general_CMD,
+	lvmsadc_general_CMD,
+	lvmsar_general_CMD,
+	COMMAND_ID_COUNT,
+};

tools/command.h

@@ -51,8 +51,8 @@ struct command_name {
  */
 
 /* arg_def flags */
-#define ARG_DEF_FLAG_NEW                1 << 0
-#define ARG_DEF_FLAG_MAY_REPEAT         1 << 1
+#define ARG_DEF_FLAG_NEW        1 << 0
+#define ARG_DEF_FLAG_MAY_REPEAT 1 << 1
 
 /* arg_def lv_types */
 enum {
@@ -123,8 +123,7 @@ struct pos_arg {
  * one or more from required_opt_args is required,
  * then the rest are optional.
  */
-#define CMD_FLAG_ONE_REQUIRED_OPT   1
-#define CMD_FLAG_SECONDARY_SYNTAX   2
+#define CMD_FLAG_ONE_REQUIRED_OPT 1
 
 /* a register of the lvm commands */
 struct command {

tools/lvmcmdline.c

@@ -1237,14 +1237,17 @@ static void _print_description(int ci)
 	int bi = 0;
 
 	for (di = 0; di < strlen(desc); di++) {
+		if (desc[di] == '\0')
+			break;
+		if (desc[di] == '\n')
+			continue;
+
 		if (!strncmp(&desc[di], "DESC:", 5)) {
 			if (bi) {
-				buf[bi] = '\0';
 				log_print("%s", buf);
 				memset(buf, 0, sizeof(buf));
 				bi = 0;
 			}
-			/* skip DESC: */
 			di += 5;
 			continue;
 		}
@@ -1258,10 +1261,8 @@ static void _print_description(int ci)
 			break;
 	}
 
-	if (bi) {
-		buf[bi] = '\0';
+	if (bi)
 		log_print("%s", buf);
-	}
 }
 
 /*
@@ -1548,9 +1549,6 @@ static int _usage(const char *name, int help_count)
 		if (strcmp(_cmdline.commands[i].name, name))
 			continue;
 
-		if ((_cmdline.commands[i].cmd_flags & CMD_FLAG_SECONDARY_SYNTAX) && (help_count < 3))
-			continue;
-
 		if (strlen(_cmdline.commands[i].desc))
 			_print_description(i);
 
@@ -1587,11 +1585,6 @@ static int _usage(const char *name, int help_count)
 		log_print(". The _new suffix indicates the VG or LV must not yet exist.");
 		log_print(". LV followed by _<type> indicates that an LV of the given type");
 		log_print("  is required.  (raid represents any raid<N> type.)");
-		log_print(". The default output unit is specified by letter, followed by |unit");
-		log_print("  which represents other possible units: hHbBsSkKmMgGtTpPeE.");
-		log_print(". Output units are 1024 SI base, regardless of unit capitalization.");
-		log_print(". Use --help --help --help to print secondary command syntax");
-		log_print("  formats that are recognized, e.g. for compatibility.");
 		log_print(". See man pages for short option equivalents of long option names,");
 		log_print("  and for more detailed descriptions of variable parameters.");
 	}