-
Notifications
You must be signed in to change notification settings - Fork 154
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
Support batchmode with "quit" flag on #29
Open
VMaldecoasago
wants to merge
4
commits into
apple:main
Choose a base branch
from
SagoSago:fix/load-default-profile-after-importing-assets
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Support batchmode with "quit" flag on #29
VMaldecoasago
wants to merge
4
commits into
apple:main
from
SagoSago:fix/load-default-profile-after-importing-assets
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…y are going to be required
VMaldecoasago
changed the title
Fix creating assets during asset import
Support batchmode
Jul 5, 2024
VMaldecoasago
force-pushed
the
fix/load-default-profile-after-importing-assets
branch
from
July 8, 2024 21:36
53f2403
to
06244e1
Compare
VMaldecoasago
added a commit
to SagoSago/apple-unityplugins
that referenced
this pull request
Jul 8, 2024
DodgyTim
pushed a commit
to dodgy-ltd/apple-unityplugins
that referenced
this pull request
Jul 14, 2024
Really helpful, thank you! |
VMaldecoasago
added a commit
to SagoSago/apple-unityplugins
that referenced
this pull request
Nov 27, 2024
Up @AdamSzApple |
Thanks for raising this issue. I'm not allowed to make any forward-looking statements about fixes or future updates, but we're aware of the problem. Thank you. 😉 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
Currently, compiling a game with Apple.Core while in batch mode results in native libraries not being found. That occurs due to the following issues:
1. Creating assets during InitializeOnLoad
According to this Unity issue, assets cannot be created on InitializeOnLoad, because the asset importing is not yet completed at that stage. This was causing a UnityException when building my game due to Apple.Core not being able to load the Default Profile asset, even though it is present in the repository. All the assets creation and retrival were moved to OnPostprocessAllAssets, which, according to the above mentioned Unity issue, is also called after domain reload, but only after all assets have been imported.
2. Using asynchronous coroutines while using
-quit
argumentApplePlugInEnvironment
relies on EditorUpdate to wait for the PackageManager to complete, which will not work on -batchmode -quit. In this case, we have to call EditorUpdate manually