Skip to content

RDKEMW-15910: bcm bb/bbappend analysis for sysint - #91

Open
Abhinavpv28 wants to merge 2 commits into
developfrom
Abhinavpv28-patch-1
Open

RDKEMW-15910: bcm bb/bbappend analysis for sysint#91
Abhinavpv28 wants to merge 2 commits into
developfrom
Abhinavpv28-patch-1

Conversation

@Abhinavpv28

Copy link
Copy Markdown

No description provided.

Copilot AI review requested due to automatic review settings May 19, 2026 06:15
@Abhinavpv28
Abhinavpv28 requested review from a team as code owners May 19, 2026 06:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Yocto image recipe to add a new rootfs postprocess step that prefers vendor-provided “legacy entos” properties/scripts and otherwise falls back to the generic middleware versions.

Changes:

  • Appends a new legacy_entos_support_patch hook to ROOTFS_POSTPROCESS_COMMAND.
  • Adds a legacy_entos_support_patch() postprocess function to remove generic files when vendor versions exist, or rename generic files into place otherwise.
Comments suppressed due to low confidence (2)

recipes-images/rdk-fullstack-image.bb:67

  • In the else branch, mv runs whenever imageFlasher.sh is missing, but it doesn’t verify that imageFlasher_generic.sh exists. If neither script is present, this will fail the postprocess step. Add a source-file existence check (and/or a warning) before attempting the rename.
    if [ -f "${IMAGE_ROOTFS}/lib/rdk/imageFlasher.sh" ]; then
        rm -f ${IMAGE_ROOTFS}/lib/rdk/imageFlasher_generic.sh
    else
        mv ${IMAGE_ROOTFS}/lib/rdk/imageFlasher_generic.sh ${IMAGE_ROOTFS}/lib/rdk/imageFlasher.sh
    fi

recipes-images/rdk-fullstack-image.bb:73

  • Similarly, the else branch renames init-zram_generic.sh without checking it exists. If both the vendor and generic scripts are absent, mv will fail and can break image creation. Please guard with a -f check (and/or warn) before renaming.
    if [ -f "${IMAGE_ROOTFS}/lib/rdk/init-zram.sh" ]; then
        rm -f ${IMAGE_ROOTFS}/lib/rdk/init-zram_generic.sh
    else
        mv ${IMAGE_ROOTFS}/lib/rdk/init-zram_generic.sh ${IMAGE_ROOTFS}/lib/rdk/init-zram.sh
    fi    

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

Comment thread recipes-images/rdk-fullstack-image.bb Outdated
Comment thread recipes-images/rdk-fullstack-image.bb
Comment thread recipes-images/rdk-fullstack-image.bb
@svc-rdkeportal01

Copy link
Copy Markdown

CCI-Build-Verified +1
sha: cc9a8ee

@svc-rdkeportal01

Copy link
Copy Markdown

CCI-Build-Verified -1
sha: cc9a8ee

2 similar comments
@svc-rdkeportal01

Copy link
Copy Markdown

CCI-Build-Verified -1
sha: cc9a8ee

@svc-rdkeportal01

Copy link
Copy Markdown

CCI-Build-Verified -1
sha: cc9a8ee

@svc-rdkeportal01

Copy link
Copy Markdown

CCI-Test-Verified -1
sha: cc9a8ee

@svc-rdkeportal01

Copy link
Copy Markdown

CCI-Build-Verified +1
sha: cc9a8ee

@svc-rdkeportal01

Copy link
Copy Markdown

CCI-Test-Verified +1
sha: cc9a8ee

@nhanasi nhanasi changed the title Update rdk-fullstack-image.bb RDKEMW-15910: bcm bb/bbappend analysis for sysint May 27, 2026
nhanasi
nhanasi previously approved these changes May 27, 2026
goruklu
goruklu previously approved these changes May 27, 2026
@nhanasi
nhanasi dismissed stale reviews from goruklu and themself via 00b1aed May 27, 2026 14:40
Copilot AI review requested due to automatic review settings May 27, 2026 14:40
@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

Copilot AI left a comment

Copy link
Copy Markdown

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 4 comments.

Comment thread recipes-images/rdk-fullstack-image.bb
Comment thread recipes-images/rdk-fullstack-image.bb
Comment thread recipes-images/rdk-fullstack-image.bb
Comment thread recipes-images/rdk-fullstack-image.bb
@svc-rdkeportal01

Copy link
Copy Markdown

CCI-Build-Verified +1
sha: 00b1aed

1 similar comment
@svc-rdkeportal01

Copy link
Copy Markdown

CCI-Build-Verified +1
sha: 00b1aed

@svc-rdkeportal01

Copy link
Copy Markdown

CCI-Test-Verified +1
sha: 00b1aed

1 similar comment
@svc-rdkeportal01

Copy link
Copy Markdown

CCI-Test-Verified +1
sha: 00b1aed

@svc-rdkeportal01

Copy link
Copy Markdown

CCI-Build-Verified -1
sha: 00b1aed

@svc-rdkeportal01

Copy link
Copy Markdown

CCI-Build-Verified -1
sha: 00b1aed

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.

6 participants