Skip to content

vapoursynth-vszip 22.1.0 (new formula) - #298871

Merged
BrewTestBot merged 2 commits into
mainfrom
vapoursynth-vszip
Aug 14, 2026
Merged

vapoursynth-vszip 22.1.0 (new formula)#298871
BrewTestBot merged 2 commits into
mainfrom
vapoursynth-vszip

Conversation

@cho-m

@cho-m cho-m commented Aug 14, 2026

Copy link
Copy Markdown
Member

This doesn't meet notability but considering an exception to simplify

Zig makes it harder to pass our -headerpad_max_install_names (usually don't see an issue as most Zig dependents are executables rather than libraries).

Here the default @rpath/... is actually okay to use since the library is a plugin loaded by VapourSynth so can use preserve_rpath. Seemed safer to isolate this DSL usage to a separate formula and also allows scoping the Zig dependency, which is fragile and will keep breaking across releases until zig 1.0.

The parent package that introduces dependency does meet our notability requirement: https://github.com/Jaded-Encoding-Thaumaturgy/vs-jetpack (just barely at 77 stars)


Also, loading VapourSynth plugins is a bit complicated and won't work correctly in virtualenv. We will likely need to add VAPOURSYNTH_EXTRA_PLUGIN_PATH but this only accepts a single path and a user may want to override this at runtime.


  • 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>?
  • Is your test running fine brew test <formula>?
  • 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>?

  • I did not use AI/LLM to create this PR, or I disclosed the tool/model below and reviewed its output; I did not attribute commits to AI and will answer maintainer questions and review comments myself without AI/LLM.

@cho-m cho-m added notability Project is not notable enough for inclusion CI-skip-new-formulae Pass --skip-new to brew test-bot. labels Aug 14, 2026
@github-actions github-actions Bot added python Python use is a significant feature of the PR or issue new formula PR adds a new formula to Homebrew/homebrew-core zig Zig use is a significant feature of the PR or issue labels Aug 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

馃 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 Aug 14, 2026
@BrewTestBot
BrewTestBot enabled auto-merge August 14, 2026 22:52
@BrewTestBot
BrewTestBot added this pull request to the merge queue Aug 14, 2026
Merged via the queue into main with commit de4e4fa Aug 14, 2026
22 checks passed
@BrewTestBot
BrewTestBot deleted the vapoursynth-vszip branch August 14, 2026 23:00
@chenrui333 chenrui333 mentioned this pull request Aug 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. CI-skip-new-formulae Pass --skip-new to brew test-bot. new formula PR adds a new formula to Homebrew/homebrew-core notability Project is not notable enough for inclusion python Python use is a significant feature of the PR or issue zig Zig use is a significant feature of the PR or issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants