Commit 79ef8f84 authored by Kent Tamura's avatar Kent Tamura Committed by Commit Bot

Update _CheckBuildConfigMacrosWithoutInclude() so that it excludes OS_CHROMEOS.

Bug: 742692
Change-Id: I3a64ed5870d6a1769bfccab353ad261ce24de17d
Reviewed-on: https://chromium-review.googlesource.com/571521Reviewed-by: default avatarDirk Pranke <dpranke@chromium.org>
Reviewed-by: default avatarNico Weber <thakis@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487294}
parent e4d48f77
......@@ -2134,8 +2134,9 @@ def _CheckPatchFiles(input_api, output_api):
def _CheckBuildConfigMacrosWithoutInclude(input_api, output_api):
macro_re = input_api.re.compile(
r'^\s*#(el)?if.*\bdefined\(((OS_|COMPILER_|ARCH_CPU_|WCHAR_T_IS_)[^)]*)')
# Excludes OS_CHROMEOS, which is not defined in build_config.h.
macro_re = input_api.re.compile(r'^\s*#(el)?if.*\bdefined\(((OS_(?!CHROMEOS)|'
'COMPILER_|ARCH_CPU_|WCHAR_T_IS_)[^)]*)')
include_re = input_api.re.compile(
r'^#include\s+"build/build_config.h"', input_api.re.MULTILINE)
extension_re = input_api.re.compile(r'\.[a-z]+$')
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment