dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land.
Ah, the free space check in CopyJob doesn't take into account that it's an overwrite? Heh, two bugs canceling each other, in a way ;-) In the situation I describe, there *is* space to do the copying, just not with a temporary .part file. So I didn't expect an error in that case, but rather directly overwriting without using .part. But OK that's a separate issue then, the check itself could be improved. REPOSITORY R241 KIO BRANCH arcpatch-D18904 (branched from master) REVISION DETAIL https://phabricator.kde.org/D18904 To: chinmoyr, dfaure, dmitrio, ngraham Cc: kde-frameworks-devel, ngraham, michaelh, bruns