Maven 2.x Eclipse Plugin

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

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.6
  • Fix Version/s: 2.7
  • Component/s: AJDT support
  • Labels:
    None
  • Number of attachments :
    0

Description

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.

Issue Links

This issue is duplicated by:
MECLIPSE-553 Maven Eciipse Plugin 2.6 excludes *.aj files from src/main/java Major Closed
This issue is related to:
MECLIPSE-551 [regression] Resources are excluded if resource dir equals source dir Major Closed
This issue is superceded by:
MECLIPSE-104 Add the ability to specify source inclusions/exclusions Minor Closed

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Barrie Treloar added a comment -

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.

Show
Barrie Treloar added a comment - 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.
Hide
Permalink
Andrew Eisenberg added a comment -

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

Show
Andrew Eisenberg added a comment - Will this fix be included in 2.6.1 (if this build occurs)?
Hide
Permalink
Barrie Treloar added a comment -

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.

Show
Barrie Treloar added a comment - 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.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.