Skip to content

Export MATERIALX_INSTALL_STDLIB_PATH in CMake config - #2724

Merged
jstone-lucasfilm merged 1 commit into
AcademySoftwareFoundation:mainfrom
ld-kerley:cmake/fix-cmake-config-export-for-data-libraries
Dec 13, 2025
Merged

Export MATERIALX_INSTALL_STDLIB_PATH in CMake config#2724
jstone-lucasfilm merged 1 commit into
AcademySoftwareFoundation:mainfrom
ld-kerley:cmake/fix-cmake-config-export-for-data-libraries

Conversation

@ld-kerley

Copy link
Copy Markdown
Contributor

MATERIALX_INSTALL_STDLIB_PATH can be used to control where the data library is installed, but this needs to be reflected in the exported cmake config file as well.

@jfpanisset reported this in this slack conversation.

…ibrary is installed, but this needs to be reflected in the exported cmake config file as well.
@ld-kerley
ld-kerley requested a review from jfpanisset December 12, 2025 21:56

@jfpanisset jfpanisset left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One less local patch to apply when building MaterialX for aswf-docker, thank you!

@jstone-lucasfilm jstone-lucasfilm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, good catch, @ld-kerley and @jfpanisset, and this seems like a clear improvement.

@jstone-lucasfilm jstone-lucasfilm changed the title Export MATERIALX_INSTALL_STDLIB_PATH in cmake config Export MATERIALX_INSTALL_STDLIB_PATH in CMake config Dec 13, 2025
@jstone-lucasfilm
jstone-lucasfilm merged commit 973f986 into AcademySoftwareFoundation:main Dec 13, 2025
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants