欢迎!请参阅关于页面以获取更多关于如何使用本网站的信息。
core.match clojurescript 应该是自宿主兼容的
评论者:viebel
基本上我做了的是从https://github.com/clojure/core.match/blob/master/src/main/clojure/clojure/core/match.clj
重命名为match.cljc,并为:cljs创建一个分支。
match.cljc
:cljs
补丁已附上
我用KLIPSE测试了它 - http://app.klipse.tech/?cljs_in.gist=viebel/10da8338c4a1573fab8c04989e471704&external-libs=[https://raw.githubusercontent.com/viebel/core.match/master/src/main/clojure/]
评论者:alexmiller
请注意,目前我们的自动化CI构建箱无法构建或发布包含cljc文件的项目。我正在积极解决这个问题,但在解决问题之前,这个问题无法推进。(问题解决后,我会很高兴这样做。)
无论如何,我很乐意收到对我的补丁的反馈。是大量的代码处理 cljs 和 clojure 类型之间的差异。
现在 cljc 已经完全支持,能否有人审查这个补丁?Alex?David?
另一个问题是,我们是否希望将 Clojure 1.7 作为 core.match 的依赖项,这意味着要从支持中移除许多旧版本。我认为我对此是认同的,但还想听听社区的意见。
评论人:cgrand
如果这个补丁的目标只是让 {{cljs.core.match}} 成为自托宿兼容的(而不是统一 {{cljs.core.match}} 和 {{clojure.core.match}}),那么我认为其中有几个对 clojure 的引用不应该存在,而且在 clojure 编译的 clojurescript 中运行时,{{backtrack-expr}} 和 {{FailureNode}} 应该发出 clojurescript 代码而不是 clojure 代码。
(链接:~viebel)请参阅 Christophe 的评论。如果其他人能把这个补丁修改到Clojure、ClojureScript 和自托宿 ClojureScript 上都能工作,我会很高兴帮忙推进到发布。