Skip to content

Disable dll copies by vcpkg to avoid race condition - #3412

Merged
vrabaud merged 2 commits into
AOMediaCodec:mainfrom
vrabaud:main
Oct 13, 2026
Merged

vrabaud merged 2 commits into
AOMediaCodec:mainfrom
vrabaud:main

Conversation

@vrabaud

@vrabaud vrabaud commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

This variable is explained at:
https://learn.microsoft.com/en-us/vcpkg/users/buildsystems/cmake-integration#vcpkg_applocal_deps

Parallel ninja builds call "vcpkg z-applocal" in parallel which can creaate collisions.

X_VCPKG_APPLOCAL_DEPS_SERIALIZED (from #3408) did not resolve this because CMake's Ninja generator inlines target post-build commands directly into the linker rule and does not place them in the terminal/console pool.

VCPKG_APPLOCAL_DEPS=OFF disables the post build copies. We still need to find the dlls so the the build folder is added to the path.

It should fix the issues we've been seeing on the Windows CI with vcpkg.

This variable is explained at:
 https://learn.microsoft.com/en-us/vcpkg/users/buildsystems/cmake-integration#vcpkg_applocal_deps

Parallel ninja builds call "vcpkg z-applocal" in parallel which
can creaate collisions.

X_VCPKG_APPLOCAL_DEPS_SERIALIZED (from AOMediaCodec#3408) did not resolve this
because CMake's Ninja generator inlines target post-build commands
directly into the linker rule and does not place them in the
terminal/console pool.

VCPKG_APPLOCAL_DEPS=OFF disables the post build copies. We still
need to find the dlls so the the build folder is added to the
path.

It should fix the issues we've been seeing on the Windows CI
with vcpkg.
Comment thread .github/actions/setup-windows/action.yml Outdated
@vrabaud
vrabaud merged commit ab37e29 into AOMediaCodec:main Oct 13, 2026
44 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants