diff options
| author | Nick Mathewson <nickm@torproject.org> | 2020-01-23 10:26:09 -0500 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2020-01-23 10:28:09 -0500 |
| commit | cda10a2e893da45eea7e742155682efcf66afb22 (patch) | |
| tree | f3abaeb194cd56f0e8319ca19ca01d814fb0e059 /scripts/git | |
| parent | 7c76cedf0fea5117612018bf8b07e6fe91e967e7 (diff) | |
| download | tor-cda10a2e893da45eea7e742155682efcf66afb22.tar.gz tor-cda10a2e893da45eea7e742155682efcf66afb22.zip | |
Git hooks: remove support for 0.2.9 directory layout.
We no longer will be pushing or committing things in branches that
use src/common or src/or.
Resolves ticket 32957.
Diffstat (limited to 'scripts/git')
| -rwxr-xr-x | scripts/git/pre-commit.git-hook | 27 | ||||
| -rwxr-xr-x | scripts/git/pre-push.git-hook | 25 |
2 files changed, 16 insertions, 36 deletions
diff --git a/scripts/git/pre-commit.git-hook b/scripts/git/pre-commit.git-hook index d5a75f2794..f630a242bd 100755 --- a/scripts/git/pre-commit.git-hook +++ b/scripts/git/pre-commit.git-hook @@ -17,13 +17,14 @@ if [ $# -eq 0 ]; then # When called in pre-commit, check the files modified in this commit CHECK_FILTER="git diff --cached --name-only --diff-filter=ACMR" # Use the appropriate owned tor source list to filter the changed files - if [ -d src/lib ]; then - # This is the layout in 0.3.5 - # Keep these lists consistent: - # - OWNED_TOR_C_FILES in Makefile.am - # - CHECK_FILES in pre-commit.git-hook and pre-push.git-hook - # - try_parse in check_cocci_parse.sh - CHECK_FILES="$($CHECK_FILTER \ + + # This is the layout in 0.3.5 and later. + + # Keep these lists consistent: + # - OWNED_TOR_C_FILES in Makefile.am + # - CHECK_FILES in pre-commit.git-hook and pre-push.git-hook + # - try_parse in check_cocci_parse.sh + CHECK_FILES="$($CHECK_FILTER \ src/lib/*/*.[ch] \ src/core/*/*.[ch] \ src/feature/*/*.[ch] \ @@ -31,17 +32,7 @@ if [ $# -eq 0 ]; then src/test/*.[ch] \ src/test/*/*.[ch] \ src/tools/*.[ch] \ - )" - elif [ -d src/common ]; then - # This was the layout before 0.3.5 - CHECK_FILES="$($CHECK_FILTER \ - src/common/*/*.[ch] \ - src/or/*/*.[ch] \ - src/test/*.[ch] \ - src/test/*/*.[ch] \ - src/tools/*.[ch] - )" - fi + )" else # When called in pre-push, concatenate the argument array # Fails on special characters in file names diff --git a/scripts/git/pre-push.git-hook b/scripts/git/pre-push.git-hook index 8b5efb0a9d..7b06f3734d 100755 --- a/scripts/git/pre-push.git-hook +++ b/scripts/git/pre-push.git-hook @@ -63,13 +63,12 @@ do CHECK_FILTER="git diff --name-only --diff-filter=ACMR $range" # Use the appropriate owned tor source list to filter the changed # files - if [ -d src/lib ]; then - # This is the layout in 0.3.5 - # Keep these lists consistent: - # - OWNED_TOR_C_FILES in Makefile.am - # - CHECK_FILES in pre-commit.git-hook and pre-push.git-hook - # - try_parse in check_cocci_parse.sh - CHECK_FILES="$($CHECK_FILTER \ + # This is the layout in 0.3.5 + # Keep these lists consistent: + # - OWNED_TOR_C_FILES in Makefile.am + # - CHECK_FILES in pre-commit.git-hook and pre-push.git-hook + # - try_parse in check_cocci_parse.sh + CHECK_FILES="$($CHECK_FILTER \ src/lib/*/*.[ch] \ src/core/*/*.[ch] \ src/feature/*/*.[ch] \ @@ -77,17 +76,7 @@ do src/test/*.[ch] \ src/test/*/*.[ch] \ src/tools/*.[ch] \ - )" - elif [ -d src/common ]; then - # This was the layout before 0.3.5 - CHECK_FILES="$($CHECK_FILTER \ - src/common/*/*.[ch] \ - src/or/*/*.[ch] \ - src/test/*.[ch] \ - src/test/*/*.[ch] \ - src/tools/*.[ch] \ - )" - fi + )" # We want word splitting here, because file names are space # separated |