Re: [ ruby-Bugs-1917 ] Pathname#unlink fails on a symlink which points to a directory.

In article <[e-mail]>,
  [e-mail] writes:

> Summary: Pathname#unlink fails on a symlink which points to a directory.
>
> Initial Comment:
> When you try to unlink a link to a directory Pathname#unlink fails like that:
>
> irb(main):001:0> Pathname.new('/tmp/bar').unlink
> Errno::ENOTDIR: Not a directory - /tmp/bar

I'll fix it as follows.

Index: lib/pathname.rb
===================================================================
RCS file: /src/ruby/lib/pathname.rb,v
retrieving revision 1.38
diff -u -p -r1.38 pathname.rb
--- lib/pathname.rb	12 Dec 2004 16:02:08 -0000	1.38
+++ lib/pathname.rb	15 May 2005 05:49:22 -0000
@@ -868,10 +868,10 @@ class Pathname    # * mixed *
   # Removes a file or directory, using <tt>File.unlink</tt> or
   # <tt>Dir.unlink</tt> as necessary.
   def unlink()
-    if FileTest.directory? @path
-      Dir.unlink @path
-    else
+    begin
       File.unlink @path
+    rescue Errno::EISDIR
+      Dir.unlink @path
     end
   end
   alias delete unlink
-- 
Tanaka Akira