Unity-Technologies / EditorXR

Author XR in XR
Other
925 stars 167 forks source link

Staging 0.3 prep #557

Closed amirebrahimi closed 4 years ago

amirebrahimi commented 4 years ago

Purpose of this PR

Fix compilation on 2017.4, so it at least shows download dialog box Re-serialize assets for 2018.4, since that is the new min version Update asmdef patch for pre-2019.1

Testing status

Manual testing on 2017.4 and 2018.4

Technical / Halo risk

[Overall product level assessment of the risk of the changes using the following criteria:

Tech Risk (not Tech Correctness, which is deferred to the reviewers)

Halo Risk (Externality risk)

Comments to reviewers

mtschoen-unity commented 4 years ago

Thanks for updating the 2019_1_OR_NEWER stuff. I agree that ENABLE_EDITORXR is the way to go.

I'm finally getting around to testing this and having some trouble importing into an empty project. I'm still trying to figure out the best path forward, but after a clean import I get a bunch of null refs trying to start EditorXR. I'm guessing that it has to do with default references not getting set up properly if the first compile fails.

I also got the version warning dialog when updating an existing checkout to this branch and opening it in 2018.4. I'm not sure what that is about, but that is what prompted me to try a clean checkout.

We can probably get rid of the TextMeshPro asmdef in the unitypackage patch thing, since 2018.4 includes the TMP package by default.

amirebrahimi commented 4 years ago

TMP asmdef is to allow it to compile in versions prior to 2018.4 only to report that you should download 2018.4 :)