Thanks Ceki, good advice! 

-----Original Message-----
From: Ceki G�lc� [mailto:[EMAIL PROTECTED] 
Sent: Friday, 16 January 2004 11:57
To: Log4J Developers List
Subject: Re: log4j jmx extension: we would like to participate


I would suggest that to work against the latest code from CVS not 1.2.8. Your best 
chances to have the code integrated to a release is to supply small, humanely 
manageable patches. Large patches require a leap of faith which we are not always 
inclined to take.

If you change the indentation of the code, then you should first supply a patch with 
indentation changes alone (no changes to the logic).  When you later enhance the code 
and supply a patch against the indented code, the changes in logic come out clearly. 
It is much more difficult to spot the changes in the logic when they are intermixed 
with indentation.

At 09:50 AM 1/16/2004 +0100, Peter Widmer wrote:
>Hello
>
>What we have done so far is to patch the jmx extension of log4j 1.2.8 
>with the patch of Bill Rossi, as described by Ceki. This patch provides 
>compatibility with jmx 1.2, primarily by adapting the existing sources 
>according to the naming conventions of jmx 1.2.
>
>As soon as we have made some improvements, we will mail them.
>
>'Peter
>
>Shapira, Yoav wrote:
>>Howdy,
>>Can you email your latest version to the list?  Thanks, Yoav Shapira 
>>Millennium ChemInformatics
>>
>>
>>>-----Original Message-----
>>>From: Ana-Maria Oleski [mailto:[EMAIL PROTECTED]
>>>Sent: Thursday, January 15, 2004 9:58 AM
>>>To: Log4J Developers List
>>>Subject: RE: log4j jmx extension: we would like to participate
>>>
>>>
>>>Hi,
>>>
>>>The problem I had with it was that it didn't run at all with jmxri 2.0.
>>>I've fixed that
>>>and added a little functionality. We've been running this modified
>>version
>>
>>>productively
>>>for the last three months and I can't begin to tell you how 
>>>incredibly practical it's been.
>>>I've mailed the changes to the list some time ago  but I've improved 
>>>a couple of things in the mean time so if you're interested, I'll 
>>>mail them again. Or
>>upload
>>
>>>them to
>>>bugzilla, I think there is a already bug open for this.
>>>
>>>
>>>Ana Oleski
>>>
>>>---------------------------------------------------------------------
>>>--
>>----
>>
>>>----------------
>>>***Protect your PC from local E-Mail Application security holes*** 
>>>***Maintain your Privacy - MS Passport Free*** ***Anti SPAM 
>>>"Whitelist" feature***
>>>
>>>http://www.x-mail.net Web E-Mail, accessible anywhere, 128 bit SSL
>>Secure
>>
>>>Voice Messages, Voice Calls (VoIP), Video Conferencing, Live Chat, 
>>>X-Mail Messenger, Personal Web Hosting, Private Disk Storage, 
>>>Calendar, Bookmarks, Forwarding, Virtual Mail Map Aliasing
>>>
>>>X-Mail Premium: 20 - 250MB Storage, 20MB Messages, SMTP, POP3, Ad 
>>>Free Starting at $9.95 per year
>>>---------------------------------------------------------------------
>>>--
>>----
>>
>>>----------------
>>>Anonymous Web Surfing http://www.snoopblocker.com Search 
>>>http://www.teradex.com
>>>
>>>
>>>
>>>
>>>
>>>---------------------------------------------------------------------
>>>To unsubscribe, e-mail: [EMAIL PROTECTED]
>>>For additional commands, e-mail: [EMAIL PROTECTED]
>>
>>
>>This e-mail, including any attachments, is a confidential business 
>>communication, and may contain information that is confidential, 
>>proprietary and/or privileged.  This e-mail is intended only for the
>>individual(s) to whom it is addressed, and may not be saved, copied, 
>>printed, disclosed or used by anyone else.  If you are not the(an) 
>>intended recipient, please immediately delete this e-mail from your 
>>computer system and notify the sender.  Thank you.
>>
>>---------------------------------------------------------------------
>>To unsubscribe, e-mail: [EMAIL PROTECTED]
>>For additional commands, e-mail: [EMAIL PROTECTED]
>
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: [EMAIL PROTECTED]
>For additional commands, e-mail: [EMAIL PROTECTED]

--
Ceki G�lc�

      For log4j documentation consider "The complete log4j manual"
      ISBN: 2970036908 http://www.qos.ch/shop/products/clm_t.jsp  



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



NOTE: This e-mail message is subject to the MTN Group disclaimer see 
http://www.mtn.co.za/email_disclaimer.asp

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to