[#MECLIPSE-538] Ajdt support needs to include*.aj on the classpath. - jira.codehaus.org

Maven 2.x Eclipse Plugin

Ajdt support needs to include*.aj on the classpath.

Created: 26/Mar/09 05:43 PM Updated: 09/May/09 07:04 PM Resolved: 30/Mar/09 06:16 PM

Return to search

Component/s:

AJDT support

Affects Version/s:

2.6

Fix Version/s:

2.7

Time Tracking:

Not Specified

Issue Links:

Duplicate

This issue is duplicated by:

~~MECLIPSE-553~~ Maven Eciipse Plugin 2.6 excludes *.aj files from src/main/java

Related

This issue is related to:

~~MECLIPSE-551~~ [regression] Resources are excluded if resource dir equals source dir

Supercedes

This issue is superceded by:

~~MECLIPSE-104~~ Add the ability to specify source inclusions/exclusions

Description

« Hide

~~MECLIPSE-443~~ broken ajdt support as *.java files are now specified as "included" where previously this was empty.
This means that *.aj files are no longer on the classpath and this break ajdt support.

Sort Order:

Barrie Treloar made changes - 26/Mar/09 05:43 PM

Field	Original Value	New Value
Assignee		Barrie Treloar [ baerrach ]

Barrie Treloar made changes - 26/Mar/09 05:43 PM

Status

Open [ 1 ]

In Progress [ 3 ]

Barrie Treloar made changes - 26/Mar/09 07:48 PM

Fix Version/s

2.7 [ 14813 ]

[ Permalink | « Hide ]

Barrie Treloar added a comment - 26/Mar/09 11:46 PM

Moved inclusion/exclusion patterns out of EclipseClasspathWriter and into EclipsePlugin extract methods where the EclipseSourceDir gets created and defines the includes/excludes.

If ajdt is enable then all sources directories will include */.aj files.

The aspect directory will also include */.java files.

Added toString() to EclipseSourceDir.

The generated resource directory for manifest handling (target/generated-resources/eclipse) no longer exclude java files.
Since we create that directory and copy the MANIFEST.MF file into it there isn't much need for exclusion.
Test cases updated to reflect this.

Test case for ~~MECLIPSE-538~~ created.

Barrie Treloar made changes - 26/Mar/09 11:46 PM

Resolution		Fixed [ 1 ]
Status	In Progress [ 3 ]	Closed [ 6 ]

Barrie Treloar made changes - 30/Mar/09 06:15 PM

Status	Closed [ 6 ]	Reopened [ 4 ]
Resolution	Fixed [ 1 ]

Barrie Treloar made changes - 30/Mar/09 06:16 PM

Resolution		Fixed [ 1 ]
Fix Version/s	2.7 [ 14813 ]
Fix Version/s		2.6.1 [ 15130 ]
Status	Reopened [ 4 ]	Closed [ 6 ]

Arnaud Heritier made changes - 15/Apr/09 04:22 PM

Link

This issue is duplicated by ~~MECLIPSE-553~~ [ ~~MECLIPSE-553~~ ]

Arnaud Heritier made changes - 15/Apr/09 04:23 PM

Link

This issue is superceded by ~~MECLIPSE-104~~ [ ~~MECLIPSE-104~~ ]

[ Permalink | « Hide ]

Andrew Eisenberg added a comment - 15/Apr/09 05:38 PM

Will this fix be included in 2.6.1 (if this build occurs)?

[ Permalink | « Hide ]

Barrie Treloar added a comment - 15/Apr/09 05:42 PM

No 2.6.1 has been renamed to 2.7.
As the consensus was that even though the issues fixed resolved a regression issue they did so by adding new features.

Arnaud is working on getting 2.7 out shortly.

Ari Meyer made changes - 09/May/09 07:04 PM

Link

This issue is related to ~~MECLIPSE-551~~ [ ~~MECLIPSE-551~~ ]