HBASE-20134 dev-support scripts should use mktemp instead of hard-coding /tmp.

Signed-off-by: Mike Drob <mdrob@apache.org>
This commit is contained in:
Sean Busbey 2018-03-05 23:40:25 -06:00
parent b7fb2df154
commit ff7f5ecb0f
2 changed files with 11 additions and 13 deletions

View File

@ -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}")"

View File

@ -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}")"