Review Request: VCS: Extend the specification of checkout to encompass the DVCS case, renamed checkout() to createWorkingCopy()

[Fabian Wiesel]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/526/
-----------------------------------------------------------

Review request for KDevelop and Andreas Pakulat.


Summary
-------

As I see it, the current IBasicVersionControl::checkout() function is used to create an initial "checkout" from a remote repository.
In case for a DVCS, the closest operation is clone(), which additionally to creating a working copy, also entails the creation of a local copy of the remote repository.

For DVCS the term "checkout" maps to the operation the IBasicVersionControl::update().

In order to avoid confusion, I suggest renaming IBasicVersionControl::checkout() to createWorkingCopy().

Also, as the result IDistributedVersionControl::clone() fulfils all requirements one has on createWorkingCopy(), one can consider it 
a duplicate of createWorkingCopy() and therefore remove it.


Diffs
-----

  /trunk/KDE/kdevplatform/plugins/appwizard/appwizardplugin.cpp 949360 
  /trunk/KDE/kdevplatform/plugins/cvs/cvsplugin.h 949360 
  /trunk/KDE/kdevplatform/plugins/cvs/cvsplugin.cpp 949360 
  /trunk/KDE/kdevplatform/plugins/git/gitplugin.h 949360 
  /trunk/KDE/kdevplatform/plugins/git/gitplugin.cpp 949360 
  /trunk/KDE/kdevplatform/plugins/mercurial/mercurialplugin.h 949360 
  /trunk/KDE/kdevplatform/plugins/mercurial/mercurialplugin.cpp 949360 
  /trunk/KDE/kdevplatform/plugins/mercurial/tests/initTest.cpp 949360 
  /trunk/KDE/kdevplatform/plugins/subversion/kdevsvnplugin.h 949360 
  /trunk/KDE/kdevplatform/plugins/subversion/kdevsvnplugin.cpp 949360 
  /trunk/KDE/kdevplatform/shell/kross/krossdistributedversioncontrol.h 949360 
  /trunk/KDE/kdevplatform/shell/kross/krossdistributedversioncontrol.cpp 949360 
  /trunk/KDE/kdevplatform/vcs/dvcs/dvcsplugin.h 949360 
  /trunk/KDE/kdevplatform/vcs/dvcs/dvcsplugin.cpp 949360 
  /trunk/KDE/kdevplatform/vcs/interfaces/ibasicversioncontrol.h 949360 
  /trunk/KDE/kdevplatform/vcs/interfaces/idistributedversioncontrol.h 949360 
  /trunk/KDE/kdevplatform/vcs/tests/vcsBlackBoxTest.cpp 949360 

Diff: http://reviewboard.kde.org/r/526/diff


Testing
-------


Thanks,

Fabian