admin: improve warnings and errors for missing modules

Since we lazy-load Xapian now, some errors may become
more cryptic or buried.  Try to improve that by making
Admin show better errors.

1 files changed, 41 insertions, 12 deletions

diff --git a/script/public-inbox-purge b/script/public-inbox-purge
index 264bcdef..381826dc 100755
--- a/script/public-inbox-purge
+++ b/script/public-inbox-purge
@@ -7,16 +7,16 @@
 use strict;
 use warnings;
 use Getopt::Long qw(:config gnu_getopt no_ignore_case auto_abbrev);
-use PublicInbox::Config;
-use PublicInbox::MIME;
 use PublicInbox::Admin qw(resolve_repo_dir);
-use PublicInbox::Filter::Base;
-*REJECT = *PublicInbox::Filter::Base::REJECT;
+PublicInbox::Admin::check_require('-index');
+require PublicInbox::Filter::Base;
+require PublicInbox::Config;
+require PublicInbox::MIME;
+require PublicInbox::V2Writable;
 
-my $usage = "$0 [--all] [INBOX_DIRS] </path/to/message";
+{ no warnings 'once'; *REJECT = *PublicInbox::Filter::Base::REJECT }
 
-eval { require PublicInbox::V2Writable } or die
-        "DBI, DBD::SQLite and Search::Xapian required for purge\n";
+my $usage = "$0 [--all] [INBOX_DIRS] </path/to/message";
 my $config = eval { PublicInbox::Config->new };
 my $cfgfile = PublicInbox::Config::default_file();
 my ($all, $force);
@@ -63,11 +63,6 @@ if ($all) {
                                 mainrepo => $dir,
                         });
                 };
-
-                # somebody could "rm -r" all the Xapian directories;
-                # let them purge the overview, at least
-                $ibx->{indexlevel} ||= 'basic' unless $ibx->search;
-
                 push @inboxes, $ibx;
         }
 
@@ -77,6 +72,40 @@ if ($all) {
         }
 }
 
+foreach my $ibx (@inboxes) {
+        my $lvl = $ibx->{indexlevel};
+        if (defined $lvl) {
+                PublicInbox::Admin::indexlevel_ok_or_die($lvl);
+                next;
+        }
+
+        # Undefined indexlevel, so `full'...
+        # Search::Xapian exists and the DB can be read, at least, fine
+        $ibx->search and next;
+
+        # it's possible for a Xapian directory to exist, but Search::Xapian
+        # to go missing/broken.  Make sure it's purged in that case:
+        $ibx->over or die "no over.sqlite3 in $ibx->{mainrepo}\n";
+
+        # $ibx->{search} is populated by $ibx->over call
+        my $xdir_ro = $ibx->{search}->xdir(1);
+        my $npart = 0;
+        foreach my $part (<$xdir_ro/*>) {
+                if (-d $part && $part =~ m!/\d+\z!) {
+                        my $bytes = 0;
+                        $bytes += -s $_ foreach glob("$part/*");
+                        $npart++ if $bytes;
+                }
+        }
+        if ($npart) {
+                PublicInbox::Admin::require_or_die('-search');
+        } else {
+                # somebody could "rm -r" all the Xapian directories;
+                # let them purge the overview, at least
+                $ibx->{indexlevel} ||= 'basic';
+        }
+}
+
 my $data = do { local $/; scalar <STDIN> };
 $data =~ s/\A[\r\n]*From [^\r\n]*\r?\n//s;
 my $n_purged = 0;