Naming conventions for targets

mvukov / rules_ros

Build ROS (1) with Bazel

Apache License 2.0

24 stars 12 forks source link

Open mgrrx opened 2 years ago

mgrrx commented 2 years ago

Some targets are prefixed with cc or py, for some others it's not consistent. I'm happy to provide a PR. What would you prefer?

mvukov commented 2 years ago

Here is some background:

ros_interfacelibrary groups/provides ROS IDL, then cc and py_ros_interface_library are C++ and Python variants (doing code generation internally).
cc(binary|library) are built in rules for C++ targets. ros variants are ROS specific macros.

In this sense naming is consistent IMO, though I understand confusion. Perhaps docs can be updated. WDYT?