Fork me on GitHub
#cljs-dev
<
2019-11-14
>
mkvlr08:11:55

my compile error just popped up again, three files are affected, all missing just the namespace in one function def:

mkvlr09:11:22

it’s just missing the namespace, so somewhere swallowing com.nextjournal.journal.views.

thheller10:11:23

@mkvlr that looks very much like a threading issue. like 2 threads writing the same file. do you use :parallel-build at all?

thheller10:11:26

return cljs.core.reset_BANG_.call(null,notifications_popover_ifications_form,current_setting,map__134483,map__134483__$1,runner_log))

thheller10:11:57

one extra ) at the end. but also name overlap before, as if the output slightly shifted because some other thread got a few bytes in before

thheller10:11:48

then suddenly some other bits appear that don't belong there

thheller10:11:26

I would suspect threads. either just actually writing to the same file or just messing with *out* somehow (unlikely given its a thread-local but who knows)

thheller10:11:24

the others look like just the ns is missing but I suspect there might be more errors in the files. it just stops parsing at the first one. did you try manually adding the ns and see how far it gets?

thheller10:11:00

also note

return (new com.nextjournal.journal.views.billing.t_com$nextjournal$journal$views$billing135573(null));
})()
,new cljs.core.Keyword(null,"initial-data","initial-data",-1315709804),null,new cljs.core.Keyword(null,"options","options",99638489),new cljs.core.PersistentArrayMap(null, 1, [new cljs.core.Keyword(null,"watch-atom","watch-atom",-2134031308),false], null)], null));
})], null));

//# sourceMappingURL=billing.js.map?rel=1573720735583
65");
});

/**
 * Positional factory function for com.nextjournal.journal.views.billing/t_com$nextjournal$journal$views$billing135465.
 */
com.nextjournal.journal.views.billing.__GT_t_com$nextjournal$journal$views$billing135465 = (function com$nextjournal$journal$views$billing$__GT_t_com$nextjournal$journal$views$billing135465(meta135466){
return (new com.nextjournal.journal.views.billing.t_com$nextjournal$journal$views$billing135465(meta135466));
});

}

return (new com.nextjournal.journal.views.billing.t_com$nextjournal$journal$views$billing135465(null));
})()
,new cljs.core.Keyword(null,"initial-data","initial-data",-1315709804),null,new cljs.core.Keyword(null,"options","options",99638489),new cljs.core.PersistentArrayMap(null, 1, [new cljs.core.Keyword(null,"watch-atom","watch-atom",-2134031308),false], null)], null));
})], null));

//# sourceMappingURL=billing.js.map?rel=1573720735531

thheller10:11:13

definitely overlap in the billing.js. the same code repeating in the file pretty much makes it clear its a thread issue?

mkvlr11:11:32

@thheller yes, we’re using :parallel-build. Let me try to produce the error again and see if there’s more errors… I didn’t save the build folder 🙈

thheller13:11:26

@mkvlr note that :parallel-build is unlikely to create issues in the same file. that should still only compile once. so more likely that it is something related to your build setup in general. running multiple watches or so.

Roman Liutikov15:11:29

Tested master on Pitch code base, seems to work fine.

dnolen15:11:42

@mkvlr you're not trying to use the REPL at the same time as the build is running are you?

mkvlr15:11:01

@dnolen figwheel builds on boot and we do connect emacs automatically on boot. Should we delay that until the first compile is done?

mkvlr15:11:56

will check in what order things are happening exactly…

dnolen16:11:48

well if the REPL itself triggers some compilation this won't be coordinated with the first build pretty sure

dnolen20:11:32

fixed the node lib load-file issues at the REPL

dnolen20:11:38

applying 3077

dnolen20:11:43

anything else we should consider?

dnolen20:11:54

(for a release)

dnolen21:11:07

well that's for core.async 🙂 there's no need to coordinate that

👍 4
dnolen21:11:26

but yes, I think that looks ok will get that in my queue soon

❤️ 4
thheller21:11:52

maybe https://clojure.atlassian.net/browse/CLJS-3003 dunno about the state of the patch though

lread23:11:25

sooo... I created a few patches what seems like a long time ago... I am foggy now and forget how to tell which patches are being considered for release.

lread17:11:23

@U04VDQDDY is there a jira search for this?