My favorites | Sign in
Project Home Downloads Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 711: MyBatis Generator Maven Plugin should log to Maven log
2 people starred this issue and may be notified of changes. Back to list
Status:  New
Owner:  ----


Sign in to add a comment
 
Reported by paulkrau...@alum.mit.edu, Nov 1, 2012
What version of the MyBatis are you using?
mybatis-generator-maven-plugin:1.3.2

Please describe the problem.  Unit tests are best!
When run from Maven, DatabaseIntrospector initializes its log, causing a warning message to printed to the console.

What is the expected output? What do you see instead?
Log messages should be directed to Maven.  Instead they try to go to log4j.

Can you provide stack trace, logs, error messages that are displayed?
log4j:WARN No appenders could be found for logger (org.mybatis.generator.internal.db.DatabaseIntrospector).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.


Please provide any additional information below.
Proposed fix attached.  This patch calls LogFactory.setLogFactory from the Mojo, redirecting MBG log messages to the Maven log.
mybatis-generator-maven-log.patch
3.1 KB   View   Download
Nov 1, 2012
#1 paulkrau...@alum.mit.edu
For backwards compatibility with JDK 1.5, the @Overrides annotations in the MavenLogFactory and MavenLogImpl classes should be removed.  
Sorry about that.
Feb 13, 2013
#2 paulkrau...@alum.mit.edu
I'm not sure why this issue hasn't merited a response.  If the patch is wrong-headed, please say why.  Perhaps it was just too difficult to apply.  I'm attaching a new one, without @Override annotations, and rooted at the project level.
mybatis-generator-maven-log.patch
3.0 KB   View   Download
Feb 13, 2013
Project Member #3 jeffgbut...@gmail.com
I'll get to it.  Just been crazy busy.  Also, we're moving to github so I won't apply this until the move is complete.

It's open source man - we're all volunteers!
Sign in to add a comment

Powered by Google Project Hosting