Issue Details (XML | Word | Printable)

Key: MECLIPSE-200
Type: New Feature New Feature
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Arnaud Heritier
Reporter: Eric Berry
Votes: 10
Watchers: 10
Operations

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

Please add support for the AJDT plugin

Created: 30/Nov/06 01:31 PM   Updated: 27/Nov/09 07:44 AM   Resolved: 02/Sep/08 06:22 PM
Return to search
Component/s: AJDT support
Affects Version/s: 2.2
Fix Version/s: 2.6

Time Tracking:
Not Specified

File Attachments: 1. Text File ajdt.patch (23 kB)
2. Zip Archive maven-eclipse-plugin.zip (122 kB)
3. Zip Archive maven-eclipse-plugin.zip (14 kB)

Issue Links:
Duplicate
 
Related
dependent
 

Patch Submitted: Yes


 Description  « Hide

Please add support for the AJDT plugin.

I have modified the eclipse plugin for our site with the needed changes. I have attached the modifications that I made for someone to look at and incorporate into the plugin.



Carlos Sanchez added a comment - 13/Mar/07 05:21 PM

Previous attachment in diff format


Carlos Sanchez made changes - 13/Mar/07 05:21 PM
Field Original Value New Value
Attachment ajdt.patch [ 26256 ]
Arnaud Heritier made changes - 16/Nov/07 07:06 AM
Patch Submitted [Yes]
Arnaud Heritier made changes - 29/Nov/07 09:54 PM
Component/s AJDT support [ 13074 ]
Frank Stolle made changes - 26/Jan/08 05:35 AM
Link This issue relates to MECLIPSE-313 [ MECLIPSE-313 ]
Eric Berry added a comment - 28/May/08 02:10 PM - edited

This has been updated for maven-eclipse-plugin 2.5.1 and also ajdt version 1.5.3. I have also included tests and updates to the site build to show how to use it.

Hopefully one of the commiters will be willing to look at this and incorporate it.


Eric Berry made changes - 28/May/08 02:10 PM
Attachment maven-eclipse-plugin.zip [ 34827 ]
Eric Berry added a comment - 28/May/08 02:11 PM

Look at the 122kb maven-eclipse-plugin.zip.


Eric Berry made changes - 28/May/08 02:13 PM
Link This issue is duplicated by MECLIPSE-270 [ MECLIPSE-270 ]
David J. M. Karlsen made changes - 23/Aug/08 06:06 PM
Link This issue is duplicated by MOJO-839 [ MOJO-839 ]
Arnaud Heritier made changes - 02/Sep/08 12:16 PM
Assignee Arnaud Heritier [ aheritier ]
Arnaud Heritier made changes - 02/Sep/08 12:17 PM
Fix Version/s 2.5.2 [ 14163 ]
Arnaud Heritier made changes - 02/Sep/08 12:18 PM
Status Open [ 1 ] In Progress [ 3 ]
Arnaud Heritier added a comment - 02/Sep/08 06:22 PM

Fixed and snapshot 2.6-20080902.232029-1 deployed. If you can test it ....


Arnaud Heritier made changes - 02/Sep/08 06:22 PM
Resolution Fixed [ 1 ]
Status In Progress [ 3 ] Closed [ 6 ]
Eric Berry added a comment - 05/Sep/08 10:43 AM

Tested. It works as expected. Thanks.


David J. M. Karlsen added a comment - 09/Sep/08 10:26 AM

Is the site staged anywhere (to be able to see documentation of new parameters)


Arnaud Heritier added a comment - 09/Sep/08 05:26 PM

Alexander Hachmann made changes - 03/Apr/09 09:32 AM
Link This issue is depended upon by MECLIPSE-544 [ MECLIPSE-544 ]
Alexander Hachmann added a comment - 03/Apr/09 09:37 AM

I can understand the wish for a AJDT-Plugin support, but I cannot understand
why this must have such an impact on projects that do not use that plugin.

As we do not use that plugin we now are having the problem that our projects
do not compile for the reason of the missing jars :-/

In my eyes this AJDT-Plugin Feature MUST be implemented in a more generic way.
Read the eclipse configurations whether that plugin is installed or something else.
But do not force other Projects to use the AJDT Plugin, just because the are workin
with AOP!!!!


Arnaud Heritier added a comment - 03/Apr/09 04:21 PM

Please open an new issue if this one creates a problem for you. A 2.7 is already on rails.


Eric Berry added a comment - 03/Apr/09 05:27 PM - edited

Alexander, In the configuration of the eclipse plugin add <ajdtVersion>none</ajdtVersion>. This should fix the problem.

There is hidden documentation here http://maven.apache.org/plugins/maven-eclipse-plugin/examples/ajdt-projects.html that describes how to use the ajdt plugin.


Tan Quach made changes - 06/Apr/09 05:03 PM
Link This issue is related to MECLIPSE-547 [ MECLIPSE-547 ]
Tan Quach added a comment - 06/Apr/09 05:04 PM

I agree with Alexander, and I've created a new issue: http://jira.codehaus.org/browse/MECLIPSE-547


Eirik Rosvold Larsen added a comment - 25/Nov/09 08:37 AM

According to the comments on MECLIPSE-270, this jira task replaces/duplicates that task. MECLIPSE-270 is about adding attributes to classpathentries in .classpath.

Can someone please elaborate how this can be accomplished in the pom.xml? This feature lacks documentation on the project website.

Regards


Arnaud Heritier added a comment - 25/Nov/09 09:04 AM

The documentation about ajdt is here : http://maven.apache.org/plugins/maven-eclipse-plugin/examples/ajdt-projects.html
You can find several exemples here : http://svn.apache.org/repos/asf/maven/plugins/trunk/maven-eclipse-plugin/src/test/resources/projects/
If some docs are missing, do not hesitate to open an issue. We'll try to improve it.
We know there are many lacks in it.


Eirik Rosvold Larsen added a comment - 27/Nov/09 07:44 AM

Arnaud: thanks for the link.

After plundering around, I found that adding wtp version 2.0 to the pom automatically added the required attributes to the classpathentries like this:

<classpathentry kind="var" path="M2_REPO/log4j/log4j/1.2.15/log4j-1.2.15.jar" sourcepath="M2_REPO/log4j/log4j/1.2.15/log4j-1.2.15-sources.jar">
<attributes>
<attribute value="/WEB-INF/lib" name="org.eclipse.jst.component.dependency"/>
</attributes>
</classpathentry>

A nice feature was that the attribute was only added if the maven dependency scope was compile or runtime. Small question though: If the scope is provided, the maven eclipse plugin doesn't add any attribute to the corresponding classpathentry. The wtp plugin for eclipse seems to support the following attribute for entries that are not to be deployed:

"org.eclipse.jst.component.nondependency"

Maybe you should add this for scope provided? Just a thought..

Thanks again!