Add vcpkg dependency management with Bullet - #5
Merged
Conversation
sstarosz
changed the title
- vcpkg.json: add Bullet 3.25 (float precision) as the first C/C++ dependency. - CMakeLists.txt: fail with clear instructions when no CMake toolchain is configured, then resolve Bullet via find_package(Bullet CONFIG REQUIRED). - CMakePresets.json: add the hidden with-vcpkg preset that activates the vcpkg toolchain from \, and make all maya* presets inherit it. - .github/workflows: set up vcpkg (lukka/run-vcpkg) in pr-checks and release. - .gitignore: ignore third_party/ and vcpkg_installed/. - docs/CPPDevelopment.md: document vcpkg setup + troubleshooting. - README.md: list vcpkg as a prerequisite and note auto-install of deps. - CHANGELOG.md: note vcpkg dependency management under [Unreleased]. - tests/CMakeLists.txt: drop the stale 'Re-enable the CI preset (ci)' comment (the ci preset was removed in PR #4). Part of the feature/physics-system (PR #3) split — stacked on PR #4 (fix/cmake-build-fixes).
sstarosz
force-pushed
the
feature/physics-build-infra
branch
from
August 9, 2026 19:04
b261037 to
0f68a2e
Compare
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.
Adds vcpkg dependency management with Bullet 3.25, extracted from
feature/physics-system(PR #3).Changes
Dependency management
vcpkg.json(new) — Bullet 3.25 (float precision) as the first C/C++ dependency, with a pinnedbuiltin-baseline.CMakeLists.txt— fails with clear instructions when no CMake toolchain is configured, then resolves Bullet viafind_package(Bullet CONFIG REQUIRED).CMakePresets.json— adds the hiddenwith-vcpkgpreset that activates the vcpkg toolchain from$VCPKG_ROOT; allmaya*presets inherit it.CI & housekeeping
.github/workflows/pr-checks.yml/release.yml— set up vcpkg (lukka/run-vcpkg@v11) before configuring..gitignore— ignorethird_party/andvcpkg_installed/.docs/CPPDevelopment.md— vcpkg setup section + "vcpkg is not configured" troubleshooting entry.README.md— vcpkg prerequisite + note that deps auto-install on first configure.CHANGELOG.md—Addedentry for vcpkg dependency management.tests/CMakeLists.txt— drop the stale "Re-enable the CI preset (ci)" comment (that preset is removed in Fix Maya SDK resolution and clean up CMake presets #4).Not in this PR
add_subdirectory(mmd/core)+ the Maya-freemmd_coreengine and its C++ unit tests → next PR (core engine).Testing
pr-checks.yml) exercises the vcpkg toolchain end-to-end (configure + build with Bullet) on this PR.