summaryrefslogtreecommitdiffstats
path: root/scripts
diff options
context:
space:
mode:
authorChen Qi <Qi.Chen@windriver.com>2019-04-12 09:40:06 +0800
committerRichard Purdie <richard.purdie@linuxfoundation.org>2019-04-12 13:37:11 +0100
commitc574aaf30c82ad397c0a6567b3cb52e7fb5d5829 (patch)
tree2e0e0310aa8f0781cde99fd976e04ce060613c55 /scripts
parent1bc122a044de280d17525c8e5ee66e81775a7d8b (diff)
downloadopenembedded-core-contrib-c574aaf30c82ad397c0a6567b3cb52e7fb5d5829.tar.gz
openembedded-core-contrib-c574aaf30c82ad397c0a6567b3cb52e7fb5d5829.tar.bz2
openembedded-core-contrib-c574aaf30c82ad397c0a6567b3cb52e7fb5d5829.zip
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>
Diffstat (limited to 'scripts')
-rwxr-xr-xscripts/runqemu6
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():
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()
@@ -1351,7 +1352,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())