Comment 3 for bug 1704459

Revision history for this message
Mikael Arguedas (mikael-arguedas) wrote :

SRU:

[Impact]

 * This bug prevents users to link dynamically against pcl libraries
 * There is no upload required as rebuilding the package is enough to fix the bug

[Test Case]

 * install libpcl-dev
   * apt-get update && apt-get install libpcl-dev
 * mkdir -p /tmp/test_pcl/src && cd /tmp/test_pcl
 * echo "#include <pcl/common/io.h>" > src/foo.cpp
 * create a CMakeLists.txt
```
find_package(PCL REQUIRED)

option(BUILD_SHARED_LIBS "Build shared libs" ON)
include_directories(${PCL_INCLUDE_DIRS})
add_library(foolib src/foo.cpp)
target_link_libraries(foolib ${PCL_LIBRARIES})
```
 * mkdir build && cd build
 * cmake ..
 * make
This produces the error message:
```
make[2]: *** No rule to make target '/usr/lib/libmpi.so', needed by 'libfoolib.so'. Stop.
CMakeFiles/Makefile2:67: recipe for target 'CMakeFiles/foolib.dir/all' failed
make[1]: *** [CMakeFiles/foolib.dir/all] Error 2
```

[Patch]
 * Rebuild the package
 * Test case passes once the package rebuilt

[Regression Potential]

 * Given that the patch is just a rebuild I don't see a potential for regression