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

In Maven2 jobs, variable expansion is not done for the root POM path and alternative settings

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • maven-plugin
    • None
    • Hudson 1.349, Clearcase plugin 1.1.1

      I'm trying to convert an existing free-style job to maven2.
      Since I use also the Clearcase plugin, I use the ${CLEARCASE_VIEWNAME}/pom.xml to reference my root pom. Inside a freestyle job, no problem, but when using maven2 job, the variable is not expanded.
      The same applies to the alternative settings. If I provide it in options it is expanded correctly, but not in the dedicaced field. This is problematic because then, it cannot be used for post-build deployment for instance.

          [JENKINS-5885] In Maven2 jobs, variable expansion is not done for the root POM path and alternative settings

          Vincent Latombe created issue -
          Andrew Bayer made changes -
          Assignee New: Andrew Bayer [ abayer ]

          Andrew Bayer added a comment -

          Just to clarify - ${CLEARCASE_VIEWNAME}/pom.xml does expand properly in a freestyle job? I just want to make sure so that I don't reuse something that turns out not to work. =)

          Andrew Bayer added a comment - Just to clarify - ${CLEARCASE_VIEWNAME}/pom.xml does expand properly in a freestyle job? I just want to make sure so that I don't reuse something that turns out not to work. =)

          Vincent Latombe added a comment - - edited

          My bad.... I actually use %CLEARCASE_VIEWPATH% in my freestyle job so it is done through the shell. Should be similar with %CLEARCASE_VIEWNAME%

          Vincent Latombe added a comment - - edited My bad.... I actually use %CLEARCASE_VIEWPATH% in my freestyle job so it is done through the shell. Should be similar with %CLEARCASE_VIEWNAME%

          Carsten Klein added a comment -

          We, too, would like to be able to parameterize the Root POM field.
          See the attached patch for a proposed change that could make this happen.

          Carsten Klein added a comment - We, too, would like to be able to parameterize the Root POM field. See the attached patch for a proposed change that could make this happen.
          Carsten Klein made changes -
          Attachment New: git.patch [ 20514 ]
          Olivier Lamy made changes -
          Assignee Original: Andrew Bayer [ abayer ] New: Olivier Lamy [ olamy ]

          Hal Naut added a comment -

          Seems to me that the M2 Extra Builds step expands variables fine for the same target pom path (assuming my issue is the same), so it could be another alternative solution source to Carsten's kindly provided patch.

          Hal Naut added a comment - Seems to me that the M2 Extra Builds step expands variables fine for the same target pom path (assuming my issue is the same), so it could be another alternative solution source to Carsten's kindly provided patch.

          Carsten Klein added a comment -

          I published the fix to address this problem on github.

          There is now a pull request open for the jenkins repository.

          Carsten Klein added a comment - I published the fix to address this problem on github. There is now a pull request open for the jenkins repository.

          Code changed in jenkins
          User: Kohsuke Kawaguchi
          Path:
          changelog.html
          http://jenkins-ci.org/commit/jenkins/3192dfb59f6864fa5e782163dbca749cd80bed2b
          Log:
          [FIXED JENKINS-5885] expand rootPOM with variables.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Kohsuke Kawaguchi Path: changelog.html http://jenkins-ci.org/commit/jenkins/3192dfb59f6864fa5e782163dbca749cd80bed2b Log: [FIXED JENKINS-5885] expand rootPOM with variables.
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

            olamy Olivier Lamy
            vlatombe Vincent Latombe
            Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: