From 0e0671bd04d464c10c6fe62640374c2f0b8038a7 Mon Sep 17 00:00:00 2001 From: Kelly Kinkade Date: Mon, 9 Nov 2026 10:44:20 -0600 Subject: [PATCH] check-type-sizes: don't assume windows artifact is double-wrapped The dfhack build-windows workflow now uploads the contents of output/ directly instead of wrapping them in a .tar.bz2. Only untar the downloaded artifact when a tarball is actually present, matching the pattern used in dfhack's test-suite workflow. --- .github/workflows/check-type-sizes.yml | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/.github/workflows/check-type-sizes.yml b/.github/workflows/check-type-sizes.yml index 219741b8e9..f83262e8cf 100644 --- a/.github/workflows/check-type-sizes.yml +++ b/.github/workflows/check-type-sizes.yml @@ -83,9 +83,11 @@ jobs: shell: bash run: | for name in old new; do - cd ${name} - tar xjf sizes-${{ runner.os }}-${{ matrix.arch }}-${name}.tar.bz2 - cd .. + tarball=${name}/sizes-${{ runner.os }}-${{ matrix.arch }}-${name}.tar.bz2 + if [ -f "${tarball}" ]; then + tar xjf "${tarball}" -C ${name} + rm "${tarball}" + fi done old/xml-dump-type-sizes > sizes-old.txt new/xml-dump-type-sizes > sizes-new.txt