[PR #1892] [CLOSED] [DOCS](mac): document -system-libs build mode for Homebrew compatibility #2680

Open
opened 2026-01-29 17:23:23 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/CCExtractor/ccextractor/pull/1892
Author: @gaurav02081
Created: 12/24/2025
Status: Closed

Base: masterHead: brew-gaurav


📝 Commits (6)

  • 609a53f [BUG] -out=spupng with EIA608/teletext: offset values in XML may be not correct #893
  • 2b708c4 Enhance SPUPNG offset calculations and XML tag handling in EIA608 encoder
  • 6ed09ea SPUPNG: fix formatting to match clang-format
  • bf9841a Merge branch 'master' into gaurav-v1
  • e42bc2b fixed the merged conflict in the ccx_encoders_common.h
  • 9ab6871 docs(mac): document -system-libs build mode for Homebrew

📊 Changes

6 files changed (+213 additions, -8 deletions)

View changed files

docs/HOMEBREW.md (+158 -0)
src/_deps/corrosion-src (+1 -0)
📝 src/lib_ccx/ccx_encoders_common.c (+1 -0)
📝 src/lib_ccx/ccx_encoders_common.h (+1 -1)
📝 src/lib_ccx/ccx_encoders_spupng.c (+50 -7)
📝 src/lib_ccx/ccx_encoders_structs.h (+2 -0)

📄 Description

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.

PR Summary

This PR documents the macOS -system-libs build mode introduced in #1862, which enables building CCExtractor using system-installed libraries instead of bundled dependencies.

This mode is required for Homebrew compatibility and uses pkg-config to link against Homebrew-provided libraries (e.g. gpac, freetype, libpng, zlib). The default bundled build remains unchanged.

The documentation explains:

When to use the default vs -system-libs build

Required Homebrew dependencies

How to verify the build

CI coverage for the system-libs path

This provides a clear upstream reference for package managers and unblocks future Homebrew formula work.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/CCExtractor/ccextractor/pull/1892 **Author:** [@gaurav02081](https://github.com/gaurav02081) **Created:** 12/24/2025 **Status:** ❌ Closed **Base:** `master` ← **Head:** `brew-gaurav` --- ### 📝 Commits (6) - [`609a53f`](https://github.com/CCExtractor/ccextractor/commit/609a53f373b0ecf97f919babf6b73d7205310f34) [BUG] -out=spupng with EIA608/teletext: offset values in XML may be not correct #893 - [`2b708c4`](https://github.com/CCExtractor/ccextractor/commit/2b708c4a31100b245158ff21e0c6af5cb99fa97c) Enhance SPUPNG offset calculations and XML tag handling in EIA608 encoder - [`6ed09ea`](https://github.com/CCExtractor/ccextractor/commit/6ed09ea3970a44758f4fca0e25a6dfb75552ccea) SPUPNG: fix formatting to match clang-format - [`bf9841a`](https://github.com/CCExtractor/ccextractor/commit/bf9841a2558e038db0d5c526ec7d5a2dbea4be0b) Merge branch 'master' into gaurav-v1 - [`e42bc2b`](https://github.com/CCExtractor/ccextractor/commit/e42bc2b9f9251608a466eb3ac6c798d4fb6852f0) fixed the merged conflict in the ccx_encoders_common.h - [`9ab6871`](https://github.com/CCExtractor/ccextractor/commit/9ab6871a94f8783d3a55da76f43c9f67c548e7be) docs(mac): document -system-libs build mode for Homebrew ### 📊 Changes **6 files changed** (+213 additions, -8 deletions) <details> <summary>View changed files</summary> ➕ `docs/HOMEBREW.md` (+158 -0) ➕ `src/_deps/corrosion-src` (+1 -0) 📝 `src/lib_ccx/ccx_encoders_common.c` (+1 -0) 📝 `src/lib_ccx/ccx_encoders_common.h` (+1 -1) 📝 `src/lib_ccx/ccx_encoders_spupng.c` (+50 -7) 📝 `src/lib_ccx/ccx_encoders_structs.h` (+2 -0) </details> ### 📄 Description <!-- 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):** - [x] 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. - [ ] **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. - [ ] 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. --- PR Summary This PR documents the macOS -system-libs build mode introduced in #1862, which enables building CCExtractor using system-installed libraries instead of bundled dependencies. This mode is required for Homebrew compatibility and uses pkg-config to link against Homebrew-provided libraries (e.g. gpac, freetype, libpng, zlib). The default bundled build remains unchanged. The documentation explains: When to use the default vs -system-libs build Required Homebrew dependencies How to verify the build CI coverage for the system-libs path This provides a clear upstream reference for package managers and unblocks future Homebrew formula work. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 17:23:23 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ccextractor#2680