RDKEMW-15910: bcm bb/bbappend analysis for sysint - #91
Conversation
There was a problem hiding this comment.
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_patchhook toROOTFS_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
elsebranch,mvruns wheneverimageFlasher.shis missing, but it doesn’t verify thatimageFlasher_generic.shexists. 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
elsebranch renamesinit-zram_generic.shwithout checking it exists. If both the vendor and generic scripts are absent,mvwill fail and can break image creation. Please guard with a-fcheck (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.
|
CCI-Build-Verified +1 |
|
CCI-Build-Verified -1 |
|
CCI-Test-Verified -1 |
|
CCI-Build-Verified +1 |
|
CCI-Test-Verified +1 |
|
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:
For more information about GitHub Code Scanning, check out the documentation. |
|
CCI-Build-Verified +1 |
1 similar comment
|
CCI-Build-Verified +1 |
|
CCI-Test-Verified +1 |
1 similar comment
|
CCI-Test-Verified +1 |
|
CCI-Build-Verified -1 |
|
CCI-Build-Verified -1 |
No description provided.