search

laas / jrl_dynamics_urdf

URDF parser for jrl-dynamics
2 stars 9 forks source link

Add urdfdom_headers needed by tests/ #7

Closed aclodic closed 11 years ago

aclodic commented 11 years ago

urdfdom_headers is needed by one_dof_revolute_x

more precisely, one_dof_revulte_x include "jrl/dynamics/urdf/parser.hh" which include "urdf_model/model.h" (which is part of urdfdom_headers package)

thomas-moulard commented 11 years ago

The patch looks good but I think you are mistaken: "jrl/dynamics/urdf/parser.hh" is part of this package, not urdfdom_headers.

olivier-stasse commented 11 years ago

Agree that the wording could be improved but the message seems to say that one header file needs the urdf_dom headers. Anyway I will merge