Clojure-Contrib

Backport clojure.contrib.logging to clojure-1.0-compat branch

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None

Description

This simply backports the clojure.contrib.logging namespace to the 1.0-compatible branch.

The only change from master was to revert back to the old style of import calls.

Activity

Hide
Assembla Importer added a comment -

technomancy said: [file:dIpoK8Mm4r3Rt6eJe5aVNr]: logging backport patch

Show
Assembla Importer added a comment - technomancy said: [file:dIpoK8Mm4r3Rt6eJe5aVNr]: logging backport patch
Hide
Assembla Importer added a comment -

ataggart said: Out of curiosity, why was such an old version of c.c.logging used? I've added a number of performance improvements since then:
http://github.com/richhickey/clojure-contrib/blob/master/src/clojure/contrib/logging.clj

Show
Assembla Importer added a comment - ataggart said: Out of curiosity, why was such an old version of c.c.logging used? I've added a number of performance improvements since then: http://github.com/richhickey/clojure-contrib/blob/master/src/clojure/contrib/logging.clj
Hide
Assembla Importer added a comment -

technomancy said: Alex: just because I did the backport a few weeks ago and procrastinated on submitting it. =)

I'll create a new patch for the latest logging.clj.

Show
Assembla Importer added a comment - technomancy said: Alex: just because I did the backport a few weeks ago and procrastinated on submitting it. =) I'll create a new patch for the latest logging.clj.
Hide
Assembla Importer added a comment -

technomancy said: [file:dd6JqgTpur3RF9eJe5afGb]: Updated patch

Show
Assembla Importer added a comment - technomancy said: [file:dd6JqgTpur3RF9eJe5afGb]: Updated patch
Hide
Assembla Importer added a comment -

tomfaulhaber said: Applied patch (commit bd3c20). Seems to work.

Show
Assembla Importer added a comment - tomfaulhaber said: Applied patch (commit bd3c20). Seems to work.

People

  • Assignee:
    Unassigned
    Reporter:
    Anonymous
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: