revi-archive/llvm-premerge-checks
1
0
Fork 0
Code Activity

Allow "building" in ninja all log

Browse source 
That is a temporary workaround for #207 that caused by grep exiting with
1 as whole `ninja all` output is filtered out.
This commit is contained in:
Mikhail Goncharov 2020-07-09 15:09:58 +02:00
parent 0e36e0cf7c
commit 7667fc8004
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
scripts/premerge_checks.py
View file

@ -37,7 +37,7 @@ def ninja_all_report(step: Step, _: Report):
print('Full log will be available in Artifacts "ninja-all.log"', flush=True) print('Full log will be available in Artifacts "ninja-all.log"', flush=True)
r = subprocess.run(f'ninja all | ' r = subprocess.run(f'ninja all | '
f'tee {artifacts_dir}/ninja-all.log | ' f'tee {artifacts_dir}/ninja-all.log | '
f'grep -vE "\\[.*] (Building|Linking|Linting|Copying|Generating|Creating)"', f'grep -vE "\\[.*] (Linking|Linting|Copying|Generating|Creating)"',
shell=True, cwd=build_dir) shell=True, cwd=build_dir)
logging.debug(f'ninja all: returned {r.returncode}, stderr: "{r.stderr}"') logging.debug(f'ninja all: returned {r.returncode}, stderr: "{r.stderr}"')
step.set_status_from_exit_code(r.returncode) step.set_status_from_exit_code(r.returncode)