[FIXUP] for "cmake: Add wallet functionality"

LINK : warning LNK4098: defaultlib 'LIBCMTD' conflicts with use of other libs; use /NODEFAULTLIB:library

Author: hebasto

cmake/optional.cmake

@@ -111,9 +111,16 @@ endif()
 
 if(ENABLE_WALLET)
   if(WITH_SQLITE)
-    include(CrossPkgConfig)
-    cross_pkg_check_modules(sqlite sqlite3>=3.7.17 IMPORTED_TARGET)
-    if(sqlite_FOUND)
+    # TODO: Consider using the FindSQLite3 module after bumping
+    # the minimum required CMake version up to 3.14+.
+    if(MSVC)
+      # Use of the `unofficial::` namespace is a vcpkg package manager convention.
+      find_package(unofficial-sqlite3 CONFIG)
+    else()
+      include(CrossPkgConfig)
+      cross_pkg_check_modules(sqlite3 sqlite3>=3.7.17 IMPORTED_TARGET)
+    endif()
+    if(TARGET unofficial::sqlite3::sqlite3 OR TARGET PkgConfig::sqlite3)
       set(WITH_SQLITE ON)
       set(USE_SQLITE ON)
     elseif(WITH_SQLITE STREQUAL "AUTO")

src/wallet/CMakeLists.txt

@@ -44,7 +44,11 @@ if(NOT USE_SQLITE AND NOT USE_BDB)
 endif()
 if(USE_SQLITE)
   target_sources(bitcoin_wallet PRIVATE sqlite.cpp)
-  target_link_libraries(bitcoin_wallet PRIVATE PkgConfig::sqlite)
+  target_link_libraries(bitcoin_wallet
+    PRIVATE
+      $<TARGET_NAME_IF_EXISTS:unofficial::sqlite3::sqlite3>
+      $<TARGET_NAME_IF_EXISTS:PkgConfig::sqlite3>
+  )
 endif()
 if(USE_BDB)
   target_sources(bitcoin_wallet PRIVATE bdb.cpp salvage.cpp)