I do think that's probably an improvement from the end user perspective
since the munging isn't necessarily obvious. I'd like to hear from @ibdknox
to see if there's something in the API I'm missing to handle this use case,
or if it's just encouraged to do these things in the phase when that
information is currently available. If the latter, that's going to need to
be carefully explained in the docs, since it was a bit of stumbling block
in figuring out the issue (and actually was affecting some core code I
found).


On Tue, Jan 21, 2014 at 11:36 AM, Jeff Taggart <notifications@github.com>wrote:

> In src/lt/objs/plugins.cljs:
>
> > @@ -28,6 +28,14 @@
> >  (def plugins-url "http://plugins.lighttable.com")
> >  (def ^:dynamic *plugin-dir* nil)
> >
> > +(defn plugin-dir [name]
> > +  (let [lt-plugin (files/join plugins-dir name)
> > +        user-plugin (files/join plugins-dir name)
>
> Is this function not a better mechanism than the *plugin-dir* global var
> (after the fixes you mentioned above)?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/LightTable/LightTable/pull/1114/files#r9050770>
> .
>



-- 
Screw the environment. Please print this email immediately. And then burn
it.