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
[TIMOB-26079] Continue build on invalid Android library ABI #10084
Conversation
Generated by 🚫 dangerJS |
As the comment describes, we use the |
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.
CR'd and looks good.
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.
LGTM (except that straggler debug log to be removed)
android/cli/commands/_build.js
Outdated
store: store | ||
}); | ||
entryNames.push(entry.entryName); | ||
this.logger.debug('test3'); |
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.
Remove this straggler debug log, please
Generated by 🚫 dangerJS |
JIRA: https://jira.appcelerator.org/browse/TIMOB-26079
Optional Description:
This aligns the handling of invalid ABIs for native libraries from Android .aar files to be the same as in the other stages of creating the APK archive.
@cb1kenobi I had to remove the override for
console.error
because that caused all ourdebug
andwarn
messages to not show up. I'm sure you had your reason to temporarily override it. Do you remember why and if there is a better way to do this nowadays? The comment said something about to many event listeners but i couldn't see anything when testing this.