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).

- Brett

Reply via email to