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.

Sign up for GitHub

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

Disable hl tools by default #2313

Merged
merged 5 commits into from
Dec 17, 2022
Merged

Disable hl tools by default #2313

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
9b8e18b
Disable hl tools by default
byrnHDF Dec 16, 2022
51f35d3
identify the tools
byrnHDF Dec 16, 2022
3d9b155
Only GIF tools are depecated
byrnHDF Dec 16, 2022
f87405e
Add new option
byrnHDF Dec 16, 2022
bd80e49
Update autotools
byrnHDF Dec 16, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 2 additions & 4 deletions configure.ac
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -834,14 +834,12 @@ AC_LANG_POP(C++)
AC_SUBST([HDF5_HL])
AC_SUBST([HDF5_HL_TOOLS])

## The high-level library and high-level tools are enabled unless the build mode
## The high-level library are enabled unless the build mode
## is clean.
if test "X-$BUILD_MODE" = "X-clean" ; then
HDF5_HL=no
HDF5_HL_TOOLS=no
else
HDF5_HL=yes
HDF5_HL_TOOLS=yes
fi

## high-level library directories (set when needed, blank until then)
Expand Down Expand Up @@ -877,7 +875,7 @@ AC_MSG_CHECKING([if the high-level tools are enabled])
AC_ARG_ENABLE([hltools],
[AS_HELP_STRING([--enable-hltools],
[Enable the high-level tools.
[default=yes]
[default=no]
])],
[HDF5_HL_TOOLS=$enableval])

Expand Down
2 changes: 1 addition & 1 deletion hl/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ if (HDF5_BUILD_TOOLS)
#-- Option to build the High level Tools
#-----------------------------------------------------------------------------
if (EXISTS "${HDF5_HL_SOURCE_DIR}/tools" AND IS_DIRECTORY "${HDF5_HL_SOURCE_DIR}/tools")
option (HDF5_BUILD_HL_TOOLS "Build HDF5 HL Tools" ON)
option (HDF5_BUILD_HL_TOOLS "Build HDF5 HL Tools" OFF)
if (HDF5_BUILD_HL_TOOLS)
add_subdirectory (tools)
endif ()
Expand Down
2 changes: 1 addition & 1 deletion release_docs/INSTALL_CMake.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -740,7 +740,7 @@ HDF5_BUILD_FORTRAN "Build FORTRAN support" OFF
HDF5_BUILD_JAVA "Build JAVA support" OFF
HDF5_BUILD_HL_LIB "Build HIGH Level HDF5 Library" ON
HDF5_BUILD_TOOLS "Build HDF5 Tools" ON
HDF5_BUILD_HL_TOOLS "Build HIGH Level HDF5 Tools" ON
HDF5_BUILD_HL_TOOLS "Build HIGH Level HDF5 Tools" OFF

---------------- HDF5 Folder Build Options ---------------------
Defaults relative to $<INSTALL_PREFIX>
Expand Down
16 changes: 12 additions & 4 deletions release_docs/RELEASE.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -301,6 +301,14 @@ Bug Fixes since HDF5-1.13.3 release

Configuration
-------------
- Change the default for building the high-level tools
Copy link
Collaborator

Choose a reason for hiding this comment

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

I wonder if it might be good to call out which tools are getting disabled here by default since I imagine some folks would want to know that building of h5watch is opt-in after this.

Copy link
Member

Choose a reason for hiding this comment

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

Agree. The tools are gif2hdf and hdf2gif as well as h5watch

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done


The gif2hdf5 high-level tools are deprecated and will be removed in a future release.
The default build setting for them have been changed from enabled to disabled. A user
can enable the build of these tools if needed.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
The gif2h5 and h52gif tools were previously enabled or disabled by the --enable-hltools / HDF5_BUILD_HL_TOOLS option. That option also controlled building and installing the h5watch tool, which will continue to be built and installed by default but may be disabled using that option.

(ADB - 2022/12/16)

- Change the settings of the *pc files to use the correct format

The pkg-config files generated by CMake uses incorrect syntax for the 'Requires'
Expand Down Expand Up @@ -583,7 +591,7 @@ The issues with the gif tool are:
HDFFV-10592 CVE-2018-17433
HDFFV-10593 CVE-2018-17436
HDFFV-11048 CVE-2020-10809
These CVE issues have not yet been addressed and can be avoided by not building
the gif tool. Disable building the High-Level tools with these options:
autotools: --disable-hltools
cmake: HDF5_BUILD_HL_TOOLS=OFF
These CVE issues have not yet been addressed and are avoided by not building
the gif tool by default. Enable building the High-Level tools with these options:
autotools: --enable-hltools
cmake: HDF5_BUILD_HL_TOOLS=ON