Fix incorrect return value in help function.

Fix vgrename using UUID in case there are VGs with the same name.

Create global is_static() to eliminate from the library init function.

Very simple / crude method of removing 'is_static' from initialization.
Why should we require an application tell us whether it is linked
statically or dynamically to libLVM?  If the application is linked
statically, but libraries exist and dlopen() calls succeed, why
do we care if it's statically linked?

Remove struct arg * from struct cmd_context and create_toolcontext().

This allows us to remove one argument from create_toolcontext() and
moves it closer to a generic library init function.

In the arg_*() functions, we just use _the_args() directly.
For now we leave the first parameter to these
arg_*() functions (struct cmd_context *) because
of the number of files involved in removing the
parameter.

Move arg_* functions from toollib.c to lvmcmdline.c.

In preparation for removing cmd->args.
IMO, it makes more sense to put these accessor functions
in the same location as the static array _the_args.
Next patch will update arg_* functions to use _the_args[]
directly and remove cmd->args.

Remove status=noxfer from test, not all dd versions support this (RHEL4 for example).

Rename 'cmd' to 'clvmd_cmd' to remove ambiguity.

Signed-off-by: Dave Wysochanski <dwysocha@redhat.com>
Acked-by: James Cameron <james.cameron@hp.com>
Acked-by: Alasdair G Kergon <agk@redhat.com>

Fix segfault when invalid field given in reporting commands.

Problem is dm_report_init() may return NULL and subsequent call to
dm_report_set_output_field_name_prefix() doesn't handle NULL value.

Example:
pvs --nameprefixes --rows --unquoted --noheadings -opv_name,fred
  Logical Volume Fields
  ---------------------
    lv_uuid              - Unique identifier
    lv_name              - Name.  LVs created for internal use are enclosed in brackets.
 ...

  Physical Volume Segment Fields
  ------------------------------
    pvseg_start          - Physical Extent number of start of segment.
    pvseg_size           - Number of extents in segment.

  Unrecognised field: fred
Segmentation fault

Use dm_snprintf to avoid duplicating the snprintf compatibility code.

Create _init_globals() and call from bottom of create_toolcontext().

Move init_full_scan_done(0) and init_mirror_in_sync(0) from init_lvm()
after call to create_toolcontext() to _init_globals(), called from bottom
of create_toolcontext().  No functional change.

Author: Dave Wysochanski <dwysocha@redhat.com>
Acked-by: James Cameron <james.cameron@hp.com>
Acked-by: Alasdair G Kergon <agk@redhat.com>

Replace _dm_snprintf with EMIT_PARAMS macro for creating target lines

*** empty log message ***

Move initialization of cmd->fmt into init_formats().

init_formats() sets up the command formats, and currently sets cmd->fmt_backup
but does not set cmd->fmt to a default value.  This seems incorrect so we
set it to cmd->default_settings.fmt before returning.

The call we remove here may set cmd->fmt based on a command line setting.
But it is safe to remove this, because the only caller of init_lvm() that
cares about the cmdline override is the cmdline tools (clvmd does not care),
called from lvm2_main().  After lvm2_main() calls init_lvm(), it later calls
lvm_run_command().  In lvm_run_command(), we have a call to _apply_settings(),
which has the identical assignment of cmd->fmt that this patch removes.

Remove redundant init_msg_prefix() and init_cmd_name().

This is very obvious - _init_logging() makes the identical init_msg_prefix()
and init_cmd_name() calls with cmd->default_settings so these calls are
clearly redundant after calling create_toolcontext().

Remove redundant set_activation() call after create_toolcontext() calls.

Very similar argument to removal of init_debug() and other calls.

create_toolcontext() calls _process_config() which sets
cmd->default_settings.activation, then calls
set_activation(cmd->default_settings.activation).  Later, create_toolcontext()
sets cmd->current_settings = cmd->default_settings.  So these calls
set_activation(cmd->current_settings.activation) are redundant.

Remove backup_enable() calls after create_toolcontext() calls.

Identical argument to previous patch which removed archive_enable() calls.
We add a new parameter to backup_init() which sets the enable value based
on the cmd->default_settings.backup value.  This value was used to set
cmd->current_settings.backup, used in the removed backup_enable() call.

