robotology / blockfactory

A tiny framework to wrap algorithms for dataflow programming
https://robotology.github.io/blockfactory
GNU Lesser General Public License v2.1
40 stars 16 forks source link

It is not possible to compile the example in a PC without Matlab #17

Closed traversaro closed 5 years ago

traversaro commented 5 years ago

BlockFactoryMex is currently always required in the example (see https://github.com/robotology/blockfactory/blob/master/example/CMakeLists.txt#L35), but it should be required only if Matlab compilation is required (typically using options such as <package>_USES_MATLAB or USES_MATLAB, or in the case of the example implicitly based on the availability or not of Matlab in the system).

diegoferigo commented 5 years ago

This case should be handled as I did in the ROS demo:

https://github.com/diegoferigo/blockfactory-demo-ros/blob/9c6bac0723e1fcc479a9e1ed6522c51eafb52feb/CMakeLists.txt#L44-L46

diegoferigo commented 5 years ago

This case should have been caught from CI, I found a mistake in the script.