This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2017-07-25
Channels
- # aleph (2)
- # aws (2)
- # beginners (37)
- # boot (23)
- # cider (29)
- # clara (34)
- # cljs-dev (2)
- # cljsrn (17)
- # clojure (230)
- # clojure-dev (47)
- # clojure-italy (11)
- # clojure-nl (2)
- # clojure-poland (5)
- # clojure-russia (52)
- # clojure-sg (1)
- # clojure-spec (70)
- # clojure-uk (73)
- # clojurescript (31)
- # core-async (9)
- # cursive (15)
- # datomic (39)
- # events (1)
- # graphql (1)
- # lein-figwheel (2)
- # luminus (13)
- # off-topic (2)
- # onyx (29)
- # other-lisps (1)
- # parinfer (15)
- # pedestal (14)
- # re-frame (41)
- # reagent (24)
- # ring (4)
- # ring-swagger (12)
- # rum (1)
- # spacemacs (3)
- # specter (1)
- # test-check (13)
- # timbre (9)
- # unrepl (29)
- # vim (5)
@dnolen it looks like core.match 0.3.0-alpha4 has a dependency on tools.analyzer.jvm
and is requiring it but never actually using it (apart from defining analyze
which is never used anyway) , this is adding a needles couples of seconds in compilation times -- could that be removed?
@bronsa it could be, I planned to use it at some point - but unclear when I will have time to do that work
@alexmiller export PATH=$PATH:$PWD/src/main/resources/clj
should be export PATH=$PATH:$PWD/src/main/resources
in the readme
@bronsa I can help with that if you tell me what needs to be done
or patch etc
if you want to do it, just remove the tools.analyzer requires from match.clj, and the defn analyze
@bronsa did it - please check
tests pass :)
should I release 0.3.0-alpha5? :)
@alexmiller also while you're here, the clj
script doesn't seem to work , $tools_cp
seems to be missing the tools.deps.alpha jar
hmm, could be - I actually have a working installer but not ready to release that stuff yet so I’m kind of backing into the manual install
thx for the heads up
cp=$("$JAVA_CMD" -Xmx256m -classpath "$HOME/.m2/repository/org/clojure/tools.deps.alpha/0.1.11-SNAPSHOT/tools.deps.alpha-0.1.11-SNAPSHOT.jar:$tools_cp" clojure.main -m clojure.tools.deps.alpha.makecp "$project_deps" "$project_cache_dir" "${tools_args[@]}")
I hotpatched it like this and it works --obv not the real fixjust need to modify how the cp gets set
@alexmiller is the intent of tools.deps
to also use it for CLJS or CLJ only? eg. add a :provider
for :npm
?
for the moment, CLJ
open to expanding it to CLJS but no thought has gone into that yet
ok. I guess it doesn’t make much sense since npm doesn’t have the concept of a classpath but I suppose adding things to the classpath could be optional
well the whole point of this is to build classpaths
@alexmiller small typo in your slides: https://github.com/clojure/brew-isntaller, but I guess you know about it
that’s in the immutable past man
@bronsa @dnolen core.match 0.3.0-alpha5 released without tools.analyzer.jvm dependency
@alexmiller thanks!
@alexmiller it looks like issues are still enabled on https://github.com/clojure/tools.deps.alpha
@alexmiller It looks like deps isn’t designed to be a complete project description, correct? In particular, no mention is made of source paths. So it’s only designed for execution, and not to be used for building?
Still pending another discussion with rich
For projects that don't require special building I think there is a lot of value in supporting a way to specify what parts of the project are included in the classpath for use
@cfleming mind posting a link to the presentation?
I must’ve missed it
@anmonteiro It’s in the README at https://github.com/clojure/tools.deps.alpha
thanks