Clojure JIRA

[CONTRIB-112] clojure.java.classpath/system-classpath is incorrect when running in a classloader other than the application classloader

Fri, 27 Jan 2012 09:45:18 -0600

Because system-classpath uses java.class.path, it doesn't work when running in a classloader other than the application classloader.

This happens when running in any container that uses a custom classloader (e.g. Jetty, Cake, TomCat).

The fix is to use (.getURLs (.getClassLoader clojure.lang.RT)) instead of (System/getProperty "java.class.path"). A patch is attached.

[CONTRIB-35] clojure.contrib.trace/deftrace does not work with function definitions with doc-strings

Fri, 30 Sep 2011 13:36:40 -0500

Apparently, defttrace does not support functions with doc-strings. I tried replacing defn with deftrace in a function of mine (as the docs indicated) and I got the following error -

java.lang.IllegalArgumentException: Don't know how to create ISeq from: Symbol
</code></pre>

Sample code to reproduce the error - 

<pre><code>(deftrace fact
  "Calculate the factorial of x"
  [x]
  (if (< x 1)
    1
    (* x (fact (- x 1)))))

Would be great if this bug is fixed, all my functions have doc-strings

[CONTRIB-117] POM file for math.combinatorics

Fri, 9 Sep 2011 11:39:20 -0500

I noticed that math.combinatorics isn't being built or released yet. Here is the pom file needed to get it into CI builds.

[CONTRIB-114] java.jmx build failure due to missing JAR

Fri, 19 Aug 2011 16:54:17 -0500

Appears to be a problem with parent POM:

[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 3.663s
[INFO] Finished at: Fri Aug 19 14:58:52 EDT 2011
[INFO] Final Memory: 6M/81M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.7.2:test (default-test) on project java.jmx: Error to resolving surefire provider dependency: Missing:
[ERROR] ----------
[ERROR] 1) org.apache.maven.surefire:surefire-junit3:jar:2.7.2
[ERROR]
[ERROR] Try downloading the file manually from the project website.
[ERROR]
[ERROR] Then, install it using the command:
[ERROR] mvn install:install-file -DgroupId=org.apache.maven.surefire -DartifactId=surefire-junit3 -Dversion=2.7.2 -Dpackaging=jar -Dfile=/path/to/file
[ERROR]
[ERROR] Alternatively, if you host your own repository you can deploy the file there:
[ERROR] mvn deploy:deploy-file -DgroupId=org.apache.maven.surefire -DartifactId=surefire-junit3 -Dversion=2.7.2 -Dpackaging=jar -Dfile=/path/to/file -Durl=[url] -DrepositoryId=[id]
[ERROR]
[ERROR] Path to dependency:
[ERROR] 1) dummy:dummy:jar:1.0
[ERROR] 2) org.apache.maven.surefire:surefire-junit3:jar:2.7.2
[ERROR]
[ERROR] ----------
[ERROR] 1 required artifact is missing.
[ERROR]
[ERROR] for artifact:
[ERROR] dummy:dummy:jar:1.0

[CONTRIB-113] There needs to be a way to collect regular arguments in tools.cli

Mon, 18 Jul 2011 18:07:59 -0500

In clojure.contrib.command-line, you could specify a 'catch-all' argument at the end of your spec and all arguments after the ones that were parsed would be collected in. In tools.cli, nothing like this seems to exist.

For example, with tools.cli, there is no way to handle stuff like this: "myapp --foo bar --baz foo a b c". The last three arguments, a, b, and c will all be discarded with no way to retrieve them.

Finally, if this wasn't the right place to put this issue, I apologize. tools.cli doesn't seem to be listed as a project and you guys don't accept issues on Github, so I wasn't sure what else to do.

[CONTRIB-1] Move Clojure Contrib issues from GC to Assembla

Fri, 17 Jun 2011 20:42:18 -0500

Sorry, Rich, taking over #1 here.

All issues (even closed ones!) from Contrib's Google Code issue tracker need to be moved over to assembla.

Full list of issues at GC

Take a look at the Clojure space tickets for an example of how it was done there

Note: The tickets should be moved in order; also, please leave them unassigned to anyone or to any milestone

Hint: You can add the tickets all as "new" at first and then use the batch update feature to change their attributes
to match Google Code.

[CONTRIB-111] Build failure against edge clojure

Wed, 1 Jun 2011 21:05:59 -0500

Build failure

ashee:clojure-contrib amitava$ mvn assembly:assembly -Dclojure.jar=/Users/amitava/opt/clojure/clojure.jar

....Lots of output elided....

Testing clojure.contrib.test-logging
Exception in thread "main" java.lang.IllegalStateException: Can't dynamically bind non-dynamic var: clojure.contrib.logging/log-factory
at clojure.lang.Var.pushThreadBindings(Var.java:339)
at clojure.core$push_thread_bindings.invoke(core.clj:1716)
at clojure.contrib.test_logging$eval248$fn__249.invoke(test_logging.clj:19)
at clojure.test$compose_fixtures$fn_6529$fn_6530.invoke(test.clj:670)
at clojure.test$default_fixture.invoke(test.clj:663)
at clojure.test$compose_fixtures$fn__6529.invoke(test.clj:670)
at clojure.test$test_all_vars.invoke(test.clj:705)
at clojure.test$test_ns.invoke(test.clj:728)
at clojure.core$map$fn__3840.invoke(core.clj:2433)
at clojure.lang.LazySeq.sval(LazySeq.java:42)
at clojure.lang.LazySeq.seq(LazySeq.java:60)
at clojure.lang.Cons.next(Cons.java:39)
at clojure.lang.RT.boundedLength(RT.java:1607)
at clojure.lang.RestFn.applyTo(RestFn.java:130)
at clojure.core$apply.invoke(core.clj:602)
at clojure.test$run_tests.doInvoke(test.clj:743)
at clojure.lang.RestFn.invoke(RestFn.java:408)
at user$eval488$fn__491.invoke(run-test8724407867004279179.clj:9)
at user$eval488.invoke(run-test8724407867004279179.clj:7)
at clojure.lang.Compiler.eval(Compiler.java:6406)
at clojure.lang.Compiler.load(Compiler.java:6843)
at clojure.lang.Compiler.loadFile(Compiler.java:6804)
at clojure.main$load_script.invoke(main.clj:282)
at clojure.main$script_opt.invoke(main.clj:342)
at clojure.main$main.doInvoke(main.clj:426)
at clojure.lang.RestFn.invoke(RestFn.java:408)
at clojure.lang.Var.invoke(Var.java:401)
at clojure.lang.AFn.applyToHelper(AFn.java:161)
at clojure.lang.Var.applyTo(Var.java:518)
at clojure.main.main(main.java:37)
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Clojure Contrib parent module ..................... SKIPPED
[INFO] def ............................................... SKIPPED
[INFO] types ............................................. SKIPPED
[INFO] generic ........................................... SKIPPED
[INFO] accumulators ...................................... SKIPPED
[INFO] agent-utils ....................................... SKIPPED
[INFO] base64 ............................................ SKIPPED
[INFO] jar ............................................... SKIPPED
[INFO] classpath ......................................... SKIPPED
[INFO] combinatorics ..................................... SKIPPED
[INFO] command-line ...................................... SKIPPED
[INFO] complex-numbers ................................... SKIPPED
[INFO] cond .............................................. SKIPPED
[INFO] seq ............................................... SKIPPED
[INFO] condition ......................................... SKIPPED
[INFO] core .............................................. SKIPPED
[INFO] graph ............................................. SKIPPED
[INFO] except ............................................ SKIPPED
[INFO] dataflow .......................................... SKIPPED
[INFO] datalog ........................................... SKIPPED
[INFO] error-kit ......................................... SKIPPED
[INFO] fcase ............................................. SKIPPED
[INFO] find-namespaces ................................... SKIPPED
[INFO] fnmap ............................................. SKIPPED
[INFO] prxml ............................................. SKIPPED
[INFO] gen-html-docs ..................................... SKIPPED
[INFO] greatest-least .................................... SKIPPED
[INFO] import-static ..................................... SKIPPED
[INFO] java-utils ........................................ SKIPPED
[INFO] jmx ............................................... SKIPPED
[INFO] json .............................................. SKIPPED
[INFO] lazy-seqs ......................................... SKIPPED
[INFO] lazy-xml .......................................... SKIPPED
[INFO] logging ........................................... SKIPPED
[INFO] macro-utils ....................................... SKIPPED
[INFO] macros ............................................ SKIPPED
[INFO] map-utils ......................................... SKIPPED
[INFO] math .............................................. SKIPPED
[INFO] miglayout ......................................... SKIPPED
[INFO] mmap .............................................. SKIPPED
[INFO] ns-utils .......................................... SKIPPED
[INFO] mock .............................................. SKIPPED
[INFO] monads ............................................ SKIPPED
[INFO] monadic-io-streams ................................ SKIPPED
[INFO] priority-map ...................................... SKIPPED
[INFO] stream-utils ...................................... SKIPPED
[INFO] probabilities ..................................... SKIPPED
[INFO] profile ........................................... SKIPPED
[INFO] reflect ........................................... SKIPPED
[INFO] repl-ln ........................................... SKIPPED
[INFO] repl-utils ........................................ SKIPPED
[INFO] server-socket ..................................... SKIPPED
[INFO] set ............................................... SKIPPED
[INFO] singleton ......................................... SKIPPED
[INFO] sql ............................................... SKIPPED
[INFO] with-ns ........................................... SKIPPED
[INFO] strint ............................................ SKIPPED
[INFO] swing-utils ....................................... SKIPPED
[INFO] trace ............................................. SKIPPED
[INFO] zip-filter ........................................ SKIPPED
[INFO] complete .......................................... SKIPPED
[INFO] standalone ........................................ SKIPPED
[INFO] Clojure Contrib ................................... FAILURE [2:34.296s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 2:35.348s
[INFO] Finished at: Wed Jun 01 21:48:18 EDT 2011
[INFO] Final Memory: 10M/81M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal com.theoryinpractise:clojure-maven-plugin:1.3.3:test (test-clojure) on project logging: Clojure failed. -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR] mvn <goals> -rf :logging

