From cc77cd788deebe036180099ccca0af5ca43af3f0 Mon Sep 17 00:00:00 2001 From: ljf Date: Thu, 9 Feb 2017 13:37:51 +0100 Subject: [PATCH] [fix] Permission issue on install of some apps 778 --- data/helpers.d/filesystem | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/data/helpers.d/filesystem b/data/helpers.d/filesystem index 27a016e63..bce58b5cf 100644 --- a/data/helpers.d/filesystem +++ b/data/helpers.d/filesystem @@ -70,6 +70,13 @@ ynh_bind_or_cp() { # usage: ynh_mkdir_tmp # | ret: the created directory path ynh_mkdir_tmp() { - echo "This helper is deprecated, you should use 'mktemp -d' instead." >&2 - mktemp -d + echo "The helper ynh_mkdir_tmp is deprecated." >&2 + echo "You should use 'mktemp -d' instead and manage permissions \ +properly with chmod/chown." >&2 + local TMP_DIR=$(mktemp -d) + + # Give rights to other users could be a security risk. + # But for retrocompatibility we need it. (This helpers is deprecated) + chmod 755 $TMP_DIR + echo $TMP_DIR }