diff options
| author | Richard Lowe <richlowe@richlowe.net> | 2012-10-05 18:45:33 -0400 |
|---|---|---|
| committer | Richard Lowe <richlowe@richlowe.net> | 2012-10-10 16:15:10 -0400 |
| commit | 3cb02613df33d0be6e1b0ed7c2ecbf33525431d6 (patch) | |
| tree | b7f8f7b80943e297a17142917cf6b17e916e7bac /usr/src/tools/scripts/webrev.sh | |
| parent | 23519ef0271738bf3585268555f1c683f7b12b33 (diff) | |
| download | illumos-joyent-3cb02613df33d0be6e1b0ed7c2ecbf33525431d6.tar.gz | |
3262 webrev only includes oldest cset comment when file changes multiple times
Reviewed by: Igor Kozhukhov <ikozhukhov@gmail.com>
Reviewed by: Garrett D'Amore <garrett@damore.org>
Approved by: Dan McDonald <danmcd@nexenta.com>
Diffstat (limited to 'usr/src/tools/scripts/webrev.sh')
| -rw-r--r-- | usr/src/tools/scripts/webrev.sh | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/usr/src/tools/scripts/webrev.sh b/usr/src/tools/scripts/webrev.sh index 2d5b31f162..c667d2a381 100644 --- a/usr/src/tools/scripts/webrev.sh +++ b/usr/src/tools/scripts/webrev.sh @@ -1806,11 +1806,11 @@ function git_wxfile chomp; if (/^R(\d+)\s+([^ ]+)\s+([^ ]+)/) { # rename if ($1 >= 75) { # Probably worth treating as a rename - $realfiles{$3} = $2 + $realfiles{$3} = $2; } else { $realfiles{$3} = $3; $realfiles{$2} = $2; - } + } } else { my $f = (split /\s+/, $_)[1]; $realfiles{$f} = $f; @@ -1826,7 +1826,8 @@ function git_wxfile my $fname = (split /\t/, $_)[1]; next if !defined($realfiles{$fname}); # No real change $state = 1; - $files{$fname} = $msg; + chomp $msg; + $files{$fname} .= $msg; } else { if ($state == 1) { $state = 0; @@ -1839,9 +1840,9 @@ function git_wxfile for (sort keys %files) { if ($realfiles{$_} ne $_) { - print "$_ $realfiles{$_}\n$files{$_}\n"; + print "$_ $realfiles{$_}\n$files{$_}\n\n"; } else { - print "$_\n$files{$_}\n" + print "$_\n$files{$_}\n\n" } }' ${parent} > $TMPFLIST |