deps(macos): handle CAS's vapoursynth get-include meson probe (upstream drift)#43
Merged
Merged
Conversation
…am drift)
VapourSynth-CAS upstream switched its meson.build VS-include probe from
`import vapoursynth` to `run_command('vapoursynth', 'get-include', ...)`, which
invokes the `vapoursynth` console script not present on the clean runner. The
build_plugin patcher only fired on "import vapoursynth", so CAS now fails to
configure on arm64 (its sole source; x64 also downloads a prebuilt so it slipped
through) → packaging completeness guard fails.
Broaden the patcher trigger to also match `run_command('vapoursynth'`; the
existing `.stdout().strip()` regex already rewrites it to the from-source VS
include dir. Verified against the current CAS meson.build.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
VapourSynth-CAS upstream switched its
meson.buildVS-include probe fromimport vapoursynthtorun_command('vapoursynth', 'get-include', ...)(invokes thevapoursynthconsole script, absent on the clean runner).build_plugin's patcher only fired onimport vapoursynth, so a fresh--depth 1clone of CAS now fails to configure on arm64 (its sole source — x64 also downloads a prebuilt CAS, so it slipped through), tripping the packaging completeness guard.Broaden the patcher trigger to also match
run_command('vapoursynth'; the existing.stdout().strip()substitution already rewrites it to the from-source VS include dir.Validation
deps-v1.6.0rebuild now succeeds and uploaded.🤖 Generated with Claude Code