1
0
Fork 0

ci: Avoid deadlock while reading uncrustify output

The docu for `Popen.wait()` says:
> This will deadlock when using stdout=PIPE and/or stderr=PIPE and
> the child process generates enough output to a pipe such that it
> blocks waiting for the OS pipe buffer to accept more data.

Fixes 3caa5fea3c

Part-of: <https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/1971>
This commit is contained in:
Robert Mader 2021-08-23 17:36:30 +02:00
parent 3caa5fea3c
commit 6960b9b45a

View file

@ -76,11 +76,11 @@ def reformat_chunks(chunks, rewrite):
# uncrustify chunk
proc = subprocess.Popen(["uncrustify", "-c", uncrustify_cfg, "-f", tmp.name], stdout=subprocess.PIPE)
reindented = proc.stdout.readlines()
proc.wait()
if proc.returncode != 0:
continue
reindented = proc.stdout.readlines()
tmp.close()
# Remove INDENT-ON/OFF comments