QuestCraftPlusPlus / PojavLauncher

A Minecraft: Java Edition Launcher for Android based on Boardwalk. Fork built specifically to run on the Meta Quest platform.
GNU General Public License v3.0
52 stars 36 forks source link

[BUG] Crash when trying to add new JVM #16

Closed hellidox closed 2 years ago

hellidox commented 2 years ago

Describe the bug

Pressing "New VM" crashes the app, something to do with the alert handler

The log file and images/videos

logs can't save, no storage perms

Steps To Reproduce

Start Pojav-Settings-Java Tweaks-Runtime manager-Add New-crash

Expected Behavior

I expect to be able to add a JRE

Platform

- Device model: Quest 2
- CPU architecture: aarch64
- Android version: Quest 2 latest
- PojavLauncher version: Latest build (gh-actions)

Anything else?

No response

Flamgop commented 2 years ago

These builds contain the JVM 17, but this bug is known. We are planning on fixing this in the future.

ghost commented 2 years ago

I'm also getting what I think is a similar error, and it seems to be caused by the Launcher not having access to my files. (Sorry for Google Photos links, it was the fastest way) https://photos.app.goo.gl/zuELfCmtxzWu3vtZ8

hellidox commented 2 years ago

@GiraffeMan3125 Yes, that's the same error. Strangely, pojav doesn't request storage permissions, and trying to start a version (1.18.1) says it can't find a compatible version of Java.

Sinnn3000 commented 2 years ago

Sideload zarchiver and give it storage permissions. this is because pojav doesnt have access to the file browser built into quest so u have to use a separate one.

it worked for me let me know how it goes for you.

hellidox commented 2 years ago

Give what storage perms? Zarchiver or pojav?

Sinnn3000 commented 2 years ago

zarchiver, sorry for not specifying

edit: it should ask when you first start it

Flopgop commented 2 years ago

Closing this, you shouldn't be adding a new JVM anyway.