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

switch build to conan 1.x #100

Merged
merged 1 commit into from
Dec 7, 2023
Merged

switch build to conan 1.x #100

merged 1 commit into from
Dec 7, 2023

Conversation

copperlight
Copy link
Collaborator

@copperlight copperlight commented Dec 6, 2023

The Conan 1.x package manager for C++ is a bit easier to work with than the
Bazel system. This allows us to more easily update dependencies, to keep them
in-sync with the atlas-system-agent project, which also uses Conan.

This change includes some additional configuration in a profile, which is used
to inject necessary cxxflags to the package builds, so that the local debug
builds can be run with the address sanitizer enabled. Without this additional
setting, segfaults will occur during testing, because the address sanitizer
will not be applied to both the project and the dependencies.

Switching back and forth between Debug and Release builds requires removing
all packages and builds from the Conan cache, due to the way these flags are
injected. The ./build.sh clean script shows how this is done.

The Conan 1.x package manager for C++ is a bit easier to work with than the
Bazel system. This allows us to more easily update dependencies, to keep them
in-sync with the atlas-system-agent project, which also uses Conan.

This change includes some additional configuration in a profile, which is used
to inject necessary cxxflags to the package builds, so that the local debug
builds can be run with the address sanitizer enabled. Without this additional
setting, segfaults will occur during testing, because the address sanitizer
will not be applied to both the project and the dependencies.

Switching back and forth between Debug and Release builds requires removing
all packages and builds from the Conan cache, due to the way these flags are
injected. The `./build.sh clean` script shows how this is done.
@copperlight copperlight merged commit 61a190a into Netflix:main Dec 7, 2023
1 check passed
@copperlight copperlight deleted the conan branch December 7, 2023 04:36
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