taoensso / sente

Realtime web comms library for Clojure/Script
https://www.taoensso.com/sente
Eclipse Public License 1.0
1.73k stars 193 forks source link

Assertion failed on `receive-buffered-evs!` with ClojureScript 0.0-2261 #53

Closed giorgio-v closed 9 years ago

giorgio-v commented 10 years ago

Description

Running the example project and clicking the chsk-send! (with reply) button throws a validation error on taoensso.sente/receive-buffered-evs!.

Steps to reproduce

NB I know the above steps could sound insulting to the project maintainer but they’re meant for everybody else. :)

Notes

I’ve tried to dig further into the problem but I’m unable to use the checkouts feature of Leinengen with the example project, so the whole build cycle would be too long for me. I never used this feature before, so maybe I’m doing it wrong.

ptaoussanis commented 10 years ago

Hi Giorgio, thanks for the report!

NB I know the above steps could sound insulting to the project maintainer but they’re meant for everybody else. :)

No worries, they're helpful for the project maintainer too ;-)

button throws a validation error on taoensso.sente/receive-buffered-evs!.

Could you include the validation error you're seeing?

Update the dependencies in example-project/project.clj for clojurescript to version 0.0-2261 and for cljsbuild to 1.0.3.

Just to clarify: are you seeing this only with the updated dependencies (i.e. it doesn't happen with the default deps)?

Cheers :-)

giorgio-v commented 10 years ago

Here it is:

[Error] Error: Assert failed: (vector? clj)
    receive_buffered_evs_BANG_ (main.js, line 41620)
    (funzione anonima) (main.js, line 42028)

schermata 2014-07-04 alle 07 41 43

Also, note the timeout message

Regarding the dependencies: yes, it seems the minimum update to trigger the bug.

ptaoussanis commented 10 years ago

Okay, great! Will try look into this later today or tomorrow, just working through a bit of a priority backlog atm.

ptaoussanis commented 10 years ago

Hi Giorgio, finally had a chance to look at this (sorry for the delay!).

I'm afraid I can't reproduce what you're seeing, at least with ClojureScript 0.0-2268 (a little newer than 0.0-2261). So may have been an issue with 0.0-2261 that's since been corrected, or it may be that your build artefacts have gotten into an inconsistent state and need a lein clean call.

Could you look at [com.taoensso.examples/sente "0.15.0-SNAPSHOT"] (which has the newest dependencies) and let me know if you're still having problems after a lein clean call? Thanks a lot, cheers! :-)

giorgio-v commented 9 years ago

Seems fixed. :)

Thank you!