cblas: add include directory to target

[NeroBurner]

add include directories to target. With this change in a cmake project one can link cblas with just the following

find_package(cblas CONFIG REQUIRED)
add_executable(foo foo.cpp)
target_link_libraries(foo cblas)

[codecov]

Codecov Report

Merging #170 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #170   +/-   ##
=======================================
  Coverage   82.25%   82.25%           
=======================================
  Files        1822     1822           
  Lines      177187   177187           
=======================================
  Hits       145752   145752           
  Misses      31435    31435

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e774b43...380178e. Read the comment docs.

[ruslo]

CMake minimum version is set to 2.8.10:

• lapack/CMakeLists.txt

  Line 1 in e774b43

  cmake_minimum_required(VERSION 2.8.10)

Command target_include_directories is not available:

• https://cmake.org/cmake/help/v2.8.10/cmake.html#section_Commands

This can be fixed by updating minimum requirement to 2.8.12:

• https://cmake.org/cmake/help/v2.8.12/cmake.html#command:target_include_directories

If update to 2.8.12 is acceptable, alternatively you can use INCLUDES DESTINATION include here:

• lapack/CMakeLists.txt

  Lines 102 to 107 in e774b43

  	install(TARGETS ${lib}
  	EXPORT ${LAPACK_INSTALL_EXPORT_NAME}
  	ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR}
  	LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
  	RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
  	)

Docs:

• https://cmake.org/cmake/help/v2.8.12/cmake.html#command:install

[NeroBurner]

I think INCLUDES DESTINATION is also a v2.8.12 feature. I didn't find it in the v2.8.10 documentation
https://cmake.org/cmake/help/v2.8.10/cmake.html#command:install

[ruslo]

I think INCLUDES DESTINATION is also a v2.8.12 feature

Yes, that's what I meant by "If update to 2.8.12 is acceptable, alternatively..." 😉