Skip to content

Improve CMake package support - #182

Open
Growl1234 wants to merge 2 commits into
ddsolvation:mainfrom
Growl1234:cmake
Open

Improve CMake package support#182
Growl1234 wants to merge 2 commits into
ddsolvation:mainfrom
Growl1234:cmake

Conversation

@Growl1234

@Growl1234 Growl1234 commented Aug 13, 2026

Copy link
Copy Markdown
  • Gather options and rename with project-specific or standard naming.
  • Drop hard-coded SHARED from add_library(ddx) so BUILD_SHARED_LIBS=ON/OFF will be valid.
  • Enable position-independent code for the ddx target.
  • Drop ${BLAS_LIBRARIES} target linking, as ${LAPACK_LIBRARIES} already contains BLAS libraries, and current duplication will make statical linking against math library fail.
  • Add ddX::ddx namespace alias.
  • Better handle CMake configuration files and install under standard .../cmake/ddX.
  • Install include files under project-specific include/ddX.

@Growl1234 Growl1234 changed the title CMake: Enable static library linking Improve CMake package support Aug 18, 2026
@Growl1234
Growl1234 force-pushed the cmake branch 9 times, most recently from 47ea961 to 608044f Compare August 18, 2026 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant