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-8 Clojure <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java">Reported by richhickey, Dec 17, 2008 Was made a no-op <span class="code-keyword">for</span> AOT/gen-class - need to distinguish cases Comment 1 by richhickey, Dec 17, 2008 (No comment was entered <span class="code-keyword">for</span> <span class="code-keyword">this</span> change.) Owner: --- Comment 2 by richhickey, Dec 18, 2008 (No comment was entered <span class="code-keyword">for</span> <span class="code-keyword">this</span> change.) Labels: Priority-High Comment 3 by scgilardi, May 12, 2009 Rich, <span class="code-keyword">do</span> you have an example of the problem with AOT/gen-class that motivated disabling detection of circular loads? With that in hand, I'd like to take a look at fixing <span class="code-keyword">this</span>. Comment 4 by richhickey, May 13, 2009 You should just <span class="code-keyword">try</span> it out. I'm not sure <span class="code-keyword">if</span> it is just something I encountered on my way to the <span class="code-keyword">final</span> mechanism (and now isn't needed). However, you should hold off right at the moment, as I'm working on loading/classloader stuff.</pre> </div></div> CLJ-8

GC Issue 3: Re-enable detection of circular loads

Enhancement Minor Closed Completed Stephen C. Gilardi Stephen C. Gilardi Wed, 17 Jun 2009 14:45:00 -0500 Fri, 28 Jan 2011 09:24:22 -0600 Fri, 28 Jan 2011 09:24:22 -0600 Release 1.3 2 3 <p>Converted from <a href="http://www.assembla.com/spaces/clojure/tickets/8">http://www.assembla.com/spaces/clojure/tickets/8</a><br/> Attachments:<br/> load-tracking.diff - <a href="https://www.assembla.com/spaces/clojure/documents/arZXq-2qKr3R8teJe5aVNr/download/arZXq-2qKr3R8teJe5aVNr">https://www.assembla.com/spaces/clojure/documents/arZXq-2qKr3R8teJe5aVNr/download/arZXq-2qKr3R8teJe5aVNr</a><br/> check-cyclic-load.diff - <a href="https://www.assembla.com/spaces/clojure/documents/aUykkoZfSr35wNeJe5cbCb/download/aUykkoZfSr35wNeJe5cbCb">https://www.assembla.com/spaces/clojure/documents/aUykkoZfSr35wNeJe5cbCb/download/aUykkoZfSr35wNeJe5cbCb</a></p> <p>richhickey said: Updating tickets (#8, #19, #30, #31, #126, #17, #42, #47, #50, #61, #64, #69, #71, #77, #79, #84, #87, #89, #96, #99, #103, #107, #112, #113, #114, #115, #118, #119, #121, #122, #124)</p> <p>scgilardi said: I'm working on this now. I plan to have more info by 19 October.</p> <p>richhickey said: Updating tickets (#8, #42, #113, #2, #20, #94, #96, #104, #119, #124, #127, #149, #162)</p> <p>scgilardi said: [<a href="file:arZXq-2qKr3R8teJe5aVNr">file:arZXq-2qKr3R8teJe5aVNr</a>]: fixes tickets 8, 42, and 113</p> <p>scgilardi said: please see my posting to the developer group</p> <p>scgilardi said: [<a href="file:aUykkoZfSr35wNeJe5cbCb">file:aUykkoZfSr35wNeJe5cbCb</a>]: patch to detect/reject cyclic deps</p> <p>scgilardi said: fresh patch that detects and rejects cyclic dependencies. based on today's master. tests included.</p> <p>I provided a patch for this. Is there anything further I can or must do to to move it along?</p> <p>Had to recreate the patch almost from scratch. Haven't seen this problem before--when the git merge failed, it wouldn't create any .rej files to work with.</p> <p>Recreated patch is same as Steve's, except for a test configuration change to match current MASTER.</p> Approval Ok Global Rank Patch Code and Test