From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Fri, 31 Mar 2017 15:07:17 +0000 (+0100)
Subject: git-debrebase: wip, found
X-Git-Tag: archive/debian/5.0~371
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=dgit.git;a=commitdiff_plain;h=2edca8d928612e96bb1935ff8b7a5d4ba92f2dfc;hp=536e29d7d60fb9eda56203d3472b2b45d387bd1c

git-debrebase: wip, found
---

diff --git a/git-debrebase b/git-debrebase
index ad26d909..2a3804f3 100755
--- a/git-debrebase
+++ b/git-debrebase
@@ -286,14 +286,23 @@ sub classify ($) {
     return $unknown->("complex merge");
 }
 
-sub launder ($$$) {
-    my ($input, $pseudos_must_overwrite_this, $wantdebonly) = @_;
+sub launder ($$$;$$$) {
+    my ($input, $pseudos_must_overwrite_this, $wantdebonly,
+	$report, $depth, $report_anomaly, $nogenerate) = @_;
     # go through commits backwards
     # we generate two lists of commits to apply
+    # => ($tip, $breakwater_tip)
     my (@deb_cl, @ups_cl, @processed);
     my %found;
     my @pseudomerges;
 
+    $report //= sub { };
+    $report_anomaly //= sub {
+	my ($cl, $msg) = @_;
+	die "commit $cl: $msg\n";
+    };
+    $depth //= 0;
+
     my $cl;
     my $xmsg = sub {
 	my ($appendinfo) = @_;
@@ -312,6 +321,7 @@ sub launder ($$$) {
 	$cl = classify $cur;
 	my $ty = $cl->{Type};
 	my $st = $cl->{SubType};
+	$report->($cl);
 	$found{$ty. ( defined($st) ? "-$st" : '' )}++;
 	my $p0 = $cl->{Parents}[0]{CommitId};
 	if ($ty eq 'AddPatches') {
@@ -339,14 +349,15 @@ sub launder ($$$) {
 	    $rewrite_from_here->();
 	    next;
 	} elsif ($ty eq 'Pseudomerge') {
-	    if (defined $pseudos_must_overwrite_this) {
-		grep {
+	    if (defined $pseudos_must_overwrite_this &&
+		!grep {
 		    is_fast_fwd $pseudos_must_overwrite_this, $_->{CommitId}
 		},
-		    @{ $cl->{Overwritten} }
-		or
-		    die "Pseudomerge $cl->{CommitId} does not overwrite ".
-		        "$pseudos_must_overwrite_this as expected.";
+		@{ $cl->{Overwritten} }) {
+		$report_anomaly->($cl,
+				  "Pseudomerge should overwrite".
+				  " $pseudos_must_overwrite_this".
+				  " but does not do so");
 	    }
 	    push @pseudomerges, $cl;
 	    $rewrite_from_here->();
@@ -380,15 +391,18 @@ sub launder ($$$) {
                 $xmsg->("convert dgit import: upstream changes")
 	    };
 	    my $differs = get_differs $previous_breakwater, $cl->{Tree};
-	    $basis = launder $pseudomerges[0]{Overwritten}, undef, 1;
+	    $basis = launder $pseudomerges[0]{Overwritten}, undef, 1,
+		$report, $depth+1, $nogenerate;
 	    $rewrite_from_here->();
 	    last;
 	} else {
-	    die "Reached difficult commit $cur: ".Dumper($cl);
+	    $report_anomaly->($cl, "Cannot cope with this commit");
 	}
     }
     # Now we build it back up again
 
+    
+
     workarea_fresh();
 
     my $rewriting = 1;
@@ -484,6 +498,8 @@ sub update_head ($$) {
     runcmd @git, qw(update-ref -m), "git-debrebase $mrest", $new, $old;
 }
 
+sub cmd_analyse () {
+
 sub cmd_launder () {
     my $old = get_head();
     my ($tip,$breakwater) = launder $old, 0, undef, 0;