Skip to content

build: implement vcpkg Manifest Mode, CMakePresets and update README#22

Merged
varev-dev merged 3 commits intodevelfrom
build/vcpkg-deps
Feb 2, 2026
Merged

build: implement vcpkg Manifest Mode, CMakePresets and update README#22
varev-dev merged 3 commits intodevelfrom
build/vcpkg-deps

Conversation

@varev-dev
Copy link
Contributor

This PR refactors the project's build system by migrating dependency management from manual vendoring to vcpkg Manifest Mode (vcpkg.json). It also introduces CMakePresets to streamline the configuration process and updates the README.md with a "Development Setup" guide.

The first build will take significantly longer than usual because vcpkg needs to compile deps from source.

Hot to test

# 1. Clean old artifacts
rm -rf build

# 2. Configure project (using the new preset)
# Ensure VCPKG_ROOT is set in your env, or adjust the path in CMakePresets.json
cmake --preset default

# 3. Build
cmake --build build

# 4. Run the agent (verify it starts and loads config)
./build/source/agent/volta_agent

@varev-dev varev-dev requested a review from kox13 February 2, 2026 14:41
@varev-dev varev-dev added documentation Improvements or additions to documentation enhancement New feature or request labels Feb 2, 2026
@kox13 kox13 added this to volta Feb 2, 2026
@kox13 kox13 moved this to Done in volta Feb 2, 2026
Copy link
Collaborator

@kox13 kox13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works great, although workflow file may need changes in the near future, as the checks take too long.

@varev-dev varev-dev merged commit 7ae8bc2 into devel Feb 2, 2026
1 check passed
@varev-dev varev-dev deleted the build/vcpkg-deps branch February 2, 2026 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants