Skip to content

fix(cov_docker_script): disable dbus tests for CMake 3.17 - #30

Open
pavankumar464 wants to merge 2 commits into
developfrom
bug/native_build_failing
Open

fix(cov_docker_script): disable dbus tests for CMake 3.17#30
pavankumar464 wants to merge 2 commits into
developfrom
bug/native_build_failing

Conversation

@pavankumar464

@pavankumar464 pavankumar464 commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

The dbus dependency build was failing during CMake configuration because dbus 1.16.2 uses file(CHMOD ...) in its test setup macros, which requires CMake 3.19 while the CI environment provides CMake 3.17.

Add -DDBUS_BUILD_TESTS=OFF to disable test builds for the D-Bus dependency.

The dbus dependency build was failing during CMake configuration because dbus 1.16.2 uses file(CHMOD ...) in its test setup macros, which requires CMake 3.19 while the CI environment provides CMake 3.17.

Add -DENABLE_TESTS=OFF to the dbus cmake_flags in component_config.json so test/CMakeLists.txt is skipped during configuration. This avoids the unsupported file(CHMOD ...) path while still allowing the dbus library and headers to build and install as required by the dependency setup.
@pavankumar464
pavankumar464 requested a review from a team as a code owner July 17, 2026 17:40
Copilot AI review requested due to automatic review settings July 17, 2026 17:40
@github-actions

Copy link
Copy Markdown

📋 PR Format Reminder

  • Title: fix(cov_docker_script): disable dbus tests for CMake 3.17 — expected TICKET-123 : description
    (Multiple tickets OK: RDKCOM-5492 RDKBDEV-3336 : ... | Include US ticket + subtask for user-stories)
  • Description missing:
    • Reason for change
    • Test Procedure
    • Risks (Low / Medium / High)
    • Priority (P0 / P1 / P2)

Expected:

TICKET-123 : brief description

Reason for change: why
Test Procedure: how to verify
Risks: Low / Medium / High
Priority: P0 / P1 / P2

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Coverity/native dependency build configuration to avoid a CMake-version incompatibility when configuring the dbus dependency in CI (CMake 3.17), by skipping dbus’s test configuration.

Changes:

  • Add -DENABLE_TESTS=OFF to the dbus cmake_flags in cov_docker_script/component_config.json to prevent configuring/building dbus tests.

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

Copilot AI review requested due to automatic review settings July 17, 2026 17:50

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Comment thread cov_docker_script/component_config.json
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