Bug 77163 - Load time weaver default verbose setting
Summary: Load time weaver default verbose setting
Status: RESOLVED FIXED
Alias: None
Product: AspectJ
Classification: Tools
Component: Compiler (show other bugs)
Version: 1.2.1   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Matthew Webster CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-10-27 20:09 EDT by david.pearce CLA
Modified: 2012-04-03 15:52 EDT (History)
1 user (show)

See Also:

Attachments
Fix to WeavingAdaptor and new tests (16.62 KB, patch)
2004-11-03 12:58 EST, Matthew Webster CLA 		aclement: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description david.pearce CLA 2004-10-27 20:09:52 EDT 
The default setting for the load time weaver is (as of 1.2.1rc1) to produce
weaving messages rather than ignore them.  I think the default should be changed
to ignore them.  

Also, I cannot override the default setting with -Daj.weaver.versose="False". 
This is the implied syntax from docs/examples/build.xml.  If this has changed
then build.xml should be updated.  Otherwise, the weaver should be fixed to use it!
Comment 1 david.pearce CLA 2004-10-27 20:11:32 EDT 
Correction.  I used -Daj.weaving.verbose="False"
Comment 2 Matthew Webster CLA 2004-11-03 12:58:46 EST 
Created attachment 15623 [details]
Fix to WeavingAdaptor and new tests

1. New message handler which ignores WEAVEINFO messages by default
2. New org.aspectj.weaver.showWeaveInfo system property to show WEAVEINFO
messages
3. Replace all syserr/sysout with info/warning/error messages
4. Improved exception handling
5. New tests
Comment 3 Andrew Clement CLA 2004-11-05 11:50:06 EST 
Patch integrated, waiting for build....
Comment 4 Andrew Clement CLA 2004-11-05 11:51:38 EST 
Fix available:

BUILD COMPLETE -  build.391
Date of build: 11/05/2004 14:19:22
Time to build: 132 minutes 36 seconds
Last changed: 11/05/2004 11:00:43
Last log entry: Matthews fix for Bugzilla Bug 77163: Load time weaver default
verbose setting
Latest good AspectJ jar available at:
download.eclipse.org/technology/ajdt/dev/aspectj-DEVELOPMENT.jar

Also available in 1.2.1 final which will be released in about an hour...