Question

I want to to do like this:

<appender name="ErrorLog" class="org.apache.log4j.FileAppender">
        <param name="File" value="${error.log.path}"/>
        <param name="Append" value="true" />
        <param name="Threshold" value="ERROR"/>
        <layout class="org.apache.log4j.PatternLayout">
            <param name="ConversionPattern" value="%C{1} %L [%t] %d{dd MMM,yyyy HH:mm:ss.SSS} %-5p - %m%n" />
        </layout>
    </appender>

Notice this line: <param name="File" value="${error.log.path}"/>

I tried to set the values like this:

public static void main(String[] args) {
     System.setProperty("error.log.path", "/test/crm/log/error.log");
     ApplicationContext context = new ClassPathXmlApplicationContext("blah.xml");
     ..........
     .......... 
  }

But I don’t see any effect.

Is log4j gets configured before calling the main method?

Is there any other way to do this?

Answer 1

Look at this thread

It looks like you did everything right. I don’t think there is any difference between setting the property inside your main class with System.setProperty() and specifying it via the command line as long as it happens befor actual log4j initialization.

I think your issue is that your logging framework gets loaded before you specify the property.
I can say that the logging framework (log4j) will get configured when you call the configurator. Stuff like BasicConfigurator.configure() (in your case its xml configurator).

Otherwise the first attempt to use the logging will cause message like “log4j is not configured properly”.

The real question is whether your code snippet with ‘main’ is not oversimplified.

With this in mind, another question that I have to ask – whether you’re running inside some container or you’re running a real vanilla method main and configure everything by yourself? I’m asking because if you’re running in container, the chances are that container will by itself somehow configure its logging, for example JBoss will do so. In this case more investigation is required.

Hope this helps