PabbleDabble commented on PR #262:
URL:
https://github.com/apache/cordova-plugin-media-capture/pull/262#issuecomment-1650579944
Anyone else getting Android manifest merger errors with this? Our app uses
a bunch of different plugins and it looks like these two items are being placed
in the manifest from other plugins...
```
<uses-permission
android:name="android.permission.WRITE_EXTERNAL_STORAGE" />
<uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE"
/>
```
...and these these items appear to be coming from this
(cordova-plugin-media-capture)
```
<uses-permission android:maxSdkVersion="32"
android:name="android.permission.READ_EXTERNAL_STORAGE" />
<uses-permission android:maxSdkVersion="32"
android:name="android.permission.WRITE_EXTERNAL_STORAGE" />
```
It looked like I might be able to do something in my app's config.xml in
order to replace them (
https://stackoverflow.com/questions/25981156/tools-replace-not-replacing-in-android-manifest
) but I wasn't fully understanding it / created more errors in Android studio.
Apologies if I'm off base, I'm not blaming this change, this looks like the
better way to specify it, just not sure if there's anything else that needs
done? Or maybe it's something on my end and need to find out what other
plugins might be adding this and get them upgraded as well?
Sorry if this is off topic.
--
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]
