<table><tr><td style="">dfaure 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/D9001" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Phabricator usage: you can't upload one patch for two git repos into a single change request. This makes the "repo" field wrong, and it makes it impossible to get context. Use <tt style="background: #ebebeb; font-size: 13px;">arc diff</tt> and create two review requests (unfortunately).</p>

<p>Commit log: something like "fix crash during file copy after message box warning" would already be better than either just a bug number (v1) or a very generic sentence (v2).</p>

<p>The bug looks like a nested event loop issue, which is always nasty. I don't have time right now for full debugging, but if you could paste a backtrace of where a messagebox is shown that would be helpful. I suppose it all comes from the slave calling warning? Warnings have never been really handled well... Maybe we should collect warnings and show them at the end of the job? This might help with these issues...</p>

<p>KCompositeJob patch: that one looks ok, it reads like basic input validation, even though I would have never thought this could happen (again, nested event loops is most certainly the reason...)</p>

<p>KIO patch: If there are still any subjobs running, we should kill them rather than patiently wait for them to finish (it's pointless, if the whole job is being aborted). Which subjobs are still there, in fact? This code already kills the put job if the get job has an error, and vice-versa. Are there more jobs that should be killed in that code?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9001" rel="noreferrer">https://phabricator.kde.org/D9001</a></div></div><br /><div><strong>To: </strong>jtamate, Frameworks, dfaure<br /><strong>Cc: </strong>ltoscano<br /></div>