Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4f9e4f06 authored by Makoto Onuki's avatar Makoto Onuki
Browse files

Fix RAVENWOOD_VERBOSE for liblog 

Now RAVENWOOD_VERBOSE=1 will correctly enable Log.v and Log.d

Flag: EXEMPT host test change only
Bug: 292141694
Test: RAVENWOOD_VERBOSE=1 atest RavenwoodBivalentTest, make sure
    Log.v() output is logged.
Change-Id: Id2dd12f1004dcfc3fd10be10fcea61362c9d58fb
parent fca57103

ravenwood/junit-impl-src/android/platform/test/ravenwood/RavenwoodRuntimeEnvironmentController.java

+11 −9
Original line number Diff line number Diff line
@@ -165,6 +165,17 @@ public class RavenwoodRuntimeEnvironmentController { 
        RavenwoodSystemProperties.initialize(RAVENWOOD_BUILD_PROP);

        setSystemProperties(null);



        // Do this after loading RAVENWOOD_NATIVE_RUNTIME_NAME (which backs Os.setenv()),

        // before loadFrameworkNativeCode() (which uses $ANDROID_LOG_TAGS).

        if (RAVENWOOD_VERBOSE_LOGGING) {

            RavenwoodCommonUtils.log(TAG, "Force enabling verbose logging");

            try {

                Os.setenv("ANDROID_LOG_TAGS", "*:v", true);

            } catch (ErrnoException e) {

                throw new RuntimeException(e);

            }

        }



        // Make sure libandroid_runtime is loaded.

        RavenwoodNativeLoader.loadFrameworkNativeCode();
@@ -175,15 +186,6 @@ public class RavenwoodRuntimeEnvironmentController { 
        Objects.requireNonNull(Build.TYPE);

        Objects.requireNonNull(Build.VERSION.SDK);



        if (RAVENWOOD_VERBOSE_LOGGING) {

            RavenwoodCommonUtils.log(TAG, "Force enabling verbose logging");

            try {

                Os.setenv("ANDROID_LOG_TAGS", "*:v", true);

            } catch (ErrnoException e) {

                // Shouldn't happen.

            }

        }



        System.setProperty(RAVENWOOD_VERSION_JAVA_SYSPROP, "1");

        // This will let AndroidJUnit4 use the original runner.

        System.setProperty("android.junit.runner",