Changing macro requires restarting process and triggering reload about doo HOT 15 CLOSED

We got green light to fix this at the compiler level! Will update here any progress.

from doo.

I think this is related to #52 and how the compiler finds/watches macro files. Since this is a little more complex, can you provide an example repo?

from doo.

I solved #52 by supplying a top-level :source-paths in my project.clj, but doo still doesn't pick up changes to .clj files. Here's an example project demonstrating the problem.

from doo.

Thanks @exupero , I'll look at it shortly.

from doo.

I was able to reproduce the behavior without using doo here. I'll see if we can fix this at the compiler level.

Reading this line it is clear why it doesn't work.

from doo.

Hey @exupero

Macro reloading is not supported by the compiler's api. I think it is an important feature and doo should support it. Since we heavily rely on the compiler's api we will try to add it to the compiler. If that effort fails, we will refactor doo accordingly. In any case, it will take time :(

Sorry for the delay, I'll keep you posted.

from doo.

Thanks for your work on this. I'm in no urgent need of a fix and can work around the problem until the right solution is found.

from doo.

Cool, we'll be tracking progress on the compiler front in CLJS-1490.

from doo.

Note that I encounter this issue with .cljc too,
will this fix the issue for both .clj and .cljc?

(thanks for this very handy tool btw)

from doo.

Hi @lsenta, it seems that if I manage to solve it at the compiler level it should fix the problem for both cljc and clj files.

from doo.

Awesome that you're working on this; Status updates?

I'm working on a bunch of cljs macros, and it's sooo painful...

from doo.

Hi @metasoarous

We need to go to cljs.build.api/watch and call something like this function when clj and cljc files need to be reloaded. I'll eventually get to it, but I don't know when since I've been super busy lately.

from doo.

OK; Sounds good. Looking forward to it :-)

from doo.

I'm not experiencing this issue (default config, no magic). I can touch a testing macro and suite will reload.

Issue might have become outdated?

from doo.

CLJS-1490 has been fixed. Therefore I believe this problem can be solved by upgrading to ClojureScript 1.9.198 or later. I'll close this issue now.

from doo.