Skip to content

Loading…

Force realization of lazy sequences we pass to Aether #32

Closed
wants to merge 1 commit into from

2 participants

@cemerick cemerick added a commit that closed this pull request
@cemerick realize all lazy sequences to eliminate mentions of e.g. `LazySeq@483…
…7abc3` in error messages, fixes gh-32
6210f24
@cemerick cemerick closed this in 6210f24
@cemerick
Owner

Other larger changes made a merge unworkable, so I just added the doall's on this end. Thanks for bubbling up the issue. :-)

@technomancy technomancy pushed a commit to technomancy/pomegranate that referenced this pull request
@cemerick realize all lazy sequences to eliminate mentions of e.g. `LazySeq@483…
…7abc3` in error messages, fixes gh-32
5dd3df6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Showing with 2 additions and 2 deletions.
  1. +2 −2 src/main/clojure/cemerick/pomegranate/aether.clj
View
4 src/main/clojure/cemerick/pomegranate/aether.clj
@@ -397,10 +397,10 @@ kwarg to the repository kwarg.
(let [repositories (or repositories maven-central)
system (repository-system)
session (repository-session system local-repo offline? transfer-listener)
- deps (map dependency coordinates)
+ deps (doall (map dependency coordinates))
collect-request (CollectRequest. deps
nil
- (map #(make-repository % proxy) repositories))
+ (doall (map #(make-repository % proxy) repositories)))
_ (.setRequestContext collect-request "runtime")
result (if retrieve
(.resolveDependencies system session (DependencyRequest. collect-request nil))
Something went wrong with that request. Please try again.