Maven 2 & 3
  1. Maven 2 & 3
  2. MNG-3535

Valid properties which look self referential fail to resolve

    Details

    • Complexity:
      Intermediate
    • Number of attachments :
      1

      Description

      In 2.0.9 properties which look self referential but would otherwise resolve to a system property are failing due to fixes for MNG-2339. Current example is any version of jruby shared pom at
      http://repo1.maven.org/maven2/org/jruby/shared/1.0.1/shared-1.0.1.pom

      which contains:
      <java.specification.version>$

      {java.specification.version}

      </java.specification.version>

      The question is whether this should be valid or not, but it has worked in every version up to and including 2.0.8 because System properties were available in the first interpolate step. In 2.0.9 this first pass does not include the system props and an exception is thrown because of the self reference check.

        Issue Links

          Activity

          Hide
          Shane Isbell added a comment -

          Patch for this issue. If the system property starts with 'pom.' or 'project.' then self -references are not allowed. But if the self-reference is anything else (outside of the model), it is allowed.

          Show
          Shane Isbell added a comment - Patch for this issue. If the system property starts with 'pom.' or 'project.' then self -references are not allowed. But if the self-reference is anything else (outside of the model), it is allowed.
          Hide
          Shane Isbell added a comment -

          Bug fix here: https://svn.apache.org/repos/asf/maven/sandbox/branches/MNG-3536

          This is part of a rewrite of the interpolation code.

          Show
          Shane Isbell added a comment - Bug fix here: https://svn.apache.org/repos/asf/maven/sandbox/branches/MNG-3536 This is part of a rewrite of the interpolation code.

            People

            • Assignee:
              John Casey
              Reporter:
              Chris Custine
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: