Test MSVC instead of NOT MINGW

The condition also needs to exclude Clang on Windows. Fix AOMediaCodec#2442.
wantehchang · Sep 25, 2024 · a65e172 · a65e172
1 parent 2fe5b3a
commit a65e172
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -727,7 +727,7 @@ if(AVIF_BUILD_APPS)
         #   output:
         #   CVTRES : fatal error CVT1100: duplicate resource.  type:MANIFEST,
         #   name:1, language:0x0409
-        if(NOT MINGW)
+        if(MSVC)
             target_link_options(avifenc PRIVATE /MANIFEST:NO)
         endif()
     endif()
@@ -738,7 +738,7 @@ if(AVIF_BUILD_APPS)
     add_executable(avifdec apps/avifdec.c)
     if(WIN32)
         target_sources(avifdec PRIVATE apps/utf8.rc)
-        if(NOT MINGW)
+        if(MSVC)
             target_link_options(avifdec PRIVATE /MANIFEST:NO)
         endif()
     endif()
@@ -774,7 +774,7 @@ if(AVIF_BUILD_APPS)
         add_executable(avifgainmaputil "${AVIFGAINMAPUTIL_SRCS}")
         if(WIN32)
             target_sources(avifgainmaputil PRIVATE apps/utf8.rc)
-            if(NOT MINGW)
+            if(MSVC)
                 target_link_options(avifgainmaputil PRIVATE /MANIFEST:NO)
             endif()
         endif()

diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
@@ -240,7 +240,7 @@ if(AVIF_BUILD_APPS)
     add_executable(are_images_equal gtest/are_images_equal.cc)
     if(WIN32)
         target_sources(are_images_equal PRIVATE ${CMAKE_SOURCE_DIR}/apps/utf8.rc)
-        if(NOT MINGW)
+        if(MSVC)
             target_link_options(are_images_equal PRIVATE /MANIFEST:NO)
         endif()
     endif()