Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

build: declare dependency to use it as subproject#1451

Open
ylatuya wants to merge 1 commit into
Netflix:masterNetflix/vmaf:masterfrom
fluendo:vmaf_depfluendo/vmaf:vmaf_depCopy head branch name to clipboard
Open

build: declare dependency to use it as subproject#1451
ylatuya wants to merge 1 commit into
Netflix:masterNetflix/vmaf:masterfrom
fluendo:vmaf_depfluendo/vmaf:vmaf_depCopy head branch name to clipboard

Conversation

@ylatuya

@ylatuya ylatuya commented Dec 2, 2025

Copy link
Copy Markdown
Contributor

As part of the work we are doing to add a new vmaf element in GStreamer, we are also adding vmaf as a subproject of GStreamer. In #1442 we fixed a linker issue with symbol names when building as a subproject.

In this PR we declare the libvmaf_dep dependency following https://mesonbuild.com/Subprojects.html#naming-convention-for-dependency-variables so that it can be used from other projects like:

libvmaf_dep = dependency('libvmaf', fallback : ['libvmaf', 'libvmaf_dep'])

One remaining issue is the lack of a meson.build in the repository root directory, for which I will open a new PR.

@ylatuya
ylatuya force-pushed the vmaf_dep branch 3 times, most recently from c93a3c6 to 3446ff8 Compare December 2, 2025 15:02
Comment thread libvmaf/src/meson.build Outdated
Declare the libvmaf_dep dependency so that libvmaf
can be used as a subproject.
@ylatuya

ylatuya commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

Any chance of getting this merged? It's not possible to build libvmaf as a subproject without this patch.

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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.