Messages in this thread | ![/](/images/icornerl.gif) | | Date | Wed, 17 Apr 2019 16:35:33 -0400 | From | Masayoshi Mizuma <> | Subject | Re: [PATCH] ktest: Add support for meta characters in GRUB_MENU |
| |
On Wed, Apr 17, 2019 at 04:17:39PM -0400, Steven Rostedt wrote: > On Wed, 17 Apr 2019 16:11:14 -0400 > Masayoshi Mizuma <msys.mizuma@gmail.com> wrote: > > > From: Masayoshi Mizuma <m.mizuma@jp.fujitsu.com> > > > > ktest fails if meta characters are in GRUB_MENU, for example > > GRUB_MENU = 'Fedora (test)' > > > > Thanks for the patch! One little nit below though. > > > The failure happens because the meta characters are not escaped, > > so the menu doesn't match in any entries in GRUB_FILE. > > > > Use quotameta() to escape the meta characters. > > > > Signed-off-by: Masayoshi Mizuma <m.mizuma@jp.fujitsu.com> > > --- > > tools/testing/ktest/ktest.pl | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/tools/testing/ktest/ktest.pl b/tools/testing/ktest/ktest.pl > > index 87af8a68ab25..ea07d43856b8 100755 > > --- a/tools/testing/ktest/ktest.pl > > +++ b/tools/testing/ktest/ktest.pl > > @@ -1866,9 +1866,10 @@ sub get_grub2_index { > > or dodie "unable to get $grub_file"; > > > > my $found = 0; > > + my $_grub_menu = quotemeta($grub_menu); > > I'd like to avoid names that start with underscore. Could you call this: > > $grub_menu_qt > > or something similar, to be a bit more descriptive of what the variable > is.
Thank you for your review! I'll fix it and post the v2.
Thanks! Masa
| ![\](/images/icornerr.gif) |