Skip to content

Fixed search for matching line with java version#260

Closed
shuebnersr wants to merge 4 commits intopuppetlabs:masterfrom
shuebnersr:master
Closed

Fixed search for matching line with java version#260
shuebnersr wants to merge 4 commits intopuppetlabs:masterfrom
shuebnersr:master

Conversation

@shuebnersr
Copy link
Copy Markdown
Contributor

Use value of variable instead of return statement
Extended unit test for checking handling of JAVA_TOOL_OPTIONS

do not just try to read java version from first line, but search for the matching line from java output
…when_java_not_installed

Fact1760 test for java version when java not installed
Use value of variable instead of return statement
Extended unit test for checking handling of JAVA_TOOL_OPTIONS
@shuebnersr
Copy link
Copy Markdown
Contributor Author

shuebnersr commented Oct 1, 2017

Code improved, CI checks now succeed. Pull request #257 has updated itself, so no need to create a new one.

@shuebnersr shuebnersr closed this Oct 1, 2017
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.

1 participant