Zworkb services

[zworkb]

minor changes for service_only bootstrap:

• service_only was recjeted by build.py (fixed that thanks to @AndreMiras 's hint)
• removed dep on crystax in setup.py in testapps/setup_testapp_service.py
• create own notification channel in PythonService.java (needed for android 8+)
• catch the case that kivy is not in the requirements and thus some args are missing

service_only bootstrap now works with android (7 and 8 tested)

[AndreMiras]

Looks very good overall, thanks for making p4a a better place! We left a couple of comments if you could review/address.
Cheers!

[AndreMiras]

Hey @zworkb we're almost there! Do you mind taking a final look at the couple of comments not yet addressed so we can test and merge?

[ghost]

just a minor thing for code readability in android/__init__.py, but I think is_service_only or similar should be used rather than misusing is_webview (see also my comment)

[AndreMiras]

Thanks again for all that hard work. Would you mind rebasing/squashing your changes. I cannot really review since the diff is very confusing and hard to read. There're changes from other PR that ended up in here somehow.

[AndreMiras]

I think I've seen this change breaking things in the past. I haven't tried on device (Android 7) yet, but are we sure-ish about it?

[AndreMiras]

Thanks for rebasing/squashing. It looks pretty good overall! 👏
I still want to test it on Android 7 to see if we don't have regression. My schedule is pretty awful lately, but I'll try asap.

[AndreMiras]

I've verified that Android 7 is still happy.
Thanks for the PR

[AndreMiras]

@Jonast I think your comment was addressed, could you review/approve if things are OK to you too?

[ghost]

Looks good 👍

---

Unrelated/not merge relevant: surprise mega confusion wait what, what's that "sdl2python3" in the code? is that some super old relic? not something that is to be fixed by this pull request of course since it's in the code part left alone, but I've never seen that and I rearranged all the bootstraps so that's weird 😄 edit: and "sdl2_gradle" is from where the "normal" sdl2 bootstrap was still ant I assume, that has to refer to something removed decades ago as well 😂