Uploaded image for project: 'Magnolia'
  1. Magnolia
  2. MAGNOLIA-3396

log4j.xml in magnolia-core

    XMLWordPrintable

Details

    • Bug
    • Resolution: Won't Do
    • Minor
    • None
    • 4.3.8
    • core
    • None
    • Java6, Tomcat6, slf4j, log4j

    Description

      magnolia-core includes a log4j.xml configuration file in its root directory.

      If one wants to use other projects which itself use slf4j/log4j or log4j on their own and one wants to use a log4j.properties file instead of the XML based configuration, one needs to (well, first figure out, and then) add a "-Dlog4j.configuration=log4j.properties" system property to java options, since log4j will first search for XML based configuration files on the classpath and then for the .properties files.

      I can't see a need for magnolia-core including a log4j.xml in its root directory, expecially because magnolia has its own Log4J configurer, which only initializes log4j if a log4j configuration property has been set in magnolia.properties. In turn, not specifying it probably means, that the developer had other intentions anyway.

      Thanks
      Philipp

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              pbaerfuss Philipp Bärfuss
              philipp.gaschuetz Philipp Gaschütz
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Checklists

                  Bug DoR
                  Task DoD