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.

Sign up for GitHub

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

Jump to bottom

Deprecate ROS 1-based software and add HUMANSTATEPROVIDER_ENABLE_ROS1VISUALIZER option to disable compilation of it #420

Open

traversaro wants to merge 1 commit into master from addoptiontodisableros1support

+11 −4

Member

traversaro commented Nov 28, 2024

For now I just added an option instead of removing all the folders as:

it was easier
it permits to easily re-enable the support if we realize that we actually need a device or msg somewhere (some of these dependencies are at runtime, so ti is not easy to detect them just by compilation)


          Deprecate ROS 1-based software and add HUMANSTATEPROVIDER_ENABLE_ROS1…

23d47a9

…VISUALIZER option to disable compilation of it

traversaro requested review from S-Dafarra and dariosortino as code owners

November 28, 2024 13:35

S-Dafarra approved these changes

View reviewed changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet