[llvm] r337842 - [docker] Fix LLVM_EXTERNAL_PROJECTS cmake variable value

Azharuddin Mohammed via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 24 11:34:13 PDT 2018


Author: azharudd
Date: Tue Jul 24 11:34:13 2018
New Revision: 337842

URL: http://llvm.org/viewvc/llvm-project?rev=337842&view=rev
Log:
[docker] Fix LLVM_EXTERNAL_PROJECTS cmake variable value

Summary:
LLVM_ENABLE_PROJECTS expects a semicolon separated project list.

Fixes PR38158.

Reviewers: ilya-biryukov

Reviewed By: ilya-biryukov

Subscribers: llvm-commits

Differential Revision: https://reviews.llvm.org/D49712

Modified:
    llvm/trunk/utils/docker/build_docker_image.sh

Modified: llvm/trunk/utils/docker/build_docker_image.sh
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/docker/build_docker_image.sh?rev=337842&r1=337841&r2=337842&view=diff
==============================================================================
--- llvm/trunk/utils/docker/build_docker_image.sh (original)
+++ llvm/trunk/utils/docker/build_docker_image.sh Tue Jul 24 11:34:13 2018
@@ -117,7 +117,7 @@ while [[ $# -gt 0 ]]; do
 
       CHECKOUT_ARGS="$CHECKOUT_ARGS $1 $PROJ"
       if [ "$PROJ" != "clang-tools-extra" ]; then
-        CMAKE_ENABLED_PROJECTS="$CMAKE_ENABLED_PROJECTS:$PROJ"
+        CMAKE_ENABLED_PROJECTS="$CMAKE_ENABLED_PROJECTS;$PROJ"
       fi
 
       shift 2
@@ -142,7 +142,7 @@ done
 
 
 if [ "$CMAKE_ENABLED_PROJECTS" != "" ]; then
-  # Remove the leading ':' character.
+  # Remove the leading ';' character.
   CMAKE_ENABLED_PROJECTS="${CMAKE_ENABLED_PROJECTS:1}"
 
   if [[ $SEEN_CMAKE_ARGS -eq 0 ]]; then




More information about the llvm-commits mailing list