Issue Details (XML | Word | Printable)

Key: MPANT-16
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Carlos Sanchez
Reporter: Brent Worden
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Maven 1.x Ant Plugin

Use relative paths in directory properties.

Created: 24/Jun/04 04:36 PM   Updated: 16/Aug/04 09:13 AM   Resolved: 16/Aug/04 09:13 AM
Return to search
Component/s: None
Affects Version/s: 1.7
Fix Version/s: 1.8.1

Time Tracking:
Not Specified

File Attachments: 1. Text File maven-plugins-ant-relative.diff (2 kB)



 Description  « Hide

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.



Brent Worden made changes - 24/Jun/04 04:37 PM
Field Original Value New Value
Attachment maven-plugins-ant-relative.diff [ 12363 ]
Carlos Sanchez made changes - 03/Jul/04 01:49 PM
Resolution Fixed [ 1 ]
Status Open [ 1 ] Closed [ 6 ]
Carlos Sanchez made changes - 03/Jul/04 01:49 PM
Assignee Carlos Sanchez [ carlos ]
Arnaud Heritier added a comment - 20/Jul/04 02:55 PM

reopen to set "fix version"


Arnaud Heritier made changes - 20/Jul/04 02:55 PM
Status Closed [ 6 ] Reopened [ 4 ]
Resolution Fixed [ 1 ]
Arnaud Heritier added a comment - 20/Jul/04 02:56 PM

update fix-version


Arnaud Heritier made changes - 20/Jul/04 02:56 PM
Fix Version/s 1.8 [ 10990 ]
Status Reopened [ 4 ] Closed [ 6 ]
Resolution Fixed [ 1 ]
Brent Worden added a comment - 15/Aug/04 08:53 PM

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


Carlos Sanchez made changes - 16/Aug/04 05:43 AM
Resolution Fixed [ 1 ]
Status Closed [ 6 ] Reopened [ 4 ]
Carlos Sanchez added a comment - 16/Aug/04 06:53 AM

Arnaud: maybe you want to release 1.8.1 with this fixed


Carlos Sanchez made changes - 16/Aug/04 06:53 AM
Resolution Fixed [ 1 ]
Fix Version/s 1.9 [ 11003 ]
Fix Version/s 1.8 [ 10990 ]
Status Reopened [ 4 ] Closed [ 6 ]
Arnaud Heritier added a comment - 16/Aug/04 07:25 AM

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


Arnaud Heritier added a comment - 16/Aug/04 09:13 AM

Will be fixed in 1.8.1


Arnaud Heritier made changes - 16/Aug/04 09:13 AM
Status Closed [ 6 ] Reopened [ 4 ]
Resolution Fixed [ 1 ]
Arnaud Heritier added a comment - 16/Aug/04 09:13 AM

We will create a 1.8.1 release


Arnaud Heritier made changes - 16/Aug/04 09:13 AM
Resolution Fixed [ 1 ]
Fix Version/s 1.9 [ 11003 ]
Fix Version/s 1.8.1 [ 11104 ]
Status Reopened [ 4 ] Closed [ 6 ]