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.

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

Fix for JENKINS-16204 #11

Merged
merged 1 commit into from Mar 23, 2014

Conversation

Projects
None yet
5 participants
@ghost
Copy link

ghost commented Mar 22, 2014

List JDKs even if there is only one, to ensure a value is always set.

Fix for JENKINS-16204
List JDKs even if there is only one, to ensure a value is always set.

@ghost ghost closed this Mar 22, 2014

@ghost ghost reopened this Mar 22, 2014

@cloudbees-pull-request-builder

This comment has been minimized.

Copy link

cloudbees-pull-request-builder commented Mar 22, 2014

plugins » fitnesse-plugin #14 SUCCESS
This pull request looks good

@jenkinsadmin

This comment has been minimized.

Copy link
Member

jenkinsadmin commented Mar 22, 2014

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

lessonz added a commit that referenced this pull request Mar 23, 2014

@lessonz lessonz merged commit 3d0ac03 into jenkinsci:master Mar 23, 2014

@ghost

This comment has been minimized.

Copy link
Author

ghost commented Mar 25, 2014

Thank you for merging the pull request!

Is there any chance a new release of the plugin could be made?

@lessonz

This comment has been minimized.

Copy link
Member

lessonz commented Mar 26, 2014

Done. It may take up to 48 for it to show up on the release page, but you can see 1.9 here: http://maven.jenkins-ci.org:8081/content/repositories/releases/org/jenkins-ci/plugins/fitnesse/

@albers

This comment has been minimized.

Copy link
Member

albers commented Mar 26, 2014

Thanks for the fix. It is a great improvement and fixes the issue for me.

There is still a minor issue when no JDK is configured at all, but this is of minor importance. See updated JENKINS-16204

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.