core: change Android tmp dir, make it configurable #2605
Conversation
When Android tries to access the default `/tmp` temporary directory it crashes due to missing permissions. As a workaround, let's try to use `/data/local/tmp` instead, and if that doesn't work, make it configurable within JNI for the user.
|
There was a problem hiding this comment.
Are you sure that /data/local/tmp works at all? On newer versions of Android I would bet that it doesn't. Security has been tightened a lot, now we need to explicitly ask permission to the user in order to access files outside the app storage.
Without asking permission, it's possible to use the app storage, but the path depends on the app, so it should be configurable.
The title says "make it configurable". Does it do that? I don't see how it can be configured 🤔 😅
|
@JonasVautherin should be configurable as mentioned in the PR description ⬆️, at link time. |
|
Going to merge it and then try it out. |
When Android tries to access the default
/tmptemporary directory it crashes due to missing permissions.As a workaround, let's try to use
/data/local/tmpinstead, and if that doesn't work, make it configurable within JNI for the user.Android apps can now optionally override the temp path from JNI like:
Hopefully fixes mavlink/MAVSDK-Java#206.