<div class="gmail_quote">On Sat, May 1, 2010 at 6:44 PM, Andreas Pakulat <span dir="ltr"><<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On 01.05.10 17:24:52, Aleix Pol wrote:<br>
> As a first iteration I thought it would be a good improvement to provide VCS<br>
> checkout capability, for now I tried to add a first page for checking out in<br>
> case we want to import the project remotely, that's the UI VCS plugins are<br>
> providing:<br>
<br>
</div>I find Eclipse way of doing this very good, it allows you to setup a<br>
browser for a remote repository and then browse it and select any part of<br>
it to "checkout as project". What I don't like about this is that its<br>
hidden quite a bit (you need to add the cvs/svn/git repo browser view and<br>
then fill it), so having it available as widget and re-using it in a wizard<br>
to "Checkout project from CVS/Svn/Git/.." would be nice.<br>
<div class="im"><br>
> <a href="http://proli.net/meu/kdevelop/kdevelop_import_git.png" target="_blank">http://proli.net/meu/kdevelop/kdevelop_import_git.png</a><br>
> <a href="http://proli.net/meu/kdevelop/kdevelop_import_svn1.png" target="_blank">http://proli.net/meu/kdevelop/kdevelop_import_svn1.png</a><br>
><br>
> I guess that here the problem is that since it's some interface that wasn't<br>
> used we're just getting untested stuff. (Note that's what we get when we<br>
> call IBasicVersionControl::createImportMetadataWidget). That unveils me some<br>
> doubts on the API which I hope that someone (named apaku? :D).<br>
<br>
</div>This is actually an import wizard, not a checkout-wizard. Import means that<br>
some directory+files are being _imported into a VCS_ at some location.<br>
Thats why its part of the new-project-wizard, it automatically imports the<br>
generated project into a VCS and then automatically checks out from the VCS<br>
into the location that you specified. This is not intended to be used to<br>
checkout an existing project from a VCS and thats something not only you<br>
got wrong. However I don't think there's any way to "fix" that<br>
misconception, except by providing a way to checkout-project-from-vcs<br>
(probably as part of the open/import wizard) and documenting both ways<br>
properly in a documentation-wiki and showing the differences there.<br>
<br>
Andreas<br>
<br>
--<br>
Celebrate Hannibal Day this year.  Take an elephant to lunch.<br>
<font color="#888888"><br>
--<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
</font></blockquote></div><br>Oh I see, so we need a new interface, don't we?<br><br>We could consider this interface to be a VcsLocation creator, maybe we can reuse that somewhere else later. This interface could have a navigation view too, if the plugin provides it we're going to get it. (I don't really think it's necessary to abstract the operations to do so because it's KIO in the end).<br>
A base implementation with just a KUrlRequester would be good too I guess.<br><br>Sounds good?<br><br>Aleix<br><br><br>