Fix recording failed problem after upgrade projects. Use lower target sdk version to avoid strict system storage access check.

Author: wysaid

build.gradle

@@ -25,7 +25,7 @@ ext {
             compileSdkVersion: 30,
             buildToolsVersion: '30.0.3',
             minSdkVersion    : 16,
-            targetSdkVersion : 30,
+            targetSdkVersion : 25, // higher target version needs more storage permission request.
             versionCode      : 1,
             versionName      : "1.0.0",
             applicationId    : "org.wysaid.cgeDemo",

cgeDemo/build.gradle

@@ -24,6 +24,7 @@ android {
         targetSdkVersion rootProject.ext.android.targetSdkVersion
         versionCode rootProject.ext.android.versionCode
         versionName rootProject.ext.android.versionName
+        signingConfig signingConfigs.debug
     }
     buildTypes {
         release {

cgeDemo/src/main/AndroidManifest.xml

@@ -1,8 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
 <manifest xmlns:android="http://schemas.android.com/apk/res/android">
-    <!-- <uses-sdk android:minSdkVersion="14" -->
-    <!-- android:targetSdkVersion="22" -->
-    <!-- /> -->
     <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />
     <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />
     <uses-permission android:name="android.permission.CAMERA" />

gradle.properties

@@ -16,4 +16,5 @@ org.gradle.jvmargs=-Xmx2048m -XX:MaxPermSize=512m -XX:+HeapDumpOnOutOfMemoryErro
 # org.gradle.parallel=true
 
 android.useAndroidX=true
-android.enableJetifier=true
+android.enableJetifier=true
+android.injected.testOnly=false

library/build.gradle

@@ -31,10 +31,10 @@ android {
 
                     if (usingCMakeCompileDebug()) {
                         cmakeBuildType = "-DCMAKE_BUILD_TYPE=Debug"
-                        cppExtraFlags = "-DDEBUG=1 -D_DEBUG=1"
+                        cppExtraFlags = "-DDEBUG=1 -D_DEBUG=1 -Od -g"
                     } else {
                         cmakeBuildType = "-DCMAKE_BUILD_TYPE=Release"
-                        cppExtraFlags = "-Os -DNDEBUG=1 -D_NDEBUG=1"
+                        cppExtraFlags = "-O2 -DNDEBUG=1 -D_NDEBUG=1"
                     }
 
                     arguments "-DCGE_USE_VIDEO_MODULE=ON",

library/src/main/java/org/wysaid/myUtils/PermissionUtil.java

@@ -14,6 +14,7 @@ public class PermissionUtil {
     private static final int REQUEST_PERMISSION = 0;
     private static String[] PERMISSIONS_STORAGE = {
             Manifest.permission.CAMERA,
+            Manifest.permission.RECORD_AUDIO,
             Manifest.permission.WRITE_EXTERNAL_STORAGE,
             Manifest.permission.READ_EXTERNAL_STORAGE};
 

vscode_tasks.sh

@@ -23,8 +23,23 @@ function setupProject() {
 }
 
 function runAndroidApp() {
-    . "$ADB_COMMAND" -d shell am start -n "$PACKAGE_NAME/$PACKAGE_NAME.$LAUNCH_ACTIVITY" &&
-        . "$ADB_COMMAND" -d logcat | grep -iE "(cge|demo|wysaid)"
+
+    if . "$ADB_COMMAND" -d shell am start -n "$PACKAGE_NAME/$PACKAGE_NAME.$LAUNCH_ACTIVITY"; then
+        if [[ -z "$(ps -ef | grep -i adb | grep -v grep | grep logcat)" ]]; then
+            if [[ $(uname -s) == "Linux" ]] || [[ $(uname -s) == "Darwin" ]]; then
+                APP_PROC_ID=$(adb shell ps | grep org.wysaid.cgeDemo | tr -s ' ' | cut -d' ' -f2)
+                if [[ -n "$APP_PROC_ID" ]]; then
+                    . "$ADB_COMMAND" -d logcat | grep -F "$APP_PROC_ID"
+                else
+                    echo "Can not find proc id of org.wysaid.cgeDemo"
+                fi
+            else
+                . "$ADB_COMMAND" -d logcat | grep -iE "(cge|demo|wysaid| E |crash)"
+            fi
+        fi
+    else
+        echo "Launch $PACKAGE_NAME/$PACKAGE_NAME.$LAUNCH_ACTIVITY failed."
+    fi
 }
 
 function cleanProject() {
@@ -42,12 +57,17 @@ function buildProject() {
         exit 1
     fi
 
-    echo "apk generated at:"
-    find "$THIS_DIR/cgeDemo/build" -iname "*.apk" | grep -i "${ANDROID_BUILD_TYPE/assemble/}"
+    GENERATED_APK_FILE=$(find "$THIS_DIR/cgeDemo/build" -iname "*.apk" | grep -i "${ANDROID_BUILD_TYPE/assemble/}")
+    echo "apk generated at: $GENERATED_APK_FILE"
 
-    if [[ -n "$GRADLEW_RUN_TASK" ]] && [[ $(. "$ADB_COMMAND" -d devices | grep -v 'List' | tr -d ' \n' | wc -l) -ne 0 ]]; then
-        if ! ./gradlew -p cgeDemo "$GRADLEW_RUN_TASK"; then
-            echo "Install failed." >&2
+    if [[ -n "$GRADLEW_RUN_TASK" ]] && [[ $(. "$ADB_COMMAND" -d devices | grep -v 'List' | grep -vE '^$' | wc -l | tr -d ' ') -ne 0 ]]; then
+        if [[ "$GRADLEW_RUN_TASK" == "installRelease" ]]; then
+            # release can not be installed directly. do adb install.
+            . $ADB_COMMAND -d install "$GENERATED_APK_FILE"
+        else
+            if ! ./gradlew -p cgeDemo "$GRADLEW_RUN_TASK"; then
+                echo "Install failed." >&2
+            fi
         fi
     else
         echo "No device connected, skip installation."
@@ -66,6 +86,23 @@ if [[ ! -f "local.properties" ]]; then
     fi
 fi
 
+function changeProperty() {
+    TARGET_FILE="$1"
+    MATCH_PATTERN="$2"
+    SED_PATTERN="$3"
+    DEFAULT_VALUE="$4"
+    if [[ -f "$TARGET_FILE" ]] && grep -E "$MATCH_PATTERN" "$TARGET_FILE" >/dev/null; then
+        # Stupid diff between the command.
+        if [[ "$(uname -s)" == "Darwin" ]]; then # Mac OS
+            sed -I "" -E "$SED_PATTERN" "$TARGET_FILE"
+        else
+            sed -i"" -E "$SED_PATTERN" "$TARGET_FILE"
+        fi
+    else
+        echo "$DEFAULT_VALUE" >>"$TARGET_FILE"
+    fi
+}
+
 while [[ $# > 0 ]]; do
 
     PARSE_KEY="$1"
@@ -84,12 +121,15 @@ while [[ $# > 0 ]]; do
         ;;
     --release)
         ANDROID_BUILD_TYPE="assembleRelease"
+        # ANDROID_BUILD_TYPE="assembleDebug" # use this if the release apk can not be installed.
         GRADLEW_RUN_TASK="installRelease"
+        changeProperty "local.properties" '^usingCMakeCompileDebug=' 's/usingCMakeCompileDebug=.*/usingCMakeCompileDebug=false/' 'usingCMakeCompileDebug=false'
         shift
         ;;
     --debug)
         ANDROID_BUILD_TYPE="assembleDebug"
         GRADLEW_RUN_TASK="installDebug"
+        changeProperty "local.properties" '^usingCMakeCompileDebug=' 's/usingCMakeCompileDebug=.*/usingCMakeCompileDebug=true/' 'usingCMakeCompileDebug=true'
         shift
         ;;
     --setup-project)
@@ -113,16 +153,7 @@ while [[ $# > 0 ]]; do
         shift # past argument
         ;;
     --enable-cmake)
-        if [[ -f "local.properties" ]] && grep -E '^usingCMakeCompile=' "local.properties" >/dev/null; then
-            # Stupid diff between the command.
-            if [[ "$(uname -s)" == "Darwin" ]]; then # Mac OS
-                sed -I "" -E 's/usingCMakeCompile=.*/usingCMakeCompile=true/' "local.properties"
-            else
-                sed -i"" -E 's/usingCMakeCompile=.*/usingCMakeCompile=true/' "local.properties"
-            fi
-        else
-            echo "usingCMakeCompile=true" >>"local.properties"
-        fi
+        changeProperty "local.properties" '^usingCMakeCompile=' 's/usingCMakeCompile=.*/usingCMakeCompile=true/' 'usingCMakeCompile=true'
         shift # past argument
         ;;
     *)