[OS-BUILD PATCHv5 23/26] [redhat] Makefile: using test -x to test for bats

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Ben Crocker <bcrocker@xxxxxxxxxx>

Using 'test -x' instead of 'dnf list installed' to test for presence
of bats package, don't need to redirect stdout & stderr.

Signed-off-by: Ben Crocker <bcrocker@xxxxxxxxxx>
---
 redhat/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/redhat/Makefile b/redhat/Makefile
index e7ba07066177..84950bf876af 100644
--- a/redhat/Makefile
+++ b/redhat/Makefile
@@ -393,7 +393,7 @@ dist-dump-variables:
 	@$(REDHAT)/dist-dump-variables.sh
 
 dist-self-test:
-	@if test -x /usr/bin/bats >& /dev/null; then \
+	@if test -x /usr/bin/bats; then \
 		bats $(REDHAT)/self-test/*.bats ; \
 	else \
 		echo "dist-self-test: The bats package is not installed" ; \
-- 
GitLab
_______________________________________________
kernel mailing list -- kernel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to kernel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/kernel@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux