Skip to content

Commit

Permalink
[docker] Fix LLVM_EXTERNAL_PROJECTS cmake variable value
Browse files Browse the repository at this point in the history
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

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@337842 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
azharudd committed Jul 24, 2018
1 parent 264d803 commit 3e6b79a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions utils/docker/build_docker_image.sh
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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
Expand Down

0 comments on commit 3e6b79a

Please sign in to comment.