Bug 48495 – webkit-patch land -g HEAD started failing, leaving uncommitted ChangeLog's

• Status: NEW
• Component: Tools / Tests

[Yuzo Fujishima]

At r70736, webkit-patch land -g HEAD started failing for me.
(The bug for which I tried to land the patch is a security bug. Can this be a factor?)

$ git status
# On branch *********
nothing to commit (working directory clean)
$ webkit-patch land -g HEAD
Parsing ChangeLog: <WebKitSrc>/WebKit/LayoutTests/ChangeLog
Parsing ChangeLog: <WebKitSrc>/WebKit/WebCore/ChangeLog

WARNING: Builders ["Leopard Intel Release (Tests)", "GTK Linux 32-bit Release", "GTK Linux 32-bit Debug"] are red, please watch your commit carefully.
See http://build.webkit.org/console?category=core

Fetching: https://bugs.webkit.org/show_bug.cgi?id=*****&ctype=xml
Logging in as [e-mail]...
Fetching: https://bugs.webkit.org/show_bug.cgi?id=*****&ctype=xml
Guessing "Eric Seidel" as reviewer from attachment ***** on bug *****.
Parsing ChangeLog: <WebKitSrc>/WebKit/LayoutTests/ChangeLog
Parsing ChangeLog: <WebKitSrc>/WebKit/WebCore/ChangeLog
Working copy is modified. Cannot commit individual git_commits.
Working copy is modified. Cannot commit individual git_commits.
$ git status
# On branch *********
# Changed but not updated:
#   (use "git add <file>..." to update what will be committed)
#   (use "git checkout -- <file>..." to discard changes in working directory)
#
#	modified:   LayoutTests/ChangeLog
#	modified:   WebCore/ChangeLog
#
no changes added to commit (use "git add" and/or "git commit -a")
$ git diff
diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index 93707d9..322ba96 100644
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,6 +1,6 @@
 2010-10-26  Yuzo Fujishima  <[e-mail]>
 
-        Reviewed by NOBODY (OOPS!).
+        Reviewed by Eric Seidel.
 
         Fix for Bug ***** - ********************************************************
         https://bugs.webkit.org/show_bug.cgi?id=*****
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 08fad35..d278f56 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,6 +1,6 @@
 2010-10-26  Yuzo Fujishima  <[e-mail]>
 
-        Reviewed by NOBODY (OOPS!).
+        Reviewed by Eric Seidel.
 
         Fix for Bug ***** - ********************************************************
         https://bugs.webkit.org/show_bug.cgi?id=*****

[Ojan Vafai]

Unfortunately, webkit-patch land has always had this issue. webkit-patch land -g only works if the changelog doesn't need to be modified.

It occurs to me that this could be fixed by reordering some code. Right now, the way it works for committing a specific commit:
1. Update the changelogs based off the bug
2. Create a dummy branch
3. Cherry pick the given commit to the dummy branch
4. Commit the dummy branch
5. Switch back to the original branch

In theory, we could do step 1 after step 3 and everything would work right. As the code is currently structured, that might be a bit tricky, but it's possible.

I don't have time to fix this, but I'd be happy to review a patch that did.

[Thiago Marcos P. Santos]

I did this from the master branch:

$ webkit-patch land -g mybranch

It committed mybranch but used as SVN message the ChangeLog of the master branch's HEAD. :|

[Adam Barth]

Perhaps we should remove support for -g from land?