Revert patch

Caller of exec must report log_error when rstatus is passed.

diff --git a/WHATS_NEW b/WHATS_NEW
index 4e02bdc065b3381a58c5a4a7d794b8522ff5ab89..8d20f569ac8acfbe4e8b7426301b04a28ca4b765 100644 (file)
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -1,6 +1,5 @@
 Version 2.02.89 - 
 ==================================
-  Use log_error instead of log_verbose when executed command fails.
   Add support for non /dev device paths into fsadm script.
   Support different PATH setting for fsadm script testing.
   Surround all executed commands with quotes in fsadm script.

diff --git a/lib/misc/lvm-exec.c b/lib/misc/lvm-exec.c
index c565ddd1dfc2c8a85d74636978d201149c8be6fd..56a9815337b24c3abb514dcafe0a5bd00a864b56 100644 (file)
--- a/lib/misc/lvm-exec.c
+++ b/lib/misc/lvm-exec.c
@@ -96,7 +96,7 @@ int exec_cmd(struct cmd_context *cmd, const char *const argv[],
        if (WEXITSTATUS(status)) {
                if (rstatus) {
                        *rstatus = WEXITSTATUS(status);
-                       log_error("%s failed: %u", argv[0], *rstatus);
+                       log_verbose("%s failed: %u", argv[0], *rstatus);
                } else
                        log_error("%s failed: %u", argv[0], WEXITSTATUS(status));
                return 0;