Olical / depot

Find newer versions of your dependencies in your deps.edn file
https://clojars.org/olical/depot
The Unlicense
256 stars 14 forks source link

Issue with :deps key in alias when running without --update #43

Open orestis opened 4 years ago

orestis commented 4 years ago

I have an alias like this:

 :clj-kondo {:deps {clj-kondo {:mvn/version "2020.01.27"}}
              :main-opts ["-m" "clj-kondo.main"]}

According to the guide:

:deps - a deps map from lib to coordinate of deps to REPLACE the project :deps

When running without the --update flag, I get this error:

Execution error (ExceptionInfo) at clojure.tools.deps.alpha/choose-rule (alpha.clj:39).
Unknown alias key: :deps

Full trace:

{:clojure.main/message
 "Execution error (ExceptionInfo) at clojure.tools.deps.alpha/choose-rule (alpha.clj:39).\nUnknown alias key: :deps\n",
 :clojure.main/triage
 {:clojure.error/class clojure.lang.ExceptionInfo,
  :clojure.error/line 39,
  :clojure.error/cause "Unknown alias key: :deps",
  :clojure.error/symbol clojure.tools.deps.alpha/choose-rule,
  :clojure.error/source "alpha.clj",
  :clojure.error/phase :execution},
 :clojure.main/trace
 {:via
  [{:type clojure.lang.ExceptionInfo,
    :message "Unknown alias key: :deps",
    :data {:key :deps},
    :at
    [clojure.tools.deps.alpha$choose_rule
     invokeStatic
     "alpha.clj"
     39]}],
  :trace
  [[clojure.tools.deps.alpha$choose_rule invokeStatic "alpha.clj" 39]
   [clojure.tools.deps.alpha$choose_rule invoke "alpha.clj" 37]
   [clojure.tools.deps.alpha$merge_alias_maps$fn__1076$fn__1078
    invoke
    "alpha.clj"
    46]
   [clojure.core.protocols$iter_reduce invokeStatic "protocols.clj" 49]
   [clojure.core.protocols$fn__8140 invokeStatic "protocols.clj" 75]
   [clojure.core.protocols$fn__8140 invoke "protocols.clj" 75]
   [clojure.core.protocols$fn__8088$G__8083__8101
    invoke
    "protocols.clj"
    13]
   [clojure.core$reduce invokeStatic "core.clj" 6828]
   [clojure.core$reduce invoke "core.clj" 6810]
   [clojure.tools.deps.alpha$merge_alias_maps$fn__1076
    invoke
    "alpha.clj"
    45]
   [clojure.core.protocols$fn__8159 invokeStatic "protocols.clj" 168]
   [clojure.core.protocols$fn__8159 invoke "protocols.clj" 124]
   [clojure.core.protocols$fn__8114$G__8109__8123
    invoke
    "protocols.clj"
    19]
   [clojure.core.protocols$seq_reduce invokeStatic "protocols.clj" 31]
   [clojure.core.protocols$fn__8144 invokeStatic "protocols.clj" 75]
   [clojure.core.protocols$fn__8144 invoke "protocols.clj" 75]
   [clojure.core.protocols$fn__8088$G__8083__8101
    invoke
    "protocols.clj"
    13]
   [clojure.core$reduce invokeStatic "core.clj" 6828]
   [clojure.core$reduce invoke "core.clj" 6810]
   [clojure.tools.deps.alpha$merge_alias_maps
    invokeStatic
    "alpha.clj"
    44]
   [clojure.tools.deps.alpha$merge_alias_maps doInvoke "alpha.clj" 41]
   [clojure.lang.RestFn applyTo "RestFn.java" 137]
   [clojure.core$apply invokeStatic "core.clj" 665]
   [clojure.core$apply invoke "core.clj" 660]
   [clojure.tools.deps.alpha$combine_aliases
    invokeStatic
    "alpha.clj"
    65]
   [clojure.tools.deps.alpha$combine_aliases invoke "alpha.clj" 58]
   [depot.outdated$gather_outdated invokeStatic "outdated.clj" 107]
   [depot.outdated$gather_outdated invoke "outdated.clj" 104]
   [depot.outdated.main$_main invokeStatic "main.clj" 55]
   [depot.outdated.main$_main doInvoke "main.clj" 33]
   [clojure.lang.RestFn applyTo "RestFn.java" 137]
   [clojure.lang.Var applyTo "Var.java" 705]
   [clojure.core$apply invokeStatic "core.clj" 665]
   [clojure.main$main_opt invokeStatic "main.clj" 514]
   [clojure.main$main_opt invoke "main.clj" 510]
   [clojure.main$main invokeStatic "main.clj" 664]
   [clojure.main$main doInvoke "main.clj" 616]
   [clojure.lang.RestFn applyTo "RestFn.java" 137]
   [clojure.lang.Var applyTo "Var.java" 705]
   [clojure.main main "main.java" 40]],
  :cause "Unknown alias key: :deps",
  :data {:key :deps}}}
Olical commented 4 years ago

Ah, I guess it's a path that was never accounted for. I'll try to take a look soon but I'm pretty buried right now. Thank you for the detailed report!