Made that grep for profile parameter in build_{tv,mobile} now always succeeds.
authorTomasz Olszak <t.olszak@samsung.com>
Fri, 24 Oct 2014 07:31:03 +0000 (09:31 +0200)
committerYoungsoo Choi <kenshin.choi@samsung.com>
Tue, 10 Jul 2018 06:57:09 +0000 (06:57 +0000)
grep returns non 0 status when it doesn't find any matching string. If someone
used set -e in parent script which invoked build_{tv,mobile} then got error and
script exited.

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=7957
Reviewed by:  Kangil Han

Change-Id: I9dfa0c026f65a25db071f89a4d10d871290db52a
Signed-off-by: Tomasz Olszak <t.olszak@samsung.com>
tizen_src/build/build_mobile.sh
tizen_src/build/build_tv.sh

index a24eb53..207f65c 100755 (executable)
@@ -7,7 +7,8 @@ SCRIPTDIR=$(cd $(dirname $0); pwd -P)
 
 ${SCRIPTDIR}/apply_patches.sh
 
-PROFILE_NAME=$(echo "$@" | grep -Po "(?<=\-P\s)[^\s]*")
+# "|| :" means "or always succeeding built-in command"
+PROFILE_NAME=$(echo "$@" | grep -Po "(?<=\-P\s)[^\s]*" || :)
 
 if [ "$PROFILE_NAME" == "" ]; then
   PROFILE_NAME=tizenmb_v2.3
index 4b4609e..9546a60 100755 (executable)
@@ -7,7 +7,8 @@ SCRIPTDIR=$(cd $(dirname $0); pwd -P)
 
 ${SCRIPTDIR}/apply_patches.sh
 
-PROFILE_NAME=$(echo "$@" | grep -Po "(?<=\-P\s)[^\s]*")
+# "|| :" means "or always succeeding built-in command"
+PROFILE_NAME=$(echo "$@" | grep -Po "(?<=\-P\s)[^\s]*" || :)
 
 if [ "$PROFILE_NAME" == "" ]; then
   PROFILE_NAME=tztv_v2.2.1_prehawk