From: Eric Wong <e@80x24.org>
Date: Mon, 23 Nov 2020 07:06:00 +0000 (+0000)
Subject: extsearchidx: do not short-circuit MiscIdx on no-op v2 prepare
X-Git-Tag: v1.7.0~1619
X-Git-Url: http://www.git.stargrave.org/?a=commitdiff_plain;h=616a08457175b442d4caeb67f9ccd9d3e69f50f5;p=public-inbox.git

extsearchidx: do not short-circuit MiscIdx on no-op v2 prepare

This was intended to make development easier; but also allows us
description, URL, and address changes to be picked up
independently of message history.
---

diff --git a/lib/PublicInbox/ExtSearchIdx.pm b/lib/PublicInbox/ExtSearchIdx.pm
index 890ac282..2cdc31cb 100644
--- a/lib/PublicInbox/ExtSearchIdx.pm
+++ b/lib/PublicInbox/ExtSearchIdx.pm
@@ -298,7 +298,7 @@ sub _sync_inbox ($$$) {
 		my $epoch_max;
 		defined($ibx->git_dir_latest(\$epoch_max)) or return;
 		$sync->{epoch_max} = $epoch_max;
-		sync_prepare($self, $sync) or return; # fills $sync->{todo}
+		sync_prepare($self, $sync); # or return # TODO: once MiscIdx is stable
 	} elsif ($v == 1) {
 		my $uv = $ibx->uidvalidity;
 		my $lc = $self->{oidx}->eidx_meta("lc-v1:$ekey//$uv");
@@ -309,8 +309,10 @@ sub _sync_inbox ($$$) {
 		warn "E: $ekey unsupported inbox version (v$v)\n";
 		return;
 	}
-	index_todo($self, $sync, $_) for @{delete($sync->{todo}) // []};
-	$self->{midx}->index_ibx($ibx);
+	unless ($sync->{quit}) {
+		index_todo($self, $sync, $_) for @{delete($sync->{todo}) // []};
+		$self->{midx}->index_ibx($ibx) unless $sync->{quit};
+	}
 	$ibx->git->cleanup; # done with this inbox, now
 }