Messages in this thread Patch in this message | | | Date | Wed, 04 Jan 2012 22:48:13 -0500 | From | Steven Rostedt <> | Subject | [PATCH 18/20] ktest: Fix compare script to test if options are not documented |
| |
From: Steven Rostedt <srostedt@redhat.com>
The compare script compare-ktest-sample.pl checks for options that are defined in ktest.pl and not documented in samples.conf, as well as samples in samples.conf that are not used in ktest.pl.
With the switch to the hash format to initialize the ktest variables the compare script needs to be updated to handle the change.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org> --- tools/testing/ktest/compare-ktest-sample.pl | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/tools/testing/ktest/compare-ktest-sample.pl b/tools/testing/ktest/compare-ktest-sample.pl index 9a571e7..a373a5b 100755 --- a/tools/testing/ktest/compare-ktest-sample.pl +++ b/tools/testing/ktest/compare-ktest-sample.pl @@ -2,7 +2,9 @@ open (IN,"ktest.pl"); while (<IN>) { + # hashes are now used if (/\$opt\{"?([A-Z].*?)(\[.*\])?"?\}/ || + /^\s*"?([A-Z].*?)"?\s*=>\s*/ || /set_test_option\("(.*?)"/) { $opt{$1} = 1; } @@ -11,7 +13,7 @@ close IN; open (IN, "sample.conf"); while (<IN>) { - if (/^\s*#?\s*(\S+)\s*=/) { + if (/^\s*#?\s*([A-Z]\S*)\s*=/) { $samp{$1} = 1; } } -- 1.7.7.3
[unhandled content-type:application/pgp-signature] | |