Hello,

please find attached patch.
it improves experience by using generally available posix "command" instead
of "which"


thanks,
Ilya
From d1070681d7d06f4865bf64319299550294138fd8 Mon Sep 17 00:00:00 2001
From: Ilya Shipitsin <chipits...@gmail.com>
Date: Sun, 9 May 2021 11:50:22 +0500
Subject: [PATCH] use "command" instead of "which" for better POSIX
 compatibility

for example, "which" is not installed by default in Fedora docker image.
---
 lib/Test/Nginx.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/Test/Nginx.pm b/lib/Test/Nginx.pm
index fb70eda..c5f0df8 100644
--- a/lib/Test/Nginx.pm
+++ b/lib/Test/Nginx.pm
@@ -282,7 +282,7 @@ sub has_daemon($) {
 	}
 
 	Test::More::plan(skip_all => "$daemon not found")
-		unless `which $daemon`;
+		unless `command -v $daemon`;
 
 	return $self;
 }
-- 
2.31.1

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel

Reply via email to