diff options
| author | Juergen Funk <juergen.funk_ml@cib.de> | 2019-11-18 10:33:13 +0100 |
|---|---|---|
| committer | Thorsten Behrens <Thorsten.Behrens@CIB.de> | 2019-11-19 11:17:57 +0100 |
| commit | 7f8c9ddbbd26594bb9249cc6295d0170e4bb58e7 (patch) | |
| tree | 9b9c65ba55840339558e559d8bc14124484fa228 /bin | |
| parent | 83e3ba6fba486a5bf68427f0cf9aa9de7d97a391 (diff) | |
symstore.sh: use logical operators for test statements
No need to spawn two subshells, test can do and/or logical ops itself.
Change-Id: I2abba303383f9f0053515088d4fa32753a777a1d
Reviewed-on: https://gerrit.libreoffice.org/83066
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
Diffstat (limited to 'bin')
| -rwxr-xr-x | bin/symstore.sh | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/bin/symstore.sh b/bin/symstore.sh index f37ea0f4e7a0..2f7ad2f77610 100755 --- a/bin/symstore.sh +++ b/bin/symstore.sh @@ -20,11 +20,11 @@ add_pdb() } # check preconditions -if [ -z "${INSTDIR}" ] || [ -z "${WORKDIR}" ]; then +if [ -z "${INSTDIR}" -o -z "${WORKDIR}" ]; then echo "INSTDIR or WORKDIR not set - script expects calling inside buildenv" exit 1 fi -if [ ! -d "${INSTDIR}" ] || [ ! -d "${WORKDIR}" ]; then +if [ ! -d "${INSTDIR}" -o ! -d "${WORKDIR}" ]; then echo "INSTDIR or WORKDIR not present - script expects calling after full build" exit 1 fi @@ -84,7 +84,7 @@ rm -f "${TMPFILE}" # Cleanup symstore, older revisions will be removed. Unless the # .dll/.exe changes, the .pdb should be shared, so with incremental # tinderbox several revisions should not be that space-demanding. -if [ "${MAX_KEEP}" -gt 0 ] && [ -d "${SYM_PATH}/000Admin" ]; then +if [ "${MAX_KEEP}" -gt 0 -a -d "${SYM_PATH}/000Admin" ]; then to_remove=$(ls -1 "${SYM_PATH}/000Admin" | grep -v '\.txt' | grep -v '\.deleted' | sort | head -n "-${MAX_KEEP}") for revision in $to_remove; do symstore.exe del /i "${revision}" /s "$(cygpath -w "${SYM_PATH}")" |