Skip to content

Add missing thread include#24

Merged
eriksven merged 1 commit intoeclipse-kuksa:mainfrom
boschglobal:fix/missing-include
Feb 28, 2025
Merged

Add missing thread include#24
eriksven merged 1 commit intoeclipse-kuksa:mainfrom
boschglobal:fix/missing-include

Conversation

@SebastianSchildt
Copy link
Contributor

This include is mising. That does not allow building on bookworm, it fails with

[ 85%] Building CXX object src/CMakeFiles/kuksaclient.dir/kuksa_client.cpp.o
/build/kuksa-incubation/kuksa-cpp-client/src/kuksa_client.cpp:418:8: error: 'thread' in namespace 'std' does not name a type
  418 |   std::thread mSubscribeThread;
      |        ^~~~~~
/build/kuksa-incubation/kuksa-cpp-client/src/kuksa_client.cpp:24:1: note: 'std::thread' is defined in header '<thread>'; did you forget to '#include <thread>'?
   23 | #include <spdlog/spdlog.h>
  +++ |+#include <thread>
   24 | #include <vector>

This fixes that

Signed-off-by: Sebastian Schildt <sebastian.schildt@de.bosch.com>
Copy link
Contributor

@eriksven eriksven left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@eriksven eriksven merged commit 5d25abe into eclipse-kuksa:main Feb 28, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants