Add SLANG_INCLUDE_TOOLS cmake option to allow disabling tools targets

MikePopoloski · Aug 4, 2022 · 3329c1a · 3329c1a
1 parent 5ddd06d
commit 3329c1a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -7,6 +7,7 @@ project(slang VERSION 2.0.0 LANGUAGES CXX
 option(SLANG_COVERAGE "Enable code coverage" OFF)
 option(CI_BUILD "Enable longer running tests for CI builds" OFF)
 option(FUZZ_TARGET "Enables changes to make binaries easier to fuzz test" OFF)
+option(SLANG_INCLUDE_TOOLS "Include tools targets in the build" ON)
 option(SLANG_INCLUDE_TESTS "Include test targets in the build" ON)
 option(SLANG_INCLUDE_DOCS "Include documentation targets in the build" OFF)
 option(SLANG_INCLUDE_LLVM "Include LLVM in the build for code generation" OFF)
@@ -211,7 +212,10 @@ if(NOT SLANG_USE_SYSTEM_LIBS)
 endif()
 
 add_subdirectory(source)
-add_subdirectory(tools)
+
+if(SLANG_INCLUDE_TOOLS)
+    add_subdirectory(tools)
+endif()
 
 if(SLANG_INCLUDE_TESTS)
     include(CTest)

diff --git a/pyslang/setup.py b/pyslang/setup.py
@@ -12,8 +12,8 @@
     cmake_source_dir='../',
     cmake_install_dir="source/pyslang",
     cmake_install_target="pyslang-install-pylib",
-    cmake_args=['-DSLANG_INCLUDE_TESTS=OFF', '-DSLANG_INCLUDE_PYLIB=ON'],
+    cmake_args=['-DSLANG_INCLUDE_TESTS=OFF', '-DSLANG_INCLUDE_TOOLS=OFF', '-DSLANG_INCLUDE_PYLIB=ON'],
     include_package_data=True,
     extras_require={"test": ["pytest"]},
     python_requires=">=3.6",
-)
+)