Skip to content

Commit e976220

Browse files
committed
Fix build-winxp.yml
1 parent ee8b570 commit e976220

1 file changed

Lines changed: 11 additions & 7 deletions

File tree

.github/workflows/build-winxp.yml

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -82,17 +82,12 @@ jobs:
8282
echo "=== DEBUG: key libs present in MXE_PREFIX/lib ==="
8383
ls -la "$MXE_PREFIX/lib" | grep -Ei "json|sdl|jpeg|png|webp|smpeg|modplug|vorbis|ogg|ws2|iphlp|winmm|dxguid" || true
8484
85-
# IMPORTANT:
86-
# Do NOT use -I$MXE_PREFIX/include/jsoncpp because it doesn't exist in your MXE layout.
87-
# Put MXE include FIRST so <json/json.h> resolves to MXE headers (matching libjsoncpp.a),
88-
# and keep deps/include later (for other headers).
85+
# Put MXE include FIRST so <json/json.h> resolves to MXE headers (matching libjsoncpp.a)
8986
export CPPFLAGS="-I$MXE_PREFIX/include -I$MXE_PREFIX/include/SDL -I$GITHUB_WORKSPACE/deps/include/SDL_net -I$GITHUB_WORKSPACE/deps/include"
9087
export CFLAGS="$CPPFLAGS"
9188
export CXXFLAGS="$CPPFLAGS"
9289
93-
# NOTE (Option A):
94-
# Do NOT force -l... libs via LDFLAGS here.
95-
# The corrected src/CMakeLists.txt links required libs with correct ordering.
90+
# Option A: do not force -l... via LDFLAGS (CMakeLists handles link order)
9691
unset LDFLAGS || true
9792
9893
rm -rf build
@@ -108,6 +103,15 @@ jobs:
108103
109104
make -j"$(nproc)" VERBOSE=1
110105
106+
- name: Upload link.txt
107+
uses: actions/upload-artifact@v4
108+
with:
109+
name: linktxt-${{ env.SAFE_REF }}
110+
path: |
111+
build/src/CMakeFiles/openxcom.dir/link.txt
112+
build/CMakeFiles/openxcom.dir/link.txt
113+
if-no-files-found: warn
114+
111115
- name: Package zip
112116
run: |
113117
cd build/bin

0 commit comments

Comments
 (0)