encounter
/
dawn-cmake
mirror of https://github.com/encounter/dawn-cmake.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix platform.system check in clang-format presubmit 

I did this incorrectly in
https://dawn-review.googlesource.com/c/dawn/+/24640
because I forgot to actually look up what the platform.system() return
values were.

Bug: none
Change-Id: Ib81060d14339cf8cf52764ea779e4448ac92a148
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/24686
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
This commit is contained in:
Kai Ninomiya 2020-07-12 16:05:48 +00:00 committed by Commit Bot service account
parent 51bbbefa02
commit eafdef3818
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
PRESUBMIT.py
View File

@ -19,7 +19,7 @@ import subprocess
def _DoClangFormat(input_api, output_api): def _DoClangFormat(input_api, output_api):
if platform.system() == 'Linux' and platform.architecture()[0] == '64bit': if platform.system() == 'Linux' and platform.architecture()[0] == '64bit':
clang_format_path = 'buildtools/linux64/clang-format' clang_format_path = 'buildtools/linux64/clang-format'
elif platform.system() == 'Mac': elif platform.system() == 'Darwin':
clang_format_path = 'buildtools/mac/clang-format' clang_format_path = 'buildtools/mac/clang-format'
elif platform.system() == 'Windows': elif platform.system() == 'Windows':
clang_format_path = 'buildtools/win/clang-format.exe' clang_format_path = 'buildtools/win/clang-format.exe'