MichaelDrogalis / dire

Erlang-style supervisor error handling for Clojure
483 stars 19 forks source link

This line is bogus #36

Open hiredman opened 8 years ago

hiredman commented 8 years ago

I was helping someone debug an issue in the clojure irc chat and happened to take a look at dire.

https://github.com/MichaelDrogalis/dire/commit/ccb90f244bb1b4d98400b5a44d264f0124fbb983

This commit(and the line) is problematic.

# doesn't give you a gensym outside of syntax quote

def is always globally scoped, so hook-supervisor-to-fn is repeatedly setting the value of the global var dire.core/supervisor# to (partial supervised-meta (meta task-var))