Skip to content

VPAAMP-588: Disable pre-tune - #1618

Merged
pstroffolino merged 3 commits into
dev_sprint_25_2from
feature/VPAAMP-588-sprint
Jun 18, 2026
Merged

VPAAMP-588: Disable pre-tune#1618
pstroffolino merged 3 commits into
dev_sprint_25_2from
feature/VPAAMP-588-sprint

Conversation

@Gnanesha

Copy link
Copy Markdown
Contributor

Reason for Change: fake tune disabled for all versions of AAMP
Test Procedure: refer ticket
Risks: Low
Priority: P1

Reason for Change: fake tune disabled for all versions of AAMP
Test Procedure: refer ticket
Risks: Low
Priority: P1
@Gnanesha
Gnanesha requested a review from a team as a code owner June 16, 2026 21:02
@Gnanesha
Gnanesha requested a review from Copilot June 16, 2026 21:02

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

This PR aims to disable AAMP “pre-tune” (fake tune / preinit decoding) behavior by removing the CMake-time propagation of -DUSE_PREINIT_DECODING from the top-level build configuration.

Changes:

  • Removed the top-level CMAKE_PREINIT_DECODER block that previously appended -DUSE_PREINIT_DECODING=1 to AAMP and JS bindings compile flags.

Comment thread CMakeLists.txt
Reason for Change: fake tune disabled for all versions of AAMP
Test Procedure: refer ticket
Risks: Low
Priority: P1

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 2 out of 2 changed files in this pull request and generated no new comments.

@pstroffolino
pstroffolino merged commit c46b53a into dev_sprint_25_2 Jun 18, 2026
8 checks passed
@pstroffolino
pstroffolino deleted the feature/VPAAMP-588-sprint branch June 18, 2026 20:03
shripadbpersonal pushed a commit that referenced this pull request Jul 20, 2026
* VPAAMP-588: Disable pre-tune

Reason for Change: fake tune disabled for all versions of AAMP
Test Procedure: refer ticket
Risks: Low
Priority: P1

---------

Co-authored-by: pstroffolino <Philip_Stroffolino@cable.comcast.com>
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.

3 participants