Remove archive_enable() calls after create_toolcontext() calls.

_init_backup() calls archive_init(), which originally set 'enabled' to
a hardcoded '1' value.  This seems incorrect based on my read of other
areas of the code so here we add a 'enabled' paramter to archive_init().
We pass in cmd->default_settings.archive, which is obtained from the
config tree.  Later in create_toolcontext, cmd->current_settings is
set to cmd->default_settings.  The archive_enable() call we remove
here was using cmd->current_settings to set the 'archive' enable
value.  The final value of cmd->archive_params->enabled should thus
be equivalent to the original code.

Move init_test() from _apply_settings into _init_logging().

This one we actually need to move.  _init_logging() is called from
    create_toolcontext(), which makes this call:
        /* Test mode */
        cmd->default_settings.test =
            find_config_tree_int(cmd, "global/test", 0);

But it does not call init_test().  So we need an init_test() somewhere.
The most logical place is to put it inside _init_logging(), since this
is where the config value is read and default_settings are set.  Placing
the init_test() call here matches what is done with other variables and
seems to make sense.

Remove handles_missing_pvs assignment after call to create_toolcontext().

This variable is set at the top of create_toolcontext() to 0.
Nothing later in create_toolcontext() changes the value.
In init_lvm(), nothing between create_toolcontext() call and this assignment
changes the value.  Thus, the assignment is redundant.

Remove init_verbose() calls immediately after create_toolcontext() calls.

The rationale for removing init_verbose() call is very similar to removing
init_debug() call.  create_toolcontext() calls _init_logging() which
makes these calls:
        /* Verbose level for tty output */
        cmd->default_settings.verbose =
            find_config_tree_int(cmd, "log/verbose", DEFAULT_VERBOSE);
        init_verbose(cmd->default_settings.verbose + VERBOSE_BASE_LEVEL);

And being that create_toolcontext() copies default_settings into
current_settings at the bottom, the init_verbose() call we are removing:
        init_verbose(cmd->current_settings.verbose + VERBOSE_BASE_LEVEL);

is redundant.

Remove init_debug() calls immediately after create_toolcontext() call.

We can safely remove because create_toolcontext() calls _init_logging(),
which makes these calls:
        /* Debug level for log file output */
        cmd->default_settings.debug =
            find_config_tree_int(cmd, "log/level", DEFAULT_LOGLEVEL);
        init_debug(cmd->default_settings.debug);

Then at the bottom of create_toolcontext() we do this:
        cmd->current_settings = cmd->default_settings;

So the call we are removing from init_lvm() functions (clvmd and lvmcmdline):
        init_debug(cmd->current_settings.debug);

Just sets the value of debug based on 'cmd->current_settings.debug'.
Since cmd->current_settings is equivalent to cmd->default_settings, and
init_debug() was called with cmd->default_settings, the call we remove is
redundant.

Replace _apply_settings() after create_toolcontext() with equivalent inline.

Subsequent patches will refactor / remove each of these lines, as many of
them are redundant when called immediately after create_toolcontext().

operate on test subdirectory instead on main /dev

add simple test for dev node usability (detect devices mounted with nodev)

Add liblvm interactive test infrastructure to build.

Make _init_rand() thread safe - use rand_r() instead of rand().

Use good entropy for seed value if possible.

Add generic function to read /dev/urandom, used in uuid calculation.

Fix test-utils.h for rhel4 backward compatability.

for losetup, break out of the loop when successful setup of loop device,
and only look at 7 loop devices (default loop module setting)
for blockdev, use old option if new one is not available

Added displayable_lvs_in_vg and lv_is_displayable functions to deal with
the counts of visible LVs from user's perspective consistently throughout
the code.

Fix vgcreate race which could allow two parallel vgcreates to succeed,
with the second vgcreate overwriting the first.

Obtain lock before calling vg_create(), which checks for existence of vgname
and fails if it already exists.

Fix uninitialised lv_count in vgdisplay -c

Suppress 'duplicate PV' message when it's only a cache update not a real
duplicate.

Don't skip updating pvid hash when lvmcache_info struct got swapped.

Add tinfo to termcap search path for pld-linux.

