Bug 93560 - ASPECTJ5_DEVELOPMENT branch LTW can't find aop.xml files
Summary: ASPECTJ5_DEVELOPMENT branch LTW can't find aop.xml files
Status: RESOLVED FIXED
Alias: None
Product: AspectJ
Classification: Tools
Component: Compiler (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 1.5.0   Edit
Assignee: Adrian Colyer CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-05-03 12:57 EDT by Ron Bodkin CLA
Modified: 2006-05-30 10:00 EDT (History)
0 users

See Also:


Attachments
updated ClassLoaderWeavingAdaptor file (11.59 KB, text/plain)
2005-05-03 12:58 EDT, Ron Bodkin CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ron Bodkin CLA 2005-05-03 12:57:50 EDT
I created a META-INF/aop.xml file, but the definitions weren't getting picked 
up by the loadtime5 support. 

The following change in ClassLoaderWeavingAdaptor.java fixed the problem:


    private void registerDefinitions(final BcelWeaver weaver, final 
ClassLoader loader) {
-            Enumeration xmls = loader.getResources("/META-INF/aop.xml");
+            Enumeration xmls = loader.getResources("META-INF/aop.xml");

i.e., removing the leading slash fixed the problem.

I will also attach my version of the file that loads the default Xlint 
properties (I needed that to avoid a lot of warnings) and that adds a few 
comments & some error handling.
Comment 1 Ron Bodkin CLA 2005-05-03 12:58:54 EDT
Created attachment 20647 [details]
updated ClassLoaderWeavingAdaptor file
Comment 2 Andrew Clement CLA 2006-05-30 10:00:38 EDT
i believe this change is already in (see constant AOP_XML in ClassLoaderWeavingAdaptor).