Mailing-List: contact user-help@geronimo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: user@geronimo.apache.org
Received-SPF: pass (athena.apache.org: domain of kevan.miller@gmail.com
 designates 209.85.212.54 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=content-type:mime-version:subject:from:in-reply-to:date
         :content-transfer-encoding:message-id:references:to:x-mailer;
        b=jrwWlYT6bj3r+LoYRrBfp+myaM4w/khckDeNCjDPXstAFgsFnRdSlYbtepAOuYT2Vi
         eMSDW+49I0qlg0IGtIhloxtZt1TKS71osRLZ/btD1BhMjpsJi1ByCwj4w7sem5PYodyd
         pt7Zwp7vYE0CJsy7pfjOxr7su13WUP+FxYmE8=
Content-Type: text/plain; charset=us-ascii
Mime-Version: 1.0 (Apple Message framework v1081)
Subject: Re: XML Log4j configuration
From: Kevan Miller <kevan.miller@gmail.com>
In-Reply-To: <1283257657821-1393987.post@n3.nabble.com>
Date: Tue, 31 Aug 2010 09:37:59 -0400
Content-Transfer-Encoding: quoted-printable
Message-Id: <63047D5E-9500-45C5-80E1-8C5B4818DB82@gmail.com>
References: <1283168825592-1388271.post@n3.nabble.com>
 <AANLkTik3sB0ZAHCsd2SyFp6auY3_7rRin_bzN2BGL7=L@mail.gmail.com>
 <1283190705611-1389968.post@n3.nabble.com>
 <A4B2EB6F-0FA4-4356-BAC9-CDBD5340BBE0@gmail.com>
 <1283257657821-1393987.post@n3.nabble.com>
To: user@geronimo.apache.org


On Aug 31, 2010, at 8:27 AM, Mik wrote:

>=20
> Hi Kevan
>=20
> Thanks for your reply. Looking through the code in Log4jService.java =
made
> the difference..
>=20
> // Allow users to override the configurationFile which is hardcoded
> // in config.ser and cannot be updated by config.xml, as the
> // AttrbiuteManager comes up after this GBean
> String cfgFile =3D System
>        .getProperty(LOG4JSERVICE_CONFIG_PROPERTY);
> if ((cfgFile !=3D null) && (!cfgFile.equals(""))) {
>    this .configurationFile =3D cfgFile;
> }
>=20
> So the config file value can be overridden at server startup by using
> org.apache.geronimo.log4jservice.configuration system property, and it =
also
> seems to accept the xml format :-)

Hi Mik,
Great. Good work digging through the source!. Forgot that this GBean =
attribute couldn't be overridden... Normally, the config.xml override =
would work. As indicated by above comment, this is an edge case due to =
order of server startup...

--kevan