breautek commented on issue #871: URL: https://github.com/apache/cordova-plugin-camera/issues/871#issuecomment-1815494652
There's likely nothing in the camera plugin specifically (although wouldn't be tested) that requires API 24+ but the camera plugin requires cordova-android@12 since that's the version that sets the compile SDK to API 33, which is needed to reference the new API 33 permissions. > Is it too complex to apply that cordova-android@12 code conditionally by means of simply checking the current SDK level? Doing this requires using reflection to access bits needed at compile time, and reflection introduces harder to maintain code, so yes. cordova-android@12 platform introduces changes that requires API 24. If you have a specific need to support API 21+ devices, then you can likely fork cordova-android@11 and this plugin and make the necessary modifications to target and compile with API 33 (and remove the cordova-android@12 requirement on the camera plugin). Might be necessary to review the commit log to bring in important things for API 33 support for the fork. I'm closing this issue because I don't foresee any action to be taken by Apache. Should you go with the fork route and have further questions, [slack community members](https://cordova.slack.com/) ([signup](https://s.apache.org/cordova-slack)) might be able to provide insight. -- 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]
