Skip to content

RDKEMW-18231: Bring RDKEMW-18129 changes to 8.5#3921

Open
psiva01 wants to merge 1 commit intosupport/8.5.3.0from
feature/RDKEMW-18231
Open

RDKEMW-18231: Bring RDKEMW-18129 changes to 8.5#3921
psiva01 wants to merge 1 commit intosupport/8.5.3.0from
feature/RDKEMW-18231

Conversation

@psiva01
Copy link
Copy Markdown
Contributor

@psiva01 psiva01 commented May 7, 2026

RDKEMW-18231: Bring RDKEMW-18129 changes to 8.5

Co-authored-by: Copilot Autofix powered by AI 175728472+Copilot@users.noreply.github.com

RDKEMW-18129: Fix incorrect aamp version issue in diff report

Copilot AI review requested due to automatic review settings May 7, 2026 13:07
@psiva01 psiva01 requested review from a team as code owners May 7, 2026 13:07
Copy link
Copy Markdown
Contributor

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

Ports the RDKEMW-18129 fix onto the 8.5 branch, aiming to correct AAMP version/branch reporting in generated artifacts/diff reports by adjusting recipe metadata and emitted build info.

Changes:

  • Switches PV/PR from conditional (?=) to fixed (=) assignment in the AAMP recipe.
  • Removes the unused AAMP_RELEASE_TAG_NAME variable and stops injecting it into CXXFLAGS.
  • Updates artifacts.info generation to emit AAMP_BRANCH instead of AAMP_RELEASE_TAG_NAME.

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

Comment thread recipes-extended/aamp/aamp_git.bb
Comment thread recipes-extended/aamp/aamp_git.bb Outdated
RDKEMW-18231: Bring RDKEMW-18129 changes to 8.5

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

RDKEMW-18129: Fix incorrect aamp version issue in diff report
@psiva01 psiva01 force-pushed the feature/RDKEMW-18231 branch from df1b6f3 to 0c33468 Compare May 7, 2026 13:24
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