Skip to content
This repository has been archived by the owner on Aug 22, 2023. It is now read-only.

[Feature Request]: Pojav Launcher support #41

Open
1 task done
Madis0 opened this issue Nov 18, 2022 · 10 comments
Open
1 task done

[Feature Request]: Pojav Launcher support #41

Madis0 opened this issue Nov 18, 2022 · 10 comments
Labels
enhancement New feature or request on hold Issues or pull requests that are waiting for another issue.
Milestone

Comments

@Madis0
Copy link
Member

Madis0 commented Nov 18, 2022

Code of Conduct

  • I agree to follow this project's Code of Conduct

Potential Enhancement

A checkbox users can enable to indicate they want to install to Pojav launcher

How it could be implemented

The user would download the installer as jar and check the checkbox. The installer would install to assigned path as before, but also unpack a set of patches to specified folders.

Anything else?

Low priority, get the main thing running first

@Madis0 Madis0 added the enhancement New feature or request label Nov 18, 2022
@osfanbuff63
Copy link
Member

osfanbuff63 commented Nov 18, 2022

Just to clarify, we don't need to run this on the Android device running Pojav, this just simplifies the process of getting all the files ready correct? If we're running on Android, that might be a bit harder because I don't know if Qt for Python runs on Android, meaning we're stuck to the CLI (which ofc isn't ideal).

@osfanbuff63 osfanbuff63 added this to the Backlog milestone Nov 18, 2022
@Madis0
Copy link
Member Author

Madis0 commented Nov 18, 2022

  • It needs to be a jar
  • It needs to detect the correct path somehow (Fabric installer detected the correct one, but maybe Pojav gave it as a parameter)
  • I can do the testing on Pojav itself

@nsde
Copy link
Contributor

nsde commented Nov 18, 2022

Shouldn't we create a separate Kotlin/Java Project for Pojav?

@Madis0
Copy link
Member Author

Madis0 commented Nov 18, 2022

Well, I thought you could compile it with something like Jython.

But if not, scrap this idea, as it's not worth creating a new project for, then it'd just wait for packwiz's implementation.

@osfanbuff63
Copy link
Member

Well, I thought you could compile it with something like Jython.

But if not, scrap this idea, as it's not worth creating a new project for, then it'd just wait for packwiz's implementation.

We could also compile to an APK, allowing us to use the existing VI code base. Not sure if that's idea though.

@nsde
Copy link
Contributor

nsde commented Nov 18, 2022

The current release (a Jython 2.7.x) only supports Python 2 (sorry)

Sorry, but I don't think its not a good idea to simply compile it to an APK. It's not as easy as that. Python is just not made for mobile app developement.

The closest thing to that would be that we could program an adb feature, which would install the files via USB. But maybe even that would be worth creating a new repository for, because it's a completely different system.

This repo's name is VanillaInstaller, so it wouldn't make sense to add any unrelated functionality. We should keep it lightweight.

@osfanbuff63
Copy link
Member

You can compile PySide6 to C++ which can then be compiled to an android APK, allowing this functionality. So I wouldn't rule it out in the future, but not for now.

@osfanbuff63
Copy link
Member

We could also make this somewhat of a monorepo if we wanted, although I'm not sure.

TL;DR: Maybe at some point, but not high priority right now.

@osfanbuff63 osfanbuff63 added the on hold Issues or pull requests that are waiting for another issue. label Nov 18, 2022
@nsde
Copy link
Contributor

nsde commented Nov 19, 2022

You can compile PySide6 to C++ which can then be compiled to an android APK, allowing this functionality

This is probably way too cumbersome...

@osfanbuff63
Copy link
Member

osfanbuff63 commented Nov 19, 2022

You can compile PySide6 to C++ which can then be compiled to an android APK, allowing this functionality

This is probably way too cumbersome...

Who knows, it's still an early idea :P
Correction, https://pypi.org/project/pyqtdeploy can do this it seems. Again, still an early idea.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request on hold Issues or pull requests that are waiting for another issue.
Projects
None yet
Development

No branches or pull requests

3 participants