patchtest: add stronger indication for failed tests

Do not only log that there has been an issue but add WARNING before for local
runs. Hopefully this helps to avoid that people reading the log to quickly miss
issues.

Fixes [YOCTO #15389]

Signed-off-by: Simone Weiß <simone.p.weiss@posteo.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>

diff --git a/scripts/patchtest b/scripts/patchtest
index d0dc6e8fac5164b1e8743145cf672d0a2fd812b2..316342022009a3d4c306d23636cbb339e721a24d 100755 (executable)
--- a/scripts/patchtest
+++ b/scripts/patchtest
@@ -164,9 +164,9 @@ def run(patch, logfile=None):
     if premerge_result == 2 and postmerge_result == 2:
         logger.error('patchtest: No test cases found - did you specify the correct suite directory?')
     if premerge_result == 1 or postmerge_result == 1:
-        logger.error('patchtest: At least one patchtest caused a failure or an error - please check')
+        logger.error('WARNING: patchtest: At least one patchtest caused a failure or an error - please check')
     else:
-        logger.error('patchtest: All patchtests passed')
+        logger.error('OK: patchtest: All patchtests passed')
     print('----------------------------------------------------------------------\n')
     return premerge_result or postmerge_result