aboutsummaryrefslogtreecommitdiffstats
path: root/tools
diff options
context:
space:
mode:
authorSteven Rostedt (Red Hat) <rostedt@goodmis.org>2013-01-30 12:28:15 -0500
committerSteven Rostedt <rostedt@goodmis.org>2013-01-30 12:28:15 -0500
commit35275685bf6123529e67c1dc91b8c05e479124e8 (patch)
tree06622dcbb6a7d47a7b6fd679cef45bf8a31c6f74 /tools
parent319ab14f051aeb04d6da20d82b389ad875083041 (diff)
ktest: Strip off '\n' when reading which files were modified
The patchcheck test looks at what files are modified for each patch it checks and makes sure that those files do not produce any warnings. Unfortunately, when it read the diffstat, the newlines were added on the files and this made compares miss warnings, and commits that should not have passed, ktest let pass. Fix this by using the perl command "chomp" that strips off whitespace at the end of lines. Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'tools')
-rwxr-xr-xtools/testing/ktest/ktest.pl4
1 files changed, 4 insertions, 0 deletions
diff --git a/tools/testing/ktest/ktest.pl b/tools/testing/ktest/ktest.pl
index d6690df5e7f1..cc9925a68f2a 100755
--- a/tools/testing/ktest/ktest.pl
+++ b/tools/testing/ktest/ktest.pl
@@ -1925,6 +1925,10 @@ sub check_buildlog {
1925 1925
1926 my @files = `git show $patch | diffstat -l`; 1926 my @files = `git show $patch | diffstat -l`;
1927 1927
1928 foreach my $file (@files) {
1929 chomp $file;
1930 }
1931
1928 open(IN, "git show $patch |") or 1932 open(IN, "git show $patch |") or
1929 dodie "failed to show $patch"; 1933 dodie "failed to show $patch";
1930 while (<IN>) { 1934 while (<IN>) {