diff options
author | Andreas Jaeger <aj@suse.de> | 2014-09-03 22:23:20 +0200 |
---|---|---|
committer | Andreas Jaeger <aj@suse.de> | 2014-09-11 18:54:13 +0200 |
commit | 85d665e51185cd0c6fe2985fa4c657b929a4019b (patch) | |
tree | a49648b84b6012dde450dae2d5ce2dd9dcba7eba /tools | |
parent | 407a69995dbc13494c1183d46145ec07b1f4850c (diff) | |
download | python-heatclient-85d665e51185cd0c6fe2985fa4c657b929a4019b.tar.gz |
warn against sorting requirements
And do not check for sorted requirements.
Change-Id: I64ae9191863564e278a35d42ec9cd743a233028e
Closes-Bug: 1365061
Diffstat (limited to 'tools')
-rwxr-xr-x | tools/requirements_style_check.sh | 32 |
1 files changed, 0 insertions, 32 deletions
diff --git a/tools/requirements_style_check.sh b/tools/requirements_style_check.sh deleted file mode 100755 index ccbff3b..0000000 --- a/tools/requirements_style_check.sh +++ /dev/null @@ -1,32 +0,0 @@ -#!/bin/bash -# -# Enforce the requirement that dependencies are listed in the input -# files in alphabetical order. - -# FIXME(dhellmann): This doesn't deal with URL requirements very -# well. We should probably sort those on the egg-name, rather than the -# full line. - -function check_file() { - typeset f=$1 - - # We don't care about comment lines. - grep -v '^#' $f > ${f}.unsorted - sort -i -f ${f}.unsorted > ${f}.sorted - diff -c ${f}.unsorted ${f}.sorted - rc=$? - rm -f ${f}.sorted ${f}.unsorted - return $rc -} - -exit_code=0 -for filename in $@ -do - check_file $filename - if [ $? -ne 0 ] - then - echo "Please list requirements in $filename in alphabetical order" 1>&2 - exit_code=1 - fi -done -exit $exit_code |