Remove logging - ivan will follow up in a separate PR

Author: steveisok

src/coreclr/dlls/mscordbi/CMakeLists.txt

@@ -106,12 +106,6 @@ elseif(CLR_CMAKE_HOST_UNIX)
         )
     endif(CLR_CMAKE_HOST_HAIKU)
 
-    if(CLR_CMAKE_TARGET_ANDROID)
-        list(APPEND COREDBI_LIBRARIES
-            log
-        )
-    endif(CLR_CMAKE_TARGET_ANDROID)
-
     # Before llvm 16, lld was setting `--undefined-version` by default. The default was
     # flipped to `--no-undefined-version` in lld 16, so we will explicitly set it to
     # `--undefined-version` for our use-case.

src/coreclr/pal/src/CMakeLists.txt

@@ -300,10 +300,7 @@ if(CLR_CMAKE_TARGET_LINUX)
   else(NOT CLR_CMAKE_TARGET_ANDROID)
     target_link_libraries(coreclrpal
       PUBLIC
-      ${ANDROID_GLOB}
-      ${LZMA}
-      PRIVATE
-      log)
+      ${ANDROID_GLOB})
   endif(NOT CLR_CMAKE_TARGET_ANDROID)
 
   target_link_libraries(coreclrpal

src/coreclr/pal/src/thread/process.cpp

@@ -71,10 +71,6 @@ SET_DEFAULT_DEBUG_CHANNEL(PROCESS); // some headers have code with asserts, so d
 #include <sys/membarrier.h>
 #endif
 
-#if defined(TARGET_ANDROID)
-#include <android/log.h>
-#endif // defined(TARGET_ANDROID)
-
 #ifdef __APPLE__
 #include <pwd.h>
 #include <sys/sysctl.h>
@@ -2549,12 +2545,6 @@ PROCAbort(int signal, siginfo_t* siginfo)
     // the others to prevent multiple core dumps from being generated.
     SEHCleanupSignals(false /* isChildProcess */);
 
-#if defined(TARGET_ANDROID)
-    // Ideally, MAIN_CLR_MODULE_NAME_A would be used, but cor.h can't
-    // currently be included in the pal.
-    __android_log_print(ANDROID_LOG_FATAL, "coreclr", "Aborting in %s:%u", __FILE_NAME__, __LINE__);
-#endif // defined(TARGET_ANDROID)
-
     // Abort the process after waiting for the core dump to complete
     abort();
 }

src/coreclr/utilcode/log.cpp

@@ -18,10 +18,6 @@
 #include "log.h"
 #include "utilcode.h"
 
-#if defined(TARGET_ANDROID)
-#include <android/log.h>
-#endif // defined(TARGET_ANDROID)
-
 #ifdef LOGGING
 
 #define DEFAULT_LOGFILE_NAME    W("COMPLUS.LOG")
@@ -375,16 +371,10 @@ VOID LogSpewAlwaysValist(const char *fmt, va_list args)
 
     if (LogFlags & LOG_ENABLE_CONSOLE_LOGGING)
     {
-#if defined(TARGET_ANDROID)
-        // TODO: priority should be configurable here (best, passed via a parameter)
-        //       likewise for the tag
-        __android_log_write(ANDROID_LOG_INFO, MAIN_CLR_MODULE_NAME_A, pBuffer);
-#else
         WriteFile(GetStdHandle(STD_OUTPUT_HANDLE), pBuffer, buflen, &written, 0);
         //<TODO>@TODO ...Unnecessary to flush console?</TODO>
         if (LogFlags & LOG_ENABLE_FLUSH_FILE)
             FlushFileBuffers( GetStdHandle(STD_OUTPUT_HANDLE) );
-#endif // defined(TARGET_ANDROID)
     }
 
     if (LogFlags & LOG_ENABLE_DEBUGGER_LOGGING)

src/native/corehost/apphost/static/CMakeLists.txt

@@ -309,8 +309,6 @@ target_link_libraries(
 if(CLR_CMAKE_TARGET_ANDROID)
     target_link_libraries(
         singlefilehost
-        PRIVATE
-        log
     )
 endif()