<div dir="ltr">Kevin,<div><br></div><div>Thanks for the guidance.  I did decide the actions and such do belong inside KompareModelList afterall, and found a way to make KomparePart get the actions from the KompareModelList and add them to it's own actionCollection.  I've pushed a branch to kompare called movelibdiff2 if you want to take a look.  It works ok here, but I'll let you guys see what you think of my solution.</div>
<div><br></div><div>My libdiff2 is at scratch/whiting/libdiff2 and builds by itself and installs a LibKompareDiff2Config.cmake file that libdialogpages on my branch uses to find LibKompareDiff2 variables to build against.</div>
<div><br></div><div>thanks,</div><div>Jeremy</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 18, 2013 at 5:05 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"><div class="im">On Thursday 18 July 2013 at 14:04:52, Jeremy Whiting wrote:<br>
> One other question, do we want the library to be called libdiff2 as the<br>
> directory is named? or libkomparediff2 as the .so is named?  I'd like to<br>
> make them match, so I'd change one or the other to make them match.<br>
<br>
</div>I'd go with libkomparediff2. (libdiff2 is too generic, it'd make sense if this<br>
were a really general diff library without any KDE dependencies and used by<br>
many diffing apps, but that's not what this library is.)<br>
<br>
(As for your other question, I'm going to defer the answer until we know<br>
whether option A is even implementable, as per your followup. If not, there's<br>
no decision to be made there. ;-) )<br>
<span class="HOEnZb"><font color="#888888"><br>
        Kevin Kofler<br>
</font></span></blockquote></div><br></div>