Conor, Magesh's code did allow for user customization - those in the properties file were just the initial defaults and could be overridden with Ant properties. My MailLogger uses Ant properties for configuration also, as well as a properties file, although there are no defaults for that one.
Erik ----- Original Message ----- From: "Conor MacNeill" <[EMAIL PROTECTED]> To: "Ant Developers List" <[EMAIL PROTECTED]> Sent: Wednesday, January 30, 2002 6:10 PM Subject: RE: [SUBMIT] AnsiColorLogger > Magesh, > > Looks good. My only comment would be the name of the properties file. Should > it be more closely identified with the logger in question, say > ansilogger.properties, or should config info for all loggers go in > defaults.properties? > > In fact, there is a more general issue, regarding how loggers can be > configured, since I guess users may want to change the colours without > having to rebuild the ant jar. > > Conor > > > > -----Original Message----- > > From: Magesh Umasankar [mailto:[EMAIL PROTECTED] > > Sent: Thursday, 31 January 2002 6:39 AM > > To: Ant Developers List > > Subject: [SUBMIT] AnsiColorLogger > > > > > > Hi, > > > > AnsiColorLogger uses ANSI color code sequences > > to display output to the console in color. > > Different colors are used to classify messages > > based on message level. > > > > This submission is to provide you an opportunity > > to take a look at the implementation before I > > commit it (planning to do it same time tomorrow). > > Please send your comments, if any, my way. > > > > NOTE: > > The defaults.properties file must go into the > > org/apache/tools/ant/listener directory. > > > > Cheers, > > Magesh > > > > ************************************************* > > * Politician: One who shakes your hand before * > > * elections and your confidence after. * > > ************************************************* > > > > > -- > To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> > For additional commands, e-mail: <mailto:[EMAIL PROTECTED]> > > -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>