[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] configure: Remove spurious [] from tr
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [PATCH] configure: Remove spurious [] from tr |
Date: |
Thu, 12 Aug 2021 18:54:38 +0100 |
User-agent: |
Mutt/2.0.7 (2021-05-04) |
* Paolo Bonzini (pbonzini@redhat.com) wrote:
> On 12/08/21 13:09, Dr. David Alan Gilbert (git) wrote:
> > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
> >
> > ShellCheck points out that tr '[a-z]' actually replaces the []'s
> > and only the a-z is needed.
> >
> > Remove the spurious [] - although in this use it will make no
> > difference.
> >
> > Fixes: bb55b712e8dc4d4eb515144d5c26798fea178cba
> > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> > ---
> > configure | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/configure b/configure
> > index 9a79a004d7..5bb8c2a59d 100755
> > --- a/configure
> > +++ b/configure
> > @@ -4549,7 +4549,7 @@ if test "$gprof" = "yes" ; then
> > fi
> > echo "CONFIG_AUDIO_DRIVERS=$audio_drv_list" >> $config_host_mak
> > for drv in $audio_drv_list; do
> > - def=CONFIG_AUDIO_$(echo $drv | LC_ALL=C tr '[a-z]' '[A-Z]')
> > + def=CONFIG_AUDIO_$(echo $drv | LC_ALL=C tr 'a-z' 'A-Z')
> > echo "$def=y" >> $config_host_mak
> > done
> > if test "$alsa" = "yes" ; then
> >
>
> Do we want this in 6.1? For the next release I'm moving all audio stuff to
> meson anyway. :)
Bah! Not bothered; I just tripped over ShellCheck and that
looked like a trivial one.
Dave
> Paolo
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK