This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2022-10-18
Channels
- # announcements (18)
- # babashka (34)
- # beginners (47)
- # biff (1)
- # calva (99)
- # cider (1)
- # clojure (99)
- # clojure-austin (13)
- # clojure-australia (1)
- # clojure-dev (53)
- # clojure-europe (30)
- # clojure-nl (1)
- # clojure-norway (7)
- # clojure-spec (7)
- # clojure-uk (1)
- # clojurescript (50)
- # cursive (11)
- # datahike (20)
- # datomic (10)
- # fulcro (7)
- # graalvm (8)
- # gratitude (1)
- # introduce-yourself (7)
- # jackdaw (1)
- # joyride (1)
- # lambdaisland (7)
- # lsp (2)
- # malli (7)
- # nbb (1)
- # off-topic (1)
- # portal (24)
- # re-frame (8)
- # reagent (13)
- # reitit (7)
- # releases (2)
- # remote-jobs (8)
- # rewrite-clj (3)
- # sci (1)
- # scittle (21)
- # shadow-cljs (2)
- # testing (3)
- # tools-deps (22)
- # web-security (19)
@alexmiller When trying to figure out how to invoke clojure.tools.deps.alpha/prep-libs!
directly, I got confused by its config
argument. It appears to be unused and oddly, the prep
CLI API function passes the basis
function as its value: https://github.com/clojure/tools.deps.alpha/blob/master/src/main/clojure/clojure/tools/cli/api.clj#L102 -- maybe some leftover from a refactoring?
that is confusing looking but also right
inside tools.deps, "config" usually refers to the top level procurer config (currently just :mvn/repos and :mvn/local-repo) which is part of the basis
it's passed through a lot of layers in case it is needed in some future evolution so may be unused in many cases
pretty much anywhere you see config
in those apis, that's what it means
I see, makes sense!
So passing the function value of basis
is a legit bug then?
no, that's correct - it is the map containing those keys if they exist
it contains other stuff too
Hm? No it's the value of (defn basis ...)
oh, then that's bad
Ah I found the culprit: https://github.com/clojure/tools.deps.alpha/commit/b04ac4f25508df1d8e643baa1baec7abe91e2f75
it used to be shadowed by the destructuring binding of basis
which was removed by that commit
yeah, will fix
Thanks for the clarifications 🙂
now I have to remember if I intended to remove it entirely or what I was doing then :)
Just take a reference of your brain as of that time! 😄
We need a cybernetic Datomic implant
pushed fix for next release