Fix a starup race in clvmd that could result in huge waits for the first command to be processed.

generate init script

regenerate

Generate clvmd init script at configuration time for Red Hat-based distros.

post-release (retrospective)

daemons depends on liblvm2cmd - fixes parallel builds

fix missing const char*

Fix "lvremove -f vgname" when vgname contains snapshots.

Prior to this patch, "lvremove -f vgname" would fail if vgname contained
one or more snapshot LVs.  Now this passes, but has a side-effect.
If you issue "lvremove vgname" where vgname contains one or more snaps,
you will get an extra "y/n" prompt to remove the same snapshot.
Example:
$ lvs
  LV     VG     Attr   LSize  Origin Snap%  Move Log Copy%  Convert
  lvsnap vgtest swi-a- 16.00M lvtest   0.05
  lvtest vgtest owi-a- 64.00M
$ lvremove vgtest
Do you really want to remove active logical volume "lvsnap"? [y/n]: n
  Logical volume "lvsnap" not removed
Do you really want to remove active logical volume "lvsnap"? [y/n]: n
  Logical volume "lvsnap" not removed
  Command failed with status code 5.

Fixing this will most likely require modification of the iterator
function, process_each_lvs_in_vg() to iterate over snaps in some
cases (e.g. lvs, vgdisplay -v) but not in others (lvremove).

some makefile fixes for liblvm2cmd & remove some hardcoded .so

More man page cleanup - convert "+/-" to "+|-"

Make man pages consistent with either-or options (use "|" everywhere).

cleaner const char* usage for last_path_component()

minor compiler warning fix for function declaration prototype

removed redundant redeclaration of init_indent() and init_msg_prefix() from lvm-logging.h

* minor compilator warning fix for improper function declaration.

Add a bunch of sub-tests aimed at testing various bugzillas, by jstava.

Import a bunch of new tests by jstava. Mostly everything pass, they however
tend to expose the lvremove -ff vg ordering bug.

clarify

no need for libdevmapper.h on system any more

Include libdm in lcov reports.

Fix t-pool-labels to pass again after the vgdisplay error return fix from
last Friday.

Fix lvm2.static build.

Minor code and documentation cleanups and changes by jstava.

add bz264241 check
remove lvconvert progress notifications (-i100)
(by jstava)

First test release of the merged code base.

bz429342 lvcreate --nosync
bz192865 lvconvert logtype of an inactive mirror lv
(both by jstava)

Add bz186013 check -- lvcreate rejects an invalid regionsize (jstava).

Use separate locking directory for each of the tests, as they never share a
volume group, or physical devices for that matter. Hopefully fixes occasional
test errors due to locking failures. Patch by jstava.

Explicitly check after lvremove -ff vg that it did its job. By jstava.

Fix lvhange and lvcreate man pages to properly describe permissions option.

James Youngman <jay@gnu.org>

Fix vgdisplay return code and exit status.

Set LD_LIBRARY_PATH in init.sh, as it is required to run dmsetup now as well,
which otherwise picks up the system version of libdevmapper, which is a pretty
bad idea.

quick review of docs
Note that we'll continue to use WHATS_NEW_DM for device-mapper changes
until we can switch to unified version numbering.

make install_device-mapper

clvmd

dmeventd

more fixes

more missing bits

Fix typo.

Also report the tests that have been skipped, at the end of harness.sh.

Use $(abs_top_builddir) instead of @DMDIR@ and update paths appropriately.

more tweaks for dmeventd - not finished yet

Right, a simple build (without options) is working again.

add list fns

more tweaking to get things to compile - dmlib.h for log fns, list.h

Rename a couple of variables that matched function names.

autoreconf

Use simple x.y.z library version in libdevmapper.pc

autoreconf

further progress

export DM_LIB_VERSION

add libdevmapper-event.h

autoreconf

no DMDIR

dm+lvm2 macros

dmeventd plugins move

drop configure --with-dmdir

autoreconf

first attempt at merging configure

libdevmapper version - will change once the merger's settled in

remove ancient debian dir

Add dm-logging.h ready for libdevmapper.

drop explicit libdevmapper.h

separate lvm-globals

Separate out globals from the logging code.

Split out lvm-logging.h from log.h.