Revert "fix: preserve debuggability even in minimized release mode" #17266
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.
Purpose / Description
This reverts commit aa271f9.
-dontobfuscate
appears to cause problems, and there were crashes in 2.19beta4-6 that appear to be related to itIt should work, and the failure has not been characterized yet
Fixes
Approach
A lot of detailed investigation and reading on my part, but really it was just a bisect and then a one-line change on an alternate release pathway (#17259) that allowed me to zero in on the
-dontobfuscate
change to proguard configHow Has This Been Tested?
Diff indicated that proguard was only thing between beta3 and beta4 that should have caused this, and disabling it worked for user. So I assume that reverting that change will then work.
There is a slight chance it will not work.
Learning (optional, can help others)
-dontobfuscate
must have some interaction between app code and library code but I couldn't pin it downChecklist
Please, go through these checks before submitting the PR.