Allow pipefail when filtering collected msvc symbols

aws · Jul 31, 2024 · 101f06c · 101f06c
1 parent 6fb34c9
commit 101f06c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/.github/workflows/sys-bindings-generator.yml b/.github/workflows/sys-bindings-generator.yml
@@ -49,7 +49,7 @@ jobs:
     if: github.repository == 'aws/aws-lc-rs'
     runs-on: windows-latest
     strategy:
-      fail-fast: true
+      fail-fast: false
       matrix:
         target:
           - 'aarch64-pc-windows-msvc'

diff --git a/scripts/build/collect_symbols.sh b/scripts/build/collect_symbols.sh
@@ -93,8 +93,11 @@ if [[ "${PLATFORM}" = *-msvc ]]; then
   VS_INSTALL_PATH="$("$(echo "${PFx86//\\/\/}//Microsoft Visual Studio/Installer/vswhere.exe")" | grep 'resolvedInstallationPath:' | sed -e 's/[^:]*: \(.*\)$/\1/')"
 
   DUMPBIN="$(ls -1 "${VS_INSTALL_PATH//\\/\/}"/VC/Tools/MSVC/*/bin/Hostx64/${MSVC_ARCH}/dumpbin.exe | tail -n 1)"
-  PATH="$(dirname "${DUMPBIN}")":"${PATH}"
-  dumpbin //EXPORTS //SYMBOLS  ${LIBCRYPTO_PATH} | grep External | grep SECT1 | sed -e 's/.*External\s*|\s*\(.*\)$/\1/'| egrep '^\w' | sort | uniq | filter_symbols >"${SYMBOLS_FILE}"
+  PATH="$(dirname "${DUMPBIN/C:/\/c}")":"${PATH}"
+set +o pipefail
+  dumpbin //EXPORTS //SYMBOLS  "${LIBCRYPTO_PATH}" | grep External | grep SECT1 | sed -e 's/.*External\s*|\s*\(.*\)$/\1/'| grep -E '^\w' | sort | uniq | filter_symbols >"${SYMBOLS_FILE}"
+  echo "dumpbin pipes: ${PIPESTATUS[@]}"
+set -o pipefail
 elif [[ "${LIBCRYPTO_PATH}" = *.dylib ]]; then
   nm --extern-only --defined-only -j  "${LIBCRYPTO_PATH}" | grep -v "${REPO_ROOT}" | sort | uniq | filter_macho_symbols | filter_nm_symbols |  filter_symbols >"${SYMBOLS_FILE}"
 elif [[ "${LIBCRYPTO_PATH}" = *.so || "${LIBCRYPTO_PATH}" = *.lib ]]; then