<table><tr><td style="">ltoscano added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D15524">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D15524#330685" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D15524#330685</a>, <a href="https://phabricator.kde.org/p/andrewgr/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@andrewgr</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Dear Luigi,<br />
     I should inform you that I've been in touch with Andreas Sturnlechner (you were copied in) - who has pointed out that there is a KF5 version already. It's a pity I did not discover this earlier - but I am new to a number of the software tools, including GitHub.</p>

<p> <a href="https://github.com/kmorwinski/cirkuit/tree/frameworks" class="remarkup-link" target="_blank" rel="noreferrer">https://github.com/kmorwinski/cirkuit/tree/frameworks</a></p></div>
</blockquote>

<p>Yep, I'm aware of that, <span class="phabricator-remarkup-mention-unknown">@asturm</span> kindly told me about this and copied me in the answer.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>   A significant difference between the versions is that the "frameworks" kmorwinski version uses KDELibs4Support, whereas the version that I produced does not. I'm sure that the authors of the "frameworks" version are more experienced than me and will have written better code. I had lot of learning to do as I had not used KF5 or cmake before. On the other hand, I am a scientific user, and I have made a small number of changes that in my view make it work better.</p></blockquote>

<p>That's a minor issue: as you did a complete port to KF5, you are certainly in position of applying the missing bits to the existing codebase to remove the usage of KDELibs4Support.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>     When I am back from holiday I can put a little effort into comparing versions (the Meld program is very helpful for comparing source files). A general statement of "release focus" and an overview of code changes would be straightforward - but presenting changes atomically (as Andreas suggested) so that they can be reviewed individually would be rather onerous .. especially if intermediate versions that can be compiled are needed as "frameworks" uses KDELibs4Support.</p></blockquote>

<p>It may be onerous, but that's a general rule: big code dump are not easy to review and makes life more complicated when tracing back the history to understand what happened. Moreover, this specific repository is already partially ported, so the missing changes (removing KDELibs4Support, adding new features) will need to be for sure smaller.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>    I have been a frequent user of cirkuit (diagrams prepared with cirkuit are in several of my papers) and would like to help - but I think some thought into the best strategy is needed. We have to decide whether it's worth trying to merge the two versions - or just stick to "frameworks" if it works well. Either way, better signposting to a KF5 version of cirkuit is needed.</p></blockquote>

<p>A Frameworks version is needed, and that's the reason why other people invested time on the frameworks branch, and I see no contingent reason to throw away the existing work.</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15524">https://phabricator.kde.org/D15524</a></div></div><br /><div><strong>To: </strong>andrewgr, ltoscano<br /><strong>Cc: </strong>ngraham, ltoscano, kde-doc-english, skadinna<br /></div>