mirror of
https://github.com/kk7ds/chirp
synced 2026-08-13 17:51:36 -04:00
6301c58554
This method should not be implemented by new drivers, so fail the check if it looks like it is present.
100 lines
2.6 KiB
Bash
Executable file
100 lines
2.6 KiB
Bash
Executable file
#!/bin/bash
|
|
|
|
BASE=${1:-origin/master}
|
|
RETCODE=0
|
|
|
|
RED='\033[1;31m'
|
|
GREEN='\033[1;32m'
|
|
NC='\033[0m'
|
|
|
|
function fail() {
|
|
echo -e "${RED}$*${NC}"
|
|
RETCODE=1
|
|
}
|
|
|
|
echo -e "${GREEN}Checking from $(git rev-parse --short ${BASE}):${NC}"
|
|
git log --pretty=oneline --no-merges --abbrev-commit ${BASE}..
|
|
echo
|
|
|
|
git diff ${BASE}.. '*.py' | grep '^+' > added_lines
|
|
|
|
if grep -E '(from|import).*six' added_lines; then
|
|
fail No new uses of future
|
|
fi
|
|
|
|
if grep -E '\wsix\w' added_lines; then
|
|
fail No new uses of six
|
|
fi
|
|
|
|
if grep -E '(from|import).*builtins' added_lines; then
|
|
fail No new uses of future
|
|
fi
|
|
|
|
if grep -E '\wfuture\w' added_lines; then
|
|
fail No new uses of future
|
|
fi
|
|
|
|
if grep -E '(from|import).*past' added_lines; then
|
|
fail Use of past library not allowed
|
|
fi
|
|
|
|
if grep -E 'MemoryMap\(' added_lines; then
|
|
fail New uses of MemoryMap should be MemoryMapBytes
|
|
fi
|
|
|
|
if grep -E "[^_]_\([^\"']" added_lines; then
|
|
fail 'Translated strings must be literals!'
|
|
fi
|
|
|
|
if grep '/cpep8.manifest' added_lines; then
|
|
fail 'Do not add new files to cpep8.manifest; no longer needed'
|
|
fi
|
|
|
|
if grep '/cpep8.blacklist' added_lines; then
|
|
fail 'Do not add new files to cpep8.blacklist'
|
|
fi
|
|
|
|
grep -i 'license' added_lines > license_lines
|
|
if grep -ivE '(GNU General Public License|Free Software Foundation|gnu.org.licenses)' license_lines; then
|
|
fail 'Files must be GPLv3 licensed (or not contain any license language)'
|
|
fi
|
|
|
|
for file in $(git diff --name-only ${BASE}..); do
|
|
if file $file | grep -q CRLF; then
|
|
fail "$file : Files should be LF (Unix) format, not CR (Mac) or CRLF (Windows)"
|
|
fi
|
|
done
|
|
|
|
if grep 'def match_model' added_lines; then
|
|
fail 'New drivers should not have match_model() implemented as it is not needed'
|
|
fi
|
|
|
|
if git log ${BASE}.. --merges | grep .; then
|
|
fail Please do not include merge commits in your PR
|
|
fi
|
|
|
|
make -C chirp/locale clean all >/dev/null 2>&1
|
|
if git diff chirp/locale | grep '^+[^#+]' | grep -v POT-Creation; then
|
|
fail Locale files need updating
|
|
fi
|
|
|
|
added_files=$(git diff --name-only --diff-filter=A ${BASE}.. | grep '\.py$' 2>&1)
|
|
if echo $added_files | grep -q chirp.drivers && ! echo $added_files | grep -q tests.images; then
|
|
fail All new drivers should include a test image
|
|
fi
|
|
|
|
existing_drivers=$(git ls-tree --name-only $BASE chirp/drivers/)
|
|
limit=20
|
|
for nf in $added_files; do
|
|
for of in $existing_drivers; do
|
|
change=$(wdiff -s $of $nf | grep -I $of | sed -r 's/.* ([0-9]+)% changed/\1/')
|
|
if [ ! "$change" ]; then
|
|
continue
|
|
fi
|
|
if [ "$change" -lt "$limit" ]; then
|
|
fail "New file $nf shares at least $((100 - $change))% with $of!"
|
|
fi
|
|
done
|
|
done
|
|
|
|
exit $RETCODE
|