…g log4j twice Without these changes, IsisSystem is setting up log4j which is too late since other classes which bootstrap Isis require logging to work. Currently, HeadlessWithBootstrappingAbstract configures log4j in addition to IsisSystem. IsisSystem sets up logging either through logging.properties or a fallback. But logging.properties does not always exist (integration tests). Without logging.properties the fallback adds a console appender to the root logger in addition to any configuration in logging-integtest.properties, resulting in duplicated logs in integration tests.
This commit removes any logging setup from IsisSystem. HeadlessWithBootstrappingAbstract now uses the IsisLoggingConfigurer which is normally used to set up logging. This requires IsisLoggingConfigurer to be flexible as to the basename of the configuration file. Also, some code duplication is refactored into LeveledLogger. [ Full content available at: https://github.com/apache/isis/pull/122 ] This message was relayed via gitbox.apache.org for [email protected]
