From 37679efe82686b5fb1ba4d198c6df41a9e1fa52d Mon Sep 17 00:00:00 2001 From: Ian Jackson Date: Sat, 8 Jul 2017 21:11:52 +0100 Subject: [PATCH] dgit: clone multisuite works even without --no-rm-on-error. This is slightly subtle. We need $rmonerror in cmd_clone to be cleared. cmd_clone does that only when clone itself returns. The multisuite plumbing means that currently, clone returns only in the child which set up the tree and set up the first suite. Unsetting $rmonerror there is correct because we want to keep the tree on success, and the parent will remove it if there is a later failure. (Ie, the child remains responsible for removing the tree if it itself fails.) In the parent, we get $multi_fetched==1 if not only our original clone child succeeded, but all the other fetches worked too. We are now responsible for the tree. If our final tasks are successful, again, we need not to delete the tree. Closes:#867434. Signed-off-by: Ian Jackson --- debian/changelog | 2 ++ dgit | 5 ++++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/debian/changelog b/debian/changelog index 729985c3..686c69cc 100644 --- a/debian/changelog +++ b/debian/changelog @@ -15,6 +15,8 @@ dgit (3.11~) unstable; urgency=medium core.sharedRepository. Prompted by #867603. * dgit-badcommit-fixup: Honour core.sharedRepository. Closes:#867603. + * dgit: clone multisuite works even without --no-rm-on-error. + Closes:#867434. * infrastructure: Cope with new git-receive-pack which has quarantine feature: ie, work around #867702. * test suite: Cope with git restricting ext:: protocols. diff --git a/dgit b/dgit index be277f3e..40b8e32a 100755 --- a/dgit +++ b/dgit @@ -3612,6 +3612,9 @@ END } sub clone ($) { + # in multisuite, returns twice! + # once in parent after first suite fetched, + # and then again in child after everything is finished my ($dstdir) = @_; badusage "dry run makes no sense with clone" unless act_local(); @@ -3623,7 +3626,7 @@ sub clone ($) { printdebug "multi clone after fetch merge\n"; clone_set_head(); clone_finish($dstdir); - exit 0; + return; } printdebug "clone main body\n"; -- 2.30.2