[PR #2058] feat: implement DVB subtitle splitting (--split-dvb-subs) #2870

New Issue

claunia · 2026-01-29T17:24:20Z

claunia commented

2026-01-29 17:24:20 +00:00

Original Pull Request: https://github.com/CCExtractor/ccextractor/pull/2058

State: closed
Merged: No

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.
I have mentioned this change in the changelog

.

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

I have never used CCExtractor.
I have used CCExtractor just a couple of times.
I absolutely love CCExtractor, but have not contributed previously.
I am an active contributor to CCExtractor.

Summary

This PR completes verification of Issue #447 and applies a few small but necessary fixes identified during review. The --split-dvb-subs feature is confirmed to work correctly with real DVB broadcast samples.

Features

--split-dvb-subs: Extract each DVB stream to separate files (e.g., output_eng_0x0000.srt, output_spa_0x006F.srt)
--no-dvb-dedup: Disable ring buffer deduplication to retain all subtitles (useful for debugging)
Multi-pipeline: Parallel processing of multiple DVB streams
Post-EOF handling: Captures DVB packets after video EOF for complete extraction

Bug Fix: GPAC fallback URL

Fixed Windows CI failures when Chocolatey returns 503 errors. Corrected the fallback download URL from:

❌ release/2.4.0/gpac-2.4.0-rev0-gc05db8ba-master-x64.exe (404)
✅ release/2.4/gpac-2.4-rev0-g5d70253a-master-x64.exe (200 OK)

Test Results

Sample	Streams	Before	After	Result
multiprogram_spain.ts	5	1 file	5 files	✅
tnt-uhf30-546MHz…	6	1 file	6 files	✅
BBC1.ts	1	1 file	1 file	✅
All other samples	1-2	✓	✓	✅

Coverage: Split, deduplication (on/off), multi-pipeline, Windows CI, Linux CI

**Original Pull Request:** https://github.com/CCExtractor/ccextractor/pull/2058 **State:** closed **Merged:** No --- In raising this pull request, I confirm the following (please check boxes): I have read and understood the [contributors guide](https://github.com/CCExtractor/ccextractor/blob/master/.github/CONTRIBUTING.md) . - [x] I have checked that another pull request for this purpose does not exist. - [x] I have considered, and confirmed that this submission will be valuable to others. - [x] I accept that this submission may not be used, and the pull request closed at the will of the maintainer. - [x] I give this submission freely, and claim no ownership to its content. - [x] I have mentioned this change in the [changelog](https://github.com/CCExtractor/ccextractor/blob/master/docs/CHANGES.TXT) . My familiarity with the project is as follows (check one): - [ ] I have never used CCExtractor. - [x] I have used CCExtractor just a couple of times. - [ ] I absolutely love CCExtractor, but have not contributed previously. - [x] I am an active contributor to CCExtractor. ___________________________________________________________________________________________________________ ### **Summary** This PR completes verification of Issue #447 and applies a few small but necessary fixes identified during review. The --split-dvb-subs feature is confirmed to work correctly with real DVB broadcast samples. ## Features - **`--split-dvb-subs`**: Extract each DVB stream to separate files (e.g., `output_eng_0x0000.srt`, `output_spa_0x006F.srt`) - **`--no-dvb-dedup`**: Disable ring buffer deduplication to retain all subtitles (useful for debugging) - **Multi-pipeline**: Parallel processing of multiple DVB streams - **Post-EOF handling**: Captures DVB packets after video EOF for complete extraction ## Bug Fix: GPAC fallback URL Fixed Windows CI failures when Chocolatey returns 503 errors. Corrected the fallback download URL from: - ❌ `release/2.4.0/gpac-2.4.0-rev0-gc05db8ba-master-x64.exe` (404) - ✅ `release/2.4/gpac-2.4-rev0-g5d70253a-master-x64.exe` (200 OK) ## Test Results | Sample | Streams | Before | After | Result | |--------|----------|---------|--------|---------| | multiprogram_spain.ts | 5 | 1 file | 5 files | ✅ | | tnt-uhf30-546MHz… | 6 | 1 file | 6 files | ✅ | | BBC1.ts | 1 | 1 file | 1 file | ✅ | | All other samples | 1-2 | ✓ | ✓ | ✅ | **Coverage**: Split, deduplication (on/off), multi-pipeline, Windows CI, Linux CI

claunia added the pull-request label 2026-01-29 17:24:20 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/ccextractor#2870