[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 5/7] configure: Remove use of backtick `...` syntax
From: |
Peter Maydell |
Subject: |
Re: [PATCH 5/7] configure: Remove use of backtick `...` syntax |
Date: |
Thu, 22 Sep 2022 15:14:31 +0100 |
On Tue, 30 Aug 2022 at 13:13, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>
> On 25/8/22 17:07, Peter Maydell wrote:
> > There's only one place in configure where we use `...` to execute a
> > command and capture the result. Switch to $() to match the rest of
> > the script. This silences a shellcheck warning.
> >
> > Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> > ---
> > configure | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/configure b/configure
> > index d5b6546ae81..5c1992d5bce 100755
> > --- a/configure
> > +++ b/configure
> > @@ -2317,7 +2317,7 @@ LINKS="$LINKS python"
> > LINKS="$LINKS contrib/plugins/Makefile "
> > for f in $LINKS ; do
> > if [ -e "$source_path/$f" ]; then
> > - mkdir -p `dirname ./$f`
> > + mkdir -p "$(dirname ./"$f")"
>
> Nitpicking, easier to read as "$(dirname ./${f})"
That would be missing the quoting on the inner level, I think.
-- PMM
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH 5/7] configure: Remove use of backtick `...` syntax,
Peter Maydell <=