2009-06-26 Eric Seidel <eric@webkit.org>
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Jun 2009 10:07:57 +0000 (10:07 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Jun 2009 10:07:57 +0000 (10:07 +0000)
        Reviewed by Tor Arne Vestbø.

        Remove non-sense --update option to land-patches
        and make land-patches update before every patch application.
        This makes it slightly less likely that multi-patch landings will fail.

        Also updated git diff command to include staged modifications.

        * Scripts/bugzilla-tool:
        * Scripts/modules/scm.py:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@45258 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKitTools/ChangeLog
WebKitTools/Scripts/bugzilla-tool
WebKitTools/Scripts/modules/scm.py

index 1326326..5300b37 100644 (file)
@@ -1,3 +1,16 @@
+2009-06-26  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by Tor Arne Vestbø.
+
+        Remove non-sense --update option to land-patches
+        and make land-patches update before every patch application.
+        This makes it slightly less likely that multi-patch landings will fail.
+
+        Also updated git diff command to include staged modifications.
+
+        * Scripts/bugzilla-tool:
+        * Scripts/modules/scm.py:
+
 2009-06-25  Eric Seidel  <eric@webkit.org>
 
         Reviewed by Jan Alonzo.
index 4c4357d..443307e 100755 (executable)
@@ -225,7 +225,6 @@ class LandAndUpdateBug(Command):
 class LandPatchesFromBugs(Command):
     def __init__(self):
         options = [
-            make_option("--no-update", action="store_false", dest="update", default=True, help="Don't update the working directory before applying patches"),
             make_option("--force-clean", action="store_true", dest="force_clean", default=False, help="Clean working directory before applying patches (removes local changes and commits)"),
             make_option("--no-clean", action="store_false", dest="clean", default=True, help="Don't check if the working directory is clean before applying patches"),
             make_option("--no-build", action="store_false", dest="build", default=True, help="Commit without building first, implies --no-test."),
@@ -259,8 +258,6 @@ class LandPatchesFromBugs(Command):
         scm.ensure_no_local_commits(options.force_clean)
         if options.clean:
             scm.ensure_clean_working_directory(options.force_clean)
-        if options.update:
-            scm.update_webkit()
 
     @classmethod
     def build_and_commit(cls, scm, options):
@@ -277,6 +274,7 @@ class LandPatchesFromBugs(Command):
         try:
             comment_text = ""
             for patch in patches:
+                tool.scm().update_webkit() # Update before every patch in case the tree has changed
                 tool.scm().apply_patch(patch)
                 comment_text = cls.build_and_commit(tool.scm(), options)
 
index 80d5220..9177f76 100644 (file)
@@ -257,7 +257,7 @@ class Git(SCM):
         return 'git status'
 
     def changed_files(self):
-        status_command = 'git diff -r --name-status -C -M'
+        status_command = 'git diff -r --name-status -C -M HEAD'
         status_regexp = '^(?P<status>[ADM])\t(?P<filename>.+)$'
         return self.run_status_and_extract_filenames(status_command, status_regexp)