Skip to content

Conversation

@x15sr71
Copy link
Contributor

@x15sr71 x15sr71 commented Dec 27, 2025

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

Re-introduces CCExtractor to homebrew-core with full system library support on both macOS and Linux.

CCExtractor is a command-line tool for extracting closed captions and subtitles from video streams, supporting EIA-608/CEA-708, DVB, Teletext, and multiple container formats.

Background

CCExtractor was removed from homebrew-core in December 2023 (#158204) due to bundled third-party libraries. A
previous reintroduction attempt (#260163) identified that, while macOS supported -system-libs, the Linux build script did not yet provide an equivalent option. Upstream has now resolved this in v0.96.2.

What Changed Upstream

Upstream has added explicit support for system library builds on both platforms:

macOS:

Linux:

Build Details

  • macOS: Builds using ./build.command -system-libs to avoid bundled dependencies and comply with Homebrew policies.
  • Linux: Builds using ./build -system-libs to link against system libraries via pkg-config.

Both platforms link exclusively against system-provided libraries with no vendored code.

Verification

Tested on macOS 15.2 (Apple Silicon):

  • brew install --build-from-source ccextractor
  • brew test ccextractor
  • brew audit --strict ccextractor
  • brew linkage --test ccextractor — Confirms system library usage

Tested on Linux (Ubuntu 22.04):

  • brew install --build-from-source ccextractor
  • ldd $(brew --prefix)/bin/ccextractor | grep -E "libpng|freetype|utf8proc"
  • readelf -d confirms no bundled libraries
  • brew audit --strict ccextractor
  • brew test ccextractor

The resulting binaries on both platforms link against Homebrew-provided libraries and run correctly.

@github-actions github-actions bot added new formula PR adds a new formula to Homebrew/homebrew-core rust Rust use is a significant feature of the PR or issue labels Dec 27, 2025
@github-actions
Copy link
Contributor

Thanks for contributing to Homebrew! 🎉 It looks like you're having trouble with a CI failure. See our contribution guide for help. You may be most interested in the section on dealing with CI failures. You can find the CI logs in the Checks tab of your pull request.

@chenrui333 chenrui333 marked this pull request as draft December 27, 2025 05:25
@chenrui333 chenrui333 added the CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. label Dec 27, 2025
@x15sr71 x15sr71 force-pushed the ccextractor-0.96.2 branch 7 times, most recently from 220e76a to bdbea8e Compare December 29, 2025 06:42
@SMillerDev
Copy link
Member

  <command-line>: note: this is the location of the previous definition
  ../src/lib_ccx/params.c:16:10: fatal error: utf8proc/utf8proc.h: No such file or directory
     16 | #include <utf8proc/utf8proc.h>
        |          ^~~~~~~~~~~~~~~~~~~~~
  compilation terminated.
  In file included from /home/linuxbrew/.linuxbrew/Cellar/gpac/2.4.0_4/include/gpac/setup.h:59,
                   from /home/linuxbrew/.linuxbrew/Cellar/gpac/2.4.0_4/include/gpac/tools.h:33,
                   from /home/linuxbrew/.linuxbrew/Cellar/gpac/2.4.0_4/include/gpac/isomedia.h:74,
                   from ../src/lib_ccx/mp4.c:5:

@x15sr71 x15sr71 force-pushed the ccextractor-0.96.2 branch 3 times, most recently from b871a17 to 21e2463 Compare December 29, 2025 16:17
@x15sr71 x15sr71 changed the title ccextractor 0.96.2 (new formula) ccextractor 0.96.3 (new formula) Dec 29, 2025
@x15sr71
Copy link
Contributor Author

x15sr71 commented Dec 29, 2025

@SMillerDev, Thanks for flagging this — fixed upstream.

The utf8proc header path issue is resolved in v0.96.3.

Changes:

I’ve marked the PR as ready for review. Please let me know if you’d like any further adjustments.

@x15sr71
Copy link
Contributor Author

x15sr71 commented Jan 5, 2026

Hi @bevanjkay,
I noticed you recently reviewed/merged a few similar formula PRs, so I was wondering if you might have time to take a look at this one as well. All checks are passing and it’s been marked ready for review. Happy to make any changes if needed.
Thanks for your time!

@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2026

🤖 An automated task has requested bottles to be published to this PR.

Caution

Please do not push to this PR branch before the bottle commits have been pushed, as this results in a state that is difficult to recover from. If you need to resolve a merge conflict, please use a merge commit. Do not force-push to this PR branch.

@github-actions github-actions bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label Jan 5, 2026
@BrewTestBot BrewTestBot enabled auto-merge January 5, 2026 21:53
@BrewTestBot BrewTestBot added this pull request to the merge queue Jan 5, 2026
Merged via the queue into Homebrew:main with commit 33727a0 Jan 5, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. new formula PR adds a new formula to Homebrew/homebrew-core rust Rust use is a significant feature of the PR or issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants