Skip to content

Commit

Permalink
Fix Bug 14403: prepare-ChangeLog --git-commit doesn't support --diff
Browse files Browse the repository at this point in the history
http://bugs.webkit.org/show_bug.cgi?id=14403

Reviewed by Mark Rowe.

* Scripts/prepare-ChangeLog: Don't spew to stdout, use stderr instead.
(sub createPatchCommand): Generate a correct diff in the $gitCommit
case.


Canonical link: https://commits.webkit.org/18561@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@23778 268f45cc-cd09-0410-ab3c-d52691b4dbfc
  • Loading branch information
aroben committed Jun 26, 2007
1 parent d2f29a5 commit 0618d4b
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 1 deletion.
11 changes: 11 additions & 0 deletions WebKitTools/ChangeLog
@@ -1,3 +1,14 @@
2007-06-25 Adam Roben <aroben@apple.com>

Fix Bug 14403: prepare-ChangeLog --git-commit doesn't support --diff
http://bugs.webkit.org/show_bug.cgi?id=14403

Reviewed by Mark Rowe.

* Scripts/prepare-ChangeLog: Don't spew to stdout, use stderr instead.
(sub createPatchCommand): Generate a correct diff in the $gitCommit
case.

2007-06-25 Adam Roben <aroben@apple.com>

Land pdevenv and supporting scripts/programs
Expand Down
3 changes: 2 additions & 1 deletion WebKitTools/Scripts/prepare-ChangeLog
Expand Up @@ -303,7 +303,7 @@ foreach my $prefix (sort keys %files) {
}

if (!$reviewer) {
print "WARNING!!! Change was not reviewed!\n";
print STDERR "WARNING!!! Change was not reviewed!\n";
$reviewer = "NOBODY (OO" . "PS!)";
}

Expand Down Expand Up @@ -961,6 +961,7 @@ sub createPatchCommand($)
my ($changedFilesString) = @_;

return "'$FindBin::Bin/svn-create-patch' $changedFilesString" if isSVN();
return "$GIT diff -C -M \"$gitCommit^\" \"$gitCommit\"" if $gitCommit;
return "$GIT diff -C -M HEAD -- $changedFilesString" if isGit();
}

Expand Down

0 comments on commit 0618d4b

Please sign in to comment.