<div class="gmail_quote">On Sun, May 2, 2010 at 7:48 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5">On 02.05.10 01:24:47, Aleix Pol wrote:<br>
> On Sat, May 1, 2010 at 6:44 PM, Andreas Pakulat <<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>> wrote:<br>
><br>
> > 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<br>
> > VCS<br>
> > > checkout capability, for now I tried to add a first page for checking out<br>
> > in<br>
> > > case we want to import the project remotely, that's the UI VCS plugins<br>
> > are<br>
> > > providing:<br>
> ><br>
> > 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>
> ><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<br>
> > 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<br>
> > some<br>
> > > doubts on the API which I hope that someone (named apaku? :D).<br>
> ><br>
> > 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>
</div></div><div class="im">> Oh I see, so we need a new interface, don't we?<br>
<br>
</div>Rather a new method on an existing interface IMHO to get a similar<br>
widget (i.e sharing some of the code) that allows to select a repository<br>
location and specify a place on disk where to put the checkout (or<br>
clone). Thats if you want to have some kind of wizard for checking out a<br>
project from a vcs, which will certainly be easier than implementing a<br>
repository browser (as I don't think any of the vcs plugins implement<br>
the repo-browser interface currently).<br>
<div class="im"><br>
> We could consider this interface to be a VcsLocation creator, maybe we can<br>
> reuse that somewhere else later. This interface could have a navigation view<br>
> too, if the plugin provides it we're going to get it. (I don't really think<br>
> it's necessary to abstract the operations to do so because it's KIO in the<br>
> end).<br>
<br>
</div>Uhm, KIO?? I don't understand how kio gets into all this.<br>
<br>
Andreas<br>
<br>
--<br>
Try to relax and enjoy the crisis.<br>
                -- Ashleigh Brilliant<br>
<font color="#888888"><br>
--<br>
</font><div><div></div><div class="h5">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>
</div></div></blockquote></div><br><div>Sorry, by interface I meant some new method "LocationWidget* IBasicVersionControl::locationWidget();" where:</div><div><br></div><div>class LocationWidget: public QWidget {</div>
<div> //...</div><div> virtual VcsLocation location() const=0;</div><div>};</div><div><br></div><div>About the KIO thing I meant that I don't think we want to create some structure to make it possible to create some interface.</div>
<div><br></div><div>Hope it's more clear.</div><div>I think that here the best will be that I start to create some stuff so that we can check later if it's what we want.</div><div><br></div><div>Aleix</div>