FYI Perforce will error on adding files already under source control, rejecting the add request.
-----Original Message----- From: Emmanuel Venisse [mailto:[EMAIL PROTECTED] Sent: Wednesday, April 26, 2006 3:58 AM To: scm-dev@maven.apache.org Subject: Re: Adding a directory recursively Brett Porter a écrit : > Emmanuel Venisse wrote: > >> >> Brett Porter a écrit : >> >>> Emmanuel Venisse wrote: >>> >>>> I think it would be good to add this method and it's easy to do. >>>> For svn, we must check if : >>>> - .svn directory exist in directory for subversion >>>> - CVS directory exist in directory for CVS >>>> - .bzr directory exist in directory for bazaar >>>> - nothing to do for file provider >>>> - don't know for clearcase, perforce and starteam >>> >>> >>> >>> Is that really necessary? Shouldn't it error out if you try to add >>> something that is already added? >> >> >> If you don't have .svn, CVS... directories, they aren't already added. >> >> Emmanuel >> > > Exactly - maybe I misunderstood but what I meant was that you don't > need to check for it - if you do an addDirectory, you attempt to add > the whole directory (and if something is already added, with .svn, it > errors, but if none of them have .svn it is all happy). > If I understand correctly what you mean, we call addDirectory for all parents directory and we catch error for all already added directory, right? svn doesn't fail when you try to add a directory already added, it just send a warning. Emmanuel
