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

[pipeline-maven-plugin] maven tool installation requires maven plugin installation.

XMLWordPrintable

      pipeline maven plugin requires the maven plugin to be able to specify the Maven installation to use - but it does not have a dependency on it

       

      the maven-pipeline plugin should not require the maven plugin to be installed to be able to use the tool installations as the maven job type is considered evil - so ideally the maven tool installations should be pulled out of the maven-plugin and moved to a separate plugin. so the tool installations can be used without including the evil job type.

          [JENKINS-43650] [pipeline-maven-plugin] maven tool installation requires maven plugin installation.

          James Nord created issue -
          James Nord made changes -
          Summary Original: maven tool installation requires maven plugin installation. New: [pipeline-maven-plugin] maven tool installation requires maven plugin installation.

          Arnaud Héritier added a comment -

          To fix with JENKINS-26462 and https://github.com/jenkinsci/maven-plugin/pull/72 ??

          I'm surprised about it, the installer isn't in core ?

          Arnaud Héritier added a comment - To fix with  JENKINS-26462 and https://github.com/jenkinsci/maven-plugin/pull/72  ?? I'm surprised about it, the installer isn't in core ?

          James Nord added a comment -

          I will retest...

          James Nord added a comment - I will retest...

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Cyrille Le Clerc
          Path:
          jenkins-plugin/pom.xml
          http://jenkins-ci.org/commit/pipeline-maven-plugin/975e511dec7ecb00392a93270cda857f405936db
          Log:
          JENKINS-43650 Add dependency to org.jenkins-ci.main:maven-plugin. Will be needed to manage global-level and folder-level settings

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Cyrille Le Clerc Path: jenkins-plugin/pom.xml http://jenkins-ci.org/commit/pipeline-maven-plugin/975e511dec7ecb00392a93270cda857f405936db Log: JENKINS-43650 Add dependency to org.jenkins-ci.main:maven-plugin. Will be needed to manage global-level and folder-level settings
          Cyrille Le Clerc made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Cyrille Le Clerc
          Path:
          jenkins-plugin/pom.xml
          http://jenkins-ci.org/commit/pipeline-maven-plugin/6e180c4d016af5f4cf8d6668fdb9d2ed241c98e2
          Log:
          Merge pull request #48 from jenkinsci/JENKINS-43650

          JENKINS-43650 Add dependency to org.jenkins-ci.main:maven-plugin.

          Compare: https://github.com/jenkinsci/pipeline-maven-plugin/compare/6c51e3ad17ce...6e180c4d016a

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Cyrille Le Clerc Path: jenkins-plugin/pom.xml http://jenkins-ci.org/commit/pipeline-maven-plugin/6e180c4d016af5f4cf8d6668fdb9d2ed241c98e2 Log: Merge pull request #48 from jenkinsci/ JENKINS-43650 JENKINS-43650 Add dependency to org.jenkins-ci.main:maven-plugin. Compare: https://github.com/jenkinsci/pipeline-maven-plugin/compare/6c51e3ad17ce...6e180c4d016a

          James Nord added a comment -

          > I will retest...

          retested... the maven installation option was there...

          James Nord added a comment - > I will retest... retested... the maven installation option was there...

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Cyrille Le Clerc
          Path:
          jenkins-plugin/pom.xml
          http://jenkins-ci.org/commit/pipeline-maven-plugin/ebd8da1dd32c1778be7bc2764ad1bfda0609b92d
          Log:
          JENKINS-43650 Dependency to org.jenkins-ci.main:maven-plugin should be optional because this plugin is too tied to the Maven Job Type.
          See discussion on https://github.com/jenkinsci/pipeline-maven-plugin/pull/48

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Cyrille Le Clerc Path: jenkins-plugin/pom.xml http://jenkins-ci.org/commit/pipeline-maven-plugin/ebd8da1dd32c1778be7bc2764ad1bfda0609b92d Log: JENKINS-43650 Dependency to org.jenkins-ci.main:maven-plugin should be optional because this plugin is too tied to the Maven Job Type. See discussion on https://github.com/jenkinsci/pipeline-maven-plugin/pull/48

          Cyrille Le Clerc added a comment -

          Eventually, as discussed on https://github.com/jenkinsci/pipeline-maven-plugin/pull/48 , keep the dependency optional.

          Cyrille Le Clerc added a comment - Eventually, as discussed on https://github.com/jenkinsci/pipeline-maven-plugin/pull/48  , keep the dependency optional.

            alobato Alvaro Lobato
            teilo James Nord
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: