<p dir="ltr">Actually, vlc-android needs ndk11, and is not compliant with ndk12.<br>
We still have to make our code respect ndk12 rules</p>
<br><div class="gmail_quote"><div dir="ltr">Le ven. 5 août 2016 21:31, Filip Roséen <<a href="mailto:filip@atch.se">filip@atch.se</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The diagnostic certainly implies that the shell script shall abort if<br>
the version of the NDK is less than 11, though the previous case-check<br>
only matched versions matching "11*".<br>
<br>
This commit changes the implementation so that we check whether "$REL"<br>
is greater-than-or-equal compared to 11.<br>
<br>
---<br>
<br>
The current implementation caused be a slight headache when trying to<br>
build using "Pkg.Revision = 12.1.2977051".<br>
---<br>
 compile-libvlc.sh | 20 +++++++++-----------<br>
 1 file changed, 9 insertions(+), 11 deletions(-)<br>
<br>
diff --git a/compile-libvlc.sh b/compile-libvlc.sh<br>
index 381e4ee..b9a3bbd 100755<br>
--- a/compile-libvlc.sh<br>
+++ b/compile-libvlc.sh<br>
@@ -279,19 +279,17 @@ fi<br>
 # try to detect NDK version<br>
 GCCVER=4.9<br>
 REL=$(grep -o '^Pkg.Revision.*[0-9]*.*' $ANDROID_NDK/source.properties |cut -d " " -f 3 | cut -d "." -f 1)<br>
-case "$REL" in<br>
-    11*)<br>
-        if [ "${HAVE_64}" = 1 ];then<br>
-            ANDROID_API=android-21<br>
-        else<br>
-            ANDROID_API=android-9<br>
-        fi<br>
-    ;;<br>
-    *)<br>
+<br>
+if [ "$REL" -ge 11 ]; then<br>
+    if [ "${HAVE_64}" = 1 ];then<br>
+        ANDROID_API=android-21<br>
+    else<br>
+        ANDROID_API=android-9<br>
+    fi<br>
+else<br>
     echo "You need the NDKv11 or later"<br>
     exit 1<br>
-    ;;<br>
-esac<br>
+fi<br>
<br>
 SYSROOT=$ANDROID_NDK/platforms/$ANDROID_API/arch-$PLATFORM_SHORT_ARCH<br>
 SRC_DIR=$PWD<br>
--<br>
2.9.2<br>
<br>
_______________________________________________<br>
Android mailing list<br>
<a href="mailto:Android@videolan.org" target="_blank">Android@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/android" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/android</a><br>
</blockquote></div>