update review notes

Author: Cynthia Jiang

.github/workflows/sdk_build.yml

@@ -65,6 +65,7 @@ jobs:
         run: git config --system core.longpaths true
 
       # Set up the requirements, and install Unity
+      # Ruby setup has to happen before python install and setup, to keep the absl config.
       - uses: ruby/setup-ruby@v1
         with:
           ruby-version: 3.0.2

cmake/external_rules.cmake

@@ -60,8 +60,6 @@ function(download_external_sources)
         "${FIREBASE_BINARY_DIR}/external/src/firebase_cpp_sdk" PARENT_SCOPE)
   endif()
 
-  message(WARNING "${ENV_COMMAND} cmake --debug-output")
-
   execute_process(
     COMMAND
       ${ENV_COMMAND} cmake --debug-output

cmake/unity_mono.cmake

@@ -100,8 +100,6 @@ macro(mono_add_external_library name projpath)
     set(absprojpath ${CMAKE_CURRENT_LIST_DIR}/${projpath})
   endif()
 
-  message(WARNING "Defining build of ${DLL_NAME} using ${MONO_CSHARP_BUILD_EXE}")
-
   set(CSHARP_BUILD_EXE "${UNITY_CSHARP_BUILD_EXE}")
   if(MSVC)
     set(CSHARP_BUILD_EXE "${MONO_CSHARP_BUILD_EXE}")

cmake/unity_pack.cmake

@@ -156,8 +156,6 @@ function(unity_pack_cs name)
 
   string(REPLACE ".dll" "" dllbasename ${dllname})
   string(REPLACE ".dll" "" dllbaserename ${dllrename})
-
-  message(WARNING "Setting up install: FILE:${dllpath}/${dllname} DEST:${UNITY_PACK_PACK_PATH} RENAME:${dllrename}")
   # Dll file
   install(
     FILES "${dllpath}/${dllname}"
@@ -247,7 +245,6 @@ function(unity_pack_native name)
     set(lib_dest "")
   endif()
 
-  message(WARNING "unity_pack_native TARGETS:${name} LIBRARY:${dll_dest} ARCHIVE:${lib_dest}")
   # Win32 treats Dll's as 'runtime' other platforms as 'library'
   install(
     TARGETS