From: Heiko Schlittermann (HS12-RIPE) <hs@schlittermann.de>
Date: Sat, 5 Nov 2016 23:13:55 +0000 (+0100)
Subject: Do not die if the auto-update does not work
X-Git-Url: https://git.exim.org/buildfarm-client.git/commitdiff_plain/128475f6273172b482ec6d5142a7a51f00833c3f

Do not die if the auto-update does not work
---

diff --git a/run_branches b/run_branches
index bf377fb..d45024d 100755
--- a/run_branches
+++ b/run_branches
@@ -85,23 +85,26 @@ if (not exists $EximBuild::Conf{auto_update} or $EximBuild::Conf{auto_update})
     die "$0: auto-update not possible: need write permissions in @{[cwd]}\n"
 	if not -w '.';
 
+    # Get information about our remote and calculate the chance for a
+    # successfull auto-updat. Based on:
     # http://stackoverflow.com/questions/3258243/check-if-pull-needed-in-git
     system("git fetch $remote") == 0 or die $? >> 8;
     my ($upstream, $local, $base) = qx'git rev-parse ...@{upstream}';   die $? >> 8 if $?;
     $base =~ s/^\^//;
 
     if ($upstream ne $local) {
-	die "$0: the merge base is not local anymore. Refusing to `git pull`\n"
-	    if $base ne $local;
 
-	say "Doing git pull ..." if $verbose;
-
-	# if we're the merge base, the ff-only should workkkkkk
-	system 'git pull --ff-only' == 0 or die $? >> 8;
-	say "$0: re-execute after update";
-	chdir $CALLED{cwd} or die "$0: Can't chdir to $CALLED{cwd}: $!\n";
-	exec $CALLED{argv0}, @{$CALLED{argv}};
-	die "Can't re-exec\n";
+	if ($base ne $local) {
+	    warn "$0: the merge base is not local anymore. Refusing to `git pull`\n"
+	}
+	else {
+	    # if we're the merge base, the ff-only should work
+	    system 'git pull --ff-only' == 0 or die $? >> 8;
+	    say "$0: re-execute after update";
+	    chdir $CALLED{cwd} or die "$0: Can't chdir to $CALLED{cwd}: $!\n";
+	    exec $CALLED{argv0}, @{$CALLED{argv}};
+	    die "Can't re-exec\n";
+	}
     }
 }