Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Cannot upgrade to Clojure(Script) 1.11 due to inlined garden dependency #363

Closed
colinkahn opened this issue May 31, 2022 · 0 comments · Fixed by #369 or #370
Closed

[Bug]: Cannot upgrade to Clojure(Script) 1.11 due to inlined garden dependency #363

colinkahn opened this issue May 31, 2022 · 0 comments · Fixed by #369 or #370
Labels

Comments

@colinkahn
Copy link

What happened?

We're trying to upgrade our version of re-frame-10x as well as Clojure(Script) to 1.11. There seems to be a dependency issue though caused by the inlined garden dep in re-frame 10x. See this issue, which resulted in a merged PR - noprompt/garden#194

garden hasn't had a new version released since March 2020, and we can't exclude / point to a SHA for the latest garden because the dep is inlined in re-frame 10x.

Is it possible to release a new version of re-frame 10x that somehow addresses the issue, making it use the latest garden code available in their git repo?

Note that while you would think that the shadowing of abs would just be a warning, this causes our build to outright fail (using shadow-cljs version 2.19.0). It means we have to choose to be on an older Clojure version or stop using re-frame-10x.

10x Version

1.3.0

Reagent Version

1.1.1

React Version

18.1.0

re-frame Version

1.2.0

What browsers are you seeing the problem on?

No response

Relevant console output

WARNING: abs already refers to: #'clojure.core/abs in namespace: day8.re-frame-10x.inlined-deps.garden.v1v3v10.garden.color, being replaced by: #'day8.re-frame-10x.inlined-deps.garden.v1v3v10.garden.color/abs
[:app] Build failure:
------ ERROR -------------------------------------------------------------------
 File: jar:file:/Users/---/.m2/repository/day8/re-frame/re-frame-10x/1.3.0/re-frame-10x-1.3.0.jar!/day8/re_frame_10x/inlined_deps/garden/v1v3v10/garden/color.cljc:390:1
--------------------------------------------------------------------------------
 387 |      (let [d (util/clip 1 179 distance-from-complement)]
 388 |          (hue-rotations color 0 d (- d)))))
 389 |
 390 | (defn- abs
-------^------------------------------------------------------------------------
null
abs already refers to: cljs.core/abs being replaced by: day8.re-frame-10x.inlined-deps.garden.v1v3v10.garden.color/abs
{:warning :redef, :line 390, :column 1, :msg "abs already refers to: cljs.core/abs being replaced by: day8.re-frame-10x.inlined-deps.garden.v1v3v10.garden.color/abs"}
ExceptionInfo: abs already refers to: cljs.core/abs being replaced by: day8.re-frame-10x.inlined-deps.garden.v1v3v10.garden.color/abs
        shadow.build.compiler/warning-collector (compiler.clj:490)
        shadow.build.compiler/warning-collector (compiler.clj:462)
        clojure.core/partial/fn--5910 (core.clj:2650)
        cljs.analyzer/warning (analyzer.cljc:752)
        cljs.analyzer/warning (analyzer.cljc:750)
        cljs.analyzer/fn--2861 (analyzer.cljc:1995)
        cljs.analyzer/fn--2861 (analyzer.cljc:1931)
        clojure.lang.MultiFn.invoke (MultiFn.java:252)
        cljs.analyzer/analyze-seq* (analyzer.cljc:4069)
        cljs.analyzer/analyze-seq* (analyzer.cljc:4067)
        cljs.analyzer/analyze-seq*-wrap (analyzer.cljc:4074)
        cljs.analyzer/analyze-seq*-wrap (analyzer.cljc:4072)
        cljs.analyzer/analyze-seq (analyzer.cljc:4098)
        cljs.analyzer/analyze-seq (analyzer.cljc:4076)
        cljs.analyzer/analyze-form (analyzer.cljc:4285)
        cljs.analyzer/analyze-form (analyzer.cljc:4282)
        cljs.analyzer/analyze* (analyzer.cljc:4338)
        cljs.analyzer/analyze* (analyzer.cljc:4330)
        cljs.analyzer/analyze (analyzer.cljc:4358)
        cljs.analyzer/analyze (analyzer.cljc:4341)
        cljs.analyzer/analyze-seq (analyzer.cljc:4099)
        cljs.analyzer/analyze-seq (analyzer.cljc:4076)
        cljs.analyzer/analyze-form (analyzer.cljc:4285)
        cljs.analyzer/analyze-form (analyzer.cljc:4282)
        cljs.analyzer/analyze* (analyzer.cljc:4338)
        cljs.analyzer/analyze* (analyzer.cljc:4330)
        cljs.analyzer/analyze (analyzer.cljc:4358)
        cljs.analyzer/analyze (analyzer.cljc:4341)
        cljs.analyzer/analyze-seq (analyzer.cljc:4099)
        cljs.analyzer/analyze-seq (analyzer.cljc:4076)
        cljs.analyzer/analyze-form (analyzer.cljc:4285)
        cljs.analyzer/analyze-form (analyzer.cljc:4282)
        cljs.analyzer/analyze* (analyzer.cljc:4338)
        cljs.analyzer/analyze* (analyzer.cljc:4330)
        shadow.build.compiler/analyze/fn--55637 (compiler.clj:264)
        shadow.build.compiler/analyze (compiler.clj:252)
        shadow.build.compiler/analyze (compiler.clj:211)
        shadow.build.compiler/analyze (compiler.clj:213)
        shadow.build.compiler/analyze (compiler.clj:211)
        shadow.build.compiler/default-analyze-cljs (compiler.clj:408)
        shadow.build.compiler/default-analyze-cljs (compiler.clj:397)
        clojure.core/partial/fn--5908 (core.clj:2642)
        shadow.build.compiler/do-analyze-cljs-string (compiler.clj:318)
        shadow.build.compiler/do-analyze-cljs-string (compiler.clj:278)
        shadow.build.compiler/analyze-cljs-string/fn--55731 (compiler.clj:511)
        shadow.build.compiler/analyze-cljs-string (compiler.clj:510)
        shadow.build.compiler/analyze-cljs-string (compiler.clj:508)
        shadow.build.compiler/do-compile-cljs-resource/fn--55759 (compiler.clj:626)
        shadow.build.compiler/do-compile-cljs-resource (compiler.clj:607)
        shadow.build.compiler/do-compile-cljs-resource (compiler.clj:565)
        shadow.build.compiler/maybe-compile-cljs/fn--55862 (compiler.clj:958)
        shadow.build.compiler/maybe-compile-cljs (compiler.clj:957)
        shadow.build.compiler/maybe-compile-cljs (compiler.clj:933)
        shadow.build.compiler/par-compile-one (compiler.clj:1066)
        shadow.build.compiler/par-compile-one (compiler.clj:1021)
        shadow.build.compiler/par-compile-cljs-sources/fn--55898/iter--55920--55924/fn--55925/fn--55926/fn--55927 (compiler.clj:1139)
        clojure.core/apply (core.clj:667)
        clojure.core/with-bindings* (core.clj:1990)
        clojure.core/with-bindings* (core.clj:1990)
        clojure.core/apply (core.clj:671)
        clojure.core/bound-fn*/fn--5818 (core.clj:2020)
        java.util.concurrent.FutureTask.run (FutureTask.java:264)
        java.util.concurrent.ThreadPoolExecutor.runWorker (ThreadPoolExecutor.java:1128)
        java.util.concurrent.ThreadPoolExecutor$Worker.run (ThreadPoolExecutor.java:628)
        java.lang.Thread.run (Thread.java:830)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants