Merge pull request #29363 from compnerd/bs-check

build-script: reduce duplication of check

Author: compnerd

utils/build-script-impl

@@ -2650,6 +2650,10 @@ for host in "${ALL_HOSTS[@]}"; do
         if ! [[ $(should_execute_action "${host}-${tmp_product}-install") ]]; then
             continue
         fi
+        if [[ -z "${INSTALL_DESTDIR}" ]] ; then
+            echo "--install-destdir is required to install products."
+            exit 1
+        fi
 
         INSTALL_TARGETS="install"
 
@@ -2692,20 +2696,12 @@ for host in "${ALL_HOSTS[@]}"; do
                 if [[ -z "${INSTALL_LLDB}" ]] ; then
                     continue
                 fi
-                if [[ -z "${INSTALL_DESTDIR}" ]] ; then
-                    echo "--install-destdir is required to install products."
-                    exit 1
-                fi
                 INSTALL_TARGETS="install-distribution"
                 ;;
             xctest)
                 if [[ -z "${INSTALL_XCTEST}" ]] ; then
                     continue
                 fi
-                if [[ -z "${INSTALL_DESTDIR}" ]] ; then
-                    echo "--install-destdir is required to install products."
-                    exit 1
-                fi
 
                 case ${host} in
                   linux-*|freebsd-*|cygwin-*|haiku-*|android-*) ;;
@@ -2727,11 +2723,6 @@ for host in "${ALL_HOSTS[@]}"; do
                     continue
                 fi
 
-                if [[ -z "${INSTALL_DESTDIR}" ]] ; then
-                    echo "--install-destdir is required to install products."
-                    exit 1
-                fi
-
                 ;;
             libdispatch|libdispatch_static)
                 if [[ -z "${INSTALL_LIBDISPATCH}" ]] ; then
@@ -2765,10 +2756,6 @@ for host in "${ALL_HOSTS[@]}"; do
                 if [[ -z "${INSTALL_LIBICU}" ]]; then
                     continue
                 fi
-                if [[ -z "${INSTALL_DESTDIR}" ]] ; then
-                    echo "--install-destdir is required to install products."
-                    exit 1
-                fi
                 echo "--- Installing ${product} ---"
                 ICU_BUILD_DIR=$(build_directory ${host} ${product})
                 ICU_INSTALL_DIR="$(get_host_install_destdir ${host})$(get_host_install_prefix ${host})"
@@ -2806,10 +2793,6 @@ for host in "${ALL_HOSTS[@]}"; do
                 if [[ -z "${INSTALL_PLAYGROUNDSUPPORT}" ]] ; then
                     continue
                 fi
-                if [[ -z "${INSTALL_DESTDIR}" ]] ; then
-                    echo "--install-destdir is required to install products."
-                    exit 1
-                fi
 
                 echo "--- Installing ${product} ---"
 
@@ -2845,11 +2828,6 @@ for host in "${ALL_HOSTS[@]}"; do
                 ;;
         esac
 
-        if [[ -z "${INSTALL_DESTDIR}" ]] ; then
-            echo "--install-destdir is required to install products."
-            exit 1
-        fi
-
         echo "--- Installing ${product} ---"
         build_dir=$(build_directory ${host} ${product})