log output continuously from hook runs

fixes #2939

Author: Vladimir Kotal

opengrok-tools/src/main/python/opengrok_tools/utils/command.py

@@ -18,7 +18,7 @@
 #
 
 #
-# Copyright (c) 2017, 2018, Oracle and/or its affiliates. All rights reserved.
+# Copyright (c) 2017, 2019, Oracle and/or its affiliates. All rights reserved.
 #
 
 import logging
@@ -184,10 +184,10 @@ def run(self):
                     self.out.append(line)
 
                     if self.doprint:
-                        # Even if print() fails the thread has to keep
+                        # Even if logging below fails, the thread has to keep
                         # running to avoid hangups of the executed command.
                         try:
-                            print(line.rstrip())
+                            self.logger.info(line.rstrip())
                         except Exception as print_exc:
                             self.logger.error(print_exc)
 

opengrok-tools/src/main/python/opengrok_tools/utils/hook.py

@@ -18,7 +18,7 @@
 #
 
 #
-# Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved.
+# Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved.
 #
 
 from .command import Command
@@ -40,12 +40,10 @@ def run_hook(logger, script, path, env, timeout):
     logger.debug("Running hook '{}' in directory {}".
                  format(script, path))
     cmd = Command([script], logger=logger, work_dir=path, env_vars=env,
-                  timeout=timeout)
+                  timeout=timeout, doprint=True)
     cmd.execute()
     if cmd.state != "finished" or cmd.getretcode() != SUCCESS_EXITVAL:
         logger.error("command failed: {} -> {}".format(cmd, cmd.getretcode()))
         ret = FAILURE_EXITVAL
 
-    logger.info("command output:\n{}".format(cmd.getoutputstr()))
-
     return ret