Skip to content
This repository was archived by the owner on Apr 23, 2020. It is now read-only.

Commit 19e7bfe

Browse files
committed
Merging r322081:
------------------------------------------------------------------------ r322081 | mgorny | 2018-01-09 06:44:04 -0800 (Tue, 09 Jan 2018) | 10 lines [test] Fix tests to use more portable LLVM_ENABLE_ZLIB The HAVE_LIBZ variable is not exported by LLVM, and therefore is not available in stand-alone builds of other tools. Use LLVM_ENABLE_ZLIB which is the name under which the effective value is exported. Additional, use llvm_canonicalize_cmake_booleans() to make sure that a correct (Python-safe) boolean value is passed down to lit. Differential Revision: https://reviews.llvm.org/D41725 ------------------------------------------------------------------------ git-svn-id: https://llvm.org/svn/llvm-project/lldb/branches/release_60@322833 91177308-0d34-0410-b5e6-96231b3b80d8
1 parent 3e3e524 commit 19e7bfe

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

lit/CMakeLists.txt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,10 @@ else()
1111
set(ENABLE_SHARED 0)
1212
endif(BUILD_SHARED_LIBS)
1313

14+
# the value is not canonicalized within LLVM
15+
llvm_canonicalize_cmake_booleans(
16+
LLVM_ENABLE_ZLIB)
17+
1418
configure_lit_site_cfg(
1519
${CMAKE_CURRENT_SOURCE_DIR}/lit.site.cfg.in
1620
${CMAKE_CURRENT_BINARY_DIR}/lit.site.cfg)

lit/lit.site.cfg.in

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ config.target_triple = "@TARGET_TRIPLE@"
1212
config.python_executable = "@PYTHON_EXECUTABLE@"
1313
config.cc = "@LLDB_TEST_C_COMPILER@"
1414
config.cxx = "@LLDB_TEST_CXX_COMPILER@"
15-
config.have_zlib = @HAVE_LIBZ@
15+
config.have_zlib = @LLVM_ENABLE_ZLIB@
1616

1717
# Support substitution of the tools and libs dirs with user parameters. This is
1818
# used when we can't determine the tool dir at configuration time.

0 commit comments

Comments
 (0)