ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 33534] - AspectJ compiler
Date Sat, 12 Feb 2005 16:06:15 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=33534>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33534





------- Additional Comments From kevin.jackson@it.fts-vn.com  2005-02-12 17:06 -------
Ok, so after a quick break to watch the football...

I've implemented a compiler for iajc, but I've only tested with an old version of AspectJ,
this has one 
major problem:

Old versions (ajc) require that aspect source files be called foo.aj, not foo.java.  New versions
of AspectJ 
compiler (iajc) don't have this deficiency.  To get it working with the old (ajc) I replaced
the 
GlobPatternMapper with a RegexpPatternMapper so that it could pull in *.aj and *.java files.
 I'm now 
going to download the newer compiler (iajc) and rename my files, revert the RegexpPatternMapper
to 
the GlobPatternMapper and see if this works.

Apart from this problem (which should be resolved with the newer compiler), all the changes
will be 
local to taskdefs.compilers.

Obviously this implementation hasn't been thoroughly tested, and as yet I've only got a barebones

(compile, no fancy debug options etc) implemetation completed.

Pros:
- enables compilation of AspectJ code within Ant without having to use an external taskdef
- iajc is based on the eclipse compiler and will happily compile 'normal' Java code along
with aspects
- minimal impact on the codebase (one change to CompilerAdapterFactory, and a new Iajc class)

Cons:
- still requires aspectjtools.jar on the classpath for the target eg:

  <classpath>
            <pathelement location="${ajlibs}/aspectjtools.jar"/>
            <pathelement location="${build}"/>
  </classpath>

Again sorry for using Bugzilla for this, but my ISP is still not happy.
Kev

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org


Mime
View raw message