👋 are there any known gotchyas with using gitlibs via t.d.a in CircleCI? I just switched a lib from Maven coords to git coords; it worked fine locally for me but over in CircleCI the deps were downloaded but then the files couldn’t be found on the classpath during the test run.

@aviflax hard to say without seeing what bin/tests-with-coverage is doing

in general if you have private ssh repos, you'll need to use an ssh-agent

clojure -J-Xmx2g -A:test:test/coverage

It’s not a private repo

I was able to reproduce the problem just now locally via Docker so I should 😅 be able to figure it out

It looks like the problem is that when the classpath is generated, there’s an extra segment main at the end; that dir doesn’t exist in the repo hosting this particular library

Here’s the cp entry: /home/circleci/.gitlibs/libs/clj-chrome-devtools/clj-chrome-devtools/58a396f54f469e98b5aae92b9357d65434914228/src/main

Anyone have any suggestions on why that might be the case? I’ve been looking through https://clojure.org/reference/deps_and_cli and so far haven’t found any clues

Seems like that lib has an invalid paths inside its deps.edn?

ah, maybe, I’ll take a look

Nope

https://github.com/tatut/clj-chrome-devtools/blob/master/deps.edn

It needs to specify paths explicitly

Ohh wait I remember this

:paths ["src"]. I think there is a bug when it isn't set

the dep is inheriting the source path that was specified in my project root

Yup

I mean, my project’s deps.edn

IIRC this is a known bug

I’ll see if I can find the JIRA issue

Make sure you vote!

will do, if/when I find it

so far no dice

wasn't it fixed?

aha! yes, it was fixed: https://clojure.atlassian.net/browse/TDEPS-52

It must be that the Docker image I’m using hasn’t picked up the fix yet

I’m using circleci/clojure:openjdk-11-tools-deps-browsers

@ghadi you reported that bug!

It's the circle of life

😂

Thanks for the help!

Alex commented on the issue: > Released in clj 1.10.1.458 And I’ve confirmed that the CircleCI image I’m using has 1.10.0.442 installed. Interestingly, even the official Docker images for Clojure don’t appear to have any variant with a version of Clojure more recent than 1.10.0.442. This is surprising…

there are no official Docker images for Clojure

ah, well that explains it

who maintains the images here? https://hub.docker.com/_/clojure

that's Docker owned, like the company.

So if I think they should be updated, I guess I pester Docker, the company 😬

OK thanks all!

yup! if you're interested in custom images and interested in sponsoring some sort of support model, I'd love to chat though.

we made a #cloud-native channel lately.

so I'm in a process of taking in problems/opinions and trying to distill something

Next question: • I’ve been using Cambada to AOT my code and build an uberjar containing my code and all my dependencies. • Now that I’ve switched one of my dependencies to a gitlib, Cambada is having trouble building the uberjar. So: Does anyone happen to know whether there’s an existing solution to this somewhere? (I mean either some way to continue using Cambada, maybe a fork, etc; or some alternative to Cambada (whether a single tool or a combination of tools) that I might want to consider switching to.)

@aviflax Is AOT a requirement?

Here's all the packaging tools available: https://github.com/clojure/tools.deps.alpha/wiki/Tools#packaging

I guess it’s not a 100% hard requirement but I’d prefer to preserve it. The project is a CLI tool so it needs to start quickly.

Thanks, I was just reviewing that list. I was just wondering if anyone had an answer off the top of their head. The descriptions of those tools don’t always include lots of details.

I don't know how many of those do AOT although, in theory, you could always do that step separately and then package stuff: -e "(compile 'my.main.ns)"

I am fairly certain that depstar works with git deps and local deps but it does not do AOT.

I believe Cambada is AOT compiling all the Clojure code in my project, including in dependencies. I like that.

Would calling compile do the same? I’d guess not...

Since AOT is transitive, I suspect just compiling the main ns will get you much the same result...?

It’s worth trying out for sure!

Gotta get on my bike now, will check back in shortly. Thanks!

You'll need to ensure classes exists as a top-level folder in your repo and add it to the :paths vector so it's on the classpath.