From 21e0a34fd16b61449460267769fd2394c8a7d617 Mon Sep 17 00:00:00 2001 From: Benjamin Dauvergne Date: Tue, 18 Aug 2020 15:17:57 +0200 Subject: [PATCH] check-migrations: use mktemp instead of tempfile --- check-migrations.sh | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/check-migrations.sh b/check-migrations.sh index f36e39957..1bb1398e8 100755 --- a/check-migrations.sh +++ b/check-migrations.sh @@ -2,10 +2,9 @@ set -e -trap "rm -f ${TEMPFILE} ${CHECK_MIGRATIONS_SETTINGS}" EXIT - # https://stackoverflow.com/questions/49778988/makemigrations-in-dev-machine-without-database-instance -CHECK_MIGRATIONS_SETTINGS=`tempfile` +CHECK_MIGRATIONS_SETTINGS=`mktemp` +trap "rm -f ${CHECK_MIGRATIONS_SETTINGS}" EXIT cat <${CHECK_MIGRATIONS_SETTINGS} DATABASES = { 'default': { @@ -14,7 +13,8 @@ DATABASES = { } EOF APPS=$(env DJANGO_SETTINGS_MODULE=authentic2.settings python -c 'from django.conf import settings; print(" ".join(p for p in (p.split(".")[-1] for p in settings.INSTALLED_APPS if not p.startswith("django.")) if p not in ["admin", "auth", "contenttypes"]))') -TEMPFILE=`tempfile` +TEMPFILE=`mktemp` +trap "rm -f ${TEMPFILE} ${CHECK_MIGRATIONS_SETTINGS}" EXIT AUTHENTIC2_SETTINGS_FILE=${CHECK_MIGRATIONS_SETTINGS} ./authentic2-ctl makemigrations --dry-run --noinput ${APPS} >${TEMPFILE} 2>&1 || true