On Tue, May 26, 2020 at 8:59 AM David Howells <[email protected]> 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. >
Acked-by: Justin M. Forbes <[email protected]> > 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 > _______________________________________________ > 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] _______________________________________________ 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]
