Maven 1.x Ant Plugin
  1. Maven 1.x Ant Plugin
  2. MPANT-16

Use relative paths in directory properties.

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7
    • Fix Version/s: 1.8.1
    • Labels:
      None
    • Number of attachments :
      1

      Description

      The generated Ant script currently contains absolute paths in some of the declared properties:
      <property name="defaulttargetdir" value="$

      {maven.build.dir}"/>
      <property name="libdir" value="${maven.build.dir}

      /lib" />
      <property name="classesdir" value="$

      {maven.build.dest}

      " />
      <property name="testclassesdir" value="$

      {maven.test.dest}

      " />

      I would like to see those properties made relative paths.

      I'll apply a patch the generate relative path properties.

        Activity

        Hide
        Arnaud Heritier added a comment -

        reopen to set "fix version"

        Show
        Arnaud Heritier added a comment - reopen to set "fix version"
        Hide
        Arnaud Heritier added a comment -

        update fix-version

        Show
        Arnaud Heritier added a comment - update fix-version
        Hide
        Brent Worden added a comment -

        I missed one of the absolute paths. The fix is simple. Change line 285 of build.jelly to:
        <fileset dir="$

        {testSrcDir}

        ">

        Show
        Brent Worden added a comment - I missed one of the absolute paths. The fix is simple. Change line 285 of build.jelly to: <fileset dir="$ {testSrcDir} ">
        Hide
        Carlos Sanchez added a comment -

        Arnaud: maybe you want to release 1.8.1 with this fixed

        Show
        Carlos Sanchez added a comment - Arnaud: maybe you want to release 1.8.1 with this fixed
        Hide
        Arnaud Heritier added a comment -

        Thanks Carlos. I'll do it today or tomorrow.

        Show
        Arnaud Heritier added a comment - Thanks Carlos. I'll do it today or tomorrow.
        Hide
        Arnaud Heritier added a comment -

        Will be fixed in 1.8.1

        Show
        Arnaud Heritier added a comment - Will be fixed in 1.8.1
        Hide
        Arnaud Heritier added a comment -

        We will create a 1.8.1 release

        Show
        Arnaud Heritier added a comment - We will create a 1.8.1 release

          People

          • Assignee:
            Carlos Sanchez
            Reporter:
            Brent Worden
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: