Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added option to install resource folder #1419

Merged
merged 3 commits into from
Jul 23, 2023

Conversation

whaiao
Copy link

@whaiao whaiao commented Jul 18, 2023

Fixes #1167

Added an option in the CMake file which toggles the installation of resources if the renderer is built and the user wants to install the resources.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jul 18, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: benjaminbeilharz / name: Benjamin Beilharz (bfa0be5)
  • ✅ login: jstone-lucasfilm / name: Jonathan Stone (6dedbe8, 34b39fc)

Signed-off-by: Jonathan Stone <[email protected]>
Signed-off-by: Jonathan Stone <[email protected]>
Copy link
Member

@jstone-lucasfilm jstone-lucasfilm left a comment

Choose a reason for hiding this comment

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

This looks good to me, thanks @benjaminbeilharz!

@jstone-lucasfilm jstone-lucasfilm merged commit 42e8b76 into AcademySoftwareFoundation:main Jul 23, 2023
13 checks passed
Michaelredaa pushed a commit to Michaelredaa/MaterialX that referenced this pull request Oct 21, 2023
Added an option in the CMake file which toggles the installation of resources if the renderer is built and the user wants to install the resources.
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.

Installation and requirement of 'resources' directory
2 participants