Skip to content

Commit

Permalink
Remove MaterialXDocs target
Browse files Browse the repository at this point in the history
Following the pattern of removing the MATERIALX_BUILD_DOCS option, I think it makes the most sense to remove the MaterialXDocs target rather than deprecating it.

Signed-off-by: Jonathan Stone <[email protected]>
  • Loading branch information
jstone-lucasfilm authored Oct 21, 2024
1 parent 8e7071e commit 8391d2f
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions documents/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -41,12 +41,5 @@ add_custom_command(OUTPUT ${DOXYGEN_HTML_OUTPUT_DIR}/index.html
COMMENT "Building MaterialX C++ API Documentation in ${DOXYGEN_HTML_OUTPUT_DIR}/ ...")
add_custom_target(MaterialXDocsCpp ALL DEPENDS ${DOXYGEN_HTML_OUTPUT_DIR}/index.html)

# Display a deprecation warning if the old name of the target is used, but
# build the target using its new name instead, by making MaterialXDocs depend
# on MaterialXDocsCpp
add_custom_target(MaterialXDocs
DEPENDS MaterialXDocsCpp
COMMENT "WARNING: The MaterialXDocs target name is deprecated. The MaterialXDocsCpp target was built instead.")

install(DIRECTORY ${DOXYGEN_HTML_OUTPUT_DIR}
DESTINATION "documents" MESSAGE_NEVER)

0 comments on commit 8391d2f

Please sign in to comment.