Skip to content

Commit

Permalink
Ignore _gopath in verify scripts
Browse files Browse the repository at this point in the history
The _gopath directory is used in Jenkins to install extra dependencies,
and any files in it should be ignored by the various verification
scripts.
  • Loading branch information
ixdy committed Oct 19, 2015
1 parent 4d18186 commit 4cafa3e
Show file tree
Hide file tree
Showing 6 changed files with 9 additions and 3 deletions.
1 change: 1 addition & 0 deletions hack/after-build/verify-description.sh
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ find_files() {
\( \
-wholename './output' \
-o -wholename './_output' \
-o -wholename './_gopath' \
-o -wholename './release' \
-o -wholename './target' \
-o -wholename '*/third_party/*' \
Expand Down
2 changes: 1 addition & 1 deletion hack/boilerplate/boilerplate.py
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ def file_passes(filename, refs, regexs):
def file_extension(filename):
return os.path.splitext(filename)[1].split(".")[-1].lower()

skipped_dirs = ['Godeps', 'third_party', '_output', '.git']
skipped_dirs = ['Godeps', 'third_party', '_gopath', '_output', '.git']
def normalize_files(files):
newfiles = []
for pathname in files:
Expand Down
3 changes: 2 additions & 1 deletion hack/lib/util.sh
Original file line number Diff line number Diff line change
Expand Up @@ -241,11 +241,12 @@ kube::util::gen-analytics() {
else
dir="${path}"
fi
# We don't touch files in Godeps|third_party, and the kubectl
# We don't touch files in Godeps|third_party|_gopath, and the kubectl
# docs are autogenerated by gendocs.
mdfiles=($( find "${dir}" -name "*.md" -type f \
-not -path "${path}/Godeps/*" \
-not -path "${path}/third_party/*" \
-not -path "${path}/_gopath/*" \
-not -path "${path}/_output/*" \
-not -path "${path}/docs/user-guide/kubectl/kubectl*" ))
for f in "${mdfiles[@]}"; do
Expand Down
1 change: 1 addition & 0 deletions hack/update-gofmt.sh
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@ find_files() {
\( \
-wholename './output' \
-o -wholename './_output' \
-o -wholename './_gopath' \
-o -wholename './release' \
-o -wholename './target' \
-o -wholename '*/third_party/*' \
Expand Down
4 changes: 3 additions & 1 deletion hack/verify-flags-underscore.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,8 @@ def get_all_files(rootdir):
# don't visit certain dirs
if 'Godeps' in dirs:
dirs.remove('Godeps')
if '_gopath' in dirs:
dirs.remove('_gopath')
if 'third_party' in dirs:
dirs.remove('third_party')
if '.git' in dirs:
Expand All @@ -77,7 +79,7 @@ def get_all_files(rootdir):

def normalize_files(rootdir, files):
newfiles = []
a = ['Godeps', 'third_party', 'exceptions.txt', 'known-flags.txt']
a = ['Godeps', '_gopath', 'third_party', '.git', 'exceptions.txt', 'known-flags.txt']
for f in files:
if any(x in f for x in a):
continue
Expand Down
1 change: 1 addition & 0 deletions hack/verify-gofmt.sh
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@ find_files() {
\( \
-wholename './output' \
-o -wholename './_output' \
-o -wholename './_gopath' \
-o -wholename './release' \
-o -wholename './target' \
-o -wholename '*/third_party/*' \
Expand Down

0 comments on commit 4cafa3e

Please sign in to comment.