Re: [aspectwerkz-dev] : new version

Quote [Jonas Bonér]:
> Hi.
> 
> thanks a lot for your contributions, I like it. committed to head.

10x

> I was just wondering if it would be more intuitive to use 'classpath'
> instead of 'cls' or 'clsdir' and to use 'srcdir' and not 'src'.

+1 for classpath.
As for src vs srcdir I have used the notations according to Ant provided tasks (see javac). I think 
for the moment we can keep the attribute name srcdir and inner element src.

> 
> Ps. We are in high need of an ant task for the offline compiler
> (AspectWerkzC) as well, care to help out with one? We have a maven
> goal, but it might be outdated by now. Thanks a lot. Ds.

It will be my pleasure to take care of it. As for maven goals I must tell you that I don't know too 
much about it, but I can start learning.

./the_mindstorm

PS: I have tried many times to give maven a chance but till now I didn't like it. maybe now it's the 
moment :-).

PPS: what does it mean Ds?

> 
> 
> 
> 
> 
> On Thu, 07 Oct 2004 05:46:28 +0300, the_mindstorm
> <[email protected]> wrote:
>> Hi!
>> 
>> Attached are the latest versions for the AnnotationCTask (and related).
>> 
>> Modifications:
>> 1. the verbose parameter was not transmitted to AnnotationC: FIXED
>> 2. the execute method never threw BuildException (a stupid printStackTrace()): FIXED
>> 3. AnnotationC.printUsage printed old information: FIXED
>> 
>> Features:
>> 1. The sources can be provided in 4 ways:
>>         - srcdir attribute (1 directory)
>>         - inner element (<src path>): multiple directories
>>         - srcfileset: fileset of sources (the corresponding in AnnotationC main is comma separated list)
>>         - srcincludes: a file containing the source names (one per line)
>> 
>> 2. The classes dir can be provided in 2 ways:
>>         - clsdir attribute: 1 directory
>>         - inner element (<cls path>): multiple directories
>> This feature required a little change in AttributeEnhancer (respectively AsmAttributeEnhancer). The
>> method initialize(String, String) was changed to initialize(String, URL[]).
>> 
>> Fixes:
>> 1. Fixed the build.xml to produce the same result as the initial test:annotationc
>> 
>> Unfortunately I had no time to write down the doc for the task. I like xdocs (even if I dislike
>> maven ;-) ). I will come back later with the doc and a complete set of examples of task usage.
>> 
>> ./the_mindstorm
>> 
>> 
>>
>