Skip to content

Commit cda2950

Browse files
committed
Install SDL2 headers in CI tests job
1 parent f3b4019 commit cda2950

1 file changed

Lines changed: 12 additions & 6 deletions

File tree

.github/workflows/ci.yml

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,15 +13,21 @@ jobs:
1313
steps:
1414
- uses: actions/checkout@v4
1515

16-
# The tests mock libcurl symbols (see tests/curl_mock.cpp), but the
17-
# production code under test still includes <curl/curl.h> for the type
18-
# definitions. ZLIB is needed by ZipExtractor. zlib1g-dev is usually
19-
# pre-installed on ubuntu-latest, but make it explicit so this doesn't
20-
# silently break on a future runner image.
16+
# The tests mock libcurl + SDL symbols, but the production code under
17+
# test still pulls the headers for type definitions:
18+
# - <curl/curl.h> via DownloadManager.h
19+
# - <SDL2/SDL.h> + <SDL2/SDL_ttf.h> via TextCache.h
20+
# - <zlib.h> via ZipExtractor (find_package(ZLIB REQUIRED))
21+
# zlib1g-dev is usually pre-installed on ubuntu-latest but make it
22+
# explicit so a future runner image doesn't silently break us.
2123
- name: Install build dependencies
2224
run: |
2325
sudo apt-get update
24-
sudo apt-get install -y libcurl4-openssl-dev zlib1g-dev
26+
sudo apt-get install -y \
27+
libcurl4-openssl-dev \
28+
zlib1g-dev \
29+
libsdl2-dev \
30+
libsdl2-ttf-dev
2531
2632
- name: Configure
2733
run: |

0 commit comments

Comments
 (0)