Adding separate exit codes for success or failure, and printing the n…

…umber of reachable hosts
schweikert · Oct 27, 2018 · af9b3bf · af9b3bf
1 parent 9812557
commit af9b3bf
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/ci/test-10-option-u-x.pl b/ci/test-10-option-u-x.pl
@@ -27,14 +27,14 @@
 {
 my $cmd = Test::Command->new(cmd => "fping -x 1 8.8.0.0 127.0.0.1");
 $cmd->exit_is_num(0);
-$cmd->stdout_is_eq(">=1 hosts are reachable\n");
+$cmd->stdout_is_eq("Number of reachable hosts: 1\n");
 $cmd->stderr_is_eq("");
 }
 
 # fping -x
 {
 my $cmd = Test::Command->new(cmd => "fping -x 2 8.8.0.0 127.0.0.1");
-$cmd->exit_is_num(0);
+$cmd->exit_is_num(1);
 $cmd->stdout_is_eq("<2 hosts are reachable\n");
 $cmd->stderr_is_eq("");
 }
diff --git a/src/fping.c b/src/fping.c
@@ -1334,11 +1334,12 @@ void finish()
 
     if (reachable_flag) {
         if ((num_hosts-num_unreachable >= reachable)) {
-            printf(">=%d hosts are reachable\n", reachable);
+            printf("Number of reachable hosts: %d\n", num_hosts-num_unreachable);
+            exit(0);
         } else {
             printf("<%d hosts are reachable\n", reachable);
+            exit(1);
         }
-        exit(0);
     }
 
     if (num_noaddress)