search

multiformats / clj-multihash

Clojure implementation of the Multihash spec
The Unlicense
40 stars 15 forks source link

tools.analyzer.jvm trips over extension to byte array #18

Closed borkdude closed 1 year ago

borkdude commented 1 year ago

Hi @greglook !

Clerk cannot analyze this library using tools.analyzer.jvm since tools.analyzer trips over this expression:

https://github.com/multiformats/clj-multihash/blob/e3696c5341bcfb315589fbce020e89febbfe73bd/src/multihash/core.cljc#L248

Repro:

$ clj -Sdeps '{:deps {org.clojure/tools.analyzer.jvm {:mvn/version "RELEASE"} mvxcvi/alphabase {:mvn/version "RELEASE"}  mvxcvi/multihash {:mvn/version "RELEASE"}}}'
Downloading: mvxcvi/alphabase/maven-metadata.xml from clojars
Clojure 1.11.0
user=> (require '[clojure.tools.analyzer.jvm :as ana :as j])
nil
user=> (do (j/analyze-ns 'multihash.core) nil)
Execution error (ExceptionInfo) at clojure.tools.analyzer.passes.jvm.validate/validate-tag (validate.clj:235).
Class not found: (class (byte-array 0))

Are you open to a PR which fixes this problem?

BigLep commented 1 year ago

2023-04-25 maintainer conversation: we're using this issue to track archiving this repo.

rvagg commented 1 year ago

closing due to inactivity, archiving repo