Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consolidated "Platforms Supported" and "More Platforms Tested" #1023

Closed
wants to merge 4 commits into from
Closed

Consolidated "Platforms Supported" and "More Platforms Tested" #1023

wants to merge 4 commits into from

Conversation

lrknox
Copy link
Collaborator

@lrknox lrknox commented Sep 16, 2021

sections of RELEASE.txt into "Platforms Tested".

@lrknox
Copy link
Collaborator Author

lrknox commented Sep 16, 2021

I removed the Solaris and Windows 10 (32 bit) lines from the "Tested Configuration Features Summary" table. There may be further additions or consolidations to that table.

@derobins
Copy link
Member

I feel like it would be useful to have a paragraph that describes what should work and what we know doesn't work. For example, we know that certain versions of Windows won't work due to missing C99 support. I'm not sure if that sort of thing belongs here or elsewhere, though.

Copy link
Contributor

@byrnHDF byrnHDF left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like it would be useful to have a paragraph that describes what should work and what we know doesn't work. For example, we know that certain versions of Windows won't work due to missing C99 support. I'm not sure if that sort of thing belongs here or elsewhere, though.

But our minimum is VS2015 and I think that is fine?

release_docs/RELEASE.txt Outdated Show resolved Hide resolved
release_docs/RELEASE.txt Show resolved Hide resolved
release_docs/RELEASE.txt Outdated Show resolved Hide resolved
release_docs/RELEASE.txt Outdated Show resolved Hide resolved
release_docs/RELEASE.txt Outdated Show resolved Hide resolved
@byrnHDF
Copy link
Contributor

byrnHDF commented Sep 20, 2021

BB defaults and StdShar config:
CMake
BUILD_SHARED_LIBS: BOOL: ON HDF5_BUILD_JAVA: BOOL: ON HDF5_BUILD_FORTRAN: BOOL: ON HDF5_BUILD_CPP_LIB: BOOL: ON HDF5_BUILD_DOC: BOOL: ON
autotools
fortran: enable cxx: enable java: enable doxygen: enable

@byrnHDF
Copy link
Contributor

byrnHDF commented Sep 20, 2021

BB configs
StdStat:
CMake BUILD_SHARED_LIBS: BOOL: OFF HDF5_BUILD_JAVA: BOOL: OFF
autotools: static: enable shared: disable java: disable

@byrnHDF
Copy link
Contributor

byrnHDF commented Sep 20, 2021

Other significant configs:
TSShar (StdShar with threadsafe)
CMake: HDF5_ENABLE_THREADSAFE: BOOL: ON ALLOW_UNSUPPORTED: BOOL: ON
autotools: threadsafe: enable unsupported: enable

StdMPIShar (StdShar with MPI)
CMake: HDF5_ENABLE_PARALLEL: BOOL: ON HDF5_BUILD_CPP_LIB: BOOL: OFF HDF5_BUILD_JAVA: BOOL: OFF
autotools: parallel: enable cxx: disable java: disable
StdClang (CMake sanitizers)
Address: ONLY_SHARED_LIBS: BOOL: ON HDF5_BUILD_JAVA: BOOL: OFF HDF5_BUILD_FORTRAN: BOOL: OFF HDF5_ENABLE_SANITIZERS: BOOL: ON USE_SANITIZER: STRING: Address CMAKE_TOOLCHAIN_FILE: STRING: 'config/toolchain/clang.cmake'
Leak: ONLY_SHARED_LIBS: BOOL: ON HDF5_BUILD_JAVA: BOOL: OFF HDF5_BUILD_FORTRAN: BOOL: OFF HDF5_ENABLE_SANITIZERS: BOOL: ON USE_SANITIZER: STRING: Leak CMAKE_TOOLCHAIN_FILE: STRING: 'config/toolchain/clang.cmake'
Memory: ONLY_SHARED_LIBS: BOOL: ON HDF5_BUILD_JAVA: BOOL: OFF HDF5_BUILD_FORTRAN: BOOL: OFF HDF5_ENABLE_SANITIZERS: BOOL: ON USE_SANITIZER: STRING: Memory CMAKE_TOOLCHAIN_FILE: STRING: 'config/toolchain/clang.cmake'
Undefined: ONLY_SHARED_LIBS: BOOL: ON HDF5_BUILD_JAVA: BOOL: OFF HDF5_BUILD_FORTRAN: BOOL: OFF HDF5_ENABLE_SANITIZERS: BOOL: ON USE_SANITIZER: STRING: Undefined CMAKE_TOOLCHAIN_FILE: STRING: 'config/toolchain/clang.cmake'

@lrknox lrknox closed this Oct 9, 2021
@lrknox lrknox deleted the 1_10_update_release_notes branch March 31, 2022 15:05
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.

4 participants