[CONTRIB-110] clojure.contrib.json does not parse JSON strings containing back slashes on Windows

Fri, 27 May 2011 08:19:35 -0500

This bug does not seem to occur on my Linux box, but when run under Windows I get the following:

user> (read-json "\"C:\\Users\"")

"C:nullsers"

On Linux I get what I would expect:

user> (read-json "\"C:\\Users\"")

"C:\Users"

[CONTRIB-109] Incorrect line separator handling in spy macro

Wed, 18 May 2011 11:45:01 -0500

The spy macro removes a trailing newline added by pprint, but windows systems have two-character line separators.

Also applies to clojure.tools.logging.

[CONTRIB-108] c.c.pprint pprint-logical-block macro using private var

Thu, 5 May 2011 17:30:07 -0500

pprint-logical-block macro can't be used from the outside:
java.lang.IllegalStateException: var: clojure.contrib.pprint/current-level is not public

[CONTRIB-55] clojure.contrib.sql expects err to be a PrintWriter

Sun, 24 Apr 2011 22:49:44 -0500

The problem is that c.c.sql.internal calls `.println` on a "stream" to print update counts and errors. That method only exists on a `PrintWriter`. This isn't true within Slime, where it's a `StringWriter`.

The attached patch binds `out` and uses `println` instead, which does the right thing. I've augmented the docstrings to request a `Writer` as the value of `stream`, which aligns with the docstring for `print`.

[CONTRIB-105] c.c.logging tests rely on hardcoded line-terminator

Wed, 20 Apr 2011 03:29:32 -0500

See https://github.com/clojure/clojure-contrib/commit/330807c6d93f9a3c2bad4c8f4dd6295b6cf2053e#commitcomment-286017

[CONTRIB-107] Build Broken since April 8th

Sun, 17 Apr 2011 08:38:00 -0500

Since the April 8th commit to no longer recognize "earmuff" variables as dynamic by default, clojure-contrib/logging and clojure-contrib/trace are broken.

[CONTRIB-106] variable name colision in java-utils/set-system-properties

Wed, 9 Mar 2011 09:45:50 -0600

(defn set-system-properties
"Set some system properties. Nil clears a property."
[settings]
(doseq [[name val] settings]
(if val
(System/setProperty (name name) (name val))
(System/clearProperty (name name)))))

Somebody did a find/replace as-str -> name and created fn/variable colision with "name"

[CONTRIB-104] prxml attribute value to support internal entity reference

Fri, 18 Feb 2011 17:15:34 -0600

Now there is no way to support internal entities in xml, as value of attribute is always beings xml-escaped and '&' (used as part of entity reference) is being replaced with '&'.

[CONTRIB-99] Bug in clojure.contrib.json/read-json

Wed, 26 Jan 2011 06:26:09 -0600

Pulled this from clojure assembla ticket #436 to clojure-contrib
See clojure-436

