Background location permission on Android 10

[leso-kn]

Android 10 introduced a new 'background location' permission.

As mentioned here UnifiedNlp needs to request this permission in order to function propperly.

Being a very simple fix i would love to see it in Microg Installer Revived :)

---

To fix it add

<uses-permission android:name="android.permission.ACCESS_BACKGROUND_LOCATION"/>

to the AndroidManifest.xml of GmsCore.

There's also a open pull request on microg that does exactly that.

[nift4]

Hi. Thanks for using my module :)
Sadly I don't understand what do you want. microG Installer Revived uses the offical prebuilt APKs and I can't simply change the AndroidManifest of that apk. Is your suggestion to include an APK of an fork with it fixed?

[leso-kn]

Hi! :)
I see. Sorry, I was not sure if the official or a custom apk is bundled with the package.

I've read that the developer of microG recently stated he would pick up the work on it soon. But that's already some time ago again.

If including an apk of a fork is an option I can check if I find one that works. Since there's no updates to the official apk right now it might be worth including a fixed version for the time being. What do you think?

[nift4]

I agree. When you find an fork with it fixed and an apk I change it. I would be happier if you change it via a pull request through, you only need to swapout apk. I already added the permission in the other (module) xml as it's needed anyway

[nift4]

Nice, I wait for the apk or pull request :) (In meantime I work on Play Store)

[nift4]

Hi again; would it be for you possible to test something? (#10)

[leso-kn]

Well, I decided to use the fork of @eplinux as it already contains the fixes for both GmsCore and UnifiedNlp.
After taking a break of some time I am currently fighting with serval docker setups in order to compile the repo; the gradle files seem to be a bit outdated.

Once that is running it should be no problem to test it :)

[leso-kn]

I already thought about just de- and recompiling the official apk with apktool.
But this would be a rather dirty solution. I might re-consider it for testing in case fixing the gradle stuff takes very long.

[nift4]

From module site this is fixed and UnifiedNlp 2.0 is worked on, so ig I can close this