Missing lib

Issue #170 resolved
Damien Genet created an issue

The parsec_distribution_matrix.so is built by PaRSEC, but the only ways to find it are to either use the dplasma pkg-config or to know what you're doing and explicitly link against it.

Is it reasonable to add (-lparsec_distribution_matrix) to the parsec.pc.in?

Comments (3)

  1. George Bosilca

    It is added to PARSEC_EXTRA_LIB by our FindPARSEC.cmake, but it is not technically speaking part of PaRSEC. One need to use one of the matrix types in order to need it, and this drives the fact that the lib must be explicitly added by a knowledgeable user.

  2. George Bosilca

    A quick patch to add the lib in parsec.pc would be

    diff --git a/parsec/include/parsec.pc.in b/parsec/include/parsec.pc.in
    index a6714ce2..abc1f01a 100644
    --- a/parsec/include/parsec.pc.in
    +++ b/parsec/include/parsec.pc.in
    @@ -7,7 +7,7 @@ Name: PaRSEC
     Description: Parallel Runtime Scheduling and Execution Controller
     Version: @PARSEC_VERSION_MAJOR@.@PARSEC_VERSION_MINOR@
     Requires: @PARSEC_PKG_REQUIRE@
    -Libs: -L${libdir} -lparsec -lparsec-base @EXTRA_LIBS_EXPANDED@
    +Libs: -L${libdir} -lparsec -lparsec-base -lparsec_distribution_matrix @EXTRA_LIBS_EXPANDED@
     Libs.private: @PAPI_LIBRARIES@
     Cflags: -I${includedir} -I@MPI_C_INCLUDE_PATH@
    

    The only benefit I can see if that users will only have to include a single lib to get access to all PaRSEC capabilities.

  3. Log in to comment