Maven 1.x Ant Plugin

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
  • Component/s: None
  • 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

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: