Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable building with vcpkg #1942

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Enable building with vcpkg #1942

wants to merge 3 commits into from

Conversation

Molter73
Copy link
Collaborator

@Molter73 Molter73 commented Nov 5, 2024

Description

This enables building collector outside of the builder container by leveraging the vcpkg package manager. This is not ideal for CI, since vcpkg pulls from the internet so isolated builds are not possible, but it is more convenient than having to install all libraries to the system.

A simple CMakePresets.json file was added to enable even easier use. Command for using this build is: cmake --preset=vcpkg

Checklist

  • Investigated and inspected CI test results
  • Updated documentation accordingly

Automated testing

  • Added unit tests
  • Added integration tests
  • Added regression tests

If any of these don't apply, please comment below.

Testing Performed

Manually checked building collector with vcpkg.
CI should check the regular build for collector is not broken.

@Molter73 Molter73 requested a review from a team as a code owner November 5, 2024 16:30
This enables building collector outside of the builder container by
leveraging the [vcpkg](https://vcpkg.io) package manager. This is not
ideal for CI, since vcpkg pulls from the internet so isolated builds
are not possible, but it is more convenient than having to install all
libraries to the system.

A simple CMakePresets.json file was added to enable even easier use.
Command for using this build is: cmake --preset=vcpkg
@Molter73 Molter73 force-pushed the mauro/vcpkg-build branch 2 times, most recently from 87f810d to 48e4c0f Compare November 7, 2024 17:35
@@ -68,16 +63,12 @@ add_definitions(-DSCAP_SOCKET_ONLY_FD)
# Filter out some cgroup subsys
add_definitions("-DINTERESTING_SUBSYS=\"perf_event\", \"cpu\", \"cpuset\", \"memory\"")

add_definitions(-DASSERT_TO_LOG)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This line is duplicated in line 63

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.

1 participant