This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2016-06-13
Channels
- # admin-announcements (1)
- # beginners (10)
- # boot (15)
- # cider (9)
- # clara (195)
- # cljsrn (24)
- # clojars (20)
- # clojure (46)
- # clojure-android (1)
- # clojure-germany (15)
- # clojure-greece (16)
- # clojure-nl (1)
- # clojure-russia (13)
- # clojure-spec (28)
- # clojure-uk (44)
- # clojurescript (104)
- # clojurex (1)
- # component (7)
- # css (2)
- # cursive (27)
- # datomic (92)
- # dirac (12)
- # emacs (5)
- # lambdaisland (3)
- # lein-figwheel (36)
- # mount (87)
- # off-topic (8)
- # om (102)
- # om-next (3)
- # onyx (30)
- # pedestal (3)
- # re-frame (26)
- # reagent (20)
- # robots (4)
- # specter (18)
- # spirituality-ethics (1)
- # untangled (127)
- # yada (11)
@mfikes: do you think it's possible to redo your macroexpand'
implementation in terms of cljs.analyzer/macroexpand-1
?
does it still need to do the eval stuff?
basically I need a macroexpand-all implementation for non-bootstrap clojurescript that's distinct from the clojure version of macroexpand-all, and I want to see if the bootstrap and non-bootstrap impls can be the same
Hmm… I think the need to eval was simply to cope with not having the form being directly passed to macroexpand. (looking at my post http://blog.fikesfarm.com/posts/2015-09-05-runtime-macroexpand.html)
If there is a use case that you’d like it to work for, I wouldn’t mind taking a look perhaps later on tonight to see if it can be made to work in bootstrap.
i'll get it working for non-boostrap cljs, and then I'll ping you to see if the new impl works for bootstrap
otherwise specter will just use a different macroexpand-all for each of the 3 cases
By the way, Specter is the only major lib I’m aware of that is straddling all 3 targets. It feels like it ain’t easy to do, but it can be done. 🙂
indeed
sometimes I yell out the occasional profanity, but it's doable
@mfikes: alright, finally got it working with this commit https://github.com/nathanmarz/specter/commit/8c128816f58d8e2188a8a19b7d336dc5a128f7d3
let me know if it's still bootstrap-compatible
@nathanmarz: will do
@nathanmarz: Gotta run but was at least able to see a problem with riddley.walk
:
$ planck -c target/specter-0.12.0-SNAPSHOT.jar:/Users/mfikes/.m2/repository/riddley/riddley/0.1.12/riddley-0.1.12.jar
Planck 1.14
ClojureScript 1.9.14
Docs: (doc function-name-here)
(find-doc "part-of-name-here")
Source: (source function-name-here)
Exit: Control+D or :cljs/quit or exit or quit
Results: Stored in vars *1, *2, *3, an exception in *e
cljs.user=> (require-macros '[com.rpl.specter.macros :refer [transform]])
No such namespace: riddley.walk, could not locate riddley/walk.cljs, riddley/walk.cljc, or Closure namespace "riddley.walk"
nil
@mfikes: ok, changed master so that shouldn't happen anymore with bootstrap