diff --git a/dev-support/hbase_nightly_source-artifact.sh b/dev-support/hbase_nightly_source-artifact.sh index b334fa51617..945832498e0 100755 --- a/dev-support/hbase_nightly_source-artifact.sh +++ b/dev-support/hbase_nightly_source-artifact.sh @@ -21,7 +21,7 @@ function usage { echo "Usage: ${0} [options] /path/to/component/checkout" echo "" echo " --intermediate-file-dir /path/to/use Path for writing listings and diffs. must exist." - echo " defaults to making a directory in /tmp." + echo " defaults to making a directory via mktemp." echo " --unpack-temp-dir /path/to/use Path for unpacking tarball. default to" echo " 'unpacked_src_tarball' in intermediate directory." echo " --maven-m2-initial /path/to/use Path for maven artifacts while building in" @@ -67,11 +67,10 @@ fi component_dir="$(cd "$(dirname "$1")"; pwd)/$(basename "$1")" if [ -z "${working_dir}" ]; then - working_dir=/tmp - while [[ -e ${working_dir} ]]; do - working_dir=/tmp/hbase-srctarball-test-${RANDOM}.${RANDOM} - done - mkdir "${working_dir}" + if ! working_dir="$(mktemp -d -t hbase-srctarball-test)" ; then + echo "Failed to create temporary working directory. Please specify via --unpack-temp-dir" + exit 1 + fi else # absolutes please working_dir="$(cd "$(dirname "${working_dir}")"; pwd)/$(basename "${working_dir}")" diff --git a/dev-support/jenkins-scripts/generate-hbase-website.sh b/dev-support/jenkins-scripts/generate-hbase-website.sh index c25c895fa67..b6277d0b2c6 100644 --- a/dev-support/jenkins-scripts/generate-hbase-website.sh +++ b/dev-support/jenkins-scripts/generate-hbase-website.sh @@ -40,7 +40,7 @@ function usage { echo "" echo " --working-dir /path/to/use Path for writing logs and a local checkout of hbase-site repo." echo " if given must exist." - echo " defaults to making a directory in /tmp." + echo " defaults to making a directory via mktemp." echo " --local-repo /path/for/maven/.m2 Path for putting local maven repo." echo " if given must exist." echo " defaults to making a clean directory in --working-dir." @@ -77,12 +77,11 @@ fi component_dir="$(cd "$(dirname "$1")"; pwd)/$(basename "$1")" if [ -z "${working_dir}" ]; then - echo "[DEBUG] defaulting to creating a directory in /tmp" - working_dir=/tmp - while [[ -e ${working_dir} ]]; do - working_dir=/tmp/hbase-generate-website-${RANDOM}.${RANDOM} - done - mkdir "${working_dir}" + echo "[DEBUG] defaulting to creating a directory via mktemp" + if ! working_dir="$(mktemp -d -t hbase-generate-website)" ; then + echo "Failed to create temporary working directory. Please specify via --working-dir" + exit 1 + fi else # absolutes please working_dir="$(cd "$(dirname "${working_dir}")"; pwd)/$(basename "${working_dir}")"