Remove Windows support for hostname and username query again due to extended compiler and linker errors in Windows internal headers.

Author: breyerml

src/plssvm/detail/tracking/CMakeLists.txt

@@ -12,11 +12,6 @@ target_sources(
     ${PLSSVM_BASE_LIBRARY_NAME} PRIVATE $<BUILD_INTERFACE: ${CMAKE_CURRENT_SOURCE_DIR}/performance_tracker.cpp; ${CMAKE_CURRENT_SOURCE_DIR}/events.cpp; >
 )
 
-# need to link against special library under Windows
-if (WIN32)
-    target_link_libraries(${PLSSVM_BASE_LIBRARY_NAME} PRIVATE Ws2_32)
-endif ()
-
 # add target definitions
 target_compile_definitions(${PLSSVM_BASE_LIBRARY_NAME} PUBLIC PLSSVM_PERFORMANCE_TRACKER_ENABLED)
 

src/plssvm/detail/tracking/performance_tracker.cpp

@@ -48,13 +48,6 @@
     #define PLSSVM_HAS_POSIX_INCLUDES
 #endif
 
-#if __has_include(<winsock.h>)
-    #include <Lmcons.h>    // UNLEN
-    #include <windows.h>   // GetUserNameA, DWORD
-    #include <winsock2.h>  // gethostname
-    #define PLSSVM_HAS_WINDOWS_INCLUDES
-#endif
-
 #if defined(PLSSVM_STDPAR_BACKEND_HAS_GNU_TBB)
     #include "boost/version.hpp"  // BOOST_VERSION
 #endif
@@ -94,17 +87,8 @@ namespace {
  * @return the hostname of the current machine (`[[nodiscard]]`)
  */
 [[nodiscard]] std::string get_hostname() {
-    // maximum number of characters is given as a constant under Linux
 #if defined(PLSSVM_HAS_POSIX_INCLUDES)
     const auto host_name_max = static_cast<std::size_t>(sysconf(_SC_HOST_NAME_MAX));
-#endif
-
-    // maximum number of characters is given in the documentation (256)
-#if defined(PLSSVM_HAS_WINDOWS_INCLUDES)
-    const std::size_t host_name_max = 256;
-#endif
-
-#if defined(PLSSVM_HAS_POSIX_INCLUDES) || defined(PLSSVM_HAS_WINDOWS_INCLUDES)
     std::string hostname(host_name_max, '\0');
     if (gethostname(hostname.data(), host_name_max) != 0) {
         hostname = "not available";
@@ -137,18 +121,6 @@ namespace {
         username = "not available";
     }
     return username;
-#elif defined(PLSSVM_HAS_WINDOWS_INCLUDES)
-    auto size = static_cast<DWORD>(UNLEN + 1);
-    std::string username(size, '\0');
-    if (!GetUserNameA(username.data(), &size)) {
-        username = "not available";
-    }
-    // resize to actual string length (truncate trailing '\0's)
-    username.resize(size - 1);
-    if (username.empty()) {
-        username = "not available";
-    }
-    return username;
 #else
     return "not available";
 #endif
@@ -538,4 +510,3 @@ performance_tracker &global_performance_tracker() {
 }  // namespace plssvm::detail::tracking
 
 #undef PLSSVM_HAS_POSIX_INCLUDES
-#undef PLSSVM_HAS_WINDOWS_INCLUDES