Created attachment 372660 [details] [review]
Android tutorials: add android top-level folder

This in preparation for the next commit, adding gradle as a
build system

Created attachment 372661 [details] [review]
Android tutorials: add gradle build system

Comment on attachment 372660 [details] [review]
Android tutorials: add android top-level folder

Good idea independent of anything else

Review of attachment 372661 [details] [review]:

Looks mostly good, just fix the below and merge :)

::: examples/tutorials/android/android-tutorial-5/build.gradle
@@ +30,3 @@
+
+                // All archs except MIPS and MIPS64 are supported
+                abiFilters  'armeabi-v7a', 'arm64-v8a', 'x86'

Should stop building armeabi (non-v7a) for the 1.15/1.16 releases

::: markdown/installing/for-android-development.md
@@ +206,3 @@
+```bash
+$ cd examples/tutorials/android
+$ GSTREAMER_ROOT_ANDROID=/path/to/gst-android-1.14/ PATH=~/dev/android/tools/bin:~/dev/android/ndk-bundle:$PATH ANDROID_HOME="$HOME/dev/android/" ./gradlew :android-tutorial-1:installDebug

Should always put the path into gradle.properties instead. Environment variables are awkward for Windows users.

Maybe also add an paragraph for how to do all this in Android Studio?

@@ -195,0 +195,18 @@
+#### Using gradle from the command-line
+
+To build and deploy the tutorials to your device, use a command similar to:
... 15 more ...

Or alternatively just start the app from the app launcher on the device :)

(In reply to Sebastian Dröge (slomo) from comment #4)

> Should stop building armeabi (non-v7a) for the 1.15/1.16 releases

Fixed the rest of the points, but I don't get this one, if I leave armeabi in there it simply doesn't build with gradle I'm afraid :)

(In reply to Mathieu Duponchelle from comment #5)
> (In reply to Sebastian Dröge (slomo) from comment #4)
> 
> > Should stop building armeabi (non-v7a) for the 1.15/1.16 releases
> 
> Fixed the rest of the points, but I don't get this one, if I leave armeabi
> in there it simply doesn't build with gradle I'm afraid :)

Nothing for you to do here. I meant we should drop armeabi from the cerbero build so that it is not included in the binaries anymore.

(In reply to Sebastian Dröge (slomo) from comment #6)
> Nothing for you to do here. I meant we should drop armeabi from the cerbero
> build so that it is not included in the binaries anymore.

OK got it, pushing then, thanks!

Attachment 372660 [details] pushed as 225c23f - Android tutorials: add android top-level folder
Attachment 372661 [details] pushed as 82dd7c3 - Android tutorials: add gradle build system