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

add version defines so that UniGLTF, UnityGLTF and glTFast can co-exist #2461

Merged

Conversation

hybridherbst
Copy link
Contributor

@hybridherbst hybridherbst commented Oct 15, 2024

Hi, I'm the maintainer of https://github.com/khronosgroup/unitygltf. We've got some reports of incompatibility when both UnityGLTF and UniGLTF are in the same project, leading to users believing that either project is broken while they're not.

This PR builds on

With this PR in place, all three can co-exist and will all be available as importer overrides.

Before this PR, users had to manually add UNIGLTF_DISABLE_DEFAULT_GLB_IMPORTER and UNIGLTF_DISABLE_DEFAULT_GLTF_IMPORTER to their scripting settings which was confusing – with this PR it's automatic. And if neither glTFast nor UnityGLTF are present, there is no change in behaviour either.

Copy link
Contributor

@ousttrue ousttrue left a comment

Choose a reason for hiding this comment

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

👍

@ousttrue ousttrue added editor Package unitypackage, download, import, UPM Non breaking changes labels Oct 16, 2024
@ousttrue ousttrue added this to the next SpringBone export bake milestone Oct 16, 2024
@ousttrue ousttrue merged commit 49dd34f into vrm-c:master Oct 16, 2024
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editor Non breaking changes Package unitypackage, download, import, UPM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants