icubmod and icubmoddev are NO longer generated

[pattacini]

Hi icubers,

Please take a note of this important announcement.

Since robotology/icub-main@ef3a6a6, icubmod library so as its related binary icubmoddev are NO longer generated during the compilation of icub-main.

Therefore, three consequences are given for code/packages depending on icub-main:

1. The dependency from icubmod in the CMakeLists.txt file needs to be removed. Such dependency was not required since long, due to the new dynamic plugins system.
2. The following C++ directives need to be removed from the code as well:
   • YARP_DECLARE_DEVICES(icubmod)
   • YARP_REGISTER_DEVICES(icubmod)
   • YARP_DECLARE_PLUGINS(icubmod)
   • YARP_REGISTER_PLUGINS(icubmod)
3. References to icubmoddev (e.g. in xml applications) need to be replaced with references to yarpdev binary.

See #33 for further details.

[pattacini]

As @Tobias-Fischer correctly pointed out, it is also required to remove the following lines from the C++ code:

YARP_DECLARE_DEVICES(icubmod)
YARP_REGISTER_DEVICES(icubmod)

[traversaro]

@pattacini @mbrunettini can we increase the minor version number of icub-main such that we can cmake depend on the latest version of icub-main (after this modification)?

[traversaro]

Even the patch version was ok, thanks!

[pattacini]

Semantically speaking, it was not a patch but a sort of change in the "interface", hence "minor" is more appropriate.

[traversaro]

👍

[traversaro]

I was just scared since the minor version was not updated since 2010 : https://github.com/robotology/icub-main/tags .

[pattacini]

Time has come to update it, then 😏
I'm fine with changing the patch too, so whoever admin wants to kick in and do that, there's no problem.

[jeljaik]

Bit more details can also be found here: #33 (comment)
The original post is missing YARP_DECLARE_PLUGINS(icubmod) and YARP_REGISTER_PLUGINS(icubmod).

[pattacini]

Thanks @jeljaik; I've updated the post.