tools.logging

tools.logging ns form :require and :use subforms unrecognized by tools.namespace

Details

  • Type: Enhancement Enhancement
  • Status: Closed Closed
  • Priority: Trivial Trivial
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None

Description

Cross-reference: TNS-21

I am not sure whether this is something that should be changed in tools.logging or tools.namespace. To be sure, at most one of them should change, if either. Stuart Sierra has mentioned in TNS-21 that the ns doc string shows :require :use etc. subforms always in parentheses, and not vectors. It might be best to change tools.logging's ns form.

tools.namespace expects (:require ...) and (:use ...) subforms of the ns form to be in lists, not vectors, otherwise it ignores them.

tools.logging has ns forms that use vectors like [:require ...] and [:use ...], and are thus ignored by tools.namespace.

The Clojure compiler seems to handle these dependencies just fine, but this may be just another case where it accepts more things than it is documented to accept.

Activity

Hide
Andy Fingerhut added a comment -

Patch tlog-13-1.patch dated Aug 29 2014 simply replaces [] with () around subforms of ns forms.

Show
Andy Fingerhut added a comment - Patch tlog-13-1.patch dated Aug 29 2014 simply replaces [] with () around subforms of ns forms.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: