Skip to content

Fixes #6171#6218

Open
Growl1234 wants to merge 1 commit intoHDFGroup:developfrom
Growl1234:develop
Open

Fixes #6171#6218
Growl1234 wants to merge 1 commit intoHDFGroup:developfrom
Growl1234:develop

Conversation

@Growl1234
Copy link
Contributor

@Growl1234 Growl1234 commented Feb 14, 2026

Generate "Extra libraries" in libhdf5.settings using a new variable LINK_LIBS_SETTINGS rather than LINK_LIBS.

Fixes #6171.


Important

Generate "Extra libraries" in libhdf5.settings using LINK_LIBS_SETTINGS with formatted linker flags in CMakeLists.txt.

  • Behavior:
    • Generate "Extra libraries" in libhdf5.settings using LINK_LIBS_SETTINGS instead of LINK_LIBS.
    • Introduces _hdf5_resolve_lib_to_flag function in CMakeLists.txt to format linker flags.
  • CMake:
    • Adds LINK_LIBS_SETTINGS variable in CMakeLists.txt to store formatted linker flags.
    • Converts semicolon-separated list to space-separated string for LINK_LIBS_SETTINGS.
  • Settings:
    • Updates src/libhdf5.settings.in to use @LINK_LIBS_SETTINGS@ for "Extra libraries".

This description was created by Ellipsis for 2c69ea3. You can customize this summary. It will automatically update as commits are pushed.

… `LINK_LIBS_SETTINGS` rather than `LINK_LIBS`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: To be triaged

Development

Successfully merging this pull request may close these issues.

hdf5 2.0.0 libhdf5.settings lists a cmake target name (Threads::Threads) instead of threading link line

1 participant