do not log warning if the command actually worked

Author: vladak

opengrok-indexer/src/main/java/org/opengrok/indexer/history/MercurialRepository.java

@@ -540,8 +540,10 @@ boolean isNestable() {
     private static boolean isHgWorking() {
         String repoCommand = getCommand(MercurialRepository.class, CMD_PROPERTY_KEY, CMD_FALLBACK);
         boolean works = checkCmd(repoCommand);
-        LOGGER.log(Level.WARNING, "Command ''{0}'' does not work. " +
-                "Some operations with Mercurial repositories will fail as a result.", repoCommand);
+        if (!works) {
+            LOGGER.log(Level.WARNING, "Command ''{0}'' does not work. " +
+                    "Some operations with Mercurial repositories will fail as a result.", repoCommand);
+        }
         return works;
     }
 

opengrok-indexer/src/main/java/org/opengrok/indexer/history/PerforceRepository.java

@@ -211,8 +211,10 @@ boolean isRepositoryFor(File file, CommandTimeoutType cmdType) {
     private static boolean isP4Working() {
         String repoCommand = getCommand(PerforceRepository.class, CMD_PROPERTY_KEY, CMD_FALLBACK);
         boolean works = checkCmd(repoCommand, "-V");
-        LOGGER.log(Level.WARNING, "Command ''{0}'' does not work. " +
-                "Some operations with Perforce repositories will fail as a result.", repoCommand);
+        if (!works) {
+            LOGGER.log(Level.WARNING, "Command ''{0}'' does not work. " +
+                    "Some operations with Perforce repositories will fail as a result.", repoCommand);
+        }
         return works;
     }