Skip to content

Remove webkit dep#479

Merged
mattjala merged 2 commits into
HDFGroup:masterfrom
mattjala:remove_webkit_dep
Jul 21, 2026
Merged

Remove webkit dep#479
mattjala merged 2 commits into
HDFGroup:masterfrom
mattjala:remove_webkit_dep

Conversation

@mattjala

Copy link
Copy Markdown
Collaborator

The New Dataset/Attribute help popups used an SWT Browser (a native web engine), which pulled in a hard webkit package dependency. That made the generated RPM uninstallable on RHEL 9 (it Requires webkit2gtk4.1, which RHEL 9 does not ship) and the DEB fragile across Ubuntu releases. The Browser was only used to render a few static help messages.

  • Render the help with a read-only, word-wrapped StyledText widget and inline the help text. The .html help resources were never packaged into the jar, so the Browser popups already failed to load in shipped builds.
  • Drop the webkit entry from the RPM and DEB jpackage --linux-package-deps.
  • Delete the now-unused NewAttrHelp.html / NewDatasetHelp.html.

The New Dataset/Attribute help popups used an SWT Browser (a native web
engine), which pulled in a hard webkit package dependency. That made the
generated RPM uninstallable on RHEL 9 (it Requires webkit2gtk4.1, which
RHEL 9 does not ship) and the DEB fragile across Ubuntu releases. The
Browser was only used to render a few static help messages.

- Render the help with a read-only, word-wrapped StyledText widget and
  inline the help text. The .html help resources were never packaged into
  the jar, so the Browser popups already failed to load in shipped builds.
- Drop the webkit entry from the RPM and DEB jpackage --linux-package-deps.
- Delete the now-unused NewAttrHelp.html / NewDatasetHelp.html.

@jhendersonHDF jhendersonHDF left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Seems reasonable; a bit unfortunate that we (presumably) lose HTML support, but for such a small bit of information that wasn't previously being displayed anyway that seems like a minor detail.

@mattjala
mattjala merged commit e8994cb into HDFGroup:master Jul 21, 2026
10 checks passed
@mattjala
mattjala deleted the remove_webkit_dep branch July 21, 2026 15:40
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.

2 participants