http://dev.clojure.org/jira This file is an XML representation of an issue en-us 4.4 649 25-07-2011 http://dev.clojure.org/jira/browse/CLJ-940 Clojure <p>Compiling the following code results in a <tt>Don't know how to create ISeq from: clojure.lang.Symbol</tt> exception</p> <p><tt>code</tt><br/> (ns clj14.myns<br/> (:use<br/> <span class="error">&#91;clojure.core :only seq&#93;</span>))<br/> <tt>code</tt></p> CLJ-940

Passing a non-sequence to refer :only results in uninformative exception

Defect Major Closed Completed Stuart Sierra Hugo Duncan Fri, 24 Feb 2012 12:20:40 -0600 Sat, 1 Sep 2012 08:37:48 -0500 Sat, 1 Sep 2012 08:37:48 -0500 Release 1.3 Release 1.4 Release 1.5 0 3 <p>Hugo, your patch doesn't apply cleanly to latest master due to some changed lines of context around it that are from before Nov 2011, which confuses me given that your patch was created recently. I could mechanically update it, but if you could take a look and create an updated patch yourself it would be safer.</p> <p>Patch clj-940-add-exception-for-non-sequence-in-refer-only-patch.txt dated Apr 26 2012 is same as Hugo Duncan's, except it applies cleanly to latest master as of today.</p> <p>Previous patch does not accurately reflect new <tt>:refer</tt> syntax in <tt>ns</tt>:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java">(:require [... :refer ...])</pre> </div></div> <p>I offer a new patch, 0003-<a href="http://dev.clojure.org/jira/browse/CLJ-940" title="Passing a non-sequence to refer :only results in uninformative exception"><del>CLJ-940</del></a>-check-for-sequential.patch, as an alternative.</p> <p>This patch also checks for the more specific clojure.lang.Sequential instead of IPersistentCollection (which includes sets and maps).</p> <p>If I had my druthers, I'd check for IPersistentList, but I can't face the screaming that would result.</p> <p>Neither patch provides file/line information in the error, but there isn't much affordance for that in core.clj right now.</p> <p>Applies cleanly against d4170e65d001c8c2976f1bd7159484056b9a9d6d. This looks good to me. We should at some point talk more about the implications of checking IPersistenList, but I think there is enough value here to push it forward.</p> Approval Ok Global Rank Patch Code Waiting On richhickey