aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChen Qi <Qi.Chen@windriver.com>2019-04-12 09:40:06 +0800
committerArmin Kuster <akuster808@gmail.com>2019-04-22 18:25:02 -0600
commitced3c75fa75d9b9373d695d9204b197b98ea3bd9 (patch)
treec2b2836d490edd1b3993beb05baaee985dbe6432
parent5e900f2c9319843c8905713dd3dd12a1ad435976 (diff)
downloadopenembedded-core-contrib-ced3c75fa75d9b9373d695d9204b197b98ea3bd9.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. Signed-off-by: Chen Qi <Qi.Chen@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> Signed-off-by: Armin Kuster <akuster808@gmail.com>
-rwxr-xr-xscripts/runqemu6
1 files changed, 4 insertions, 2 deletions
diff --git a/scripts/runqemu b/scripts/runqemu
index 1c96b29a40..f83e05728b 100755
--- a/scripts/runqemu
+++ b/scripts/runqemu
@@ -1333,7 +1333,8 @@ def main():
logger.info("SIGTERM received")
os.kill(config.qemupid, signal.SIGTERM)
config.cleanup()
- subprocess.check_call(["tput", "smam"])
+ # Deliberately ignore the return code of 'tput smam'.
+ subprocess.call(["tput", "smam"])
signal.signal(signal.SIGTERM, sigterm_handler)
config.check_args()
@@ -1355,7 +1356,8 @@ def main():
return 1
finally:
config.cleanup()
- subprocess.check_call(["tput", "smam"])
+ # Deliberately ignore the return code of 'tput smam'.
+ subprocess.call(["tput", "smam"])
if __name__ == "__main__":
sys.exit(main())