Browse Source

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]

(From OE-Core rev: 661c0a8fd8fe7bff61cea82778c25cf24d791267)

Signed-off-by: Simone Weiß <simone.p.weiss@posteo.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Simone Weiß 1 year ago
parent
commit
0ba7038c09
1 changed files with 2 additions and 2 deletions
  1. 2 2
      scripts/patchtest

+ 2 - 2
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