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

SkinPrototype Error when compiling icub-main v1.14.0 #631

Closed
dcam0050 opened this issue Dec 18, 2019 · 5 comments
Closed

SkinPrototype Error when compiling icub-main v1.14.0 #631

dcam0050 opened this issue Dec 18, 2019 · 5 comments
Assignees

Comments

@dcam0050
Copy link

Tried compiling icub-main v1.14.0 and received the following error during compilation:

Scanning dependencies of target skinprototype
[ 17%] Built target skinWrapper
[ 17%] Building CXX object src/libraries/icubmod/skinprototype/CMakeFiles/skinprototype.dir/skinprototype.cpp.o
[ 17%] Built target canBusAnalogSensor
[ 17%] Building CXX object src/libraries/icubmod/skinprototype/CMakeFiles/skinprototype.dir/yarp_plugin_icubmod_skinprototype.cpp.o
In file included from /usr/local/src/robot/icub-main/src/libraries/icubmod/skinprototype/skinprototype.cpp:7:
/usr/local/src/robot/icub-main/src/libraries/icubmod/skinprototype/skinprototype.h: In constructor ‘SkinPrototype::SkinPrototype(int)’:
/usr/local/src/robot/icub-main/src/libraries/icubmod/skinprototype/skinprototype.h:40:74: error: class ‘SkinPrototype’ does not have any field named ‘mutex’
     SkinPrototype(int period=20) : PeriodicThread((double)period/1000.0),mutex(1)
                                                                          ^~~~~
In file included from /usr/local/src/robot/icub-main/build-pc104/src/libraries/icubmod/skinprototype/yarp_plugin_icubmod_skinprototype.cpp:13:
/usr/local/src/robot/icub-main/src/libraries/icubmod/skinprototype/skinprototype.h: In constructor ‘SkinPrototype::SkinPrototype(int)’:
/usr/local/src/robot/icub-main/src/libraries/icubmod/skinprototype/skinprototype.h:40:74: error: class ‘SkinPrototype’ does not have any field named ‘mutex’
     SkinPrototype(int period=20) : PeriodicThread((double)period/1000.0),mutex(1)
                                                                          ^~~~~
make[2]: *** [src/libraries/icubmod/skinprototype/CMakeFiles/skinprototype.dir/build.make:76: src/libraries/icubmod/skinprototype/CMakeFiles/skinprototype.dir/yarp_plugin_icubmod_skinprototype.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: *** [src/libraries/icubmod/skinprototype/CMakeFiles/skinprototype.dir/build.make:63: src/libraries/icubmod/skinprototype/CMakeFiles/skinprototype.dir/skinprototype.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:2009: src/libraries/icubmod/skinprototype/CMakeFiles/skinprototype.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[ 18%] Built target canBusDoubleFTSensor
make: *** [Makefile:130: all] Error 2

Any pointers? Thanks.

@dcam0050
Copy link
Author

Deleting 'mutex(1)' from the initialiser seems to work. Please advise on repercussions?

@pattacini pattacini self-assigned this Dec 18, 2019
@pattacini
Copy link
Member

Hi @dcam0050 hope you're doing well 😄

Thanks for pointing out this compilation bug that we didn't catch with our CI.
skinprototype is a deprecated device so you can safely switch it off at CMake configuration time.

We're also going to patch this in the distro.

@pattacini
Copy link
Member

By the way, I assume you're upgrading your own machine and not the robot iCubSheffield01 since it is not eligible for receiving the distro 2019.12.feat-01.

@pattacini
Copy link
Member

@dcam0050 we've just produced the tag v1.14.1, which has yielded the distro 2019.12.feat-02.

@dcam0050
Copy link
Author

@pattacini sorry for the silence. Can you advise which is the latest distro compatible with iCubSheffield01?

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