[#MPANT-16] Use relative paths in directory properties. - jira.codehaus.org

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:

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.

Sort Order:

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 ]

[ Permalink | « Hide ]

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 ]

[ Permalink | « Hide ]

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 ]

[ Permalink | « Hide ]

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 ]

[ Permalink | « Hide ]

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 ]

[ Permalink | « Hide ]

Arnaud Heritier added a comment - 16/Aug/04 07:25 AM

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

[ Permalink | « Hide ]

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 ]

[ Permalink | « Hide ]

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 ]