From adb39ea50c881a9f36d4a9fd2f8693eaec06b5d1 Mon Sep 17 00:00:00 2001 From: Bruno Faccini Date: Fri, 6 Feb 2015 13:13:02 +0100 Subject: [PATCH] LU-6221 utils: hsm_root is also required for --dry-run Not specifying hsm_root in copytool/lhsmtool_posix command line for --dry-run mode can lead to failure/error. This path ensures that hsm_root will be required even for --dry-run. Signed-off-by: Bruno Faccini Change-Id: Icaa2af6d1365751d9e77b2be3f60aacc9c1f6a5c Reviewed-on: http://review.whamcloud.com/13673 Tested-by: Jenkins Tested-by: Maloo Reviewed-by: Henri Doreau Reviewed-by: Hongchao Zhang Reviewed-by: Oleg Drokin --- lustre/utils/lhsmtool_posix.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lustre/utils/lhsmtool_posix.c b/lustre/utils/lhsmtool_posix.c index d1fc084..d19f173 100644 --- a/lustre/utils/lhsmtool_posix.c +++ b/lustre/utils/lhsmtool_posix.c @@ -347,7 +347,7 @@ static int ct_parseopts(int argc, char * const *argv) CT_TRACE("action=%d src=%s dst=%s mount_point=%s", opt.o_action, opt.o_src, opt.o_dst, opt.o_mnt); - if (!opt.o_dry_run && opt.o_hsm_root == NULL) { + if (opt.o_hsm_root == NULL) { rc = -EINVAL; CT_ERROR(rc, "must specify a root directory for the backend"); return rc; -- 1.8.3.1