Skip to content

more options for splitting tabs and moving to external view, fixes #6708#7580

Merged
hansva merged 3 commits into
apache:mainfrom
hansva:6708
Jul 20, 2026
Merged

more options for splitting tabs and moving to external view, fixes #6708#7580
hansva merged 3 commits into
apache:mainfrom
hansva:6708

Conversation

@hansva

@hansva hansva commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

When running hop-gui in debug mode you wil now also see debug logging in the console from the GENERAL and GUI logchannels


Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Run mvn clean install apache-rat:check to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
  • If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.
  • Mention the appropriate issue in your description (for example: addresses #123), if applicable.

To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

@hansva
hansva marked this pull request as draft July 20, 2026 10:19
@hansva
hansva marked this pull request as ready for review July 20, 2026 10:35
Match historical Runtime.exec(String) whitespace tokenization via
StringTokenizer before exec(String[]), so single-command fields like
"/bin/echo hop-single" work again. Based on @leehaut's fix in apache#7581.
@mattcasters

Copy link
Copy Markdown
Contributor

Pushed a small follow-up to this branch (maintainer edit) to unblock CI: ExecProcess single-command tokenization after #7565 was failing ExecProcessTest.processRow_singleCommandString_capturesOutput with an NPE on null stdout.

Approach from @leehaut in #7581: tokenize with StringTokenizer, then Runtime.exec(String[]) (avoids the deprecated String overload).

Commit: bedba3f

@hansva
hansva merged commit 6850757 into apache:main Jul 20, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants