Skip to content

Adjust libs to new Common Folder#337

Open
nxtum wants to merge 2 commits into
rdkcentral:masterfrom
nxtum:ThunderCommonFolder
Open

Adjust libs to new Common Folder#337
nxtum wants to merge 2 commits into
rdkcentral:masterfrom
nxtum:ThunderCommonFolder

Conversation

@nxtum
Copy link
Copy Markdown
Contributor

@nxtum nxtum commented May 27, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 27, 2026 08:43
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates client libraries to use Thunder’s new Common headers/package (moving Types.h from plugins to common) and wires the corresponding CMake dependencies so targets can compile against the new layout.

Changes:

  • Replace #include <plugins/Types.h> with #include <common/Types.h> where applicable.
  • Add ${NAMESPACE}Common find_package(...) and link ${NAMESPACE}Common::${NAMESPACE}Common for updated components.
  • Update the compositorclient test plugin to include/link Thunder Common.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
Source/playerinfo/PlayerInfo.cpp Switch Types.h include to common/Types.h.
Source/playerinfo/CMakeLists.txt Add Thunder Common package + link dependency.
Source/displayinfo/DisplayInfo.cpp Switch Types.h include to common/Types.h.
Source/displayinfo/CMakeLists.txt Add Thunder Common package + link dependency.
Source/deviceinfo/DeviceInfo.cpp Add common/Types.h include (but missing corresponding CMake dependency update).
Source/cryptography/Cryptography.cpp Switch Types.h include to common/Types.h.
Source/cryptography/CMakeLists.txt Add Thunder Common package + link dependency (incl. software variant).
Source/compositorclient/test/client-renderer/plugin/Module.h Include Thunder Common header umbrella.
Source/compositorclient/test/client-renderer/plugin/CMakeLists.txt Add Thunder Common package + link dependency.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Source/deviceinfo/DeviceInfo.cpp
@nxtum nxtum requested a review from sebaszm May 27, 2026 09:42
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