[go: up one dir, main page]
Menu ▾ ▴

#18 Missing lib in CMakeLists

0.99
closed
Ravishankar Sundararaman
None
2015-04-29
2015-04-28
Michele Renda
No

Hello

While compiling JDFTx in Debian 8, I noticed there is needed a change in CMakeLists.txt to get the compilation to succeed:

#External libraries to link to
-set(EXTERNAL_LIBS ${MPI_CXX_LIBRARIES} ${CBLAS_LAPACK_FFT_LIBRARIES} ${GSL_LIBRARY} ${LIBXC_LIBRARY})
+set(EXTERNAL_LIBS ${MPI_CXX_LIBRARIES} ${CBLAS_LAPACK_FFT_LIBRARIES} ${GSL_LIBRARY} ${LIBXC_LIBRARY} ${CMAKE_THREAD_LIBS_INIT})

Can you please correct that file so other user can benefit?

Thank you
Michele Renda

PS. In addition it is needed a nice flag to disable the CUDE stuff (--disable-cuda).

Discussion

  • Deniz Gunceler

    Deniz Gunceler - 2015-04-28

    Hi Michele, thanks for the information!

    Did you need to include find_package( Threads ) in the CMakeLists.txt as well?

     

  • Deniz Gunceler

    Deniz Gunceler - 2015-04-29

    Hi Michele, please check out the latest version of JDFTx from svn to see if your issue has been resolved (revision 1180)

     

  • Deniz Gunceler

    Deniz Gunceler - 2015-04-29
    • status: open --> closed
    • assigned_to: Ravishankar Sundararaman
     

  • Ravishankar Sundararaman

    Ravishankar Sundararaman - 2015-04-29

    Hi Michele,

    Thanks for reporting this issue. It seems to be specific to the LAPACK link line on Debian. So we have added ${CMAKE_THREAD_LIBS_INIT} as you suggested, but inside the CBLAS_LAPACK_FFT_LIBRARIES flag, rather than the global link line, to localize the change.

    We have also make GPU compilation optional as of rev1180; this has been requested by several others as well!

    Thanks again for your feedback and bug report.

    Best,
    Shankar