Skip to content

Integrate rdkchronylibctrl library with systimemgr and add verification toolkit#598

Draft
Copilot wants to merge 11 commits intodevelopfrom
copilot/add-rdkchronylibctrl-linking
Draft

Integrate rdkchronylibctrl library with systimemgr and add verification toolkit#598
Copilot wants to merge 11 commits intodevelopfrom
copilot/add-rdkchronylibctrl-linking

Conversation

Copy link

Copilot AI commented Feb 23, 2026

Adds rdkchronylibctrl (chrony control library) to systimemgr with complete build/runtime dependencies. Includes security hardening and comprehensive verification tools.

Integration

recipes-common/systimemgr/systimemgr_git.bb

DEPENDS = "... rdkchronylibctrl"           # Build-time: headers available
LDFLAGS:append = " -lchronyctl"            # Link-time: binary linkage
RDEPENDS:${PN} += "... rdkchronylibctrl"   # Runtime: .so deployed to target

All three required for complete shared library integration in Yocto.

recipes-support/rdkchronylibctrl/

  • Library recipe and sources from topic/chrony-prd branch
  • Fixes applied:
    • Socket permissions: 0666 → 0600
    • Removed duplicate FILES/RDEPENDS definitions
    • Fixed include path: ${S}/include → ${S}
    • Explicit null termination in strncpy

Verification Tools

Added 17+ verification methods across 4 docs (25KB total):

verify_library_linkage.sh - Automated 8-test script with color-coded output

./verify_library_linkage.sh
# Tests: binary deps, file existence, /proc/maps, pmap, package install

LIBRARY_LINKAGE_VERIFICATION.md - Complete reference covering:

  • Static analysis: ldd, readelf, objdump, nm
  • Runtime checks: /proc/PID/maps, pmap, pldd
  • Build system: BitBake logs, dependency graphs
  • Troubleshooting scenarios

Quick verification:

ls /usr/lib/libchronyctl.so && \
ldd /usr/bin/sysTimeMgr | grep chrony && \
cat /proc/$(pidof sysTimeMgr)/maps | grep chrony

Comparison with topic/chrony-prd

Base integration identical. This PR adds:

  • SRCREV pinning, version metadata (1.4.0, PR=r0)
  • Security fixes (permissions, string handling)
  • Code quality (no duplicates, correct paths)
  • Verification toolkit

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 7 commits February 23, 2026 12:35
Co-authored-by: sindhu-krishnan <102755514+sindhu-krishnan@users.noreply.github.com>
Co-authored-by: sindhu-krishnan <102755514+sindhu-krishnan@users.noreply.github.com>
Co-authored-by: sindhu-krishnan <102755514+sindhu-krishnan@users.noreply.github.com>
Co-authored-by: sindhu-krishnan <102755514+sindhu-krishnan@users.noreply.github.com>
Co-authored-by: sindhu-krishnan <102755514+sindhu-krishnan@users.noreply.github.com>
Co-authored-by: sindhu-krishnan <102755514+sindhu-krishnan@users.noreply.github.com>
Copilot AI changed the title [WIP] Integrate rdkchronylibctrl library into systemmgr_git.bb Integrate rdkchronylibctrl library with systimemgr Feb 23, 2026
Co-authored-by: sindhu-krishnan <102755514+sindhu-krishnan@users.noreply.github.com>
Copilot AI changed the title Integrate rdkchronylibctrl library with systimemgr Add runtime dependency for rdkchronylibctrl in systimemgr Feb 23, 2026
Copilot AI changed the title Add runtime dependency for rdkchronylibctrl in systimemgr Integrate rdkchronylibctrl with security and quality fixes Feb 23, 2026
Copilot AI and others added 2 commits February 23, 2026 16:26
Co-authored-by: sindhu-krishnan <102755514+sindhu-krishnan@users.noreply.github.com>
Co-authored-by: sindhu-krishnan <102755514+sindhu-krishnan@users.noreply.github.com>
Copilot AI changed the title Integrate rdkchronylibctrl with security and quality fixes Integrate rdkchronylibctrl library with systimemgr and add verification toolkit Feb 23, 2026
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