Skip to content

Commit

Permalink
WINDUP-73 Logging, JUL - Logging.init()
Browse files Browse the repository at this point in the history
  • Loading branch information
OndraZizka committed Jul 24, 2014
1 parent 85bf92e commit 9231f73
Show file tree
Hide file tree
Showing 21 changed files with 270 additions and 660 deletions.
Expand Up @@ -15,6 +15,7 @@
import org.jboss.forge.furnace.addons.Addon;
import org.jboss.windup.config.metadata.RuleMetadata;
import org.jboss.windup.graph.GraphContext;
import org.jboss.windup.log.jul.config.Logging;
import org.ocpsoft.rewrite.config.ConfigurationProvider;
import org.ocpsoft.rewrite.context.Context;

Expand Down Expand Up @@ -98,4 +99,8 @@ protected final List<String> generateDependencies(
{
return Arrays.asList(deps);
}

static {
Logging.init();
}
}

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

Expand Up @@ -15,6 +15,7 @@
import com.tinkerpop.frames.VertexFrame;
import com.tinkerpop.frames.modules.AbstractModule;
import com.tinkerpop.frames.modules.Module;
import org.jboss.windup.log.jul.config.Logging;

@Singleton
public class GraphTypeRegistry
Expand Down Expand Up @@ -80,4 +81,8 @@ public void doConfigure(FramedGraphConfiguration config)
}
};
}

static {
Logging.init();
}
}
9 changes: 9 additions & 0 deletions logging/src/main/java/README.txt
@@ -0,0 +1,9 @@

Logging module will be probably removed:

(17:41:04) ozizka-FN: lincolnthree: Should we really have a logging module, or can it be in the Exec Impl?
(17:41:13) LincolnBaxter: ozizka: logging module probably not required
(17:41:46) ozizka-FN: Because, logging will be everywhere, so, putting it in exec seems to be okay
(17:41:55) ozizka-FN: even with potential JBoss logging dependency
(17:43:03) LincolnBaxter: yeah. that's definitely true

0 comments on commit 9231f73

Please sign in to comment.