What (small set of) steps will reproduce the problem?
$ java -cp lib/clojure-1.2.0.jar:lib/clojure-contrib-1.2.0.jar clojure.main
Clojure 1.2.0
user=> (require 'clojure.contrib.json)
nil
user=> (clojure.contrib.json/read-json "")
java.lang.IllegalArgumentException: Value out of range for char: -1 (NO_SOURCE_FILE:0)
user=>

What is the expected output? What do you see instead?

Expected:

(if eof-error?
  (throw (EOFException. "JSON error (end-of-file)"))
  eof-value)

What version are you using?
1.2

Was this discussed on the group? If so, please provide a link to the discussion
[13:51] <na_ka_na> Hey guys I think there's a bug in clojure.contrib.json at line 116
[13:51] <na_ka_na> in clojure 1.2
[13:52] <na_ka_na> it has a (char i) and then checks for (= i -1) ... but (char -1) fails
[13:52] <na_ka_na> where can I report it ?
[13:53] <LauJensen> na_ka_na: assembla/support

[CONTRIB-34] Add zip and unzip functions to java-utils

Tue, 25 Jan 2011 14:53:07 -0600

Talked with Stuart Sierra on IRC; he agreed these would be helpful functions to have since the JDK bizarrely omits them.

I have a test for this, but it depends on #33, so I'll wait to submit it until that patch is applied.

[CONTRIB-102] clojure.contrib.core/-?> (and perhaps others?) have a quoting bug

Sun, 9 Jan 2011 04:23:48 -0600

See:

https://groups.google.com/group/clojure/browse_frm/thread/cc88ca26352bc604?hl=en#

When called via a namespace-qualified symbol and given three or more arguments, -?> gives the error:

user=> (clojure.contrib.core/-?> 1 inc inc)
CompilerException java.lang.Exception: Unable to resolve symbol: -?>
in this context, compiling:(NO_SOURCE_PATH:3)

[CONTRIB-83] clojure.contrib.shell broken with clojure commit 37d8f7a

Sat, 8 Jan 2011 13:58:47 -0600

clojure.core/byte was modified a couple weeks ago as follows:

(defn byte
"Coerce to byte"
{:tag Byte
:inline (fn [x] `(. clojure.lang.RT (byteCast ~x)))}

[^Number x] (. x (byteValue)))
+ [^Number x] (clojure.lang.RT/byteCast x))

byteValue and byteCast behave differently with values outside the
range of byte values:
user=> (. 255 (byteValue))
-1
user=> (clojure.lang.RT/byteCast 255)
java.lang.IllegalArgumentException: Value out of range for byte: 255
(NO_SOURCE_FILE:0)

The new version breaks clojure.contrib.shell/sh's :out :bytes mode (mapping byte over values returned by InputStream.read in shell.clj, line 125).
Perhaps have sh use #(.byteValue %) instead of byte? (untested)

[CONTRIB-103] type hinting for clojure.contrib.condition

Mon, 27 Dec 2010 09:57:38 -0600

I ran into the situation where reflection in clojure.contrib.condition was taking up much more time than I would have liked it too. I added the needed type hints to avoid reflection and the changes have solved the problem for me.

I've attached the patch. It is also available on github if you prefer:

https://github.com/bmabey/clojure-contrib/commit/acf0017e4b8c4259a03fba03a90050b0e9cb0890

[CONTRIB-100] Control output of non-ASCII characters in clojure-contrib.json

Tue, 30 Nov 2010 14:34:03 -0600

Current version of JSON output functions perform escaping of non-ASCII characters into \uXXXX sequences, although json allows to use any Unicode character in strings. When working with non-Latin-1 based texts, this lead to additional traffic.

Given patch fixes this problem, leaving old behavior by default, but adding new option to output functions, that now accept :escape-chars option, that control how non-ASCII characters in string will output. When this option has value 'true' (default), then hex sequences are used for these characters, and when 'false', then characters are output as-is

P.S. I sent CA at start of June, so Rich should already have it

[CONTRIB-98] c.c.json/json-str handles maps with keys containing quotes incorrectly

Fri, 26 Nov 2010 16:17:26 -0600

Consider:

> (println (json-str {"\"" 1})) 
{""":1} 
</code></pre>

...while I (and my parsers) would expect <pre><code>{"\"":1}

This happens in both contrib 1.2.0 and 1.3-alpha1.

[CONTRIB-96] Cartesian product with nil args

Fri, 26 Nov 2010 08:58:19 -0600

http://groups.google.com/group/clojure-dev/browse_thread/thread/ac3582cd1698be1

user> (cartesian-product [1 2] [true false] ) 
((1 true) (1 false) (2 true) (2 false)) 
user> (cartesian-product [1 2] [false true]) 
nil 
user> (cartesian-product [1 2] [nil 4]) 
nil 
false/nil in first position seems to cause failure.

[CONTRIB-101] eof-value not working in clojure.contrib.json

Thu, 18 Nov 2010 17:07:04 -0600

The docs of read-json-from promise eof-error? and eof-value, but these fail to work. This is caused by lines 116-121: https://github.com/richhickey/clojure-contrib/blob/master/src/main/clojure/clojure/contrib/json.clj#L116-121

At EOF read returns -1, but evaluating (char -1) on the repl reveals that this does not work, and thus an exception is raised before eof-value could be returned.

This is easily fixed by checking if i is in a valid range, or at least not -1, by doing something like (and (not= i -1) (char i))

[CONTRIB-75] tests for c.c.io

Wed, 10 Nov 2010 09:54:31 -0600

c.c.io has few tests, despite being on the most used libraries in contrib. Be a clojure here and write some unit tests for this module. Code review comments would be welcome too!

[CONTRIB-72] Replace references to duck-streams with io in clojurescript

Fri, 29 Oct 2010 00:47:29 -0500

duck-streams was renamed to io back in February 1st, but clojure-script still references duck-streams. Attached is a very simple patch that fixes this.

[CONTRIB-97] Add zero-arity body to comp function

Tue, 28 Sep 2010 19:22:00 -0500

The comp function should return the identity function should it be called with zero args. Currently there is no zero-arity form. I talked a little more about this at http://blog.fogus.me/2010/08/18/monkeying-with-clojures-comp-function/. Patch attached.

[CONTRIB-77] Refactor c.c.io/writer, reader, etc as a protocol; stop returning PrintWriters

Sat, 18 Sep 2010 01:53:00 -0500

As discussed here:

http://groups.google.com/group/clojure-dev/browse_frm/thread/3e39e9b3982f542b

[CONTRIB-91] json end-of-file handling broken

Fri, 17 Sep 2010 18:53:00 -0500

When read-json encounters an end-of-file, an IllegalArgumentException is thrown regardless of the user's eof preferences. This can easily be reproduced by running:

(read-json "")
</code></pre>
which throws the exception
<pre><code>java.lang.IllegalArgumentException: Value out of range for char: -1
</code></pre>
This is caused by the .read method of the Reader returning -1 when the end is reached. (.read returns an int.) This value is then passed to 'char', which tries to make the corresponding character from the code point number. However, -1 is not a valid code point, so it throws an exception.

A solution is to check for equality to -1 before trying to convert to a character. I also have an idea for a helper function that could be used:
<pre><code>(defn- read-char
  "Reads a character from a Reader. Returns nil if the end of the stream is reached"
  [stream]
  (let [x (.read stream)]
    (if (not= x -1)
      (char x))))

This function will always return either a Character or nil. Using it would make it possible to have the eof case along the other cases in the existing cond forms.

[CONTRIB-95] with-ns tests broken by changes to Clojure exception reporting

Fri, 17 Sep 2010 18:49:00 -0500

with-ns tests fail after Clojure commit 1c0468dbe5228bb8c228e1370133bbe1d018e058 because of how exception messages changed

[CONTRIB-84] clojure.contrib.base64 lacking decode

Fri, 17 Sep 2010 18:16:00 -0500

Namespace clojure.contrib.base64 contains a Base64 encode function but is (so far) missing decode.

I know commons-codec contains a quality implementation of both functions, but adding yet another jar in your build just for base64 decode purposes is an unnecessary complication. For symmetry's sake, decode should be added.

I will add a patch soon.

[CONTRIB-79] Forward-port deprecated 1.1 namespaces

Fri, 17 Sep 2010 00:51:00 -0500

Too many people are doing crazy stuff with conditional loading just to support both 1.1 and 1.2 simultaneously. This way lies madness.

To avoid this, lets forward-port all the 1.1 namespaces that were removed or renamed in master, e.g. duck-streams, seq-utils, str-utils, etc. Just copy the old files from the 1.1 branch, so that no behavior changes.

Add a printed deprecation warning at the top-level of each of the old namespaces, like
(println "The foo namespace is deprecated, please use bar instead.")

Post 1.2 release we can remove the deprecated namespaces.

[CONTRIB-94] c.c.repl-utils/show is currently broken

Thu, 16 Sep 2010 18:16:00 -0500

since the module reorganisation:

commit fe4ed31
Author: Ben Smith-Mannschott bsmith.occs@gmail.com">bsmith.occs@gmail.com
Date: Sat Aug 28 11:22:22 2010 +0200

Fix: http://github.com/juergenhoetzel/clojure-contrib/commit/31c33107f59c71ec00ae1429b38f2005dce374d3

[CONTRIB-93] remove deprecations on master

Fri, 3 Sep 2010 20:52:00 -0500

I noticed the recent aborted attempt to remove the deprecated modules form clojure-contrib master. Since I was in need of some busy work, I decided to try and do it myself and clean up the breakage.

The attached patches don't remove everything that has been marked as deprecated, just the bits Stuart previously indicated by trying to remove them.

clojure.contrib.apply-macro
clojure.contrib.http-agent
clojure.contrib.http-connection
clojure.contrib.javadoc: which rewrote a :use in repl_utils.clj from c.c.javadoc.browse to c.java.browse.
clojure.contrib.properties: where private versions of as-str and as-properties were defined in c.c.sql internal.clj
clojure.contrib.test-is

[CONTRIB-92] gen-html-docs build fails due to unecessary :use of c.c.def

Thu, 26 Aug 2010 23:23:00 -0500

on clojure-contrib master:
gen-html-docs :uses c.c.def without actually using anything from it.
This causes the maven build to fail because clojure.contrib:def is not listed as a dependency in gen-html-docs' pom.xml.

[CONTRIB-90] clojure.contrib.io does not support appending anymore

Thu, 26 Aug 2010 20:25:00 -0500

Reproduce the bug with:

(require 'clojure.contrib.io)
(clojure.contrib.io/append-writer "/xyz.txt")
</code></pre>
which will throw an Exception ���Cannot change an open stream to append mode���.

This is caused because of a flaw in the call chain.
*clojure.contrib.io/writer* for Strings calls again *writer* for whatever *clojure.contrib.io/output-stream*
returns. We call *append-writer* which then calls *writer* on the String which in turn then calls:
<pre><code>
output-stream for ^String which calls
    output-stream for ^URL which calls
        output-stream for ^File which binds *append* to false and calls
            output-stream for ^OutputStream which
            returns a BufferedOutputStream instance which then jumps back to the
        ^File handler, which removes its binding for *append*!! and returns the BOS to the
    ^URL handler which returns the BOS to the
^String handler

At this point the outer binding (for append) is set to true again, and assert-not-appending fails.

[CONTRIB-66] fix pprint tests and reenable

Tue, 24 Aug 2010 22:08:00 -0500

Apparently (according to ticket 65), someone has broken the pprint tests, presumably during the switch to maven and all the hacking to get around gen-class/maven bugs.

Take a look at this and fix it.

[CONTRIB-43] add wall-hack-method and wall-hack-field to java_utils.clj

Tue, 24 Aug 2010 21:06:00 -0500

fns for access private or protected methods or fields

[CONTRIB-42] Pom specifies dependency on clojure 1.0-SNAPSHOT

Tue, 24 Aug 2010 21:02:00 -0500

It seems like contrib's master should point to the same version as clojure's master. If not, it should at least depend on the final release of 1.0; there's no reason to use a snapshot.

[CONTRIB-89] Implements clojure.contrib.priority-map

Tue, 24 Aug 2010 20:25:00 -0500

Adds clojure.contrib.priority-map, a variety of sorted map that's useful for many priority queue situations.

http://groups.google.com/group/clojure-dev/browse_thread/thread/fdb000cae4f66a95

[CONTRIB-88] clojure.contrib.sql runs out of memory on very large datasets

Tue, 24 Aug 2010 20:09:00 -0500

If you have a query that results in a very large dataset you will run out of memory because by default, with-query-results pulls the entire result set into memory (even though it creates a lazy-seq using resultset-seq).

This issue has been discussed previously here:
http://groups.google.com/group/clojure/browse_thread/thread/7b0c250e0ba6c9eb/fb9001522b49c20a

The fix is simple, just call (.setFetchSize stmt 1). But, with-query-results doesn't give you a way to do that currently. I propose adding an optional parameter after sql-params that is a hash of attributes to set on the PreparedStatement before executing it.

So you can do this:

(with-connection {���} 
    (.setAutoCommit (sql/connection) false) ;; needed for postgres 
    (with-query-results results ["SELECT id, data FROM nodes"] 
      {:fetch-size 1000} 
      (doseq [r results] 
        ���)))

The new code in clojure.contrib.sql is very simple, but it depends on a new contrib namespace called clojure.contrib.bean. Here is more info on my proposed fix:
http://groups.google.com/group/clojure-dev/browse_thread/thread/d8334759f10f3f45

[CONTRIB-87] fix gvec reflection warnings

Tue, 24 Aug 2010 17:52:00 -0500

wrong assembla space

[CONTRIB-52] Improve running tests from Ant

Tue, 24 Aug 2010 17:41:00 -0500

This builds on clojure ticket 193, commit 607389029cfec50f32b73c00a6f66d0a1dbcda23.
Change code used by build.xml to exit with error code so ant fails cleanly.
Fork in ant so exit can be used.

[CONTRIB-86] c.c.memoize: thread-safe and extensible memoize

Tue, 24 Aug 2010 17:22:00 -0500

Add the thread-safe, extensible memoize sparked from the discussion here: http://groups.google.com/group/clojure/browse_thread/thread/36a13d150d830683/fe6daf462d3ebfac

[CONTRIB-76] (pr x & more) makes a self-call, which can lead to stack overflow

Tue, 24 Aug 2010 16:33:00 -0500

When 'pr', or any related functions are called with a sequence of arguments, they can cause stack overflow.

Eg:

(apply pr (range 5000))

[CONTRIB-31] Contribs profiler can run into an Exception

Tue, 24 Aug 2010 15:28:00 -0500

The profile function from clojure.contrib.profile can run into
an Exception when it's argument is a lazy sequence.
Reproduce this problem:

(use 'clojure.contrib.profile)

(defn foo [x] (prof :foo (+ x 10)))

(profile (map foo (range 5))) ==>
java.lang.IllegalArgumentException: Wrong number of args passed to: core$max

(profile (pmap foo (range 5))) ==>
java.lang.IllegalArgumentException: Wrong number of args passed to: core$max

(profile (doseq [i (range 5)] (foo i))) ==> Works fine

[CONTRIB-65] pprint depends on order of compilation

Tue, 24 Aug 2010 15:08:00 -0500

pprint causes problems with the build process:

1. c.c.pprint.ColumnWriter
2. c.c.pprint.PrettyWriter
3. c.c.pprint

The first two are gen-class'd, and PrettyWriter extends the class
generated for ColumnWriter.

Unless they are compiled in exactly the order given above, the build
fails with a ClassNotFoundException.

[CONTRIB-63] duck-streams open as resource what isn't File or URL

Tue, 24 Aug 2010 15:04:00 -0500

Currently: When duck-streams is given a string, it attempts to open it for reading as a URL and if that fails it tries the string as a path specifying a File. When the string doesn't specify a readable file either, we fail.

I'd like to see duck-streams try to resolve the string as a resource name before finally giving up.

[CONTRIB-32] Unknown severity level causes NPE

Tue, 24 Aug 2010 14:35:00 -0500

$ cat .clojure
/opt/clojure/clojure.jar:/opt/clojure-contrib/clojure-contrib.jar:lib/commons-logging-1.1.1.jar
$ clj
Clojure 1.1.0-alpha-SNAPSHOT
user=> (use 'clojure.contrib.logging)
nil
user=> (log :info "Hello, world.")   
Sep 21, 2009 3:44:03 PM clojure.contrib.logging$eval__92$impl_write_BANG___100 invoke
INFO: Hello, world.
nil
user=> (log :bar "Hello, world.") 
java.lang.NullPointerException (NO_SOURCE_FILE:0)

I hit this when using :severe as a log level, which should actually be :error.

[CONTRIB-30] append-spit should only write out an encoding marker once

Tue, 24 Aug 2010 14:28:00 -0500

In clojure.contrib.duck-streams append-spit writes out encoding
markers (for UnicodeLittle for example this is a FEFF in hex)
each time it appends to a file. This should happen only when
the file is initially created.

Test case for reproducing this behaviour:

(use 'clojure.contrib.duck-streams)

(binding [*default-encoding* "UnicodeLittle"]
  (append-spit "/foo.txt" "Line 1\n"))
(binding [*default-encoding* "UnicodeLittle"]
  (append-spit "/foo.txt" "Line 2\n"))

(slurp "c:/foo.txt" "UnicodeLittle")

The slurp outputs
"Line 1\n?Line 2\n"
The expected output is:
"Line 1\nLine 2\n"

[CONTRIB-71] patch for changing assert-args over to :pre and :post for c.c.mock

Tue, 24 Aug 2010 14:17:00 -0500

I've cleaned up the documentation and got rid of the old assert-args code from c.c.mock. The attached patches contain the two commits I made on my repository. I'm new to patches and git so let me know if I missed anything obvious!

[CONTRIB-14] Performance enhancements needed for logging

Tue, 24 Aug 2010 14:12:00 -0500

The enabled check should occur prior to sending off to an agent.
The allow-direct-logging flag should default to true.
Replace the map of implementation-specific functions with a direct def of them.
Memoize the get-log function.

[CONTRIB-69] maven downloads clojure jar file even if -Denv=local is specified

Tue, 24 Aug 2010 14:09:00 -0500

maven downloads the snapshot even if built with mvn -Denv=local and -Dclojure.jar=<absolute path to the locally built jar> package.

[CONTRIB-67] c.c.json.read: accept any java.io.Reader

Tue, 24 Aug 2010 14:08:00 -0500

not just PushbackReader

[CONTRIB-7] Logging functions

Tue, 24 Aug 2010 14:01:00 -0500

Logging functions to interact with existing java logging implementations.

Discussed in clojure group: http://groups.google.com/group/clojure/t/b21d1241d47d682f

[CONTRIB-6] Add set of regular expression functions

Tue, 24 Aug 2010 14:00:00 -0500

A couple of RE utilities I developed for Cascade that I no longer need but are generally useful.

[CONTRIB-4] Purge removed libraries from load_all.clj

Tue, 24 Aug 2010 13:51:00 -0500

(load "clojure/contrib/load_all") has not worked since the penultimate commit to clojure-contrib, as libraries are being moved to core. all-contrib-libs needs to be adjusted to match the new set of libraries.

[CONTRIB-3] Agent shutdown introduces AccessControlException when compiling via ant

Tue, 24 Aug 2010 13:46:00 -0500

The change that fixes issue Clojure Ticket #124 introduces an AccessControlException when building via ant on java 1.5 and java 1.6/osx:

[java] java.security.AccessControlException: access denied (java.lang.RuntimePermission modifyThread)
[java] at java.security.AccessControlContext.checkPermission(AccessControlContext.java:264)
[java] at java.security.AccessController.checkPermission(AccessController.java:427)
[java] at java.util.concurrent.ThreadPoolExecutor.shutdown(ThreadPoolExecutor.java:894)
...

For discussion see this post

The attached patch is a modification to "build.xml" which fixes the problem for java 1.5/osx and java 1.6/osx.

[CONTRIB-50] Allow for externally-provided db connection to c.c.sql

Tue, 24 Aug 2010 13:32:00 -0500

This small patch allows an :external key in a db-spec map, to allow the use of existing database connection.

Among other benefits, this would make it trivial to use a Proxool db connection-pool with c.c.sql.

(Suggestions for alternative key-names are welcome. Maybe :instance or :connection-object would be more clear?)

[CONTRIB-47] cl-format rounding bugs

Tue, 24 Aug 2010 13:25:00 -0500

Reported to me by Carlos Ungil (and confirmed by me):

Unfortunately there are still some issues with rounding. I've noticed that when the
mantissa is rounded up from 9 to 10 the exponent is not changed
accordingly. As a result, we get for example
clojure.contrib.pprint> (cl-format nil "~,0F" 9.4) => "9."
clojure.contrib.pprint> (cl-format nil "~,0F" 9.5) => "1.0"

I've compared the output from Clojure and Common Lisp for a handful of
cases. Below we see how "0.10" is returned instead of "1.0" when
printing 0.99 with a single digit after the decimal point. There is
also a minor problem with "01." being printed instead of "1.", but
returning a different representation for a number is of course not as
bad as returning a wrong representation.

;;(map #(cl-format nil % 0.99) ["~,0F" "~,1F" "~,2F" "~,3F"])
("01." "0.10" "0.99" "0.990")
;;(loop for fmt in '("_{,0F" "},1F" "_{,2F" "},3F") collect (format nil
fmt 0.99))
("1." "1.0" "0.99" "0.990")

There are also issues with negative numbers, the add-sign variable has
the same bug in fixed-float that it had in dollar-float but fixing
that is not enough to correct the problem.

;; > (map #(cl-format nil % -0.99) ["~,0F" "~,1F" "~,2F" "~,3F"])
("010." "0-1.0" "0-0.99" "0-0.990")

[CONTRIB-20] Report progress in http.agent

Tue, 24 Aug 2010 13:15:00 -0500

clojure.contrib.http.agent provides no way to monitor the progress of the HTTP request/response.

It would be possible to monitor the number of bytes sent and received by replacing duck-streams/copy with a function that updates an atom (or agent) with the number of bytes copied.

[CONTRIB-9] Invoking namespaced log macro is tedious

Tue, 24 Aug 2010 13:06:00 -0500

Current style is fine when using :use, but looks ugly when using :require. For example:

(ns foo.bar
    (:require (clojure.contrib [logging :as log])))
(log/log :debug "Hello World!")</code></pre>

Adding convenience macros for the 6 logging levels would mean invoking:
<pre><code>(log/debug "Hello World!")

Which looks much nicer.

[CONTRIB-38] repl_utils addition: apropos

Tue, 24 Aug 2010 11:55:00 -0500

An implementation of apropos, as found in some Scheme implementations: given a symbol or a string s, return all definitions in currently-loaded namespaces that contains s as part of their names.

[CONTRIB-39] tracing enhancement: dotrace

Tue, 24 Aug 2010 11:55:00 -0500

As discussed a few months ago on Google Groups:

dotrace lets one easily trace calls to the named functions, by rebinding them so that calls to these functions within the dotrace body invoke the trace-enabled functions instead of the originals.

[CONTRIB-36] clojure.contrib.test-is use-fixtures function composes fixture functions repeatedly (Backport clojure #194)

Tue, 24 Aug 2010 11:49:00 -0500

For the clojure-1.0-compatibility branch, copy the fix from http://www.assembla.com/spaces/clojure/tickets/194-clojure-test-use-fixtures-function-composes-fixture-functions-repeatedly

If you call use-fixtures over and over again, it will simply compose the test deeper and deeper into a nesting of the fixture function.
(use-fixtures :each my-cleanup) (deftest my-messy-test [��])

Load that a few times and you end up running the equivalent of:
(my-cleanup (my-cleanup (my-cleanup my-messy-test))).

[CONTRIB-85] build and release sublibraries

Tue, 24 Aug 2010 11:18:00 -0500

Design goals

keep key libraries in contrib, protecting provenance and license so that non-technical issues do not keep people from using code
continue to produce a monolithic contrib build for casual/one-off application consumers
start building individual libraries within contrib, so that larger apps can manage dependencies in a granular fashion

I propose we start with the new match (matchure) library, get feedback, and then take requests for what other libraries need to be bundled separately.

[CONTRIB-15] Streamline clojure.contrib.http.agent interface

Tue, 24 Aug 2010 11:14:00 -0500

Simplify the interface to clojure.contrib.http.agent as follows:

Replace separate :on-success and :on-failure callbacks with single :handler callback.
Shorten response-body-bytes/str names to "result" and "result-str."
Shorten response-status/message/headers names to status/message/headers
Provide accessors for request parameters such as URI, headers, body, etc.

[CONTRIB-12] log-system-agent state becomes nil if log level is not enabled.

Tue, 24 Aug 2010 11:08:00 -0500

Currently an invocation of do-log will return nil if the log level isn't enabled, thus gives a nil to the log-system-agent. The log-system needs to be returned every time.

[CONTRIB-82] make defalias work with macros

Tue, 24 Aug 2010 10:55:00 -0500

get back to 1.1-compatible behavior

[CONTRIB-81] Remove gen-class requirement from pprint

Tue, 24 Aug 2010 10:47:00 -0500

In preparation for moving pprint into clojure itself.

[CONTRIB-80] Add catch clauses to c.c.condition

Tue, 24 Aug 2010 10:45:00 -0500

Currently using clojure.contrib.condition is awkward if you need to handle both Conditions and Exceptions. This patch by Steve Gilardi and myself allows you to include catch clauses in your handle-case forms for regular Exceptions.

[CONTRIB-73] lazy-xml uses old-style reify syntax

Tue, 24 Aug 2010 10:23:00 -0500

lazy-xml was not updated when the reify syntax change hit Clojure master, and thus is broken. Assigning to next release milestone with a high priority, as this prevents lazy-xml from working at all.

Patch attached.

[CONTRIB-78] fix broken jmx tests on ibm jdk

Tue, 24 Aug 2010 09:45:00 -0500

fix broken jmx tests on ibm jdk

[CONTRIB-74] clojure.contrib.condition: list? not the correct predicate for detecting forms

Tue, 24 Aug 2010 09:24:00 -0500

The handler-case macro detects forms of the form "(handle ...)" by checking for items in the body for which list? returns true that begin with the symbol handle. code can be built from seqs that are not lists (e.g., clojure.lang.Cons's or lazy-seqs). The proper predicate is "seq?" not "list?".

[CONTRIB-62] duck-streams should handle bytes as well as characters

Tue, 24 Aug 2010 09:04:00 -0500

I'd like to see duck-streams learn to support byte-oriented I/O just as well as it currently supports character-oriented I/O.

I've written an implementation and made it available on my ducks-byte [1] branch of clojure-contrib for discussion and possible inclusion.

[1] http://github.com/bpsm/clojure-contrib/tree/ducks-byte

The branch itself breaks the change down into a sequence of (hopefully) logical steps, so I encourage those interested to have a look at it.

Features

input-stream is like reader, except for bytes.
output-stream is like writer, except for bytes.
reader and writer now build on input-stream and output-stream where appropriate.

Compatibility

I've renamed append-to-writer because it applies not just to writers, but also to output-streams. This might be a compatibility issue for clients that did not heed the advice given in documentation to use append-writer or append-spit instead of directly binding this var.

[CONTRIB-64] teach duck-streams to read clojure forms

Tue, 24 Aug 2010 09:04:00 -0500

I'd like an easy way to consume a text file written in Clojure syntax as pure data: as a sequence of Clojure data structures without evaluation. This allows Clojure to be used where property files, XML or JSON might otherwise be chosen.

[CONTRIB-58] Add println-style args to log macros

Tue, 24 Aug 2010 08:55:00 -0500

Per a post to the group from Phil Hagelberg:
http://groups.google.com/group/clojure/browse_thread/thread/d91e1911e3095157

[CONTRIB-56] Faster version of reductions

Tue, 24 Aug 2010 08:44:00 -0500

Faster version of reductions for seq-utils

[CONTRIB-54] with-temp-ns should clean up after exceptional condition

Tue, 24 Aug 2010 08:43:00 -0500

removing the temporary namespace needs to be in a finally block

[CONTRIB-53] AOT of logging.clj causes logging implementation to be decided at compile-time

Tue, 24 Aug 2010 08:42:00 -0500

We shouldn't AOT logging.clj since this causes the log implementation to be determined at compile-time. This is almost always the wrong thing; it should be determined at load-time since it's determined by what classes are on the classpath at runtime.

[CONTRIB-60] Add strint (string interpolation) lib

Tue, 24 Aug 2010 08:01:00 -0500

Previously discussed:

[CONTRIB-61] Add -?>> to clojure.contrib.core

Tue, 24 Aug 2010 08:01:00 -0500

Obvious companion to -?>.

[CONTRIB-41] PersistentFnMap doesn't handle both valAt arities

Tue, 24 Aug 2010 08:00:00 -0500

clojure.contrib.fnmap.PersistentFnMap implements Associative (and thus ILookup) but doesn't supply a definition for both arities of valAt - needs to add

-valAt ([this key] ...) ([this key not-found] ...)

[CONTRIB-40] bugs in cl-format

Tue, 24 Aug 2010 08:00:00 -0500

As reported in the google group (http://groups.google.com/group/clojure/browse_thread/thread/ae3e636a44c95686):

Hello,

I've encountered a couple of issues with the cl-format function
included in contrib.pprint

(cl-format nil "~1,1$" -12.0) ;; => "12.0" the sign is lost

I think the problem is the following assignment in the dollar-float
function
add-sign (and (:at params) (not (neg? arg))) ;; wrong
(the sign is only printed when the colon modifier is present and only
for positive numbers)
that should read, if I understand correctly the logic,
add-sign (or (:at params) (neg? arg))

The second issue is not so straightforward to solve:

(cl-format true "_1,1$%" 0.001) ;; => String index out of range: -1

I've tracked down the bug into the function round-str (the variable
round-pos will be negative and this case is not handled properly), but
I don't understand the code well enough to propose a fix.

Cheers,

Carlos

[CONTRIB-59] Bug in test_java_utils.clj

Tue, 24 Aug 2010 07:59:00 -0500

I got test error today (21 Jan 2009, GMT+5:30) on Clojure-contrib 1.1 (maven branch) while running "mvn assembly:assembly" on Windows XP 32-bit on JDK 1.6.0_18.

<snip>passing tests</snip>

FAIL in (test-relative-path-string) (run-test503247994263281606.clj:130)
absolute path strings are forbidden
expected: (thrown? IllegalArgumentException (relative-path-string "/baz"))
actual: nil

FAIL in (test-relative-path-string) (run-test503247994263281606.clj:130)
absolute File paths are forbidden
expected: (thrown? IllegalArgumentException (relative-path-string (File. "/quux")))
actual: nil

FAIL in (test-file) (run-test503247994263281606.clj:130)
no sneaking in absolute paths!
expected: (thrown? IllegalArgumentException (file "foo" "bar" "/boom" "baz" "quux"))
actual: nil

FAIL in (test-as-url) (run-test503247994263281606.clj:130)
expected: (= (URL. "file:/foo") (as-url (File. "/foo")))
actual: (not (= #<URL file:/foo> #<URL file:/C:/foo>))

<snip>passing tests</snip>

4 failures, 0 errors

[CONTRIB-5] def: Add defn-memo

Tue, 24 Aug 2010 07:54:00 -0500

Idea from google group thread

[CONTRIB-57] remove walk tests as now part of core

Tue, 24 Aug 2010 07:52:00 -0500

http://groups.google.com/group/clojure-dev/browse_thread/thread/51982e4091e3614d
Move tests over to core

[CONTRIB-29] Backport clojure.contrib.logging to clojure-1.0-compat branch

Tue, 24 Aug 2010 07:24:00 -0500

This simply backports the clojure.contrib.logging namespace to the 1.0-compatible branch.

The only change from master was to revert back to the old style of import calls.

[CONTRIB-28] Add c.c.table namespace

Tue, 24 Aug 2010 07:21:00 -0500

Add the namespace c.c.table, as described here:

http://groups.google.com/group/clojure-dev/browse_thread/thread/b60015723d81aa34

Also, support functions described here:
http://groups.google.com/group/clojure-dev/browse_thread/thread/4b20e40d83095c67
http://groups.google.com/group/clojure-dev/browse_thread/thread/155c8b9893d673bc

Sean

[CONTRIB-27] str-utils2: add type hints for returned values

Tue, 24 Aug 2010 07:20:00 -0500

clojure.contrib.str-utils2 is has type hints for function arguments, but not for the functions themselves, so return values are not type-hinted.

[CONTRIB-17] str-utils2: seq-like functions have different argument order

Tue, 24 Aug 2010 07:14:00 -0500

clojure.contrib.str-utils2 contains several functions with the same names as clojure.core functions: take, drop, butlast. However, their argument order is reversed from the order used by the clojure.core functions.

One of the stated design goals of str-utils2 is to have all functions take the string they are operating on as their first argument, to enable easy composition with ->

Either these functions should be renamed, as "str-take", or a note should be added to their documentation explaining the argument order.

[CONTRIB-70] c.c.string: final migration (in c.c.repl-utils, c.c.command-line)

Tue, 24 Aug 2010 07:11:00 -0500

Patch addresses remaining items to be converted to the new string lib:
�� "str-join" to "join" in c.c.command-line
�� "re-partition" to "partition" in c.c.repl-utils
�� correction to the example function call in c.c.string/partition's docs (reversed order of arguments)

[CONTRIB-13] Add test coverage for seq-utils

Tue, 24 Aug 2010 07:09:00 -0500

I realized that seq-utils is untested based on this discussion:

http://groups.google.com/group/clojure/browse_thread/thread/2be768e15d2b717a

I'll be adding unit tests to ensure coverage, for future changes.

[CONTRIB-11] Default log namespace is incorrect

Tue, 24 Aug 2010 07:08:00 -0500

Currently the runtime value of ns is used to namespace calls to log, instead of the ns of the code that calls log. For example:

user=> (ns foo (:use (clojure.contrib logging))) 
nil
foo=> (defn bar [] (log :fatal "in foo/bar")) 
#'foo/bar
foo=> (bar)
#<Agent@4f961bac...>
foo=> (ns baz (:use (clojure.contrib logging)))
nil
baz=>  (foo/bar) 
#<Agent@4f961bac...></code></pre>
and the output to the log is:
<pre><code>2009-08-06 11:05:19,817 FATAL foo - / - in foo/bar
2009-08-06 11:05:38,167 FATAL baz - / - in foo/bar

The second printed namespace is "baz" and it should be the same as the first, "foo".

[CONTRIB-68] clojure-contrib distclean maven target

Tue, 24 Aug 2010 07:08:00 -0500

It will be great if clojure-contrib can support distclean target for maven.

[CONTRIB-51] Add let-kw to c.c.macros

Tue, 24 Aug 2010 06:33:00 -0500

As discussed in this thread on the main Clojure list, I'm submitting the let-kw keyword argument processing macro. Please see the attached patch (against the master branch). This is the version from this message, which uses symbols for binding, like the normal let form (rather than keywords). The attached patch has a few tests which also provide decent usage examples.

[CONTRIB-49] update pom.xml for contrib to include build.clojure.org and ticket 208 change

Tue, 24 Aug 2010 06:31:00 -0500

The pom refers to 1.1.0-alpha, but clojure's master branch is now called 1.1.0-master. Also, the canonical maven repo for clojure is now build.clojure.org.

[CONTRIB-48] Replace instances of ^ reader macro with (meta ...)

Tue, 24 Aug 2010 06:28:00 -0500

Clojure 1.1 is deprecating the form ^x in favor of (meta x) in order to reclaim the ^ prefix.

We need to get clojure.contrib to respect this in order to avoid having deprecation warnings.

[CONTRIB-46] make duck-streams work with java.net.Socket

Tue, 24 Aug 2010 06:10:00 -0500

Add another type for the key multimethods in duck-streams.

[CONTRIB-45] Add new functions to c.c.map-utils

Tue, 24 Aug 2010 06:08:00 -0500

This is a ticket for patches related to c.c.map-utils. See the following discussions:

http://groups.google.com/group/clojure-dev/browse_thread/thread/9a518c853bfbba8b
http://groups.google.com/group/clojure-dev/browse_thread/thread/4b20e40d83095c67

[CONTRIB-8] Named agent-log function defined within log macro not found at runtime

Tue, 24 Aug 2010 06:06:00 -0500

Per an email from Richard Newman:

By adding print statements to the various 'discovery' macros I could confirm that a logging framework was being found. However, when my app (servlet in this case) actually executed a log statement, I got an error that clojure.contrib.logging/agent-log was undefined.

This confused me, because the presence of any send-log form should cause it to be defined in the macroexpansion. My code simply called (log :info "some string"), which eventually expands into (amongst other things) a send-log form.

Simply lifting the function definition out of the macro made the problem go away

[CONTRIB-37] Include math/lcm for more than 2 args

Tue, 24 Aug 2010 04:51:00 -0500

Please pull changes from neotyk/clojure-contrib.
This is very small improvement of math/lcm just to accept more than two arguments.

[CONTRIB-33] Add delete-file and delete-file-recursively functions to java-utils

Tue, 24 Aug 2010 04:46:00 -0500

Another one of those bizarre omissions from the JDK. Discussed this with Chouser on IRC, and he thinks it's a good idea.

I've added it to java-utils, though there are enough file-specific functions now that I think a new file-utils namespace could be spun off.

[CONTRIB-22] clojure.contrib.str-utils2 is not AOT compiled

Tue, 24 Aug 2010 04:17:00 -0500

build.xml is missing the line to pre-compile the namespace.

[CONTRIB-23] Clojure AOT compilation will compile unwanted classes

Tue, 24 Aug 2010 04:17:00 -0500

My build file specifies the namespaces to AOT compile but if I include another namespace (even from a JAR dependency) that is not AOT compiled, the other namespace will be compiled as well.

In my case, I was using clojure-contrib's clojure.contrib.str-utils2 namespace, and I got a bunch of clojure/contrib/str_utils2 classes in my output directory.

I think that the AOT compiler should NOT precompile any namespaces that are transitively reached, only namespaces in the set specified by the command line are appropriate.

As currently coded, you will frequently find unwanted third-party dependencies in your output JARs; further, if multiple parties depend on the same JARs, this could cause bloating and duplication in the eventual runtime classpath.

Note: I'm tracking against master for both clojure and clojure-contrib.

[CONTRIB-24] Dependency from clojure-contrib to clojure is incorrect (for Maven snapshot builds)

Tue, 24 Aug 2010 04:17:00 -0500

The version number of Clojure is still listed in pom.xml as 1.0-SNAPSHOT, but should be 1.1.0-alpha-SNAPSHOT to track the master of clojure-contrib against the master of clojure-lang.

[CONTRIB-25] Clojure API documentation should identify the date the doc was generated and the version number of the library

Tue, 24 Aug 2010 04:17:00 -0500

Failing to do this causes a lot of confusion (I've already gotten bit) when you have stable releases out in the world and the docs reflect an alpha version. This has been painful, for example, with the shift of clojure.contrib.test-is to clojure.test.

[CONTRIB-26] Compatibility from clojure.contrib.test-is to clojure.test

Tue, 24 Aug 2010 04:17:00 -0500

It would be nice if there was a compatibility layer for older code compiled against 1.0's clojure.contrib.test-is to work with 1.1's clojure.test. In other words, have a clojure.contrib .test-is namespace that re-exports the corresponding code from the new clojure.test. Handling the change in the (are) macro's semantics will be a challenge, however.

[CONTRIB-18] java-utils/as-str could be variadic

Tue, 24 Aug 2010 04:15:00 -0500

Compojure has a function called str* that works like clojure.core/str but uses the names of symbols/keywords instead of their literal representation.

clojure.contrib.java-utils/as-str fulfills the same purpose, but only accepts a single argument.

as-str could be 100% compatible with str if it became variadic:

(defn as-str
  [& args]
  (apply str (map (fn [x] (if (instance? clojure.lang.Named x)
                            (name x) x))
                  args)))

user> (as-str "Hello, " :World)
"Hello, World"

[CONTRIB-19] Re-add auto-agent.clj

Tue, 24 Aug 2010 04:15:00 -0500

This was discussed briefly on the clojure dev list, but no consensus was reached. Instead, I have attached the changes that have fixed the original library and if the clojure-contrib members wish to include it (again), then it's readily available.

The original code by Sierra required 2 changes to get working. First, @ was macro-expanding to a lazy-seq, so the filtering based on list? would fail. Second, I added a check for an explicit call to (deref). Also, I re-added auto-agent namespace for compilation to build.xml.

My live changes can be found at: http://github.com/fogus/clojure-contrib/tree/auto-agent-again. The tests can be run with (run-tests 'clojure.contrib.auto-agent).

If it is merged back into the contrib master, then I would be happy to roll the tests into the test-contrib framework.

[CONTRIB-21] Private logging macros aren't private

Tue, 24 Aug 2010 04:15:00 -0500

E.g.,

(defmacro commons-logging
  "...."
  #^{:private true}</code></pre>

should be
<pre><code>(defmacro commons-logging
  "...."
  {:private true}

[CONTRIB-16] duck-streams/copy bug on InputStream-to-Writer

Tue, 24 Aug 2010 04:14:00 -0500

clojure.contrib.duck-streams/copy works incorrectly when copying an InputStream to a Writer. It does not use the correct buffer size when the buffer is less than full.

[CONTRIB-2] Clojure tests: sets

Tue, 24 Aug 2010 00:45:00 -0500

Tracking ticket for work on Clojure contributed test suite.

First basic commit in forked GitHub repo:

http://github.com/rnewman/clojure-contrib/commit/1c2d3d6f9ae91abf5a7861706e0173f3290dd926

This covers difference, project, join.

Maintainers: I know the Clojure patch doc says "create a patch". Would a link to the commit suffice? It seems that a `git pull` and a link to a permanent record of the contents would serve better than using the issue tracker and patch files as a primitive form of version control, at least for real work-in-progress stuff��

[CONTRIB-10] Implementation-specific logging functions are using reflection

Tue, 24 Aug 2010 00:07:00 -0500

As reported by Richard Newman via email.

The lack of type hinting results in multiple reflective calls even if the logging level is disabled. Not fast.

[CONTRIB-44] AOT compilation of clojure-contrib.jar pre-sets logging implementation

Tue, 24 Aug 2010 00:06:00 -0500

Using ant to build clojure-contrib.jar causes clojure.contrib.logging to be AOT-compiled.

This means that the classpath at contrib build time affects which logging implementation is used, not the classpath in use when the code using c.c.l is compiled.

Using a standard pre-built clojure-contrib.jar one can never use anything but java.util.logging! That's bad for anyone who uses released snapshots, or who uses different log implementations on different projects, but doesn't want to maintain different builds of clojure-contrib.jar.

Furthermore, someone building clojure-contrib.jar with, say, log4j in their classpath will produce a jar in which c.c.l will fail when log4j is unavailable:

user=> (use 'clojure.contrib.logging)
    java.lang.NoClassDefFoundError: org/apache/log4j/Level (NO_SOURCE_FILE:0)

c.c.l should be adjusted so that the logging implementation is not discovered until runtime, or otherwise be removed from the AOT-compiled set.

Clojure JIRA

[CONTRIB-112] clojure.java.classpath/system-classpath is incorrect when running in a classloader other than the application classloader

[CONTRIB-35] clojure.contrib.trace/deftrace does not work with function definitions with doc-strings

[CONTRIB-117] POM file for math.combinatorics

[CONTRIB-114] java.jmx build failure due to missing JAR

[CONTRIB-113] There needs to be a way to collect regular arguments in tools.cli

[CONTRIB-1] Move Clojure Contrib issues from GC to Assembla

[CONTRIB-111] Build failure against edge clojure

[CONTRIB-110] clojure.contrib.json does not parse JSON strings containing back slashes on Windows

[CONTRIB-109] Incorrect line separator handling in spy macro

[CONTRIB-108] c.c.pprint pprint-logical-block macro using private var

[CONTRIB-55] clojure.contrib.sql expects *err* to be a PrintWriter

[CONTRIB-105] c.c.logging tests rely on hardcoded line-terminator

[CONTRIB-107] Build Broken since April 8th

[CONTRIB-106] variable name colision in java-utils/set-system-properties

[CONTRIB-104] prxml attribute value to support internal entity reference

[CONTRIB-99] Bug in clojure.contrib.json/read-json

[CONTRIB-34] Add zip and unzip functions to java-utils

[CONTRIB-102] clojure.contrib.core/-?> (and perhaps others?) have a quoting bug

[CONTRIB-83] clojure.contrib.shell broken with clojure commit 37d8f7a

[CONTRIB-103] type hinting for clojure.contrib.condition

[CONTRIB-100] Control output of non-ASCII characters in clojure-contrib.json

[CONTRIB-98] c.c.json/json-str handles maps with keys containing quotes incorrectly

[CONTRIB-96] Cartesian product with nil args

[CONTRIB-101] eof-value not working in clojure.contrib.json

[CONTRIB-75] tests for c.c.io

[CONTRIB-72] Replace references to duck-streams with io in clojurescript

[CONTRIB-97] Add zero-arity body to comp function

[CONTRIB-77] Refactor c.c.io/writer, reader, etc as a protocol; stop returning PrintWriters

[CONTRIB-91] json end-of-file handling broken

[CONTRIB-95] with-ns tests broken by changes to Clojure exception reporting

[CONTRIB-84] clojure.contrib.base64 lacking decode

[CONTRIB-79] Forward-port deprecated 1.1 namespaces

[CONTRIB-94] c.c.repl-utils/show is currently broken

[CONTRIB-93] remove deprecations on master

[CONTRIB-92] gen-html-docs build fails due to unecessary :use of c.c.def

[CONTRIB-90] clojure.contrib.io does not support appending anymore

[CONTRIB-66] fix pprint tests and reenable

[CONTRIB-43] add wall-hack-method and wall-hack-field to java_utils.clj

[CONTRIB-42] Pom specifies dependency on clojure 1.0-SNAPSHOT

[CONTRIB-89] Implements clojure.contrib.priority-map

[CONTRIB-88] clojure.contrib.sql runs out of memory on very large datasets

[CONTRIB-87] fix gvec reflection warnings

[CONTRIB-52] Improve running tests from Ant

[CONTRIB-86] c.c.memoize: thread-safe and extensible memoize

[CONTRIB-76] (pr x & more) makes a self-call, which can lead to stack overflow

[CONTRIB-31] Contribs profiler can run into an Exception

[CONTRIB-65] pprint depends on order of compilation

[CONTRIB-63] duck-streams open as resource what isn't File or URL

[CONTRIB-32] Unknown severity level causes NPE

[CONTRIB-30] append-spit should only write out an encoding marker once

[CONTRIB-71] patch for changing assert-args over to :pre and :post for c.c.mock

[CONTRIB-14] Performance enhancements needed for logging

[CONTRIB-69] maven downloads clojure jar file even if -Denv=local is specified

[CONTRIB-67] c.c.json.read: accept any java.io.Reader

[CONTRIB-7] Logging functions

[CONTRIB-6] Add set of regular expression functions

[CONTRIB-4] Purge removed libraries from load_all.clj

[CONTRIB-3] Agent shutdown introduces AccessControlException when compiling via ant

[CONTRIB-50] Allow for externally-provided db connection to c.c.sql

[CONTRIB-47] cl-format rounding bugs

[CONTRIB-20] Report progress in http.agent

[CONTRIB-9] Invoking namespaced log macro is tedious

[CONTRIB-38] repl_utils addition: apropos

[CONTRIB-39] tracing enhancement: dotrace

[CONTRIB-36] clojure.contrib.test-is use-fixtures function composes fixture functions repeatedly (Backport clojure #194)

[CONTRIB-85] build and release sublibraries

Design goals

[CONTRIB-15] Streamline clojure.contrib.http.agent interface

[CONTRIB-12] log-system-agent state becomes nil if log level is not enabled.

[CONTRIB-82] make defalias work with macros

[CONTRIB-81] Remove gen-class requirement from pprint

[CONTRIB-80] Add catch clauses to c.c.condition

[CONTRIB-73] lazy-xml uses old-style reify syntax

[CONTRIB-78] fix broken jmx tests on ibm jdk

[CONTRIB-74] clojure.contrib.condition: list? not the correct predicate for detecting forms

[CONTRIB-62] duck-streams should handle bytes as well as characters

Features

Compatibility

[CONTRIB-64] teach duck-streams to read clojure forms

[CONTRIB-55] clojure.contrib.sql expects err to be a PrintWriter