Missing lib
Issue #170
resolved
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)
-
-
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.
-
- changed status to resolved
Resolved by pull request #201.
- Log in to comment
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.