Skip to content

Add extra path to HDF5_PLUGIN_PATH for Windows binary testing#227

Merged
lrknox merged 1 commit intoHDFGroup:masterfrom
jhendersonHDF:win_testing_plugin_path
Feb 27, 2026
Merged

Add extra path to HDF5_PLUGIN_PATH for Windows binary testing#227
lrknox merged 1 commit intoHDFGroup:masterfrom
jhendersonHDF:win_testing_plugin_path

Conversation

@jhendersonHDF
Copy link
Contributor

@jhendersonHDF jhendersonHDF commented Feb 27, 2026

Needed for ZFP testing currently as all plugins except for ZFP get installed under ROOT\lib\plugin on Windows, whereas ZFP goes under ROOT\plugin.


Important

Adds an extra path to HDF5_PLUGIN_PATH in CMakeLists.txt for Windows to support ZFP testing.

  • Behavior:
    • Adds an extra path to _plugin_path for Windows in CMakeLists.txt to include ROOT/plugin for ZFP testing.
    • Ensures both ROOT/lib/plugin and ROOT/plugin are checked for plugins on Windows.
  • Testing:
    • Affects H5PL_BUILD_TESTING section to ensure ZFP plugin is found during tests.

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

@jhendersonHDF jhendersonHDF force-pushed the win_testing_plugin_path branch from 371bda7 to 6c662fc Compare February 27, 2026 20:30
@jhendersonHDF
Copy link
Contributor Author

Binary testing in https://github.com/HDFGroup/hdf5_plugins/actions/runs/22501444922 failed due to the missing plugin path

@lrknox lrknox merged commit e797c3c into HDFGroup:master Feb 27, 2026
13 checks passed
jhendersonHDF added a commit to jhendersonHDF/hdf5_plugins that referenced this pull request Feb 27, 2026
@jhendersonHDF jhendersonHDF deleted the win_testing_plugin_path branch March 1, 2026 17:44
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.

3 participants