]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/LeiExternal.pm
pop3: reduce memory use while generating the mailbox cache
[public-inbox.git] / lib / PublicInbox / LeiExternal.pm
index 35a7d68a17b5bc0bab41c267e1c385a1283d5782..30bb1a4579c74d8245d1eec7b66b9687bd526e71 100644 (file)
@@ -50,7 +50,7 @@ my %re_map = ( '*' => '[^/]*?', '?' => '[^/]',
                '[' => '[', ']' => ']', ',' => ',' );
 
 sub glob2re {
-       my $re = $_[-1];
+       my $re = $_[-1]; # $_[0] may be $lei
        my $p = '';
        my $in_bracket = 0;
        my $qm = 0;
@@ -101,88 +101,20 @@ sub get_externals {
                return (ext_canonicalize($loc));
        }
        if (scalar(@m) == 0) {
-               $self->fail("`$loc' is unknown");
+               die "`$loc' is unknown\n";
        } else {
-               $self->fail("`$loc' is ambiguous:\n", map { "\t$_\n" } @m);
+               die("`$loc' is ambiguous:\n", map { "\t$_\n" } @m, "\n");
        }
-       ();
 }
 
-# TODO: does this need JSON output?
-sub lei_ls_external {
-       my ($self, $filter) = @_;
-       my $opt = $self->{opt};
-       my $do_glob = !$opt->{globoff}; # glob by default
-       my ($OFS, $ORS) = $opt->{z} ? ("\0", "\0\0") : (" ", "\n");
-       $filter //= '*';
-       my $re = $do_glob ? glob2re($filter) : undef;
-       $re //= index($filter, '/') < 0 ?
-                       qr!/\Q$filter\E/?\z! : # exact basename match
-                       qr/\Q$filter\E/; # grep -F semantics
-       my @ext = externals_each($self, my $boost = {});
-       @ext = $opt->{'invert-match'} ? grep(!/$re/, @ext)
-                                       : grep(/$re/, @ext);
-       if ($opt->{'local'} && !$opt->{remote}) {
-               @ext = grep(!m!\A[a-z\+]+://!, @ext);
-       } elsif ($opt->{remote} && !$opt->{'local'}) {
-               @ext = grep(m!\A[a-z\+]+://!, @ext);
-       }
-       for my $loc (@ext) {
-               $self->out($loc, $OFS, 'boost=', $boost->{$loc}, $ORS);
-       }
-}
-
-sub add_external_finish {
-       my ($self, $location) = @_;
-       my $cfg = $self->_lei_cfg(1);
-       my $new_boost = $self->{opt}->{boost} // 0;
-       my $key = "external.$location.boost";
-       my $cur_boost = $cfg->{$key};
-       return if defined($cur_boost) && $cur_boost == $new_boost; # idempotent
-       $self->_config($key, $new_boost);
-}
-
-sub lei_add_external {
-       my ($self, $location) = @_;
-       my $opt = $self->{opt};
-       my $mirror = $opt->{mirror} // do {
-               my @fail;
-               for my $sw ($self->index_opt, $self->curl_opt,
-                               qw(no-torsocks torsocks inbox-version)) {
-                       my ($f) = (split(/|/, $sw, 2))[0];
-                       next unless defined $opt->{$f};
-                       $f = length($f) == 1 ? "-$f" : "--$f";
-                       push @fail, $f;
-               }
-               if (scalar(@fail) == 1) {
-                       return $self->("@fail requires --mirror");
-               } elsif (@fail) {
-                       my $last = pop @fail;
-                       my $fail = join(', ', @fail);
-                       return $self->("@fail and $last require --mirror");
-               }
-               undef;
-       };
-       my $new_boost = $opt->{boost} // 0;
-       $location = ext_canonicalize($location);
-       if (defined($mirror) && -d $location) {
-               $self->fail(<<""); # TODO: did you mean "update-external?"
---mirror destination `$location' already exists
-
-       } elsif (-d $location) {
-               index($location, "\n") >= 0 and
-                       return $self->fail("`\\n' not allowed in `$location'");
-       }
-       if ($location !~ m!\Ahttps?://! && !-d $location) {
-               $mirror // return $self->fail("$location not a directory");
-               index($location, "\n") >= 0 and
-                       return $self->fail("`\\n' not allowed in `$location'");
-               $mirror = ext_canonicalize($mirror);
-               require PublicInbox::LeiMirror;
-               PublicInbox::LeiMirror->start($self, $mirror => $location);
-       } else {
-               add_external_finish($self, $location);
+sub canonicalize_excludes {
+       my ($lei, $excludes) = @_;
+       my %x;
+       for my $loc (@$excludes) {
+               my @l = get_externals($lei, $loc, 1);
+               $x{$_} = 1 for @l;
        }
+       \%x;
 }
 
 # returns an anonymous sub which returns an array of potential results
@@ -223,17 +155,4 @@ sub complete_url_prepare {
        wantarray ? ($re, $cur, $match_cb) : $match_cb;
 }
 
-sub _complete_add_external { # for bash, this relies on "compopt -o nospace"
-       my ($self, @argv) = @_;
-       my $cfg = $self->_lei_cfg;
-       my $match_cb = complete_url_prepare(\@argv);
-       require URI;
-       map {
-               my $u = URI->new(substr($_, length('external.')));
-               my ($base) = ($u->path =~ m!((?:/?.*)?/)[^/]+/?\z!);
-               $u->path($base);
-               $match_cb->($u->as_string);
-       } grep(m!\Aexternal\.https?://!, @{$cfg->{-section_order}});
-}
-
 1;