diff options
author | gjb <gjb@FreeBSD.org> | 2014-05-05 23:36:46 +0000 |
---|---|---|
committer | gjb <gjb@FreeBSD.org> | 2014-05-05 23:36:46 +0000 |
commit | 915372aa18469a8626ff24ac7b55d07871412381 (patch) | |
tree | 1b54f84241eb64591b8b1c8f473c829e4ce35f49 | |
parent | 2b7aa883aff6419e934d79652b4801e6e77a8eeb (diff) | |
download | FreeBSD-src-915372aa18469a8626ff24ac7b55d07871412381.zip FreeBSD-src-915372aa18469a8626ff24ac7b55d07871412381.tar.gz |
Use 'return' instead of 'exit' in main(), since the script exits
with the last returned exit code.
Sponsored by: The FreeBSD Foundation
-rwxr-xr-x | release/scripts/relnotes-search.sh | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/release/scripts/relnotes-search.sh b/release/scripts/relnotes-search.sh index 42d67fc..a9a931a 100755 --- a/release/scripts/relnotes-search.sh +++ b/release/scripts/relnotes-search.sh @@ -56,7 +56,7 @@ main() { if [ -e "${where}" ]; then echo "Log file already exists:" echo " (${where})" - exit 2 + return 2 fi ;; r) @@ -64,7 +64,7 @@ main() { c=$(echo -n ${rev} | tr -d '0-9' | wc -c) if [ ${c} -ne 0 ]; then echo "Revision number must be numeric." - exit 2 + return 2 fi # Since the last specified revision is # specified, mangle the variable to @@ -101,25 +101,25 @@ main() { # Did we find svn? if [ -z "${svn}" ]; then echo "svn(1) binary not found." - exit 2 + return 2 fi # Is more than one path specified? (This should never # be triggered, because the argument count is checked # above, but better safe than sorry.) if [ $# -gt 1 ]; then echo "Cannot specify more than one working path." - exit 2 + return 2 fi # Does the directory exist? if [ ! -d "${1}" ]; then echo "Specified path (${1}) is not a directory." - exit 2 + return 2 fi # Is it a subversion repository checkout? ${svn} info ${1} >/dev/null 2>&1 if [ "$?" -ne 0 ]; then echo "Cannot determine svn repository information for ${1}" - exit 2 + return 2 fi # All tests passed. Let's see what can possibly go wrong |