-
-
Notifications
You must be signed in to change notification settings - Fork 251
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
Fix iOS Fastlane build when using Unity Cloud Diagnostics #383
Conversation
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.
Not sure about these changes.
-username "$UNITY_EMAIL" \ | ||
-password "$UNITY_PASSWORD" \ |
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.
These are already passed during the activate step. Any reason why we'd need to pass them here too?
The description is not clear enough as to what exactly it fixes under which circumstances. We currently have successful builds in iOS. What would be interesting to know is how |
The post I'm referring to is the first post in the thread.
The solution from the thread is to add the username and password arguments when calling Unity through batchmode. |
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.
Thanks for the additional context.
Since for MacOS the UNITY_EMAIL
and UNITY_PASSWORD
are always passed we don't need conditionals for whether or not to pass them to the CLI.
Therefor I think we're safe to merge this.
By the way can you confirm this solution works for you? You can try by using uses: game-ci/unity-builder@b081e3320cb00d466b3c95c649a31e7da227cf29
with:
your: vars |
Sure thing, I'll let you know when the build is done. |
Looks like it worked. 🙂 |
Fixes error described here: https://forum.unity.com/threads/ios-build-is-failing-seems-like-a-fastlane-problem-not-sure-how-to-proceed.682201/
Changes
Checklist