Skip to content

[FIX]: point Artifact_names to (with migrations) builds to restore CEA-708 output#1122

Open
x15sr71 wants to merge 3 commits into
CCExtractor:masterfrom
x15sr71:fix/artifact-names-migrations-build
Open

[FIX]: point Artifact_names to (with migrations) builds to restore CEA-708 output#1122
x15sr71 wants to merge 3 commits into
CCExtractor:masterfrom
x15sr71:fix/artifact-names-migrations-build

Conversation

@x15sr71

@x15sr71 x15sr71 commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Please prefix your pull request with one of the following: [FEATURE] [FIX] [IMPROVEMENT].

In raising this pull request, I confirm the following (please check boxes):

  • I have read and understood the contributors guide.
  • I have checked that another pull request for this purpose does not exist.
  • I have considered, and confirmed that this submission will be valuable to others.
  • I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
  • I give this submission freely, and claim no ownership to its content.

My familiarity with the project is as follows (check one):

  • I have never used the project.
  • I have used the project briefly.
  • I have used the project extensively, but have not contributed previously.
  • I am an active contributor to the project.

Problem

CEA-708 regression tests have shown "No output generated but there should be" since March 17. Root cause: the artifact names this platform downloads no longer point to the full-Rust builds.

On March 17, two CCExtractor CI commits restructured the build artifacts into a dual min-Rust / full-Rust setup:

After these commits, the artifact names Artifact_names matches against — "CCExtractor Linux build" and "CCExtractor Windows x64 Release build" — resolve to the -DDISABLE_RUST (min-Rust) builds. DISABLE_RUST compiles out the Rust DTVCC decoder (src/lib_ccx/ccx_dtvcc.h), which handles all CEA-708 --service output. The platform has been downloading and testing the wrong binary on both platforms ever since.

The Linux commit (3d0ce97) explicitly noted that this platform's Artifact_names would need updating:

The sample platform will need a corresponding update to recognize and test the new "with migrations" artifact.

Evidence

  • DB: regression tests 143/144/145/148/150 dropped to 0% file-row production at the exact commits above (RT 145: ~89% → 0%, RT 143/144/148/150: ~48% → 0%).
  • Confirmed still occurring on clean completed runs 9325/9326 (June 12): CEA-708 tests report normally but produce zero output files.
  • Verified end-to-end by building both binaries locally: the DISABLE_RUST build produces no output for --svc 1 --out=srt; the full-Rust ("with migrations") build writes out.p1.svc01.srt — exactly the path the platform expects.

Fix

Point both artifact names to the (with migrations) (full-Rust) builds, which is the build the platform is intended to test.

@sonarqubecloud

Copy link
Copy Markdown

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.

1 participant