search

stulp / dmpbbo

Python/C++ library for Dynamical Movement Primitives and Black-Box Optimization
GNU Lesser General Public License v2.1
224 stars 90 forks source link

Installing serialization headers #55

Closed gokhansolak closed 3 years ago

gokhansolak commented 4 years ago

When I install DMPBBO in a new PC using the release build: 

mkdir -p build_dir; cd build_dir; cmake .. -DCMAKE_BUILD_TYPE=Release; make; make Docs
sudo make install

It does not copy the serialization.hpp headers into /usr/local/include/.. directories. So it gives fatal error: dmp/serialization.hpp: No such file or directory at compilation.

It works fine when doing the debug build:

mkdir -p build_dir; cd build_dir; cmake .. -DCMAKE_BUILD_TYPE=Release; make; make Docs
sudo make install
stulp commented 4 years ago

It works for me (Ubuntu 18.04), and I cannot reproduce the error.

When looking at the CMakeLists.txt file, it also appears that all hpp files (including serialization.hpp) are installed:

file(GLOB HEADERS *.hpp)
install(FILES ${HEADERS} DESTINATION ${INCLUDE_INSTALL_DIR}/dynamicalsystems/)

I cheched this for the CMakeLists.txt files for all three serialization.hpp files in dynamicalsystems/, functionapproximators/, and dmp/. In your case, are all three serialization.hpp files not installed, or only one of them? (the answer to that question could help me debug).

Of course, you need sudo rights to do the install. But this you apparently have, or it would not install anythingn in /usr/

gokhansolak commented 3 years ago

I just had a chance to do a clean install, it installed the headers properly. Maybe something else was wrong with my setup then.