breautek commented on PR #1538: URL: https://github.com/apache/cordova-android/pull/1538#issuecomment-1462136053
We generally discourage people from updating the build tools version unless absolutely necessary/know what they are doing so we probably should use the latest patch available, so that cordova by default will use the latest patch available (at the time of this PR) obviously. 33.0.1 resolves a compilation issue: > AIDL fails with build-tools 33.0.0. (See [issue #236167971](https://issuetracker.google.com/issues/236167971)) It probably doesn't affect the cordova base but could affect other native libraries or third-party plugins. 33.0.2 addresses what appears to be a minor issue: > core-lambda-stubs.jar version 33.0.0 differs on macOS versus Linux/Windows. (See [issue #237299698](https://issuetracker.google.com/issues/237299698).) Nonetheless, I think we should just use the latest patch available, so that on release we can cleanly write an update guide stating they must have `33.0.2` installed, etc. It may save us having to explain that they need to set a preference to upgrade build tools to address `AIDL` issue if that issue affects them. Just my own opinion. If they absolutely need to use 33.0.0 for some reason, I believe they can still set the `android-buildToolsVersion` themselves. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
