Skip to content

Commit 92d9b14

Browse files
jeongseok-metameta-codesync[bot]
authored andcommitted
Avoid requiring TBB when not building io_usd (#845)
Summary: TBB is a transitive dependency of USD, so it should not be required when USD is not being built. This issue was mistakenly not fixed by #843. ## Checklist: - [x] Adheres to the [style guidelines](https://facebookresearch.github.io/momentum/docs/developer_guide/style_guide) - [x] Codebase formatted by running `pixi run lint` Pull Request resolved: #845 Test Plan: ``` pixi run test ``` Reviewed By: fbogo Differential Revision: D87385804 Pulled By: jeongseok-meta fbshipit-source-id: a547b96272b568e78293fe86d6b2cd3960eb4fc9
1 parent 6720cd9 commit 92d9b14

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,10 +120,10 @@ find_package(nlohmann_json CONFIG REQUIRED)
120120
find_package(openfbx CONFIG REQUIRED)
121121
if(MOMENTUM_BUILD_IO_USD)
122122
find_package(pxr CONFIG REQUIRED HINTS ${CMAKE_INSTALL_PREFIX} ${CMAKE_INSTALL_PREFIX}/cmake)
123+
find_package(TBB CONFIG REQUIRED)
123124
endif()
124125
find_package(re2 MODULE REQUIRED)
125126
find_package(spdlog CONFIG REQUIRED)
126-
find_package(TBB CONFIG REQUIRED)
127127
find_package(urdfdom CONFIG REQUIRED)
128128

129129
if(MOMENTUM_BUILD_RENDERER)

0 commit comments

Comments
 (0)