http://dev.clojure.org/jira/secure/IssueNavigator.jspa?reset=true&jqlQuery=labels+%3D+delete-file An XML representation of a search request en-us 4.4 649 25-07-2011 http://dev.clojure.org/jira/browse/CLJ-1159 Clojure <p>initially reported it here(somehow):<br/> <a href="https://groups.google.com/d/msg/clojure/T9Kvr0IL0kg/wcKBfR9w_1sJ">https://groups.google.com/d/msg/clojure/T9Kvr0IL0kg/wcKBfR9w_1sJ</a></p> <p>Basically clojure.java.io/delete-file doesn't ever return false (even when silently is true, it returns the value of silently), it's due to how it's implemented - but it's obvious from the code, so I'll stop here.</p> <p>Thanks.</p> <p>PS: this is what I'm using as my current workaround:<br/> (defn delete-file<br/> "<br/> an implementation that returns the true/false status<br/> which clojure.java.io/delete-file doesn't do(tested in 1.5.0-RC14)<br/> "<br/> [f &amp; <span class="error">&#91;silently&#93;</span>]<br/> (let <span class="error">&#91;ret (.delete (clojure.java.io/file f))&#93;</span><br/> (cond (or ret silently)<br/> ret<br/> :else<br/> (throw (java.io.IOException. (str "Couldn't delete " f)))<br/> )<br/> )<br/> )</p> <p>I'm sure you guys can find a better way, but as a clojure newbie(really!) that's what I have.</p> any CLJ-1159

clojure.java.io/delete-file doesn't return the status of the deletion(true/false)

Defect Minor Open Unresolved Unassigned AtKaaZ delete-file evil or Sun, 10 Feb 2013 13:55:52 -0600 Sun, 10 Feb 2013 14:21:28 -0600 Release 1.5 0 1 <p>I kinda just realized it affects all versions since and including 1.2, because it appears that its implementation was the same since then.</p> <p>If it's not meant to return the result of the delete, maybe it should specifically return nil and/or the doc say something?</p> <p>As noted in a thread on the Clojure ML, you can pass a known value in the second argument position to detect a delete that failed:</p> <p><tt>(clojure.java.io/delete-file some-file :not-deleted)</tt></p> <p>This returns true on success and :not-deleted on failure.</p> <p>However the docstring could be better worded to make that intention clear. Perhaps:</p> <p><tt>Delete file f. Return true if it succeeds. If silently is nil or false, raise an exception if it fails, else return the value of silently.</tt><br/> <tt>This allows you to detect whether the delete succeeded without catching an exception by passing a non-true truthy value as the second argument.</tt></p> Global Rank