utils/lstopo: don't build lstopo-win with MSVC

Clang can be used in "MSVC mode" to build hwloc using autotools.
That mode doesn't support -mwindows just like MSVC doesn't.
-mwindows is the only difference between lstopo-win and lstopo,
and we already remove -mwindows from lstopo-win if building
with MSVC, so just disable lstopo-win instead of building both
identically.

Closes #631

Signed-off-by: Brice Goglin <[email protected]>
(cherry picked from commit a2ee1db)

Author: bgoglin

NEWS

@@ -22,10 +22,12 @@ Version 2.11.3
 * Fix Linux cpukind information when some CPUs are offline.
 * Don't report multiple local NUMA nodes in hwloc-calc on heterogeneous memory
   platforms when a single one is requested, broken since 2.9.
+* Fix lstopo build with Clang in MSVC format.
 * Split hwloc-calc.1 into more sections to dsitinguish output conversion,
   I/O set/object and formatting options.
 
 
+
 Version 2.11.2
 --------------
 * Add missing CPU info attrs on aarch64 on Linux.

utils/lstopo/Makefile.am

@@ -1,4 +1,4 @@
-# Copyright © 2009-2023 Inria.  All rights reserved.
+# Copyright © 2009-2024 Inria.  All rights reserved.
 # Copyright © 2009-2012, 2014 Université Bordeaux
 # Copyright © 2009-2014 Cisco Systems, Inc.  All rights reserved.
 # Copyright © 2020 Hewlett Packard Enterprise.  All rights reserved.
@@ -52,7 +52,12 @@ lstopo_CFLAGS = $(lstopo_no_graphics_CFLAGS) $(HWLOC_CAIRO_CFLAGS)
 lstopo_LDADD += $(HWLOC_CAIRO_LIBS) $(HWLOC_X11_LIBS)
 endif
 if HWLOC_HAVE_WINDOWS
-bin_PROGRAMS += lstopo lstopo-win
+bin_PROGRAMS += lstopo
+if !HWLOC_HAVE_MSVC
+# only build lstopo-win if NOT building with MSVC (or Clang in MSVC mode) because
+# MSVC doesn't support -mwindows, and lstopo-win would be identical to lstopo
+bin_PROGRAMS += lstopo-win
+endif
 lstopo_SOURCES += lstopo-windows.c
 lstopo_CPPFLAGS += -DLSTOPO_HAVE_GRAPHICS
 if HWLOC_HAVE_USER32