Velocity
  1. Velocity
  2. VELOCITY-164

org.apache.log4j.Category is deprecated

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.3.1
    • Fix Version/s: 1.5
    • Component/s: Engine
    • Labels:
      None
    • Environment:
      Operating System: All
      Platform: All

      Description

      org/apache/velocity/runtime/log/SimpleLog4JLogSystem.java and
      org/apache/velocity/runtime/log/Log4JLogSystem.java the usage
      of Category should should be changed to Logger, since existence
      of Category after the middle of 2003 is undetermined.

      see: http://jakarta.apache.org/log4j/docs/api/org/apache/log4j/Category.html

        Activity

        Hide
        Henning Schmiedehausen added a comment -

        Close all resolved issues for Engine 1.5 release.

        Show
        Henning Schmiedehausen added a comment - Close all resolved issues for Engine 1.5 release.
        Hide
        Daniel Rall added a comment -

        I committed the following to CVS HEAD, which is now the 1.5 branch:

        Replaced Log4J's deprecated Category class with Logger.

        • TODO.txt
          Removed the "Drop use of Log4J Category class" item.
        • build/lib/log4j-1.1.3.jar
        • build/lib/log4j-core-1.1.3.jar
          Removed by upgrade from Log4J 1.1.3 to Log4J 1.2.8.
        • build/lib/log4j-1.2.8.jar
          Added for update from Log4J 1.1.3 to Log4J 1.2.8.
        • src/java/org/apache/velocity/runtime/log/Log4JLogSystem.java
        • src/java/org/apache/velocity/runtime/log/SimpleLog4JLogSystem.java
          Update from the deprecated Log4J Category class to the Logger class.
          Removal of Category from Log4J is pending.

        Sub-classes will have to replace their references with Category
        with Logger, and Category.getInstance() with Logger.getLogger().

        Show
        Daniel Rall added a comment - I committed the following to CVS HEAD, which is now the 1.5 branch: Replaced Log4J's deprecated Category class with Logger. TODO.txt Removed the "Drop use of Log4J Category class" item. build/lib/log4j-1.1.3.jar build/lib/log4j-core-1.1.3.jar Removed by upgrade from Log4J 1.1.3 to Log4J 1.2.8. build/lib/log4j-1.2.8.jar Added for update from Log4J 1.1.3 to Log4J 1.2.8. src/java/org/apache/velocity/runtime/log/Log4JLogSystem.java src/java/org/apache/velocity/runtime/log/SimpleLog4JLogSystem.java Update from the deprecated Log4J Category class to the Logger class. Removal of Category from Log4J is pending. Sub-classes will have to replace their references with Category with Logger, and Category.getInstance() with Logger.getLogger().
        Hide
        Daniel Rall added a comment -

        Geir, this seems like something which should be done before 1.4 final is cut.
        If you disagree, please bump the Target Milestone to 1.5.

        Show
        Daniel Rall added a comment - Geir, this seems like something which should be done before 1.4 final is cut. If you disagree, please bump the Target Milestone to 1.5.
        Hide
        Peter Nimmervoll added a comment -

        Created an attachment (id=6479)
        Patch (part 2)

        Show
        Peter Nimmervoll added a comment - Created an attachment (id=6479) Patch (part 2)
        Hide
        Peter Nimmervoll added a comment -

        Created an attachment (id=6478)
        Patch (part 1)

        Show
        Peter Nimmervoll added a comment - Created an attachment (id=6478) Patch (part 1)

          People

          • Assignee:
            Unassigned
            Reporter:
            Jens Elkner
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development