On 5/26/20 9:59 AM, David Howells wrote: > The update_scripts.sh script doesn't correctly strip the suffix from the > end of the pathname, but will substitute for it anywhere in the path. The > pattern really ought to have been /[.]$TARGET$//. > > Fix this by using a special shell expansion to do it instead of invoking a > sed pipeline. > > Signed-off-by: David Howells <[email protected]> > cc: Jeremy Cline <[email protected]> > --- > update_scripts.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/update_scripts.sh b/update_scripts.sh > index bbfd6f11f..5c3dbaeb9 100755 > --- a/update_scripts.sh > +++ b/update_scripts.sh > @@ -7,6 +7,6 @@ fi > TARGET=$1 > > for i in $RPM_SOURCE_DIR/*.$TARGET; do > - NEW=`echo $i | sed s/.$TARGET//` > + NEW=${i%.$TARGET} > cp $i $NEW > done
Acked-by: Prarit Bhargava <[email protected]> P. _______________________________________________ kernel mailing list -- [email protected] To unsubscribe send an email to [email protected] Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/[email protected]
