From cc8b9dd9733bb6bab3318f1913efecbe5bba200e Mon Sep 17 00:00:00 2001 From: Zdenek Kabelac Date: Mon, 19 Sep 2011 18:38:43 +0000 Subject: [PATCH] Revert patch Caller of exec must report log_error when rstatus is passed. --- WHATS_NEW | 1 - lib/misc/lvm-exec.c | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/WHATS_NEW b/WHATS_NEW index 4e02bdc06..8d20f569a 100644 --- 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 c565ddd1d..56a981533 100644 --- 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; -- 2.43.5