Fix Sentry (crash logging) for Win64.

The issue was windows defaults to wide-strings. Sadly sentry doesn't seem to take wide-string paths, so that users with non-ASCII names might be left out again :-/ part of CURA-11364
Ultimaker · Nov 22, 2023 · d020d0f · d020d0f
1 parent 5ef89b5
commit d020d0f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/include/utils/format/filesystem_path.h b/include/utils/format/filesystem_path.h
@@ -32,7 +32,7 @@ struct formatter<std::filesystem::path> : formatter<string_view>
     template<typename FormatContext>
     auto format(const std::filesystem::path& path, FormatContext& ctx)
     {
-        return formatter<string_view>::format(anonymizePath(path.native()), ctx);
+        return formatter<string_view>::format(anonymizePath(path.generic_string()), ctx);
     }
 };
 

diff --git a/src/main.cpp b/src/main.cpp
@@ -14,7 +14,7 @@
 #include <string>
 
 #include <range/v3/algorithm/contains.hpp>
-#include <range/v3/to_container.hpp>
+#include <range/v3/range/conversion.hpp>
 #include <range/v3/view/take_while.hpp>
 
 #include "utils/format/filesystem_path.h"
@@ -78,7 +78,7 @@ int main(int argc, char** argv)
         const auto config_path = std::filesystem::path(fmt::format("{}\\cura\\.sentry-native", std::getenv("APPDATA")));
 #endif
         spdlog::info("Sentry config path: {}", config_path);
-        sentry_options_set_database_path(options, static_cast<const char*>(std::filesystem::absolute(config_path).native().c_str()));
+        sentry_options_set_database_path(options, std::filesystem::absolute(config_path).generic_string().c_str());
         constexpr std::string_view cura_engine_version{ CURA_ENGINE_VERSION };
         const auto version = semver::from_string(
             cura_engine_version