kicad-source/tools/check_coding.sh
John Beard a69cdf1793 Formatting: exclude generated files from git formatting hook
Add a .gitattributes files to provide a place to store file
attributes. Add a custom attribute for files that should be under
KiCad style guidelines.

Exclude generated files from the style enforcement. So far:

* bitmap .cpp files
* wxFormBuilder base classes
* Lemon grammars

It's now trivial to add the 'generated' attribute to any such
file.

Putting this into .gitattributes also means it can be retreived
programmatically, by other scripts, git alaises or on the command
line.

Use the attributes to provide a utility script to show or apply
formatting to controlled files (plain git clang-format won't
pick up our custom git attributes). Add details for the script in
the dev docs.

Also modiify the check-format hook to follow the .gitattributes
and only apply to cached (staged for commit) changes. Then you
won't be stopped committing because of bad formatting in unstaged
changed, or uncontrolled files.

Concept and some aspects of the implementation inspired by
CMake commit d5f39a56 [1].

[1]: d5f39a56a4

tool
2019-03-29 15:06:48 +00:00

108 lines
3.1 KiB
Bash
Executable File

#!/usr/bin/env bash
# This program source code file is part of KICAD, a free EDA CAD application.
#
# Copyright (C) 2019 Kicad Developers, see AUTHORS.txt for contributors.
#
# This program is free software; you can redistribute it and/or
# modify it under the terms of the GNU General Public License
# as published by the Free Software Foundation; either version 2
# of the License, or (at your option) any later version.
#
# This program is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with this program; if not, you may find one here:
# http://www.gnu.org/licenses/old-licenses/gpl-2.0.html
# or you may search the http://www.gnu.org website for the version 2 license,
# or you may write to the Free Software Foundation, Inc.,
# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA
# Simple program to check and fix formatting in KiCad source files,
# while ignoring violations in "uncontrolled" files, such as generated files
# or 3rd party sources.
usage='usage: check_coding.sh [<options>] [--]
--help Print usage plus more detailed help.
--diff Only show proposed changes, do not format files
--cached Re-format changes currently staged for commit
--amend Re-format changes made in the previous commit
'
help="$usage"'
Example to format cached files:
check_coding.sh
To show what would be done:
check_coding.sh --diff
'
die() {
echo "$@" 1>&2; exit 1
}
# Parse command-line arguments.
diff=false
mode='cached'
while test "$#" != 0; do
case "$1" in
--diff) diff=true ;;
--amend) mode='amend' ;;
--cached) mode='cached' ;;
--) shift ; break ;;
-*) die "$usage" ;;
*) break ;;
esac
shift
done
test "$#" = 0 || die "$usage"
# This is the main KiCad formatting attribute
# used by .gitattributes to mark KiCad source files
format_attribute="format.clang-format-kicad"
format_commit='HEAD'
# Select the git file list command and the commit to check
case "${mode}" in
'') echo "$usage"; exit 0 ;;
amend)
# Files changed by the last commit
git_list_files='git diff-tree --diff-filter=ACM --name-only HEAD -r --no-commit-id'
format_commit='HEAD^'
;;
cached)
# Currently staged files
git_list_files='git diff-index --diff-filter=ACM --name-only HEAD --cached'
;;
*) die "Invalid mode: $mode" ;;
esac
if [ "${diff}" = true ]; then
# Only show the proposed changes
format_command="git clang-format --diff ${format_commit}"
else
# Actually make the changes
format_command="git clang-format ${format_commit}"
fi
${git_list_files} |
# Filter sources with the formatting attribute set
git check-attr ${format_attribute} --stdin |
# output only the file names
grep ": set$" |
cut -d: -f1 |
# Apply the formatting command
xargs -d '\n' ${format_command}