#221 itpp-config not installed in IT++ 4.3.0

[Kumar Appaiah]

Hi.

With the 4.3.0 release, itpp-config is no longer installed. Please add itpp-config in the installation process.

Thanks.

[Bogdan Cristea]

changes commited into master branch of git repo, please verify

[Kumar Appaiah]

Hi.

Thanks, but I can't complete the build successfully. This is due to this hunk:

index c20859c..2d5d5e1 100644
--- a/itpp/CMakeLists.txt
+++ b/itpp/CMakeLists.txt
@@ -51,6 +51,12 @@ else()
add_library (${libitpp_target} STATIC ${ITPP_SRCS})
endif()

+#remove duplicates in external libraries list
+if (ITPP_LIBS)
+ list(REMOVE_DUPLICATES ITPP_LIBS)
+ string (REPLACE ";" " " ITPP_LIBS "${ITPP_LIBS}")
+endif()
+
target_link_libraries (${libitpp_target} ${ITPP_LIBS})

set(LIBITPP_VERSION_MAJOR 8)

Now, this hunk causes the generated rule in build/itpp/CMakeFiles/itpp.dir/build.make to be:

itpp/libitpp.so.8.2.0: /usr/lib/x86_64-linux-gnu/libfftw3.so\ /usr/lib/liblapack.so\ /usr/lib/libf77blas.so.3gf\ /usr/lib/libatlas.so.3gf

And the build fails with:
c++: error: /usr/lib/x86_64-linux-gnu/libfftw3.so /usr/lib/liblapack.so /usr/lib/libf77blas.so.3gf /usr/lib/libatlas.so.3gf: No such file or directory

I tried to fix it myself, but I didn't find out how to do it. If I could be of further help, please do let me know.

Thanks.

[Kumar Appaiah]

If I comment out the hunk above, itpp-config seems to get installed though.

[Bogdan Cristea]

Hi
Could you tell me your system configuration ? Actually only commenting the line "string(REPLACE ..." should be enough, but itpp-config might be unusable. Have you tried to use it?
regards
Bogdan

[Kumar Appaiah]

I have Debian, with cmake 2.8.11. The environment is roughly similar to the one in the build log given here:

https://buildd.debian.org/status/fetch.php?pkg=libitpp&arch=i386&ver=4.3.0-1&stamp=1369788407

And the installed itpp-config works fine without the string replace, although I will confirm after retesting tomorrow.

Please let me know if I can help further.

[Bogdan Cristea]

Hi
I have committed a corrected version of itpp/CMakeLists.txt that uses string(REPLACE only for itpp-config generation. Please retest.
thanks
Bogdan

[Kumar Appaiah]

Thanks. This works as expected.

However, there is still a slight regression: the man page isn't installed. Earlier versions used to populate itpp-config.1.in with the PACKAGE_VERSION and put it in /usr/share/man/man1. An example of a previous version is here:

http://packages.debian.org/jessie/amd64/libitpp-dev/filelist

(note that Debian compresses man pages; the IT++ build process didn't).

[Bogdan Cristea]

add man file generation and installation. Please let me know if that suits your needs (installation path is ${install_prefix}/share/man/man1)

[Kumar Appaiah]

Yes. It works fine now. This bug can be closed.

Thank you.