Start service with start foregroundService on Android Oreo

[brvier]

And add a notification channel for Oreo permanent service notification
Fix #1785

[Ravi2232]

Yes we are

[inclement]

Thanks @brvier, the changes look basically good to me. It looks like travis is complaining about a build issue though, is this a real problem with the code?

[AndreMiras]

I'm really not getting that part 🤔 we have the if sticky in the template, but we decide to return not sticky.
There's something fishy about it no?

[AndreMiras]

@inclement @brvier I think usually the CI fails for good reasons. Here for instance we have:

:compileDebugJavaWithJavac/home/user/.local/share/python-for-android/dists/bdisttest_python3_sqlite_openssl_googlendk/src/main/java/org/kivy/android/PythonService.java:22: error: package android.support.v4.app does not exist
import android.support.v4.app.ActivityCompat;
                             ^
/home/user/.local/share/python-for-android/dists/bdisttest_python3_sqlite_openssl_googlendk/src/main/java/org/kivy/android/PythonService.java:28: error: package android.support.v4.app does not exist
import android.support.v4.app.NotificationCompat;
                             ^
/home/user/.local/share/python-for-android/dists/bdisttest_python3_sqlite_openssl_googlendk/src/main/java/org/kivy/android/PythonService.java:139: error: package NotificationCompat does not exist
            NotificationCompat.Builder notificationBuilder = new NotificationCompat.Builder(context, "persistent");
                              ^
/home/user/.local/share/python-for-android/dists/bdisttest_python3_sqlite_openssl_googlendk/src/main/java/org/kivy/android/PythonService.java:139: error: package NotificationCompat does not exist
            NotificationCompat.Builder notificationBuilder = new NotificationCompat.Builder(context, "persistent");
                                                                                   ^
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
4 errors
 FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':compileDebugJavaWithJavac'.
> Compilation failed; see the compiler error output for details.

[brvier]

I think usually the CI fails for good reasons.

Hopefully, else it s useless.

So here Travis point to dependancies i have, and which isn't default on pythonforandroid.

Is there any problem to add this deps in pythonforandroid ?

[AndreMiras]

Yes if your pull request requires a dependency, then yes add that dep as part of your pull request and also think about about updating the docs in that same PR

[AndreMiras]

Up @brvier 👋
There's an up to date PR also here #1725 can you give it a try before we merge it?

[ghost]

Is this superseded now and possibly should be closed? @inclement & @AndreMiras I'm seeing similar code already in current develop

[ghost]

Since we seem to have something similar on develop already I'll close this for now. Thanks so much for your contribution, sorry it didn't make it into the project in exactly your provided form!

If anyone thinks this is closed by mistake let me know and I'll gladly immediately reopen