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

Bisq2 - Android & Java code unification #399

Closed
niyid opened this issue Dec 5, 2022 · 7 comments
Closed

Bisq2 - Android & Java code unification #399

niyid opened this issue Dec 5, 2022 · 7 comments

Comments

@niyid
Copy link

niyid commented Dec 5, 2022

This is a Bisq Network proposal. Please familiarize yourself with the submission and review process.

I had to make changes to several sub-projects (common, tor and persistence) to the original Java project for a successful Android build.

Currently all the other issues are resolved and should work as well for Java, except for item 2 - the matter of java.lang.management.ManagementFactory. The idea is to find a O/S neutral way to start tor (or any process) that works on Both Android and Java.

Once this pending issue is resolved, there will be no need for code fragmentation where different versions of common, tor and persistence are maintained for Java and Android. Rather a unified code will be maintained.

Please find attached the issue list.

javaVSandroid

@niyid
Copy link
Author

niyid commented Dec 8, 2022

Tor Android seems to be the best method for starting Tor on Android that I could find. But this will not be compatible with Java.

If there are any suggestions that will work for both Java and Android, they will be appreciated.

Of course if the unification will not work,we should proceed with maintaining bisq2-android as a fork.

@pazza83
Copy link

pazza83 commented Dec 8, 2022

Hi @niyid thanks for creating this proposal.

Have you have a look at the proposal guidelines: https://bisq.wiki/Proposals ?

I am not understanding what you are proposing.

I am also not sure what you are looking to seek consensus on.

@niyid
Copy link
Author

niyid commented Dec 8, 2022

@pazza83

Apologies if you find it difficult to understand the thrust of this proposal.

  • I am working on a Bisq2 Android version.
  • I had to make changes to several core projects as listed in the initial post because of limitations in Android.
  • Some of these changes will work OK in Java; but the most stubborn issue is Tor.
  • Since Bisq2 is still in its infancy, there is a chance to prevent fragmentation (forking) and maintain same code for Java and Android.

@HenrikJannsen
Copy link

@niyid I think you should just continue as a fork. The current devs do not have the resources to support this work at the current state.
Also the proposal area is not the right place for such discussions.

@pazza83
Copy link

pazza83 commented Dec 29, 2022

Closing due to lack of discussion

@niyid
Copy link
Author

niyid commented Dec 30, 2022

I suppose there is an alternative pathway to building for Android?

It is my opinion that this is the right first step to prepare for an Android app.

Without the protocol being the same for all devices (PC/Mobile/Handheld), it will be impossible to build for other devices.

@niyid
Copy link
Author

niyid commented Jan 3, 2023

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

No branches or pull requests

3 participants