Skip to content

added quotation around $JAVA nohup and exec call#4607

Closed
lamchakchan wants to merge 1 commit into
apache:trunkfrom
lamchakchan:zookeeper-java-env-path-patch
Closed

added quotation around $JAVA nohup and exec call#4607
lamchakchan wants to merge 1 commit into
apache:trunkfrom
lamchakchan:zookeeper-java-env-path-patch

Conversation

@lamchakchan
Copy link
Copy Markdown

When the $JAVA environment value contains spaces, the scripting context fails to interpret the command correctly.

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@chia7712
Copy link
Copy Markdown
Member

@lamchakchan Thanks for your patch. Sorry that I'm going to close this PR since the issue was resolved by c34f3d0

@chia7712 chia7712 closed this Oct 19, 2020
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