1. Velocity
  2. VELOCITY-392

Switch to a better log system - slf4j


    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: 1.5
    • Fix Version/s: None
    • Component/s: Engine
    • Labels:
    • Environment:
      Operating System: All
      Platform: All


      Since I'm still very unhappy with the current logging workaround in velocity, I
      removed the complete velocity logsystem stuff and replaced it with slf4j (see ; So, the final thing
      is a little bit smaller and should be wrt. logging easier to use (i.e. no need
      for passing runtime svc handles back and forth just to be able to log
      something), much faster, safer and more precise and powerful than the old one...

      In case your are interested, I'll attach the patch made against the saturday
      evening svn repository. It basically contains the log changes as well as some
      code cleaning (i.e. remove/comment out unsed vars/methods, removed unneccessary
      casts as well as nested else clauses, removed throw XExceptions, which are
      actually not thrown). Finally I switched many into log.debug, since the
      appropriate messages are important for developers only and should not irritate
      the enduser (who usually doesn't even understand that freak language ).

      To get the thing running out of the box, I modified the buildfile to include in the jar, so that nlog4j aka log4j NG finds and uses it
      automatically, if one has not explictly configured it. And of course
      build/lib/log4j has been replaced with nlog4j (see


        Issue Links


          Mark Thomas made changes -
          Workflow Default workflow, editable Closed status [ 12551751 ] jira [ 12552582 ]
          Mark Thomas made changes -
          Workflow jira [ 12325267 ] Default workflow, editable Closed status [ 12551751 ]
          Benjamin Bentmann made changes -
          Link This issue relates to VELOCITY-621 [ VELOCITY-621 ]
          Benjamin Bentmann made changes -
          Link This issue is related to VELOCITY-168 [ VELOCITY-168 ]
          Henning Schmiedehausen made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Henning Schmiedehausen made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Assignee Velocity-Dev List [ ] Henning Schmiedehausen [ henning ]
          Resolution Won't Fix [ 2 ]
          Jeff Turner made changes -
          Field Original Value New Value
          issue.field.bugzillaimportkey 35677 12315262
          Jens Elkner created issue -


            • Assignee:
              Henning Schmiedehausen
              Jens Elkner
            • Votes:
              0 Vote for this issue
              1 Start watching this issue


              • Created: