Lean defaults: turn legacy engines/transports off by default#5109
Merged
Conversation
DataMan, MHS, DataSpaces, IME, and ZeroMQ now default to OFF; SSC gets its own ADIOS2_USE_SSC option, also OFF. Add a "Notes for Packagers" section to the README and a whatsnew entry.
The staging-common, common, and iotest suites created SSC tests without checking ADIOS2_HAVE_SSC, so with SSC now off by default the MPI/MPMD jobs tried to run the unbuilt ssc engine and failed. Gate them like the sibling SST tests.
pnorbert
approved these changes
Jul 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
DataMan, MHS, DataSpaces, IME, SSC, and ZeroMQ changed to default to OFF. Add a "Notes for Packagers" section to the README and a whatsnew entry.
Note: this essentially turns off building and testing Dataman, SSC and MHS. Dropping Dataman allows us to drop the zeromq dependency from all CI images. DataSpaces and IME were never tested, so this doesn't really change their state, except that they won't be build unless explicitly asked. Also note that we had to add the USE_SSC option in order to allow us to turn it OFF. This is a step towards declaring these things to be deprecated, which this PR does not do. Dropping those unmaintained packages from testing should improve CI times and cut static build image sizes...