]> code.ossystems Code Review - openembedded-core.git/commitdiff
populate_sdk_base: tarball installer: SDK overwrite warning
authorHongxu Jia <hongxu.jia@windriver.com>
Mon, 11 Mar 2013 06:55:44 +0000 (14:55 +0800)
committerRoss Burton <ross.burton@intel.com>
Tue, 23 Apr 2013 09:38:45 +0000 (10:38 +0100)
This patch contains two fixes:
* if the user wants to install the SDK in a directory that already
  contains a SDK for the same architecture, a warning will be shown;
* when the symbolic links are relocated use -n option. Otherwise,
  symbolic links to existing directories will be created in the
  directory itself;

Cherry-pick commit b751ec137d0228b40a90e9e32b24f5cb5732225b

[YOCTO #3401]

Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
meta/classes/populate_sdk_base.bbclass

index a1c45ad70a3fc93135854479c905df01a2ce923a..5e4ed51d4a6d8361c386b80faa10cfdf025dbb99 100644 (file)
@@ -132,11 +132,20 @@ else
        target_sdk_dir=$(readlink -m $target_sdk_dir)
 fi
 
-printf "You are about to install the SDK to \"$target_sdk_dir\". Proceed[Y/n]?"
+if [ -e "$target_sdk_dir/environment-setup-${REAL_MULTIMACH_TARGET_SYS}" ]; then
+       echo "The directory \"$target_sdk_dir\" already contains a SDK for this architecture."
+       printf "If you continue, existing files will be overwritten! Proceed[y/N]?"
+
+       default_answer="n"
+else
+       printf "You are about to install the SDK to \"$target_sdk_dir\". Proceed[Y/n]?"
+
+       default_answer="y"
+fi
 read answer
 
 if [ "$answer" = "" ]; then
-       answer="y"
+       answer="$default_answer"
 fi
 
 if [ "$answer" != "Y" -a "$answer" != "y" ]; then
@@ -176,7 +185,7 @@ find $native_sysroot -type f -exec file '{}' \;|grep ":.*ASCII.*text"|cut -d':'
 
 # change all symlinks pointing to ${SDKPATH}
 for l in $(find $native_sysroot -type l); do
-       ln -sf $(readlink $l|sed -e "s:$DEFAULT_INSTALL_DIR:$target_sdk_dir:") $l
+       ln -sfn $(readlink $l|sed -e "s:$DEFAULT_INSTALL_DIR:$target_sdk_dir:") $l
 done
 
 echo done