<div dir="ltr">Kevin,<div><br></div><div>Thanks for reviewing, I've created libkomparediff2 with all of libdiff2 and libdialogpages/settingsbase.* and diffsettings.* and kompare builds against it (on master branch only, 4.11 and previous still has libdiff2 inside kompare.)  I've merged my branch into kompare on git master branch.</div>
<div><br></div><div>Next I will get kdevelop using libkomparediff2 from kdesdk, including checking if there are any changes in the kdevplatform copy that we want in the main git repo.</div><div><br></div><div>thanks,</div>
<div>Jeremy<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Jul 20, 2013 at 4:25 PM, Kevin Kofler <span dir="ltr"><<a href="mailto:kevin.kofler@chello.at" target="_blank">kevin.kofler@chello.at</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jeremy,<br>
<div class="im"><br>
On Friday 19 July 2013 at 17:59:33, Jeremy Whiting wrote:<br>
> Thanks for the guidance.  I did decide the actions and such do belong<br>
> inside KompareModelList afterall, and found a way to make KomparePart get<br>
> the actions from the KompareModelList and add them to it's own<br>
> actionCollection.  I've pushed a branch to kompare called movelibdiff2 if<br>
> you want to take a look.  It works ok here, but I'll let you guys see what<br>
> you think of my solution.<br>
><br>
> My libdiff2 is at scratch/whiting/libdiff2 and builds by itself and<br>
> installs a LibKompareDiff2Config.cmake file that libdialogpages on my<br>
> branch uses to find LibKompareDiff2 variables to build against.<br>
<br>
</div>I looked at your changes, everything looks fine to me, so:<br>
<br>
Ship it!<br>
<span class="HOEnZb"><font color="#888888"><br>
        Kevin Kofler<br>
</font></span></blockquote></div><br></div></div></div>