From c72982db1aff2388b554ed5e372b5f0209ca0603 Mon Sep 17 00:00:00 2001
From: Tyler Wilding <xtvaser@gmail.com>
Date: Fri, 22 Dec 2023 01:02:26 -0500
Subject: [PATCH] ci: properly differentiate between windows build artifacts on
 PRs

---
 .../workflows/scripts/common/name-artifacts.sh | 18 ++++++++++++++++++
 .github/workflows/windows_build_qt.yml         |  1 +
 2 files changed, 19 insertions(+)

diff --git a/.github/workflows/scripts/common/name-artifacts.sh b/.github/workflows/scripts/common/name-artifacts.sh
index f803be285c34c..949483aa25d7b 100755
--- a/.github/workflows/scripts/common/name-artifacts.sh
+++ b/.github/workflows/scripts/common/name-artifacts.sh
@@ -8,6 +8,7 @@
 
 # Inputs as env-vars
 # OS
+# BUILD_CONFIGURATION
 # BUILD_SYSTEM
 # ARCH
 # SIMD
@@ -35,6 +36,23 @@ if [[ ! -z "${BUILD_SYSTEM}" ]]; then
   fi
 fi
 
+# Isolate artifacts produced with different build systems, otherwise they overwrite each other
+# and you have no idea what you got!
+if [[ ! -z "${BUILD_SYSTEM}" ]]; then
+  # differentiate between clang and msvc
+  if [[ "${BUILD_CONFIGURATION,,}" == *"clang"* ]]; then
+      NAME="${NAME}-clang"
+  else
+      NAME="${NAME}-msvc"
+  fi
+  # also differentiate between sse4 and avx2
+  if [[ "${BUILD_CONFIGURATION,,}" == *"avx2"* ]]; then
+      NAME="${NAME}-avx2"
+  else
+      NAME="${NAME}-sse4"
+  fi
+fi
+
 # Add PR / Commit Metadata
 if [ "$EVENT_NAME" == "pull_request" ]; then
   PR_SHA=$(git rev-parse --short "${PR_SHA}")
diff --git a/.github/workflows/windows_build_qt.yml b/.github/workflows/windows_build_qt.yml
index 392cc5e8fa04c..d17d9da3bcaab 100644
--- a/.github/workflows/windows_build_qt.yml
+++ b/.github/workflows/windows_build_qt.yml
@@ -63,6 +63,7 @@ jobs:
         shell: bash
         env:
           OS: windows
+          BUILD_CONFIGURATION: ${{ inputs.configuration }}
           BUILD_SYSTEM: ${{ inputs.buildSystem }}
           ARCH: ${{ inputs.platform }}
           SIMD: ${{ inputs.simd }}