| # Copyright 2022 The Dawn & Tint Authors |
| # |
| # Redistribution and use in source and binary forms, with or without |
| # modification, are permitted provided that the following conditions are met: |
| # |
| # 1. Redistributions of source code must retain the above copyright notice, this |
| # list of conditions and the following disclaimer. |
| # |
| # 2. Redistributions in binary form must reproduce the above copyright notice, |
| # this list of conditions and the following disclaimer in the documentation |
| # and/or other materials provided with the distribution. |
| # |
| # 3. Neither the name of the copyright holder nor the names of its |
| # contributors may be used to endorse or promote products derived from |
| # this software without specific prior written permission. |
| # |
| # THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" |
| # AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE |
| # IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE |
| # DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE |
| # FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL |
| # DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR |
| # SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER |
| # CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, |
| # OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE |
| # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
| |
| import re |
| import sys |
| |
| PRESUBMIT_VERSION = '2.0.0' |
| |
| NONINCLUSIVE_LANGUAGE_REGEXES = [ |
| re.compile(reg) for reg in [ |
| r"(?i)black[-_]?list", |
| r"(?i)white[-_]?list", |
| r"(?i)gr[ea]y[-_]?list", |
| r"(?i)(first class citizen)", |
| r"(?i)black[-_]?hat", |
| r"(?i)white[-_]?hat", |
| r"(?i)gr[ea]y[-_]?hat", |
| r"(?i)master", |
| r"(?i)slave", |
| r"(?i)\bhim\b", |
| r"(?i)\bhis\b", |
| r"(?i)\bshe\b", |
| r"(?i)\bher\b", |
| r"(?i)\bguys\b", |
| r"(?i)\bhers\b", |
| r"(?i)\bman\b", |
| r"(?i)\bwoman\b", |
| r"(?i)\she\s", |
| r"(?i)\she$", |
| r"(?i)^he\s", |
| r"(?i)^he$", |
| r"(?i)\she['|\u2019]d\s", |
| r"(?i)\she['|\u2019]d$", |
| r"(?i)^he['|\u2019]d\s", |
| r"(?i)^he['|\u2019]d$", |
| r"(?i)\she['|\u2019]s\s", |
| r"(?i)\she['|\u2019]s$", |
| r"(?i)^he['|\u2019]s\s", |
| r"(?i)^he['|\u2019]s$", |
| r"(?i)\she['|\u2019]ll\s", |
| r"(?i)\she['|\u2019]ll$", |
| r"(?i)^he['|\u2019]ll\s", |
| r"(?i)^he['|\u2019]ll$", |
| r"(?i)grandfather", |
| r"(?i)\bmitm\b", |
| r"(?i)\bcrazy\b", |
| r"(?i)\binsane\b", |
| r"(?i)\bblind\sto\b", |
| r"(?i)\bflying\sblind\b", |
| r"(?i)\bblind\seye\b", |
| r"(?i)\bcripple\b", |
| r"(?i)\bcrippled\b", |
| r"(?i)\bdumb\b", |
| r"(?i)\bdummy\b", |
| r"(?i)\bparanoid\b", |
| r"(?i)\bsane\b", |
| r"(?i)\bsanity\b", |
| r"(?i)red[-_]?line", |
| ] |
| ] |
| |
| LINT_FILTERS = [] |
| |
| def _NonInclusiveFileFilter(file): |
| """Filters files that are exempt from the non-inclusive language check.""" |
| filter_list = [ |
| "Doxyfile", # References to main pages |
| "PRESUBMIT.py", # Non-inclusive language check data |
| "PRESUBMIT.py.tint", # Non-inclusive language check data |
| "docs/dawn/debug_markers.md", # External URL |
| "docs/dawn/infra.md", # Infra settings |
| "docs/tint/spirv-input-output-variables.md", # External URL |
| "infra/config/global/generated/cr-buildbucket.cfg", # Infra settings |
| "infra/config/global/main.star", # Infra settings |
| "infra/kokoro/windows/build.bat", # External URL |
| "src/dawn/common/GPUInfo.cpp", # External URL |
| "src/dawn/common/ThreadLocal.cpp", # External URL |
| "src/dawn/native/metal/BackendMTL.mm", # OSX Constant |
| "src/dawn/native/metal/PhysicalDeviceMTL.mm", # OSX deprecated API |
| "src/dawn/native/vulkan/SamplerVk.cpp", # External URL |
| "src/dawn/native/vulkan/TextureVk.cpp", # External URL |
| "src/tools/src/cmd/run-cts/main.go", # Terminal type name |
| "src/dawn/samples/ComputeBoids.cpp", # External URL |
| "src/dawn/tests/end2end/DepthBiasTests.cpp", # External URL |
| "src/tint/transform/canonicalize_entry_point_io.cc", # External URL |
| "test/tint/samples/compute_boids.wgsl", # External URL |
| "third_party/gn/dxc/BUILD.gn", # Third party file |
| "third_party/khronos/EGL-Registry/api/KHR/khrplatform.h", # Third party file |
| "tools/roll-all", # Branch name |
| "tools/src/container/key.go", # External URL |
| "go.sum", # External URL |
| ] |
| return file.LocalPath().replace('\\', '/') not in filter_list |
| |
| |
| def CheckNonInclusiveLanguage(input_api, output_api): |
| """Checks the files for non-inclusive language.""" |
| matches = [] |
| for f in input_api.AffectedFiles(include_deletes=False, |
| file_filter=_NonInclusiveFileFilter): |
| for line_num, line in enumerate(f.NewContents(), start=1): |
| for reg in NONINCLUSIVE_LANGUAGE_REGEXES: |
| if match := reg.search(line): |
| matches.append( |
| f"{f.LocalPath()} ({line_num}): found non-inclusive language: {match.group(0)}" |
| ) |
| |
| if matches: |
| return [ |
| output_api.PresubmitPromptWarning("Non-inclusive language found:", |
| items=matches) |
| ] |
| |
| return [] |
| |
| |
| def _CalculateEnumeratedEntriesAndTypes(lines): |
| """Returns a dictionary of enumerated entries, and a list of all the 'types' encountered. |
| |
| The implemented parsing is unsophisticated, and assumes a readable/well-formed .proto file. |
| Things like unmatched '{}'s will cause a crash. Missing ';'s or writing something like `} message Foo {` will also |
| cause misbehaviour. |
| Constructs like this are normally bad style, so if really needed, adding support for them is left as an exercise for |
| the reader. |
| """ |
| push_re = re.compile(r'(\w+) {(.*)') |
| value_re = re.compile(r'(\w+) = (\d+);(.*)') |
| pop_re = re.compile(r'}(.*)') |
| |
| prefix_stack = [] |
| prefix_str = "" |
| enumerated_entries = {} |
| types = [] |
| for l in lines: |
| l = l.strip().rstrip() |
| l = l.split("//", 1)[0] |
| while l: |
| if match := re.search(push_re, l): |
| prefix_stack.append(match.group(1)) |
| prefix_str = '.'.join(prefix_stack) |
| types.append(prefix_str) |
| l = match.group(2) |
| continue |
| if match := re.search(value_re, l): |
| enumerated_entries[ |
| f"{prefix_str}.{match.group(1)}"] = match.group(2) |
| l = match.group(2) |
| continue |
| if match := re.search(pop_re, l): |
| prefix_stack.pop() |
| prefix_str = '_'.join(prefix_stack) |
| l = match.group(1) |
| continue |
| l = "" |
| |
| return enumerated_entries, types |
| |
| |
| def CheckIRBinaryCompatibility(input_api, output_api): |
| """Checks for changes to ir.proto that may cause compatibility issues""" |
| proto_file = None |
| old_entries, old_types = {}, [] |
| new_entries, new_types = {}, [] |
| for file in input_api.AffectedFiles( |
| include_deletes=False, |
| file_filter=lambda f: f.LocalPath().replace( |
| '\\', '/') == "src/tint/utils/protos/ir/ir.proto"): |
| if proto_file: |
| return [ |
| output_api.PresubmitError( |
| f"Unexpectedly found more than one ir.proto in change, [{file.AbsoluteLocalPath()}, {proto_file}]" |
| ) |
| ] |
| proto_file = file.AbsoluteLocalPath() |
| old_entries, old_types = _CalculateEnumeratedEntriesAndTypes( |
| file.OldContents()) |
| new_entries, new_types = _CalculateEnumeratedEntriesAndTypes( |
| file.NewContents()) |
| |
| changes = [] |
| for k in old_entries: |
| if k not in new_entries: |
| changes.append( |
| f"entry '{k}' has been removed, old={old_entries[k]}") |
| continue |
| if old_entries[k] != new_entries[k]: |
| changes.append( |
| f"entry '{k}' has changed, old={old_entries[k]}, new={new_entries[k]}" |
| ) |
| continue |
| |
| for s in old_types: |
| if s not in new_types: |
| changes.append(f"type '{s}' has been removed") |
| |
| if changes: |
| return [ |
| output_api.PresubmitError( |
| "Incompatible changes detected in ir.proto", items=changes) |
| ] |
| return [] |
| |
| |
| def CheckNoStaleGen(input_api, output_api): |
| """Checks that Tint generated files are not stale.""" |
| results = [] |
| try: |
| go = input_api.os_path.join(input_api.change.RepositoryRoot(), "tools", |
| "golang", "bin", "go") |
| if input_api.is_windows: |
| go += '.exe' |
| input_api.subprocess.check_call_out( |
| [go, "run", "tools/src/cmd/gen/main.go", "--check-stale"], |
| stdout=input_api.subprocess.PIPE, |
| stderr=input_api.subprocess.PIPE, |
| cwd=input_api.change.RepositoryRoot()) |
| except input_api.subprocess.CalledProcessError as e: |
| if input_api.is_committing: |
| results.append(output_api.PresubmitError('%s' % (e, ))) |
| else: |
| results.append(output_api.PresubmitPromptWarning('%s' % (e, ))) |
| return results |
| |
| |
| def CheckWebgpuHeaderDiff(input_api, output_api): |
| """Checks that generated WebGPU C Headers are not stale.""" |
| results = [] |
| try: |
| input_api.subprocess.check_call_out( |
| [sys.executable, "third_party/webgpu-headers/cli", "check"], |
| stdout=input_api.subprocess.PIPE, |
| stderr=input_api.subprocess.PIPE, |
| cwd=input_api.change.RepositoryRoot()) |
| except input_api.subprocess.CalledProcessError as e: |
| if input_api.is_committing: |
| results.append(output_api.PresubmitError('%s' % (e, ))) |
| else: |
| results.append(output_api.PresubmitPromptWarning('%s' % (e, ))) |
| return results |
| |
| |
| def _HasNoStrayWhitespaceFilter(file): |
| """Filters files that are exempt from the canned no stray whitespace check.""" |
| filter_list = [ |
| "third_party/webgpu-headers/webgpu.h.diff", # Generated diff file |
| ] |
| return file.LocalPath().replace('\\', '/') not in filter_list |
| |
| |
| def CheckChange(input_api, output_api): |
| results = [] |
| results.extend( |
| input_api.canned_checks.CheckForCommitObjects(input_api, output_api)) |
| |
| result_factory = output_api.PresubmitPromptWarning |
| if input_api.is_committing: |
| result_factory = output_api.PresubmitError |
| |
| # Check for formatting. |
| results.extend( |
| input_api.canned_checks.CheckPatchFormatted( |
| input_api, |
| output_api, |
| result_factory=result_factory)) |
| results.extend( |
| input_api.canned_checks.CheckGNFormatted(input_api, output_api)) |
| results.extend( |
| input_api.canned_checks.CheckChangeHasNoCrAndHasOnlyOneEol( |
| input_api, output_api)) |
| results.extend( |
| input_api.canned_checks.CheckChangeHasNoTabs(input_api, output_api)) |
| results.extend( |
| input_api.canned_checks.CheckChangeTodoHasOwner(input_api, output_api)) |
| results.extend( |
| input_api.canned_checks.CheckChangeHasNoStrayWhitespace( |
| input_api, |
| output_api, |
| source_file_filter=_HasNoStrayWhitespaceFilter)) |
| |
| results.extend( |
| input_api.canned_checks.CheckChangeHasDescription( |
| input_api, output_api)) |
| results.extend( |
| input_api.canned_checks.CheckDoNotSubmit(input_api, output_api)) |
| # Note, the verbose_level here should match what is set in tools/lint so |
| # the same set of lint errors are reported on the CQ and Kokoro bots. |
| results.extend( |
| input_api.canned_checks.CheckChangeLintsClean( |
| input_api, output_api, lint_filters=LINT_FILTERS, verbose_level=1)) |
| |
| return results |
