On 11 May 2010 23:16, Peter Schuller <peter.schul...@infidyne.com> wrote: >> Following symlinks in delete-file-recursively sounds like a recipe for >> disaster to me. > > Sorry, I should have been more explicit. The (delete-file-recursively > ...) in the patch *does* follow symlinks (implicitly, because > isDirectory() returns true for symlinks pointing to diretories).
Perhaps I should have been more explicit/clear :) Your paragraph above is what I understood from your original message. > The simplest "fix" (assuming everyone agrees it's a problem) is to > simply change it not to, but I suggested the option to allow for both > - but the default should presumably be to *not* follow symlinks. I think that treating symlinks to directories as directories when recursively deleting a directory tree is a bad idea. I am not sure allowing it as an option is a good idea either, but at least if it's not the default it's probably OK. i.e. I agree with you, but am also against allowing this behaviour as an option, unless others feel it's a good option to have. -- Michael Wood <esiot...@gmail.com> -- You received this message because you are subscribed to the Google Groups "Clojure" group. To post to this group, send email to clojure@googlegroups.com Note that posts from new members are moderated - please be patient with your first post. To unsubscribe from this group, send email to clojure+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/clojure?hl=en
