[Mono-bugs] [Bug 53290][Min] Changed - Move does not support directory as target

bugzilla-daemon@bugzilla.ximian.com bugzilla-daemon@bugzilla.ximian.com
Mon, 2 Feb 2004 08:07:16 -0500 (EST)

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by iain@mccoy.id.au.


--- shadow/53290	2004-02-02 08:06:29.000000000 -0500
+++ shadow/53290.tmp.19066	2004-02-02 08:07:16.000000000 -0500
@@ -10,13 +10,12 @@
 Component: CORLIB
 AssignedTo: mono-bugs@ximian.com                            
 ReportedBy: dave-mono@earth.li               
 QAContact: mono-bugs@ximian.com
 TargetMilestone: ---
 Summary: Move does not support directory as target
 Description of Problem:
 FileStream with FileMode.CreateNew does not object if the file already exists
 It should throw an IOException telling you that the file already exists.
@@ -59,6 +58,20 @@
 argument checking, then creates a new FileStream with FileMode.Create
 ------- Additional Comments From iain@mccoy.id.au  2004-02-02 08:06 -------
 Created an attachment (id=6587)
 put the file in the directory specified rather than throw an exception
+------- Additional Comments From iain@mccoy.id.au  2004-02-02 08:07 -------
+Look at me! I can make a fool of myself! :(
+The I did a few minutes ago is for bug 53236 - I'll put it in there
+shortly. Sorry about the stuffup.
+Anyway, I'm now attaching a patch that I think fixes this one. It
+changes File.Move so that instead of throwing an exception when the
+target is a directory, it changes the target to be the source file's
+name within the target directory.
+Why I explain what a one-line patch does is beyond me, but that's what
+it does.