-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Fixed GLTF build errors #1141
Fixed GLTF build errors #1141
Conversation
@HattMarris1 can you test and approve this for me please. Thanks. |
Will do |
@HattMarris1 do you approve? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
@@ -38,18 +38,16 @@ public class MotionControllerVisualizer : MonoBehaviour | |||
|
|||
[Tooltip("This material will be used on the loaded glTF controller model. This does not affect the above overrides.")] | |||
[SerializeField] | |||
protected UnityEngine.Material GLTFMaterial; | |||
protected Material GLTFMaterial; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will break 2017.2 building when this change is merged back into the dev branch (or when master i updated to work with 2017.2) because there's a Material in the GLTF namespace as well. Likely, renaming it to GLTFMaterial is the best fix.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll look into it and double check. Thanks for the heads up!
Fixes #1139