]> code.ossystems Code Review - openembedded-core.git/commitdiff
populate_sdk_base: mkdir was missing $SUDO_EXEC prefix
authorLaurentiu Palcu <laurentiu.palcu@intel.com>
Tue, 4 Dec 2012 12:16:12 +0000 (14:16 +0200)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Tue, 4 Dec 2012 13:01:22 +0000 (13:01 +0000)
mkdir was in the wrong place and missing sudo rights. Hence, the
installation to default location (or any other for which the user didn't
have rights) would fail. Unless the installer itself is run with sudo.

Signed-off-by: Laurentiu Palcu <laurentiu.palcu@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/classes/populate_sdk_base.bbclass

index 691ae45c93d71679fcebb07485b3cad45b8d99c3..f0b88b34c3416bb5a6424fa0f7a9270a161c09ef 100644 (file)
@@ -187,9 +187,6 @@ if [ "$answer" != "Y" -a "$answer" != "y" ]; then
        exit 1
 fi
 
-# create dir and don't care about the result. 
-mkdir -p $target_sdk_dir >/dev/null 2>&1
-
 # if don't have the right to access dir, gain by sudo 
 if [ ! -x $target_sdk_dir -o ! -w $target_sdk_dir -o ! -r $target_sdk_dir ]; then 
        SUDO_EXEC=$(which "sudo")
@@ -203,6 +200,9 @@ if [ ! -x $target_sdk_dir -o ! -w $target_sdk_dir -o ! -r $target_sdk_dir ]; the
        [ $? -ne 0 ] && echo "Sorry, you are not allowed to execute as root." && exit 1
 fi
 
+# create dir and don't care about the result.
+$SUDO_EXEC mkdir -p $target_sdk_dir >/dev/null 2>&1
+
 payload_offset=$(($(grep -na -m1 "^MARKER:$" $0|cut -d':' -f1) + 1))
 
 printf "Extracting SDK..."