D9001: Better handle of subjobs

2017-11-26 Thread Jaime Torres Amate
jtamate abandoned this revision. jtamate added a comment. Creating two review requests. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D9001 To: jtamate, #frameworks, dfaure Cc: ngraham, anthonyfieroni, ltoscano

D9001: Better handle of subjobs

2017-11-25 Thread Anthony Fieroni
anthonyfieroni added a comment. It should be two different patches, one in KIO and one in KCoreAddons. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D9001 To: jtamate, #frameworks, dfaure Cc: anthonyfieroni, ltoscano

D9001: Better handle of subjobs

2017-11-25 Thread David Faure
dfaure added a comment. 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 `arc diff` and create two review requests (unfortunately). Commit log: something like

D9001: Better handle of subjobs

2017-11-25 Thread Jaime Torres Amate
jtamate retitled this revision from "Don't remove a subjob if is not in the list of subjobs and dnoe emitResult if the job still has subjobs.." to "Better handle of subjobs". jtamate edited the summary of this revision. jtamate edited the test plan for this revision. REPOSITORY R241 KIO