提交 a2341e6b 编写于 作者: J Jesse Glick

[FIXED JENKINS-24251] Do not issue bogus warning about missing default JDK.

上级 9532b550
...@@ -350,7 +350,7 @@ public abstract class AbstractProject<P extends AbstractProject<P,R>,R extends A ...@@ -350,7 +350,7 @@ public abstract class AbstractProject<P extends AbstractProject<P,R>,R extends A
jdkTool = jdkTool.forNode(node, listener); jdkTool = jdkTool.forNode(node, listener);
} }
jdkTool.buildEnvVars(env); jdkTool.buildEnvVars(env);
} else if (jdk != null) { } else if (jdk != null && !jdk.equals(JDK.DEFAULT_NAME)) {
listener.getLogger().println("No JDK named ‘" + jdk + "’ found"); listener.getLogger().println("No JDK named ‘" + jdk + "’ found");
} }
......
...@@ -52,6 +52,13 @@ import org.kohsuke.stapler.DataBoundConstructor; ...@@ -52,6 +52,13 @@ import org.kohsuke.stapler.DataBoundConstructor;
* @author Kohsuke Kawaguchi * @author Kohsuke Kawaguchi
*/ */
public final class JDK extends ToolInstallation implements NodeSpecific<JDK>, EnvironmentSpecific<JDK> { public final class JDK extends ToolInstallation implements NodeSpecific<JDK>, EnvironmentSpecific<JDK> {
/**
* Name of the “default JDK”, meaning no specific JDK selected.
* @since 1.577
*/
public static final String DEFAULT_NAME = "(Default)";
/** /**
* @deprecated since 2009-02-25 * @deprecated since 2009-02-25
*/ */
......
...@@ -3598,7 +3598,7 @@ public class Jenkins extends AbstractCIBase implements DirectlyModifiableTopLeve ...@@ -3598,7 +3598,7 @@ public class Jenkins extends AbstractCIBase implements DirectlyModifiableTopLeve
* If the user chose the default JDK, make sure we got 'java' in PATH. * If the user chose the default JDK, make sure we got 'java' in PATH.
*/ */
public FormValidation doDefaultJDKCheck(StaplerRequest request, @QueryParameter String value) { public FormValidation doDefaultJDKCheck(StaplerRequest request, @QueryParameter String value) {
if(!value.equals("(Default)")) if(!value.equals(JDK.DEFAULT_NAME))
// assume the user configured named ones properly in system config --- // assume the user configured named ones properly in system config ---
// or else system config should have reported form field validation errors. // or else system config should have reported form field validation errors.
return FormValidation.ok(); return FormValidation.ok();
......
...@@ -37,7 +37,8 @@ THE SOFTWARE. ...@@ -37,7 +37,8 @@ THE SOFTWARE.
<f:entry title="JDK" <f:entry title="JDK"
description="${%JDK to be used for this project}"> description="${%JDK to be used for this project}">
<select class="setting-input validated" name="jdk" checkUrl="'${rootURL}/defaultJDKCheck?value='+this.value"> <select class="setting-input validated" name="jdk" checkUrl="'${rootURL}/defaultJDKCheck?value='+this.value">
<option>${%default.value}</option> <j:getStatic var="DEFAULT_NAME" className="hudson.model.JDK" field="DEFAULT_NAME"/>
<option>${DEFAULT_NAME}</option>
<j:forEach var="inst" items="${jdks}"> <j:forEach var="inst" items="${jdks}">
<f:option selected="${inst.name==it.JDK.name}" value="${inst.name}">${inst.name}</f:option> <f:option selected="${inst.name==it.JDK.name}" value="${inst.name}">${inst.name}</f:option>
</j:forEach> </j:forEach>
......
...@@ -20,6 +20,5 @@ ...@@ -20,6 +20,5 @@
# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
# THE SOFTWARE. # THE SOFTWARE.
default.value=(Default)
Advanced\ Project\ Options=Advanced Project Options Advanced\ Project\ Options=Advanced Project Options
Display\ Name=Display Name Display\ Name=Display Name
...@@ -21,8 +21,6 @@ ...@@ -21,8 +21,6 @@
# THE SOFTWARE. # THE SOFTWARE.
Advanced\ Project\ Options=Op\u00e7\u00f5es avan\u00e7adas do projeto Advanced\ Project\ Options=Op\u00e7\u00f5es avan\u00e7adas do projeto
# (Default)
default.value=Default
Node=N\u00f3 Node=N\u00f3
Tie\ this\ project\ to\ a\ node=Vincular esse projeto a um n\u00f3 Tie\ this\ project\ to\ a\ node=Vincular esse projeto a um n\u00f3
Display\ Name=Nome de exibi\u00e7\u00e3o Display\ Name=Nome de exibi\u00e7\u00e3o
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册