Merge branch 'main' into fix/conan

Author: lia-viam

CMakeLists.txt

@@ -34,7 +34,7 @@
 # constrained by the version of CMake available on target systems.
 cmake_minimum_required(VERSION 3.25 FATAL_ERROR)
 
-set(CMAKE_PROJECT_VERSION 0.6.0)
+set(CMAKE_PROJECT_VERSION 0.7.0)
 
 # Identify the project.
 project(viam-cpp-sdk
@@ -173,6 +173,10 @@ if (NOT CMAKE_CXX_STANDARD)
   set(CMAKE_CXX_STANDARD 14)
 endif()
 set(CMAKE_CXX_STANDARD_REQUIRED True)
+if(MSVC)
+  # https://discourse.cmake.org/t/set-cmake-cxx-standard-should-set-zc-cplusplus-for-msvc/1876
+  string(APPEND CMAKE_CXX_FLAGS " /Zc:__cplusplus")
+endif()
 set(CMAKE_CXX_EXTENSIONS OFF)
 
 
@@ -255,7 +259,7 @@ if (viam_rust_utils_files)
     ${viam_rust_utils_file}
     ONLY_IF_DIFFERENT
   )
-else()
+elseif(NOT WIN32) # TODO(RSDK-10366): Currently, rust_utils is not published for windows, so don't even try downloading
   set(lvru_system_name ${CMAKE_SYSTEM_NAME})
   if (CMAKE_SYSTEM_NAME STREQUAL "Darwin")
     set(lvru_system_name "macosx")
@@ -275,21 +279,23 @@ else()
 
 endif()
 
-add_library(viam_rust_utils SHARED IMPORTED)
+# TODO(RSDK-10366): Currently, rust_utils is not published for windows, so don't even declare the library
+if (NOT WIN32)
+  add_library(viam_rust_utils SHARED IMPORTED)
 
-target_link_directories(viam_rust_utils
-  INTERFACE
+  target_link_directories(viam_rust_utils
+    INTERFACE
     "$<BUILD_INTERFACE:${CMAKE_CURRENT_BINARY_DIR}>"
     "$<INSTALL_INTERFACE:${CMAKE_INSTALL_LIBDIR}>"
-)
-
-set_property(TARGET viam_rust_utils PROPERTY IMPORTED_LOCATION ${viam_rust_utils_file})
+  )
 
-install(
-  IMPORTED_RUNTIME_ARTIFACTS viam_rust_utils
-  LIBRARY COMPONENT viam-cpp-sdk_runtime
-)
+  set_property(TARGET viam_rust_utils PROPERTY IMPORTED_LOCATION ${viam_rust_utils_file})
 
+  install(
+    IMPORTED_RUNTIME_ARTIFACTS viam_rust_utils
+    LIBRARY COMPONENT viam-cpp-sdk_runtime
+  )
+endif()
 
 # Install the license file
 install(FILES

conanfile.py

@@ -82,7 +82,11 @@ def package(self):
         CMake(self).install()
 
     def package_info(self):
-        self.cpp_info.components["viam_rust_utils"].libs = ["viam_rust_utils"]
+
+        # TODO(RSDK-10366): Currently, rust_utils is not published for windows
+        # and the C++ SDK just doesn't include it as a dependency on that platform
+        if not self.settings.os == "Windows":
+            self.cpp_info.components["viam_rust_utils"].libs = ["viam_rust_utils"]
 
         self.cpp_info.components["viamsdk"].libs = ["viamsdk"]
 
@@ -103,10 +107,16 @@ def package_info(self):
         else:
             lib_folder = os.path.join(self.package_folder, "lib")
             lib_fullpath = os.path.join(lib_folder, "libviamapi.a")
+            if self.settings.os == "Windows":
+                lib_fullpath = os.path.join(lib_folder, "viamapi.lib")
+
             if is_apple_os(self):
                 whole_archive = f"-Wl,-force_load,{lib_fullpath}"
+            elif self.settings.os == "Windows":
+                whole_archive = f"/WHOLEARCHIVE:{lib_fullpath}"
             else:
                 whole_archive = f"-Wl,--push-state,--whole-archive,{lib_fullpath},--pop-state"
+
             self.cpp_info.components["viamapi"].exelinkflags.append(whole_archive)
             self.cpp_info.components["viamapi"].sharedlinkflags.append(whole_archive)
 
@@ -118,7 +128,13 @@ def package_info(self):
             "xtensor::xtensor",
 
             "viamapi",
-            "viam_rust_utils"
         ])
 
+        # TODO(RSDK-10366): Currently, rust_utils is not published for windows
+        # and the C++ SDK just doesn't include it as a dependency on that platform
+        if self.settings.os != "Windows":
+            self.cpp_info.components["viamsdk"].requires.extend([
+                "viam_rust_utils"
+            ])
+
         self.cpp_info.components["viamsdk"].frameworks = ["Security"]

src/viam/api/api_proto_tag.lock

@@ -1 +1 @@
-v0.1.417
+v0.1.424