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

Conditional ddp-actuator-solver dependency #74

Open
NoelieRamuzat opened this issue Apr 9, 2020 · 1 comment
Open

Conditional ddp-actuator-solver dependency #74

NoelieRamuzat opened this issue Apr 9, 2020 · 1 comment

Comments

@NoelieRamuzat
Copy link
Contributor

Hi !

Sot-torque-control has a required dependency to the ddp-actuator-solver package, only needed if you want to use a ddp algorithm, which is most of the time not the case.
Is it possible to make this dependency conditional like it seems to be for the sources ?
In the src/CMakeLists.txt there is this condition:

IF(DDP_ACTUATOR_SOLVER_FOUND)
  SET(plugins ${plugins} ddp-actuator-solver)
  SET(plugins ${plugins} ddp_pyrene_actuator_solver)
ENDIF(DDP_ACTUATOR_SOLVER_FOUND)

Can we have something similar in the CMakeLists.txt instead of:
ADD_PROJECT_DEPENDENCY(ddp-actuator-solver REQUIRED PKG_CONFIG_REQUIRES ddp-actuator-solver)

Thank you !

@andreadelprete
Copy link
Collaborator

Sounds very reasonable to me

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

No branches or pull requests

2 participants