Skip to content

Commit 5db41f3

Browse files
committed
Fix the USD build
- cmake syntax errors - mjUSEUSD definition in main and cmake file - change BUILD_WITH_USD to MUJOCO_WITH_USD
1 parent f703a0e commit 5db41f3

4 files changed

Lines changed: 8 additions & 9 deletions

File tree

CMakeLists.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -226,6 +226,8 @@ endif()
226226
if(MUJOCO_WITH_USD)
227227
include(third_party_deps/openusd)
228228

229+
target_compile_definitions(mujoco PUBLIC mjUSEUSD)
230+
229231
get_filename_component(USD_INSTALL_ROOT "${pxr_DIR}/lib" ABSOLUTE)
230232
install(
231233
DIRECTORY ${USD_INSTALL_ROOT}/

simulate/CMakeLists.txt

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -185,18 +185,14 @@ if(SIMULATE_BUILD_EXECUTABLE)
185185
target_link_libraries(simulate "-framework CoreVideo")
186186
endif()
187187

188-
if (BUILD_WITH_USD)
188+
if (MUJOCO_WITH_USD)
189189
# pxr/usd/tf/hashset.h might include deprecated headers so disable this check
190190
# when compiling with USD enabled.
191191
target_compile_options(
192192
simulate
193193
PRIVATE -Wno-deprecated
194194
)
195195

196-
target_link_libraries(
197-
simulate
198-
mujoco::usd::mjcf
199-
)
200196
endif()
201197

202198
target_link_options(simulate PRIVATE ${MUJOCO_SIMULATE_LINK_OPTIONS})
@@ -265,7 +261,7 @@ if(SIMULATE_BUILD_EXECUTABLE)
265261
MUJOCO_ENABLE_RPATH
266262
)
267263

268-
if (BUILD_WITH_USD)
264+
if (MUJOCO_WITH_USD)
269265
# pxr/usd/tf/hashset.h might include deprecated headers so disable this check
270266
# when compiling with USD enabled.
271267
target_compile_options(

src/experimental/usd/CMakeLists.txt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -170,15 +170,15 @@ if(APPLE)
170170
BUILD_RPATH "${USD_INSTALL_ROOT}"
171171
INSTALL_RPATH "@loader_path")
172172

173-
set_target_properties($(MJC_PHYSICS_PLUGIN_TARGET_NAME) PROPERTIES
173+
set_target_properties(${MJC_PHYSICS_PLUGIN_TARGET_NAME} PROPERTIES
174174
BUILD_RPATH "${USD_INSTALL_ROOT}"
175175
INSTALL_RPATH "@loader_path")
176176
elseif(UNIX)
177-
set_target_properties($(MJCF_PLUGIN_TARGET_NAME) PROPERTIES
177+
set_target_properties(${MJCF_PLUGIN_TARGET_NAME} PROPERTIES
178178
BUILD_RPATH "${USD_INSTALL_ROOT}"
179179
INSTALL_RPATH "$ORIGIN")
180180

181-
set_target_properties($(MJC_PHYSICS_PLUGIN_TARGET_NAME) PROPERTIES
181+
set_target_properties(${MJC_PHYSICS_PLUGIN_TARGET_NAME} PROPERTIES
182182
BUILD_RPATH "${USD_INSTALL_ROOT}"
183183
INSTALL_RPATH "$ORIGIN")
184184
endif()

usd-exchange

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Subproject commit f1c484f4095e99e5fab2e2a26746be52e81b3474

0 commit comments

Comments
 (0)