You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avalon.apache.org by Paul Balmee <pb...@whoever.com> on 2002/02/23 06:54:12 UTC

Bug in org.apache.avalon.excalibur.io.FileUtil??

Should lines 384 and 432 be changed from:

            destination.mkdirs();

to

            destination.getParentFile().mkdirs();


If so, here's the patch:


--- FileUtil.java.orig  Fri Feb 22 21:52:02 2002
+++ FileUtil.java       Fri Feb 22 21:52:17 2002
@@ -381,7 +381,7 @@
         if( destination.getParentFile() != null &&
             !destination.getParentFile().exists() )
         {
-            destination.mkdirs();
+            destination.getParentFile().mkdirs();
         }

         //make sure we can write to destination
@@ -429,7 +429,7 @@
         if( destination.getParentFile() != null &&
             !destination.getParentFile().exists() )
         {
-            destination.mkdirs();
+            destination.getParentFile().mkdirs();
         }

         //make sure we can write to destination
-- 

_______________________________________________
Sign-up for your own FREE Personalized E-mail at Mail.com
http://www.mail.com/?sr=signup


1 cent a minute calls anywhere in the U.S.!

http://www.getpennytalk.com/cgi-bin/adforward.cgi?p_key=RG9853KJ&url=http://www.getpennytalk.com



--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>


Re: Bug in org.apache.avalon.excalibur.io.FileUtil??

Posted by Peter Donald <pe...@apache.org>.
Your right. I applied the pathc - thanks.

On Sat, 23 Feb 2002 16:54, Paul Balmee wrote:
> Should lines 384 and 432 be changed from:
>
>             destination.mkdirs();
>
> to
>
>             destination.getParentFile().mkdirs();
>
>
> If so, here's the patch:
>
>
> --- FileUtil.java.orig  Fri Feb 22 21:52:02 2002
> +++ FileUtil.java       Fri Feb 22 21:52:17 2002
> @@ -381,7 +381,7 @@
>          if( destination.getParentFile() != null &&
>              !destination.getParentFile().exists() )
>          {
> -            destination.mkdirs();
> +            destination.getParentFile().mkdirs();
>          }
>
>          //make sure we can write to destination
> @@ -429,7 +429,7 @@
>          if( destination.getParentFile() != null &&
>              !destination.getParentFile().exists() )
>          {
> -            destination.mkdirs();
> +            destination.getParentFile().mkdirs();
>          }
>
>          //make sure we can write to destination

-- 
Cheers,

Pete

-------------------------------------------------
"Sometimes its better to keep your mouth shut and 
let people think your an idiot, than to open it 
and remove all doubt." 
-------------------------------------------------

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>