Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-755

Default JDK meaning in project options is confusing.

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Component/s: maven-plugin
    • Labels:
      None
    • Environment:
      Platform: HP, OS: Windows XP
    • Similar Issues:

      Description

      When the option for which JDK to use for native m2 builds is set to "default"
      and there is only one jdk configured in hudson the builds should use that jdk
      not the jdk/jre of the container.

      if there are more jdks' configured hudson should use the first one configured
      at the system level for default.

        Attachments

          Issue Links

            Activity

            Hide
            dogfood dogfood added a comment -

            Integrated in jenkins_main_trunk #4340
            JENKINS-755 Describe the built-in JDK as '(System)'. (Revision ff021d3c374a067d622fdb078a4e391149df7d41)

            Result = SUCCESS
            whughes : ff021d3c374a067d622fdb078a4e391149df7d41
            Files :

            • core/src/main/java/hudson/model/JDK.java
            Show
            dogfood dogfood added a comment - Integrated in jenkins_main_trunk #4340 JENKINS-755 Describe the built-in JDK as '(System)'. (Revision ff021d3c374a067d622fdb078a4e391149df7d41) Result = SUCCESS whughes : ff021d3c374a067d622fdb078a4e391149df7d41 Files : core/src/main/java/hudson/model/JDK.java
            Hide
            oleg_nenashev Oleg Nenashev added a comment -
            Show
            oleg_nenashev Oleg Nenashev added a comment - Fixed in https://github.com/jenkinsci/jenkins/pull/1528
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Oleg Nenashev
            Path:
            changelog.html
            http://jenkins-ci.org/commit/jenkins/0cc465e706f69b7ee0389749233ab2f33ca47eed
            Log:
            Noting fix of JENKINS-755 in https://github.com/jenkinsci/jenkins/pull/1528

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: changelog.html http://jenkins-ci.org/commit/jenkins/0cc465e706f69b7ee0389749233ab2f33ca47eed Log: Noting fix of JENKINS-755 in https://github.com/jenkinsci/jenkins/pull/1528
            Hide
            mwebber Matthew Webber added a comment -

            The fix for this problem, which was released in Jenkins 1.635, appears to have caused a new bug JENKINS-31217

            Show
            mwebber Matthew Webber added a comment - The fix for this problem, which was released in Jenkins 1.635, appears to have caused a new bug JENKINS-31217
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Oleg Nenashev
            Path:
            core/src/main/java/hudson/model/AbstractProject.java
            core/src/main/java/hudson/model/JDK.java
            core/src/main/java/jenkins/model/Jenkins.java
            core/src/test/java/jenkins/model/JDKNameTest.java
            http://jenkins-ci.org/commit/jenkins/730d6ecb2d29b25ef69f5cf49e097fb09dbfc3a6
            Log:
            Merge pull request #1528 from Wilfred/better_default_text

            JENKINS-755 Describe the built-in JDK as '(System)'.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: core/src/main/java/hudson/model/AbstractProject.java core/src/main/java/hudson/model/JDK.java core/src/main/java/jenkins/model/Jenkins.java core/src/test/java/jenkins/model/JDKNameTest.java http://jenkins-ci.org/commit/jenkins/730d6ecb2d29b25ef69f5cf49e097fb09dbfc3a6 Log: Merge pull request #1528 from Wilfred/better_default_text JENKINS-755 Describe the built-in JDK as '(System)'.

              People

              • Assignee:
                wilfredh Wilfred Hughes
                Reporter:
                teilo James Nord
              • Votes:
                48 Vote for this issue
                Watchers:
                51 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: