Re: Re: cvs: php-src /pear System.php

On 16 Sep 2003 at 16:13, Tomas V.V.Cox wrote: > On Tuesday, September 16, 2003 12:03, Jan Schneider wrote: > > > Tomas V.V.Cox wrote: > >> cox Mon Sep 15 16:17:34 2003 EDT > >> > >> Modified files: > >> /php-src/pear System.php > >> Log: > >> * Implemented the find command (currently supported args: > >> -type f, -type d, -maxdepth, -name) > > > Just a dumb question: why don't you extend File_Find? > > Because they belong to different packages and System is shipped within > the PEAR package where we try to get as less dependecies as we can. > > Btw, in the case that 2 packages share some functionality is better to > duplicate the code than require a whole package. Well, if one is part of the core pear the programmer of File_Find could base (some?) of his code on your base-package which already provides the necessary functions, right? But maybe I'm wrong - have never used File_Find up to now and so I don't know about it's API / possibilities. Just a thought ... Stefan