Attachment 343082 Details for Bug 186810 – FIX

[patch] FIX

186810.diff (text/plain), 1.54 KB, created by Basuke Suzuki on 2018-06-19 12:40:08 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Basuke Suzuki

Created: 2018-06-19 12:40:08 PDT

Size: 1.54 KB

patch

obsolete

>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index ad1ec70158d..f3e12e9b77d 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,15 @@
>+2018-06-19  Basuke Suzuki  <Basuke.Suzuki@sony.com>
>+
>+        [style] Fix --git-index option for check-webkit-style command
>+        https://bugs.webkit.org/show_bug.cgi?id=186810
>+
>+        When --git-index is specified, it should be compared with HEAD, not the origin/master.
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * Scripts/webkitpy/common/checkout/scm/git.py:
>+        (Git.create_patch):
>+
> 2018-06-19  Robin Morisset  <rmorisset@apple.com>
> 
>         [WSL] Improving the typing rules
>diff --git a/Tools/Scripts/webkitpy/common/checkout/scm/git.py b/Tools/Scripts/webkitpy/common/checkout/scm/git.py
>index 8d30ee41eab..f262f1073b6 100644
>--- a/Tools/Scripts/webkitpy/common/checkout/scm/git.py
>+++ b/Tools/Scripts/webkitpy/common/checkout/scm/git.py
>@@ -338,9 +338,11 @@ class Git(SCM, SVNRepository):
>         if self._filesystem.exists(order_file):
>             order = "-O%s" % order_file
> 
>-        command = [self.executable_name, 'diff', '--binary', '--no-color', "--no-ext-diff", "--full-index", "--no-renames", order, self.merge_base(git_commit)]
>+        command = [self.executable_name, 'diff', '--binary', '--no-color', "--no-ext-diff", "--full-index", "--no-renames", order]
>         if git_index:
>             command += ['--cached']
>+        else:
>+            command += [self.merge_base(git_commit)]
>         command += ["--"]
>         if changed_files:
>             command += changed_files

Actions: View | Formatted Diff | Diff

Attachments on bug 186810: 343082 | 343273 | 343287 | 343300 | 343360