Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug: java.lang.ClassCastException: etaoin.api$get_logs cannot be cast to clojure.lang.MultiFn #212

Closed
borkdude opened this issue Jul 26, 2019 · 6 comments
Assignees

Comments

@borkdude
Copy link
Contributor

The interesting part of the stacktrace:

[08:05:53]    Uncaught exception, not in assertion.
[08:05:53]    expected: nil
[08:05:53]    actual: java.lang.ClassCastException: etaoin.api$get_logs cannot be cast to clojure.lang.MultiFn
[08:05:53]    at clojure.core$get_method.invokeStatic (core.clj:1817)
[08:05:53]    clojure.core$get_method.invoke (core.clj:1817)
[08:05:53]    etaoin.api$implemented_QMARK_.invokeStatic (api.clj:71)
[08:05:53]    etaoin.api$implemented_QMARK_.invoke (api.clj:69)
[08:05:53]    etaoin.api$supports_logs_QMARK_.invokeStatic (api.clj:1498)
[08:05:53]    etaoin.api$supports_logs_QMARK_.invoke (api.clj:1495)
[08:05:53]    etaoin.api$postmortem_handler.invokeStatic (api.clj:2522)
[08:05:53]    etaoin.api$postmortem_handler.invoke (api.clj:2491)

It seems get-logs is no longer a multimethod.

@CamdenClark
Copy link

Any chance we can get a release with this fix?

luontola added a commit to CSCfi/rems that referenced this issue Aug 29, 2019
Fixes builds failing to fail:
java.lang.ClassCastException: class etaoin.api$get_logs cannot be cast to class clojure.lang.MultiFn
@luontola
Copy link
Contributor

An unofficial build with @borkdude's fix is available at [org.clojars.luontola/etaoin "0.3.5-patch1"]

@CamdenClark
Copy link

An unofficial build with @borkdude's fix is available at [org.clojars.luontola/etaoin "0.3.5-patch1"]

@luontola If you're ever in seattle, I owe you a drink. Thanks so much!

igrishaev pushed a commit that referenced this issue Oct 31, 2019
* [#212] fix supports-logs?
* fix misplaced docstrings (found with clj-kondo)
@igrishaev
Copy link
Collaborator

Sorry for the delay, Alex and I just got back to the library. Yes, the get-logs if now a function which relies on the internal get-logs* multimethod. Is anyone still facing troubles with that? If so, could you provide some code/examples, please?

@borkdude
Copy link
Contributor Author

I believe this was fixed by this PR: #213

@igrishaev
Copy link
Collaborator

Right, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants