[PATCH v2 compass-ci 2/2] lib/job.cr: refactor the matching string action of os_mount

Signed-off-by: Xu Xijian <hdxuxijian@163.com> --- src/lib/job.cr | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/lib/job.cr b/src/lib/job.cr index 89c92c1..597c88c 100644 --- a/src/lib/job.cr +++ b/src/lib/job.cr @@ -509,7 +509,7 @@ class Job if "#{os_mount}" == "initramfs" temp_initrds.concat(initramfs_initrds()) - elsif "#{os_mount}" == "nfs" || "#{os_mount}" == "cifs" || "#{os_mount}" == "local" + elsif ["nfs", "cifs", "local"].index "#{os_mount}" temp_initrds.concat(nfs_cifs_initrds()) end -- 2.23.0

On Sun, Feb 07, 2021 at 05:35:47PM +0800, Xu Xijian wrote:
Signed-off-by: Xu Xijian <hdxuxijian@163.com> --- src/lib/job.cr | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/lib/job.cr b/src/lib/job.cr index 89c92c1..597c88c 100644 --- a/src/lib/job.cr +++ b/src/lib/job.cr @@ -509,7 +509,7 @@ class Job
if "#{os_mount}" == "initramfs" temp_initrds.concat(initramfs_initrds()) - elsif "#{os_mount}" == "nfs" || "#{os_mount}" == "cifs" || "#{os_mount}" == "local" + elsif ["nfs", "cifs", "local"].index "#{os_mount}"
index => includes? includes? will return true of false Thanks, Xueliang
temp_initrds.concat(nfs_cifs_initrds()) end
-- 2.23.0

On Mon, Feb 08, 2021 at 09:14:10AM +0800, Cao Xueliang wrote:
On Sun, Feb 07, 2021 at 05:35:47PM +0800, Xu Xijian wrote:
Signed-off-by: Xu Xijian <hdxuxijian@163.com> --- src/lib/job.cr | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/lib/job.cr b/src/lib/job.cr index 89c92c1..597c88c 100644 --- a/src/lib/job.cr +++ b/src/lib/job.cr @@ -509,7 +509,7 @@ class Job
if "#{os_mount}" == "initramfs" temp_initrds.concat(initramfs_initrds()) - elsif "#{os_mount}" == "nfs" || "#{os_mount}" == "cifs" || "#{os_mount}" == "local" + elsif ["nfs", "cifs", "local"].index "#{os_mount}"
index => includes? includes? will return true of false
Thanks, Xueliang
OK. Thanks, Xijian
participants (2)
-
Cao Xueliang
-
Xu Xijian