From: dyamy-lee Date: Fri, 12 Apr 2024 09:18:21 +0000 (+0900) Subject: Fix migration script by review X-Git-Tag: accepted/tizen/8.0/unified/20241011.164854~11 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F87%2F309587%2F4;p=platform%2Fcore%2Fuifw%2Ftts.git Fix migration script by review It changes checking condition as SRC_PREV_ENGINE_PATH and it defines before use. It fixes missed '/' when making path between get_rw_path and usr. It changes destination path checking only from 7.0 to 8.0. It changes DST_CONF_PATH with full path. It changes checking exist of each SRC path. Change-Id: I5fba34ab441ae4347f08f70951f8afb50d05e28d --- diff --git a/script/migration/tts.sh b/script/migration/tts.sh index 1ff57e38..3319b414 100644 --- a/script/migration/tts.sh +++ b/script/migration/tts.sh @@ -12,52 +12,53 @@ source "$UPDATE_EXEC_PATH/common/error_handle.inc" function tts_migration() { log I "TTS Migration config files" - local SRC_PATH=$(get_old_rw_path)/usr/data/.voice/tts - local DST_PATH=$(get_rw_path)/usr/data/.voice/tts + local SRC_PATH=$(get_old_rw_path)/usr/data/.voice/tts/ + local DST_PATH=$(get_rw_path)/usr/data/.voice/tts/ local SRC_CONF_PATH=$(get_old_rw_path)/usr/data/.voice/tts-config.xml - local DST_CONF_PATH=$(get_rw_path)/usr/data/.voice/ - - if [ ! -e $DST_PATH ]; then - log I "TTS Migration set destination path" - mkdir -p -m 0775 $(get_rw_path)/usr/data/.voice - chsmack -t $(get_rw_path)/usr/data/.voice - chsmack -a "User::App::Shared" $(get_rw_path)/usr/data/.voice - mkdir -p $(get_rw_path)usr/data/.voice/tts/engine-info - chsmack -a "User::App::Shared" $(get_rw_path)/usr/data/.voice/tts - chsmack -t $(get_rw_path)/usr/data/.voice/tts/engine-info - chsmack -a "User::App::Shared" $(get_rw_path)/usr/data/.voice/tts/engine-info - chown -R ui_fw:users $(get_rw_path)/usr/data/.voice - fi + local DST_CONF_PATH=$(get_rw_path)/usr/data/.voice/tts-config.xml + local SRC_PREV_ENGINE_PATH=$(get_old_rw_path)/usr/home/owner/share/.voice/tts/1.0/engine-info/ - if [ -e $SRC_PREV_CONF_PATH ]; then + if [ -e $SRC_PREV_ENGINE_PATH ]; then + VERSON=$(get_old_platform_version) + log I "Previous Version is $VERSION" log I "TTS Migration 7.0 only START" - SRC_PREV_ENGINE_PATH=$(get_old_rw_path)/usr/home/owner/share/.voice/tts/1.0/engine-info - DST_NEW_ENGINE_PATH=$(get_rw_path)/usr/data/.voice/tts/engine-info SRC_PREV_CONF_PATH=$(get_old_rw_path)/usr/home/owner/share/.voice/tts-config.xml - - if [ ! -e $SRC_PREV_ENGINE_PATH ]; then - log E "TTS Migration ERROR : No orignal Data" - return - fi - - ## change backgound_volume_ratio value type from float to integer in 8.0 - background_volume_ratio=$(awk -F '<|>' '/background-volume-ratio/{print $3}' $SRC_PREV_CONF_PATH) - log I "bg volume = $background_volume_ratio" - if [[ "${background_volume_ratio}" =~ ^[0-9]+$ ]]; then - integer_volume_ratio=${background_volume_ratio} - else - integer_volume_ratio=$(echo $background_volume_ratio | awk '{printf "%.2f\n", $1}' | awk '{$1=$1*100; print $1}') + DST_NEW_ENGINE_PATH=$(get_rw_path)/usr/data/.voice/tts/engine-info/ + + # each 7.0 and after 8.0 has different location about configuration. It needs new path. + if [ ! -e $(get_rw_path)/usr/data/.voice/tts/engine-info ]; then + log I "TTS Migration set destination path for only Tizen 7.0" + mkdir -p -m 0775 $(get_rw_path)/usr/data/.voice + chsmack -t $(get_rw_path)/usr/data/.voice + chsmack -a "User::App::Shared" $(get_rw_path)/usr/data/.voice + mkdir -p $(get_rw_path)/usr/data/.voice/tts/engine-info + chsmack -a "User::App::Shared" $(get_rw_path)/usr/data/.voice/tts + chsmack -t $(get_rw_path)/usr/data/.voice/tts/engine-info + chsmack -a "User::App::Shared" $(get_rw_path)/usr/data/.voice/tts/engine-info + chown -R ui_fw:users $(get_rw_path)/usr/data/.voice fi - log I "after change bg volume = $integer_volume_ratio" - sed -i "s/[^<]*<\/background-volume-ratio>/${integer_volume_ratio}\<\/background-volume-ratio>/" $SRC_PREV_CONF_PATH - - mig_copy $SRC_PREV_ENGINE_PATH $DST_NEW_ENGINE_PATH || error_handle "default_data" $LINENO - mig_copy $SRC_PREV_CONF_PATH $DST_CONF_PATH || error_handle "default_data" $LINENO + + # The 'tts-config.xml' file may or may not exist, depending on the condition + if [ -e $SRC_PREV_CONF_PATH ]; then + ## change backgound_volume_ratio value type from float to integer in 8.0 + background_volume_ratio=$(awk -F '<|>' '/background-volume-ratio/{print $3}' $SRC_PREV_CONF_PATH) + log I "bg volume = $background_volume_ratio" + if [[ "${background_volume_ratio}" =~ ^[0-9]+$ ]]; then + integer_volume_ratio=${background_volume_ratio} + else + integer_volume_ratio=$(echo $background_volume_ratio | awk '{printf "%.2f\n", $1}' | awk '{$1=$1*100; print $1}') + fi + + log I "after change bg volume = $integer_volume_ratio" + sed -i "s/[^<]*<\/background-volume-ratio>/${integer_volume_ratio}\<\/background-volume-ratio>/" $SRC_PREV_CONF_PATH + + mig_copy $SRC_PREV_CONF_PATH $DST_CONF_PATH || error_handle "default_data" $LINENO + chmod 664 $(get_rw_path)/usr/data/.voice/tts-config.xml + fi chown -R ui_fw:users $(get_rw_path)/usr/data/.voice - chmod 664 $(get_rw_path)/usr/data/.voice/tts-config.xml log I "TTS Migration 7.0 only END" else @@ -69,7 +70,12 @@ function tts_migration() { fi mig_copy $SRC_PATH $DST_PATH || error_handle "default_data" $LINENO - mig_copy $SRC_CONF_PATH $DST_CONF_PATH || error_handle "default_data" $LINENO + + # The 'tts-config.xml' file may or may not exist, depending on the condition + if [ -e $SRC_CONF_PATH ]; then + mig_copy $SRC_CONF_PATH $DST_CONF_PATH || error_handle "default_data" $LINENO + fi + log I "TTS Migration after 8.0 END" fi }