Merge pull request #49 from stefanb2/topic-explicitly-require-threads

Explicitly require threads
bkueng · Nov 29, 2023 · aa88c2e · aa88c2e
2 parents 1dd28f2 + 40c21ed
commit aa88c2e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/.github/workflows/build-linux.yml b/.github/workflows/build-linux.yml
@@ -12,6 +12,7 @@ jobs:
     strategy:
       matrix:
         container:
+          - debian:oldstable
           - debian:stable
           - debian:testing
           - fedora:38
@@ -33,6 +34,11 @@ jobs:
           - container: fedora:rawhide
             no_x11:    true
 
+        exclude:
+          # Debian Bullseye doesn't have Qt6
+          - container: debian:oldstable
+            qt:        6
+
       fail-fast: false
 
     container:

diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -15,6 +15,7 @@ if(CMAKE_BUILD_TYPE STREQUAL Debug)
     find_package(Qt${QT_VERSION_MAJOR} REQUIRED COMPONENTS Test)
 endif()
 find_package(OpenSSL REQUIRED)
+find_package(Threads REQUIRED)
 
 # exposes ${TS_FILES}
 add_subdirectory(data/translations)
@@ -79,6 +80,7 @@ target_link_libraries(${TARGET} PRIVATE
     Qt::Gui
     Qt::Widgets
     OpenSSL::Crypto
+    Threads::Threads
 )
 target_include_directories(${TARGET} PRIVATE ${OPENSSL_INCLUDE_DIR})