Skip to content

quicklook-video: Add v2 for older macOS#275349

Draft
injust wants to merge 1 commit into
Homebrew:mainfrom
injust:patch-1
Draft

quicklook-video: Add v2 for older macOS#275349
injust wants to merge 1 commit into
Homebrew:mainfrom
injust:patch-1

Conversation

@injust

@injust injust commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

v3 is supported on Sequoia (the latest release is an exception), but doesn't work in practice.

Marginal/QuickLookVideo#201 (comment):

The APIs that 3.x uses to support thumbnails are supposed to work on macOS 15 but don't work in practice, so you're unlikely to see any thumbnails at all. Please use version 2.x.

Future (and previous) v3 releases will support Sequoia though, so add v2 to the cask for macOS versions between Monterey and Sonoma.


After making any changes to a cask, existing or new, verify:

Additionally, if adding a new cask:

  • Named the cask according to the token reference.
  • Checked the cask was not already refused (add your cask's name to the end of the search field).
  • brew audit --cask --new <cask> worked successfully.
  • HOMEBREW_NO_INSTALL_FROM_API=1 brew install --cask <cask> worked successfully.
  • brew uninstall --cask <cask> worked successfully.

  • AI was used to generate or assist with generating this PR. Please specify below how you used AI to help you, and what steps you have taken to manually verify the changes, including zap stanza paths.

Comment thread Casks/q/quicklook-video@2.rb Outdated
@injust injust changed the title quicklook-video@2: Add cask for v2 quicklook-video: Add v2 for older macOS Jul 17, 2026
@injust

injust commented Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

You know what, it's probably better to hold this PR until the next v3 release that supports Sequoia again, just to avoid churn.

Right now, Sequoia users already get an error because the latest version is Tahoe-or-newer.

@injust

injust commented Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

Failure seems correct, will mark this as a draft and come back when there's a new v3 release.

@injust
injust marked this pull request as draft July 17, 2026 07:19
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.

3 participants