geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dain Sundstrom <d...@coredevelopers.net>
Subject Re: cvs commit: incubator-geronimo/sandbox/webdav/src/test/org/apache/geronimo/datastore/impl/remote RemoteUseCaseTest.java
Date Sun, 29 Feb 2004 18:36:29 GMT
On Feb 29, 2004, at 11:14 AM, Jacek Laskowski wrote:

> gdamour@apache.org wrote:
>> gdamour     2004/02/29 05:14:11
> ...
>>   1.1                   
>> incubator-geronimo/sandbox/webdav/src/test/org/apache/geronimo/ 
>> datastore/Util.java
> ...
>>   package org.apache.geronimo.datastore;
>>     import java.io.File;
>>   import java.io.IOException;
>>     /**
>>    *
>>    * @version $Revision: 1.1 $ $Date: 2004/02/29 13:14:11 $
>>    */
>>   public class Util {
>>             public static void recursiveDelete(File aRoot) throws  
>> IOException {
>
> The method already exists in  
> ./modules/deployment/src/java/org/apache/geronimo/deployment/util/ 
> FileUtil.java.
>
> That rises a very interesting question - where do we put  
> methods/classes that are common to several modules?

I'd vote no.  It just leads to a highly coupled system, and our commons  
module depends on to much other stuff.  If we were to get our commons  
module stand-alone, then I might change my mind.

-dain


Mime
View raw message