-
-
Notifications
You must be signed in to change notification settings - Fork 210
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
Added installation steps for android 9 #187
base: master
Are you sure you want to change the base?
Conversation
This worked for me! Please 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.
This steps works fine for android 9
Would it be possible if you posted a detailed step by step of how did you made it work? did you have to build any apk from source? I would like to try it on Android 10 (LineageOS 17.1) |
this app doesn't have to be in priv-app or platform signature so allowing this perms isn't needed at all |
@erfanoabdi can you please elaborate this a bit more. To my understanding, it has to be in priv-app, otherwise the app is unable to register it self as a system-wide location provider.. |
yeah, just placed it on /system/app/ and everything works fine (all checks passed) |
README.md
Outdated
|
||
1. Make sure no gapps are installed | ||
2. Download `NetworkLocation.apk` | ||
3. Boot into recovery and copy the apk to `/system/priv-app/` using the adb-command `adb push NetworkLocation.apk /system/priv-app/UnifiedNLP/NetwortkLocation.apk` (also make sure you mount the `/system` partition in advance) |
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.
I'm not sure if it really matters for the functionality, but it'd be nice to update NetwortkLocation.apk
to be spelled correctly
Otherwise, it'd be great to see this merged - if this was already in the README a few months ago I wouldn't have had to embarrass myself a few times by not having working GPS on my phone 😅
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.
I'm not sure if it really matters for the functionality[...]
Well, at least in my case, the app didn't registered itself as system-wide location provider (Android 9, LineageOS) when installed as a "normal" app.
but it'd be nice to update NetwortkLocation.apk to be spelled correctly.
Done, thanks 👍
Otherwise, it'd be great to see this merged
Actually I don't know what @mar-v-in 's plan is here - there seems to be active development though...
Help for Android 12 installation. |
Resolves #181
Thanks to @pedrxd, @k4ps, @LadyBoonami and @bboyDmT