Bug 93560

Summary: ASPECTJ5_DEVELOPMENT branch LTW can't find aop.xml files
Product: [Tools] AspectJ Reporter: Ron Bodkin <rbodkin+LISTS>
Component: CompilerAssignee: Adrian Colyer <adrian.colyer>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: unspecified   
Target Milestone: 1.5.0   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
updated ClassLoaderWeavingAdaptor file none

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).