<div><div dir="auto">Martin,</div><div dir="auto"><br></div><div dir="auto">My patch fixed one bug (and probably created the others as you found out), this patch reverts mine, but does —new-tab works if konsole is set to multiple instances? </div><div dir="auto"><br></div><div dir="auto">I don’t mind at all the revert of my commit, I just want something that works correctly - I understand that mine doesn’t. </div></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 19 Jan 2020 at 11:11 Martin Tobias Holmedahl Sandsmark <<a href="mailto:noreply@phabricator.kde.org">noreply@phabricator.kde.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)"><table><tbody><tr><td>sandsmark created this revision.<br>sandsmark added reviewers: hindenburg, tcanabrava.<br>Herald added a project: Konsole.<br>Herald edited subscribers, added: konsole-devel; removed: Konsole.<br>sandsmark requested review of this revision.
</td><td><a style="text-decoration:none;padding:4px 8px;margin:0px 8px 8px;float:right;font-weight:bold;border-top-left-radius:3px;border-top-right-radius:3px;border-bottom-right-radius:3px;border-bottom-left-radius:3px;background-image:linear-gradient(rgb(255,255,255),rgb(241,240,241));display:inline-block;border:1px solid rgba(71,87,120,0.2);background-color:rgb(247,247,249);color:rgb(70,76,92)" href="https://phabricator.kde.org/D26761" target="_blank">View Revision</a></td></tr></tbody></table><br><div><strong>REVISION SUMMARY</strong><div><p><a href="https://phabricator.kde.org/R319:59ff5d37301adf4753b590f42787055e6695b614" style="border-top-left-radius:3px;border-top-right-radius:3px;border-bottom-right-radius:3px;border-bottom-left-radius:3px;padding:0px 4px;font-weight:bold;text-decoration:none;background-color:rgb(231,231,231);border-color:rgb(231,231,231);color:black" target="_blank">59ff5d37301adf4753b590f42787055e6695b614</a> completely ignored<br>
shouldUseNewProcess() which checks if we actually can re-use an existing<br>
process. This also broke --nofork, --separate etc.</p>

<p>See the comments in shouldUseNewProcess() for why and when Konsole<br>
actually needs to use a new process.</p>

<p>It also bypassed a crashfix because of the broken behavior of<br>
KDBusAddons because it didn't set the needToDeleteQApplication variable<br>
even though it forced reusing an existing an existing process.</p>

<p>See <a href="https://bugs.kde.org/show_bug.cgi?id=355545" rel="noreferrer" target="_blank">https://bugs.kde.org/show_bug.cgi?id=355545</a></p></div></div><br><div><strong>TEST PLAN</strong><div><p>Konsole now reuses a process if possible and wanted by the user, otherwise forces a new process.</p></div></div><br><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26761" target="_blank">https://phabricator.kde.org/D26761</a></div></div><br><div><strong>AFFECTED FILES</strong><div><div>src/main.cpp</div></div></div><br><div><strong>To: </strong>sandsmark, hindenburg, tcanabrava<br><strong>Cc: </strong>tcanabrava, hindenburg, konsole-devel, cent, EspadaV8, fbampaloukas, thsurrel, ngraham, maximilianocuria<br></div></blockquote></div></div>