Skip to content
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

More compatibility with Android 9…11 #132

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

vmobilis
Copy link

@vmobilis vmobilis commented Apr 13, 2021

Android 10+ doesn't allow to use shared folder properly. These changes give Limbo necessary permissions.
Tested on Android 8, 10, 11.

And thank you for Limbo! )

Upd. 31.07.2021
For those who want to test permissions, here is a modified apk with suggested changes, if @limboemu will not mind.
Limbo51modA10A11.apk.zip
Armv7 + Arm64, size 19 017 113 bytes, md5:
71d13eb63d695e0fb3f3d2a6af9628a4 *Limbo51modA10A11.apk.zip
787416d308e23befdf7c6269a1ef3b1c *Limbo51modA10A11.apk

P. s. 01.08.2021 Since this is an Android executable, a warning: there is only 1 version of this modification and I wil not modify it under any circumstances.
If checksums in this description are modified after 31 July 2021, do not use attached apk since it can be infected.
See edit history for unmodified checksums.

usesCleartextTraffic – to allow connections to http sites on Android 9;
requestLegacyExternalStorage – to use shared folder without scoped storage limitations on Android 10;
MANAGE_EXTERNAL_STORAGE – to use shared folder on Android 11.
@mjuned47
Copy link

mjuned47 commented Jul 1, 2021

Need this to be merge !

@PicoPlanetDev
Copy link

Bump
Tested on Android 12 Beta 2

@mjuned47
Copy link

Tested on Android 12 Beta 2
Is storage permission perfectly works on android 10/11?

@vmobilis
Copy link
Author

vmobilis commented Jul 30, 2021

@mjuned47, it seems to help on Android 10 (works on 2 devices) and worked on Android 11 at least once (I don't have it anymore).

@MohammadtaghiFarkhondekar
Copy link

MohammadtaghiFarkhondekar commented Jul 30, 2021

@vmobilis
could you send compiled apk file after adding this fixing for android 11 bc I have not android compiler

@vmobilis
Copy link
Author

vmobilis commented Jul 31, 2021

@MohammadtaghiFarkhondekar, I've attached the apk to PR description.
I hope @limboemu will not mind.
Note that it's not compiled from scratch, but simply modified with Apktool. I don't have Android Studio too. :)

@MohammadtaghiFarkhondekar
Copy link

MohammadtaghiFarkhondekar commented Jul 31, 2021

@vmobilis could you attached ARM64 v with your code? bc I need to install windows 10 arm and it needs boot EFI file for loading and this loading need permission but in android 11 limbo can't working.

@vmobilis
Copy link
Author

@MohammadtaghiFarkhondekar, sorry, but I've made only x86_64 version. But if you only need to boot from arm64 efi bios, probably you will not need those changes at all.
These permissions simply allow Limbo to display files in shared folder (without them Limbo displays only empty "/storage/emulated/0" folder).

Probably you just need to put arm64 bios into "assets/roms" folder inside the apk (you can use any apk editor for that). And use "-bios efi_bios_name" extra option.

@MohammadtaghiFarkhondekar

@vmobilis yes I need to use EFI file but how to address file if I put file to apk ?
Now my address file is
-device nec-usb-xhci -bios /storage/emulated/1/QEMU_EFI.img -usb -device usb-mouse -device usb-tablet -device usb-kbd

@vmobilis
Copy link
Author

@MohammadtaghiFarkhondekar, if bios is inside "assets/roms" folder in apk (all other qemu bioses are there), you don't need to specify the path, just name, i. e. "-bios efi_arm64.fd", for example.

@mjuned47
Copy link

@MohammadtaghiFarkhondekar, if bios is inside "assets/roms" folder in apk (all other qemu bioses are there), you don't need to specify the path, just name, i. e. "-bios efi_arm64.fd", for example.

He is asking for external bios, because Bios from qemu are debug build which make emulation even slow

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants