[EMAIL PROTECTED] writes:

>+    /** the name of the subversion metadata directory */
>+    private static final String SVN_DIRNAME = ".svn";
>+    
>     /////////////////////////////////////////////////////////////////////////
>     // Avalon Service Lifecycle Implementation
>     /////////////////////////////////////////////////////////////////////////
>@@ -457,7 +460,9 @@
> 
>             for( int i=0; i<list.length; i++ )
>             {
>-                if( list[i].isDirectory() )
>+                // skip the subversion directories otherwise we pick up 
>unexpected files 
>+                
>+                if( list[i].isDirectory() && 
>(list[i].getName().equalsIgnoreCase(SVN_DIRNAME) == false) )

Does this make sense? As we can be quite sure that noone will ever
repeat the "CVS" mistake, why not use

list[i].isHidden() 

or (if this breaks on Windows) list[i].getName().charAt(0) == '.'

So we will not stumble over it when the next meta directory arrives.

        Regards
                Henning


-- 
Dipl.-Inf. (Univ.) Henning P. Schmiedehausen          INTERMETA GmbH
[EMAIL PROTECTED]        +49 9131 50 654 0   http://www.intermeta.de/

RedHat Certified Engineer -- Jakarta Turbine Development  -- hero for hire
   Linux, Java, perl, Solaris -- Consulting, Training, Development

                      4 - 8 - 15 - 16 - 23 - 42

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to