2010-04-01 Ojan Vafai <ojan@chromium.org>
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 1 Apr 2010 23:49:10 +0000 (23:49 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 1 Apr 2010 23:49:10 +0000 (23:49 +0000)
commit808a1cfa6459191eae451e3c50bf6bab071040ca
tree9696a575cf765f84a689b9ceca679c6d02553ce2
parentf554d269cf873c7a931b2d7c8c732de4970b02d2
2010-04-01  Ojan Vafai  <ojan@chromium.org>

        Reviewed by Eric Seidel.

        webkit-patch can incorrectly think the working directory is modified
        https://bugs.webkit.org/show_bug.cgi?id=36985

        If a file's modification time is modified, but the contents are not,
        then diff-index will think the file has been modified unless you do
        some crazy update-index call. Instead, call diff --name-only, which
        has the index update builtin.

        Tried to write a test, but could not reproduce this in a unittest.
        To test manually:
            touch file-in-repo
            git diff-index HEAD
            git diff HEAD --name-only

        The diff-index call incorrectly shows file-in-repo as modified.

        * Scripts/webkitpy/common/checkout/scm.py:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@56950 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebKitTools/ChangeLog
WebKitTools/Scripts/webkitpy/common/checkout/scm.py