Fork me on GitHub
#cljdoc
<
2018-07-25
>
dominicm05:07:25

@hagmonk I'm interested in this for pack, happy to lend a hand in my spare time.

parrot 4
kennytilton16:07:45

@martinklepsch Have the instructions for local building involving muuntaja been tested recently? I just tried them with no luck (deets next). Simply sanity checking muuntaja with lein test resulted in

Caused by: java.io.FileNotFoundException: Could not locate jsonista/core__init.class or jsonista/core.clj on classpath.

kennytilton16:07:52

The ingest itself failed on

Error building classpath. Could not find artifact muuntaja:muuntaja:jar:0.6.0-alpha1 in central ()
org.eclipse.aether.resolution.ArtifactResolutionException: Could not find artifact muuntaja:muuntaja:jar:0.6.0-alpha1 in central ()

kennytilton16:07:40

btw, the --git path is supposed to be where I cloned muuntaja, yes?

martinklepsch17:07:10

@hiskennyness instructions were buggy facepalm pushed an updated version

pithyless18:07:24

@martinklepsch the update on ClojureVerse was a good idea :thumbsup: I’m really looking forward to support for Dash docsets

👍 4
kennytilton20:07:42

Exception in thread "main" clojure.lang.ExceptionInfo: Could not analyze muuntaja.core {}

kennytilton20:07:04

Caused by: java.io.FileNotFoundException: Could not locate jsonista/core__init.class or jsonista/core.clj on classpath., compiling:(muuntaja/format/json.clj:1:1)

kennytilton20:07:06

So that is progress, the error is now the same as simply running lein test in the muuntaja directory.

kennytilton20:07:43

Hang on, more wi-fi issues. Might be a factor.

martinklepsch21:07:09

I don‘t know about muuntaja’s tests and would consider it unrelated to cljdoc. Maybe @ikitommi can weigh in some time tomorrow.

martinklepsch21:07:49

I think the cljdoc instructions should work now :)

kennytilton22:07:02

@martinklepsch Sorry if I was unclear. The cljdoc example of a local build of muuntaja fails. I ran lein test as a sanity check. That fails, now with the same error as the doc build. So methinks we just need a more reliable example target? But I agree with your overall conclusion and will return to trying the local build on my own project. I was doing the muuntaja build only because mine had failed on the issue you have now corrected.

kennytilton22:07:26

Aint software — and documentation! — fun? 🙂

kennytilton23:07:04

OK, still no luck. Maybe if I post what I am doing here I will see it myself. 🙂 Here is the maven from my Clojars page:

<dependency>
  <groupId>com.tiltontec</groupId>
  <artifactId>matrix</artifactId>
  <version>0.1.5-SNAPSHOT</version>
</dependency>

kennytilton23:07:28

I have run lein install in the matrix repo.

kennytilton23:07:24

That says

Created /Users/kennethtilton/matrix/cljs/matrix/target/matrix-0.1.5-SNAPSHOT.jar
Wrote /Users/kennethtilton/matrix/cljs/matrix/pom.xml

kennytilton23:07:42

“target” in there looks like exploring.

kennytilton23:07:12

But looking at pom.xml I do not see “target”, so I wager that is in my repo but not in maven.

kennytilton23:07:36

pom.xml:

<?xml version="1.0" encoding="UTF-8"?><project xmlns="" xmlns:xsi="" xsi:schemaLocation=" 
  <modelVersion>4.0.0</modelVersion>
  <groupId>com.tiltontec</groupId>
  <artifactId>matrix</artifactId>
  <packaging>jar</packaging>
  <version>0.1.5-SNAPSHOT</version>
  <name>matrix</name>
  <description>Matrix dataflow library for CLJS</description>
  <url>http://tiltontec.com</url>
....etc

kennytilton23:07:10

This command

./script/cljdoc ingest -p com.tiltontec/matrix \
                       -v 0.1.5-SNAPSHOT \
                       --jar ~/.m2/repository/com.tiltontec/matrix/0.1.5-SNAPSHOT/matrix-0.1.5-SNAPSHOT.jar \
                       --pom ~/.m2/repository/com.tiltontec/matrix/0.1.5-SNAPSHOT/matrix-0.1.5-SNAPSHOT.pom \
                       --git ~/matrix/cljs/matrix \
                       --rev "master"

kennytilton23:07:29

OK, those com.tiltontecs get expressed in maven as com/tiltontec/… testing that.

kennytilton23:07:40

OK, I got it to build! Switching to com/tiltontec gave me a bunch of errors so I dropped the com. prefix altogether and it ran OK, with interesting exception that I am again getting the SCM complaint. I have not pushed the drop of the com to Github so will try that. Progress!

kennytilton23:07:16

The doc might want to emphasize where group, artifact and version go — I thought metosin was a Maven thing! And unless the flexibility is needed, how about having the script take the those as options and generate the rest of the command? Jes thinkin out loud, but for Java/Maven dummies like me it might save you some FAQs.

kennytilton23:07:17

Finally, I see other projects follow the “com.<website>” convention so advice on how I could have gotten it to work would be welcome.