-
Notifications
You must be signed in to change notification settings - Fork 4.9k
Add android14 support in lrs android wrapper example RS Camera #13910
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 android14 support in lrs android wrapper example RS Camera #13910
Conversation
Can one of the admins verify this patch? |
By ading #13718 changes the build.gradle(LRS) is now
Due lack of time during the weekend I will aprove the patch early next week. Thanks. |
Hi @nanotuxi |
@Nir-Az Thanks for the quick response. The patch also works without using pr 13718. The (dirty) workaround would be to set defaultTargetSdkVersion = 33. So, should we wait for pr 13718 to be rebased by the user having created it and being merged into the development branch and use it later to adapt the TargetSdk versions to the most recent stable versions? My own PR will be rebased by me on dev branch next week. Thank you. |
Sorry I thought that PR is also yours. |
44a2569
to
79b1d0a
Compare
@Nir-Az Hopefully it works for android 15. The only device which I can test is based on android 14. The update should also resolve PR 13718 as this PR is based on dev branch. |
Thanks, let's pend @mengyui review and then we can proceed |
Oh, I'm not in a hurry 😌 |
@nanotuxi I'm sorry for late.
|
Oh, that's quite o.k.
Using Android Studio "Meerkat" (2024.3.1) the automatic update mechanism of this recent version suggested to update to AGP 8.9.1 That is the only reason for the update and you are totally right to ask for the reason. In my personal experience it is sometimes useful to update to a more recent version of a software package or framework to make it more future proof and profit from a lot of improved methods and bug-cleaned code. If you do not wish to change to proceed to AGP 8.9.1 we can step back to 8.1.4.
Yes. That makes sense. As I did not have physical access to android 15 devices at the moment of writing the PR the versions are different. I will make the change to identical versions of compile/targetSDKs.
Android Studio showed an error in one of the wrapper project files that could only be solved updating to defaultMinSdkVersion from 23 to 24. That can be reverted if you want to revert to older software versions. Iam aware of the fact that the range of Android versions should be as wide as possible. But is it really neccessary to provide support for Android 6 in 2025? But if you decide to step back, we will do that. |
|
@mengyui when you approve the PR after the changes I will merge. |
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.
Thank you for your contribution.
Adding support for android 14 to example tools/camera of the librealsense wrapper for android