-
Notifications
You must be signed in to change notification settings - Fork 53
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 improved support for mobile platforms #7
Conversation
Mobile support v1
Add mobile support (WIP)
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 Vaidas for adding these features.
I only checked that there were no regressions regarding the older features, but I had an issue after trying the following steps:
- I created a new project with Editor version 2020.2.0a7.826 (trunk revision cea774d83c47), that did not have mobile support installed
- I placed a cube with a custom unlit shader, and built the scene using the script from https://docs.unity3d.com/2020.1/Documentation/ScriptReference/BuildOptions.DetailedBuildReport.html
- I added to the project BuildReportInspector revision c5164bb (I installed it from my disk, where I had checked out the revision)
- I clicked Window/Open Last Build Report
- I got the following error displayed 999+ times in the console:
The appendix file does not exist at C:\unity-projects-current\wil-projects\check-build-report-mobile-pr\Assets/BuildReports/Mobile\b9761b87b7495474dbd4e42ef065a9cd. UnityEngine.Debug:LogError (object) Unity.BuildReportInspector.Mobile.MobileHelper:LoadMobileAppendix (string) (at C:/source-code/from-github-unity/BuildReportInspector/com.unity.build-report-inspector/Editor/Mobile/MobileHelper.cs:85) Unity.BuildReportInspector.BuildReportInspector:get_mobileAppendix () (at C:/source-code/from-github-unity/BuildReportInspector/com.unity.build-report-inspector/Editor/BuildReportInspector.cs:47) Unity.BuildReportInspector.BuildReportInspector:OnInspectorGUI () (at C:/source-code/from-github-unity/BuildReportInspector/com.unity.build-report-inspector/Editor/BuildReportInspector.cs:176) UnityEngine.GUIUtility:ProcessEvent (int,intptr,bool&) (at C:/unity-pr2/Modules/IMGUI/GUIUtility.cs:189)
Can you please add something to avoid such issues?
Thanks a lot also for having added tests, that's very helpful.
Fixed this issue, thanks for taking your time to test it. |
Thanks for all the fixes, to me the pull request looks good to merge :) |
Merging. Will request additional QA testing on master branch. |
Purpose of this PR:
This PR extends the functionality of the BuildReportInspector package by adding improved support for Android and iOS platforms:
Release Notes:
Add improved support for iOS and Android.
Fix #1 - incomplete Assets list shown when there are many entries.
Fix .NET 3.5 support.
Testing status:
Manual testing - in progress (internal QA, ask @vaidas-unity for document link).
Automated tests - TBA.
Overall Product Risks:
Technical Risk: 2
Halo Effect: 2