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-994 Clojure <p>i'm working on clojure.core/repeat reducer. </p> CLJ-994

repeat reducer

Enhancement Major Open Unresolved Unassigned Jason Jackson Fri, 11 May 2012 13:46:08 -0500 Fri, 14 Sep 2012 14:37:35 -0500 Release 1.5 1 1 <p>Jason, have you tried to build this using JDK 1.6.0? I've tried on Mac OS X 10.6.8 + Oracle/Apple JDK 1.6.0 and Ubuntu 11.10 + IBM JDK 1.6.0, and on both it compiles, but during the tests fails with a ClassNotFoundException for class jsr166y.ForkJoinTask.</p> <p>It builds and tests cleanly on Ubuntu 11.10 + Oracle JDK 1.7.0 for me.</p> <p>That's an issue that applies to all of core.reducers. Alan Malloy experienced it as well. I tried fixing it, but eventually just upgraded to JDK 1.7. I don't understand why it's happening.</p> <p>This issue is isolated to mvn test afaik. </p> <p>When I include clojure inside a leiningen project, and add jsr166y.jar to lib directory, core.reducers works fine with java 1.6.</p> <p>Jason, you say it applies to all of core.reducers in your May 17, 2012 comment. I don't understand. Without your patch applied, I can run "./antsetup.sh ; ant" in a freshly-pulled Clojure git repo on either of the JDK 1.6.0 versions mentioned in my earlier comment, and do not get any errors during the tests. Are you saying perhaps that core.reducers currently has no tests that exercise the problem now, but your patch adds such tests that fail, even with no other changes to the code?</p> <p>Yah that's right. Now that you mention it, my patch is the first unit test to call r/fold (the existing tests do non-parallel reductions). </p> <p>With Stuart Halloway's commit to Clojure master on June 8, 2012 titled "let reducers tests work under ant", patch 0001-repeat-for-clojure.core.reducers.patch dated May 11, 2012 now runs correctly even the new unit tests requiring class jsr166y.ForkJoinTask with Oracle/Apple JDK 1.6 and Linux IBM JDK 1.6.</p> <p>I'm on the contributors list. Is this patch still needed?<br/> sorry for long long delay.</p> <p>This patch should wait until <a href="http://dev.clojure.org/jira/browse/CLJ-993">http://dev.clojure.org/jira/browse/CLJ-993</a> is committed. I think there's a some shared code. </p> Global Rank Patch Code and Test