|
From: | Emanuele Giuseppe Esposito |
Subject: | Re: [PATCH v8 08/16] qemu-iotests: add gdbserver option to script tests too |
Date: | Mon, 9 Aug 2021 10:27:12 +0200 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1 |
diff --git a/tests/qemu-iotests/common.qemu b/tests/qemu-iotests/common.qemuindex 0fc52d20d7..cbca757b49 100644 --- a/tests/qemu-iotests/common.qemu +++ b/tests/qemu-iotests/common.qemu @@ -85,7 +85,12 @@ _timed_wait_for() timeout=yes QEMU_STATUS[$h]=0 - while IFS= read -t ${QEMU_COMM_TIMEOUT} resp <&${QEMU_OUT[$h]} + read_timeout="-t ${QEMU_COMM_TIMEOUT}" + if [ ! -z ${GDB_OPTIONS} ]; thenShouldn’t we quote "${GDB_OPTIONS}" so that `test` won’t interpret it as its own parameters (if something in there starts with `--`, which I don’t think is the intended usage for $GDB_OPTIONS, but, well...)?(Also, `! -z` is the same as `-n`, but I suppose choosing between the two can be a matter of style.)
+ + GDB="" + if [ ! -z ${GDB_OPTIONS} ]; thenHere, too. (Sorry for not noticing in v3 already...)
Sorry, I forgot to reply to this email. I agree, I will put quotes and change `! -z` in `-n`.
I will send v9 because after all this time this serie has some minor conflicts with current QEMU version.
Hopefully this will be the last one :) Thank you, Emanuele
[Prev in Thread] | Current Thread | [Next in Thread] |