<table><tr><td style="">hallas 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/D19170">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/D19170#418234" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D19170#418234</a>, <a href="https://phabricator.kde.org/p/dfaure/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@dfaure</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I think it's an unwanted fact that the chmod job runs in parallel with the del job.<br />
 A "return" after creating the chmod job would fix all this.</p>

<p>But of course it should be possible to have two concurrent subjobs (one on dest, one on src), this whole issue just makes me realize that it makes the error handling more tricky ;-)</p>

<p>If we want to keep these two running in parallel, we need to fix FileCopyJob to kill the running subjob when emitting the error from the other one.<br />
 Would this fix it?<br />
 <a href="http://www.davidfaure.fr/2019/filecopyjob.cpp.diff" class="remarkup-link" target="_blank" rel="noreferrer">http://www.davidfaure.fr/2019/filecopyjob.cpp.diff</a></p></div>
</blockquote>

<p>Hi <a href="https://phabricator.kde.org/p/dfaure/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@dfaure</a>  I was actually thinking of the same fix ;)</p>

<p>I have updated the diff with your suggested fix, but I have also added the "reverse" case, so if the chmod reports an error and we have a del job running, kill the del job. If the del job reports an error and we have a chmod job running, kill the chmod job. This pattern of error handling is actually the same as with the getjob/putjob. I have tested the fix and can see that it solved the problem!</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/D19170">https://phabricator.kde.org/D19170</a></div></div><br /><div><strong>To: </strong>hallas, Frameworks, elvisangelaccio, dfaure<br /><strong>Cc: </strong>cfeck, dhaumann, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>