diff options
author | Chen Qi <Qi.Chen@windriver.com> | 2019-04-12 09:40:06 +0800 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2019-04-12 13:40:21 +0100 |
commit | 047aab4f458951855c38ab6e7f923d14aba62555 (patch) | |
tree | 463433b9338df0bcce35440f88c47dba5ee5cbf4 /scripts | |
parent | ec8a40e231ce67bb5502b309afeac2a4fda34fd6 (diff) | |
download | poky-047aab4f458951855c38ab6e7f923d14aba62555.tar.gz |
runqemu: do not check return code of tput
The subprocess.run was replaced by subprocess.check_call because
of compatibility support down to python 3.4. But we really don't
care about whether that command succeeds. Some user reports that
in some tmux environment, this command fails and gives some
unpleasant traceback output. So we use 'call' instead of 'check_call'
to avoid such problem.
(From OE-Core rev: c574aaf30c82ad397c0a6567b3cb52e7fb5d5829)
Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts')
-rwxr-xr-x | scripts/runqemu | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/scripts/runqemu b/scripts/runqemu index c0e569c44c..a4fc606e50 100755 --- a/scripts/runqemu +++ b/scripts/runqemu | |||
@@ -1329,7 +1329,8 @@ def main(): | |||
1329 | logger.info("SIGTERM received") | 1329 | logger.info("SIGTERM received") |
1330 | os.kill(config.qemupid, signal.SIGTERM) | 1330 | os.kill(config.qemupid, signal.SIGTERM) |
1331 | config.cleanup() | 1331 | config.cleanup() |
1332 | subprocess.check_call(["tput", "smam"]) | 1332 | # Deliberately ignore the return code of 'tput smam'. |
1333 | subprocess.call(["tput", "smam"]) | ||
1333 | signal.signal(signal.SIGTERM, sigterm_handler) | 1334 | signal.signal(signal.SIGTERM, sigterm_handler) |
1334 | 1335 | ||
1335 | config.check_args() | 1336 | config.check_args() |
@@ -1351,7 +1352,8 @@ def main(): | |||
1351 | return 1 | 1352 | return 1 |
1352 | finally: | 1353 | finally: |
1353 | config.cleanup() | 1354 | config.cleanup() |
1354 | subprocess.check_call(["tput", "smam"]) | 1355 | # Deliberately ignore the return code of 'tput smam'. |
1356 | subprocess.call(["tput", "smam"]) | ||
1355 | 1357 | ||
1356 | if __name__ == "__main__": | 1358 | if __name__ == "__main__": |
1357 | sys.exit(main()) | 1359 | sys.exit(main()) |