about summary refs log tree commit homepage
path: root/lib/PublicInbox/V2Writable.pm
diff options
context:
space:
mode:
authorEric Wong (Contractor, The Linux Foundation) <e@80x24.org>2019-06-10 21:43:52 +0000
committerEric Wong (Contractor, The Linux Foundation) <e@80x24.org>2019-06-10 21:43:52 +0000
commit8388a137ac3b13edaa361a7fb6c85fea3c409ea8 (patch)
tree04ca1e2c6ff4fcc0df553965de0ae94d2abfa6f0 /lib/PublicInbox/V2Writable.pm
parentd377a9d2f972f85edf759a6c152ca49078baba38 (diff)
downloadpublic-inbox-8388a137ac3b13edaa361a7fb6c85fea3c409ea8.tar.gz
Xapian on Linux <3.15 has trouble with coprocesses since it used
fork() for locking and would hold onto pipes used for git
unnecessarily.
Diffstat (limited to 'lib/PublicInbox/V2Writable.pm')
-rw-r--r--lib/PublicInbox/V2Writable.pm3
1 files changed, 3 insertions, 0 deletions
diff --git a/lib/PublicInbox/V2Writable.pm b/lib/PublicInbox/V2Writable.pm
index 3484807b..09ed4e7b 100644
--- a/lib/PublicInbox/V2Writable.pm
+++ b/lib/PublicInbox/V2Writable.pm
@@ -533,6 +533,9 @@ W: $list
         my ($oid, $type, $len) = $self->{-inbox}->git->check($expect_oid);
         $oid eq $expect_oid or die "BUG: $expect_oid not found after replace";
 
+        # don't leak FDs to Xapian:
+        $self->{-inbox}->git->cleanup;
+
         # reindex modified messages:
         for my $smsg (@$need_reindex) {
                 my $num = $smsg->{num};