| #!/usr/bin/env python3 |
| # |
| # Copyright (c) 2019 Inria |
| # All rights reserved |
| # |
| # Redistribution and use in source and binary forms, with or without |
| # modification, are permitted provided that the following conditions are |
| # met: redistributions of source code must retain the above copyright |
| # notice, this list of conditions and the following disclaimer; |
| # 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; |
| # neither the name of the copyright holders 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 |
| # OWNER 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. |
| |
| # If your commit has been canceled because of this file, do not panic: a |
| # copy of it has been stored in ./.git/COMMIT_EDITMSG |
| |
| import os |
| import re |
| import sys |
| from maint.lib import maintainers |
| |
| from style.repo import GitRepo |
| |
| |
| def _printErrorQuit(error_message): |
| """ |
| Print an error message, followed my a help message and inform failure. |
| |
| @param error_message A message describing the error that caused the |
| failure. |
| """ |
| print(error_message) |
| |
| print( |
| "The commit has been cancelled, but a copy of it can be found in " |
| + sys.argv[1] |
| + " : " |
| ) |
| |
| print( |
| """ |
| -------------------------------------------------------------------------- |
| """ |
| ) |
| print(open(sys.argv[1], "r").read()) |
| print( |
| """ |
| -------------------------------------------------------------------------- |
| """ |
| ) |
| |
| print( |
| """ |
| The first line of a commit must contain one or more gem5 tags separated by |
| commas (see MAINTAINERS.yaml for the possible tags), followed by a colon and |
| a commit title. There must be no leading nor trailing whitespaces. |
| |
| This header line must then be followed by an empty line. A detailed message, |
| although highly recommended, is not mandatory and can follow that empty line. |
| |
| e.g.: |
| cpu: Refactor branch predictors |
| |
| Refactor branch predictor code to improve its readability, moving functions |
| X and Y to the base class... |
| |
| e.g.: |
| mem,mem-cache: Improve packet class readability |
| |
| The packet class... |
| """ |
| ) |
| sys.exit(1) |
| |
| |
| def _validateTags(commit_header): |
| """ |
| Check if all tags in the commit header belong to the list of valid |
| gem5 tags. |
| |
| @param commit_header The first line of the commit message. |
| """ |
| |
| # List of valid tags |
| maintainer_dict = maintainers.Maintainers.from_file() |
| valid_tags = [tag for tag, _ in maintainer_dict] |
| |
| # Remove non-tag 'pmc' and add special tags not in MAINTAINERS.yaml |
| valid_tags.remove("pmc") |
| valid_tags.extend(["RFC", "WIP"]) |
| |
| tags = "".join(commit_header.split(":")[0].split()).split(",") |
| if any(tag not in valid_tags for tag in tags): |
| invalid_tag = next((tag for tag in tags if tag not in valid_tags)) |
| _printErrorQuit("Invalid Gem5 tag: " + invalid_tag) |
| |
| |
| # Go to git directory |
| os.chdir(GitRepo().repo_base()) |
| |
| # Get the commit message |
| commit_message = open(sys.argv[1]).read() |
| |
| # The first line of a commit must contain at least one valid gem5 tag, and |
| # a commit title |
| commit_message_lines = commit_message.splitlines() |
| commit_header = commit_message_lines[0] |
| commit_header_match = re.search( |
| "^(fixup! )?(\S[\w\-][,\s*[\w\-]+]*:.+\S$)", commit_header |
| ) |
| if commit_header_match is None: |
| _printErrorQuit("Invalid commit header") |
| if commit_header_match.group(1) == "fixup! ": |
| sys.exit(0) |
| _validateTags(commit_header_match.group(2)) |
| |
| # Make sure commit title does not exceed threshold. This line is limited to |
| # a smaller number because version control systems may add a prefix, causing |
| # line-wrapping for longer lines |
| commit_title = commit_header.split(":")[1] |
| max_header_size = 65 |
| if len(commit_header) > max_header_size: |
| _printErrorQuit( |
| "The commit header (tags + title) is too long (" |
| + str(len(commit_header)) |
| + " > " |
| + str(max_header_size) |
| + ")" |
| ) |
| |
| # Then there must be at least one empty line between the commit header and |
| # the commit description |
| if commit_message_lines[1] != "": |
| _printErrorQuit( |
| "Please add an empty line between the commit title and " |
| "its description" |
| ) |
| |
| # Encourage providing descriptions |
| if re.search( |
| "^(Signed-off-by|Change-Id|Reviewed-by):", commit_message_lines[2] |
| ): |
| print("Warning: Commit does not have a description") |
| |
| sys.exit(0) |