The following changes since commit 0d0275c31f00b71b49eb80bbdca2cfe244cf80fb:

Add the component regex for the new loongarch target.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

Acked-by: Paolo Bonzini <pbonzini@redhat.com>

Message-id: 20220718142310.16013-2-peter.maydell@linaro.org

---

scripts/coverity-scan/COMPONENTS.md | 3 +++

1 file changed, 3 insertions(+)

diff --git a/scripts/coverity-scan/COMPONENTS.md b/scripts/coverity-scan/COMPONENTS.md

index XXXXXXX..XXXXXXX 100644

--- a/scripts/coverity-scan/COMPONENTS.md

+++ b/scripts/coverity-scan/COMPONENTS.md

@@ -XXX,XX +XXX,XX @@ testlibs

tests

~ (/qemu)?(/tests/.*)

+

+loongarch

+ ~ (/qemu)?((/include)?/hw/(loongarch/.*|.*/loongarch.*)|/target/loongarch/.*)

--

2.25.1

Update the regex for the slirp component now that it lives

solely inside /slirp/, and note that it should be ignored in

Coverity analysis (because it's a separate upstream project

now, and they run Coverity on it themselves).

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

Acked-by: Paolo Bonzini <pbonzini@redhat.com>

Message-id: 20220718142310.16013-3-peter.maydell@linaro.org

---

scripts/coverity-scan/COMPONENTS.md | 4 ++--

1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/coverity-scan/COMPONENTS.md b/scripts/coverity-scan/COMPONENTS.md

index XXXXXXX..XXXXXXX 100644

--- a/scripts/coverity-scan/COMPONENTS.md

+++ b/scripts/coverity-scan/COMPONENTS.md

@@ -XXX,XX +XXX,XX @@ qemu-ga

scsi

~ (/qemu)?(/scsi/.*|/hw/scsi/.*|/include/hw/scsi/.*)

-slirp

- ~ (/qemu)?(/.*slirp.*)

+slirp (component should be ignored in analysis)

+ ~ (/qemu)?(/slirp/.*)

tcg

~ (/qemu)?(/accel/tcg/.*|/replay/.*|/(.*/)?softmmu.*)

--

2.25.1

In commit 7d7dbf9dc15be6e1 we added a line to the configure script

which is not valid POSIX shell syntax, because it is missing a space

after a '!' character. shellcheck diagnoses this:

if !(GIT="$git" "$source_path/scripts/git-submodule.sh" "$git_submodules_action" "$git_submodules"); then

^-- SC1035: You are missing a required space after the !.

and the OpenBSD shell will not correctly handle this without the space.

Fixes: 7d7dbf9dc15be6e1 ("configure: replace --enable/disable-git-update with --with-git-submodules")

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

Reviewed-by: Thomas Huth <thuth@redhat.com>

Tested-by: Dr. David Alan Gilbert <dgilbert@redhat.com>

Message-id: 20220720152631.450903-2-peter.maydell@linaro.org

---

configure | 2 +-

1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configure b/configure

index XXXXXXX..XXXXXXX 100755

--- a/configure

+++ b/configure

@@ -XXX,XX +XXX,XX @@ else

cxx=

fi

-if !(GIT="$git" "$source_path/scripts/git-submodule.sh" "$git_submodules_action" "$git_submodules"); then

+if ! (GIT="$git" "$source_path/scripts/git-submodule.sh" "$git_submodules_action" "$git_submodules"); then

exit 1

fi

--

2.25.1

In shell script syntax, $var[something] is not special for variable

expansion: $var is expanded. However, as it can look as if it were

intended to be an array element access (the correct syntax for which

is ${var[something]}), shellcheck recommends using explicit braces

around ${var} to clarify the intended expansion.

This fixes the warning:

In ./configure line 2346:

if "$target_ld" -verbose 2>&1 | grep -q "^[[:space:]]*$emu[[:space:]]*$"; then

^-- SC1087: Use braces when expanding arrays, e.g. ${array[idx]} (or ${var}[.. to quiet).

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

Reviewed-by: Thomas Huth <thuth@redhat.com>

Message-id: 20220720152631.450903-3-peter.maydell@linaro.org

---

configure | 2 +-

1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configure b/configure

index XXXXXXX..XXXXXXX 100755

--- a/configure

+++ b/configure

@@ -XXX,XX +XXX,XX @@ if test -n "$target_cc" &&

# emulation. Linux and OpenBSD/amd64 use 'elf_i386'; FreeBSD uses the _fbsd

# variant; OpenBSD/i386 uses the _obsd variant; and Windows uses i386pe.

for emu in elf_i386 elf_i386_fbsd elf_i386_obsd i386pe; do

- if "$target_ld" -verbose 2>&1 | grep -q "^[[:space:]]*$emu[[:space:]]*$"; then

+ if "$target_ld" -verbose 2>&1 | grep -q "^[[:space:]]*${emu}[[:space:]]*$"; then

ld_i386_emulation="$emu"

break

fi

--

2.25.1

The variable string-replacement syntax ${var/old/new} is a bashism

(though it is also supported by some other shells), and for instance

does not work with the NetBSD /bin/sh, which complains:

../src/configure: 687: Syntax error: Bad substitution

Replace it with a more portable sed-based approach, similar to

what we already do in quote_sh().

Note that shellcheck also diagnoses this:

In ./configure line 687:

e=${e/'\'/'\\'}

^-----------^ SC2039: In POSIX sh, string replacement is undefined.

^-- SC1003: Want to escape a single quote? echo 'This is how it'\''s done'.

^-- SC1003: Want to escape a single quote? echo 'This is how it'\''s done'.

In ./configure line 688:

e=${e/\"/'\"'}

^----------^ SC2039: In POSIX sh, string replacement is undefined.

Fixes: 8154f5e64b0cf ("meson: Prefix each element of firmware path")

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

Tested-by: Thomas Huth <thuth@redhat.com>

Message-id: 20220720152631.450903-4-peter.maydell@linaro.org

---

configure | 7 ++++---

1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/configure b/configure

index XXXXXXX..XXXXXXX 100755

--- a/configure

+++ b/configure

@@ -XXX,XX +XXX,XX @@ meson_option_build_array() {

IFS=:

fi

for e in $1; do

- e=${e/'\'/'\\'}

- e=${e/\"/'\"'}

- printf '"""%s""",' "$e"

+ printf '"""'

+ # backslash escape any '\' and '"' characters

+ printf "%s" "$e" | sed -e 's/\([\"]\)/\\\1/g'

+ printf '""",'

done)

printf ']\n'

}

--

2.25.1

Reviewed-by: Thomas Huth <thuth@redhat.com>

Message-id: 20220720152631.450903-5-peter.maydell@linaro.org

Reviewed-by: Thomas Huth <thuth@redhat.com>

Message-id: 20220720152631.450903-6-peter.maydell@linaro.org