Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 328779 - [jdt-weaving] Enable JDT Weaving by default
Summary: [jdt-weaving] Enable JDT Weaving by default
Status: RESOLVED FIXED
Alias: None
Product: AJDT
Classification: Tools
Component: Core (show other bugs)
Version: 2.1.1   Edit
Hardware: Macintosh All
: P3 major (vote)
Target Milestone: 2.1.2   Edit
Assignee: AJDT-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-26 19:00 EDT by Andrew Eisenberg CLA
Modified: 2010-11-23 19:31 EST (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Eisenberg CLA 2010-10-26 19:00:49 EDT 
It has been almost two years since we introduced JDT weaving into AJDT and we believe that most of the kinks have been worked out.  And thanks to the org.eclipse.equinox.weaving.cahcing plugin, the performance hit of having the weaving service turned on is negligible after the first startup.  Furthermore, the only problems we hear about regarding the weaving service are regarding problems with enabling it.  Finally, there are more features relying on JDT weaving now: STS and roo, Scala IDE, and potentially Groovy-Eclipse.

For all of these reasons, it is time to enable JDT weaving by default when AJDT is installed (and the ability to disable it through the UI is still important).
Comment 1 Andrew Eisenberg CLA 2010-10-26 19:03:41 EDT 
I just committed a first pass at this.  I was able to do some basic testing in my runtime workspace, but will have to wait until a build is avaiable before really being able to test.  

Here is what is entailed:

1. Needed to use the SimpleConfiguratorManipulator API to mark changes in the auto startup setting for the org.equinox.weaving.aspectj bundle.
2. Needed to add a p2.inf file to org.eclipse.equinox.weaving.aspectj bundle to change its auto-start setting to true on install.
Comment 2 Andrew Eisenberg CLA 2010-11-15 01:26:30 EST 
Now working in the snapshot version of AJDT.  JDT Weaving will not be enabled in an upgrade, only on a fresh install.
Comment 3 Andrew Eisenberg CLA 2010-11-23 19:31:35 EST 
Now working.