Summary: | ajc pseudo LTW not consistent with LTW | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Tools] AspectJ | Reporter: | Alexandre Vasseur <avasseur> | ||||||
Component: | Compiler | Assignee: | Matthew Webster <matthew_webster> | ||||||
Status: | RESOLVED FIXED | QA Contact: | |||||||
Severity: | normal | ||||||||
Priority: | P2 | CC: | matthew_webster | ||||||
Version: | DEVELOPMENT | ||||||||
Target Milestone: | 1.5.0RC1 | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Whiteboard: | |||||||||
Attachments: |
|
Description
Alexandre Vasseur
2005-05-17 05:19:40 EDT
moved to P2. Alex and Matthew, please can you work together to decide what we should do with "aj" in the new world. Just rebasing it on the new LTW agents so that it obeys the aop.xml etc. seems the simplest and best thing here? this won't solve the issue for java 1.3/1.4 since we decided to not have the AW LTW in there but as an extension (see my blog). I suggest to keep aj as it is (no aop.xml, based on aspect path etc)and possibly have an aj5 script for use with java5 -javaagent and aop.xml Matthew? We could have 2 scripts: the existing "aj" and "aj5". They are both installed regardless of which JDK you specify during install. We will rely on any error messages from the JDK if someone uses the wrong one although 1.3 will fail silently. - "aj" will use both aspectpath and aop.xml since my changes in 107741. We need to update the documentation and examples - "aj5" will be a simple wrapper for "java -javaagent:aspectjweaver.jar ..." using aop.xml. We also need a implementation for bug 95516. Matthew, could you close this one out for RC1 pls? Created attachment 29547 [details]
Patch
build: Modified installer to update aj.bat and add aj5.bat
docs: New README and simple HelloWorld/Tracing example to illustrate aop.xml
support and new –outxml option
lib: build/build.jar must be update. Build as follows:
- Run “..\lib\ant\bin\ant build” from “from build module”
- Copy “aj-build\jars\build.jar” to “lib\build”
- Commit lib module
Can someone please test aj and aj5 on Linux/Mac
Created attachment 29960 [details]
Patch
Now tested successfully on Mac after changing onUnix test to be true for Mac.
patch integrated - waiting on build. fix available. |