drop dependency on File::Path::Expand

[public-inbox.git] / lib / PublicInbox / Config.pm

diff --git a/lib/PublicInbox/Config.pm b/lib/PublicInbox/Config.pm
index b51163883317fea824a4b7492f1c91149747054d..4651861d215229fe6e53d5ca7f9886d395d3a2fe 100644 (file)
--- a/lib/PublicInbox/Config.pm
+++ b/lib/PublicInbox/Config.pm
@@ -5,21 +5,30 @@
 package PublicInbox::Config;
 use strict;
 use warnings;
-use base qw/Exporter/;
-our @EXPORT_OK = qw/try_cat/;
-use File::Path::Expand qw/expand_filename/;
+require PublicInbox::Inbox;
+use PublicInbox::Spawn qw(popen_rd);
 
 # returns key-value pairs of config directives in a hash
 # if keys may be multi-value, the value is an array ref containing all values
 sub new {
        my ($class, $file) = @_;
        $file = default_file() unless defined($file);
-       bless git_config_dump($file), $class;
+       $file = ref $file ? $file : git_config_dump($file);
+       my $self = bless $file, $class;
+
+       # caches
+       $self->{-by_addr} ||= {};
+       $self->{-by_name} ||= {};
+       $self->{-by_newsgroup} ||= {};
+       $self;
 }
 
 sub lookup {
        my ($self, $recipient) = @_;
        my $addr = lc($recipient);
+       my $inbox = $self->{-by_addr}->{$addr};
+       return $inbox if $inbox;
+
        my $pfx;
 
        foreach my $k (keys %$self) {
@@ -37,29 +46,42 @@ sub lookup {
                        last;
                }
        }
-
        defined $pfx or return;
+       _fill($self, $pfx);
+}
 
-       my %rv;
-       foreach my $k (qw(mainrepo address filter)) {
-               my $v = $self->{"$pfx.$k"};
-               $rv{$k} = $v if defined $v;
+sub lookup_name {
+       my ($self, $name) = @_;
+       my $rv = $self->{-by_name}->{$name};
+       return $rv if $rv;
+       $rv = _fill($self, "publicinbox.$name") or return;
+}
+
+sub lookup_newsgroup {
+       my ($self, $ng) = @_;
+       $ng = lc($ng);
+       my $rv = $self->{-by_newsgroup}->{$ng};
+       return $rv if $rv;
+
+       foreach my $k (keys %$self) {
+               $k =~ /\A(publicinbox\.[\w-]+)\.newsgroup\z/ or next;
+               my $v = $self->{$k};
+               my $pfx = $1;
+               if ($v eq $ng) {
+                       $rv = _fill($self, $pfx);
+                       return $rv;
+               }
        }
-       my $listname = $pfx;
-       $listname =~ s/\Apublicinbox\.//;
-       $rv{listname} = $listname;
-       my $v = $rv{address};
-       $rv{-primary_address} = ref($v) eq 'ARRAY' ? $v->[0] : $v;
-       \%rv;
+       undef;
 }
 
 sub get {
-       my ($self, $listname, $key) = @_;
+       my ($self, $inbox, $key) = @_;
 
-       $self->{"publicinbox.$listname.$key"};
+       $self->{"publicinbox.$inbox.$key"};
 }
 
-sub config_dir { $ENV{PI_DIR} || expand_filename('~/.public-inbox') }
+sub config_dir { $ENV{PI_DIR} || "$ENV{HOME}/.public-inbox" }
 
 sub default_file {
        my $f = $ENV{PI_CONFIG};
@@ -72,9 +94,9 @@ sub git_config_dump {
        my ($in, $out);
        my @cmd = (qw/git config/, "--file=$file", '-l');
        my $cmd = join(' ', @cmd);
-       my $pid = open(my $fh, '-|', @cmd);
-       defined $pid or die "$cmd failed: $!\n";
+       my $fh = popen_rd(\@cmd);
        my %rv;
+       local $/ = "\n";
        foreach my $line (<$fh>) {
                chomp $line;
                my ($k, $v) = split(/=/, $line, 2);
@@ -90,19 +112,35 @@ sub git_config_dump {
                        $rv{$k} = $v;
                }
        }
-       close $fh or die "failed to close ($cmd) pipe: $!\n";
-       $? and warn "$$ $cmd exited with: ($pid) $?\n";
+       close $fh or die "failed to close ($cmd) pipe: $?";
        \%rv;
 }
 
-sub try_cat {
-       my ($path) = @_;
-       my $rv;
-       if (open(my $fh, '<', $path)) {
-               local $/;
-               $rv = <$fh>;
+sub _fill {
+       my ($self, $pfx) = @_;
+       my $rv = {};
+
+       foreach my $k (qw(mainrepo address filter url newsgroup)) {
+               my $v = $self->{"$pfx.$k"};
+               $rv->{$k} = $v if defined $v;
+       }
+       return unless $rv->{mainrepo};
+       my $name = $pfx;
+       $name =~ s/\Apublicinbox\.//;
+       $rv->{name} = $name;
+       my $v = $rv->{address} ||= 'public-inbox@example.com';
+       my $p = $rv->{-primary_address} = ref($v) eq 'ARRAY' ? $v->[0] : $v;
+       $rv->{domain} = ($p =~ /\@(\S+)\z/) ? $1 : 'localhost';
+       $rv = PublicInbox::Inbox->new($rv);
+       if (ref($v) eq 'ARRAY') {
+               $self->{-by_addr}->{lc($_)} = $rv foreach @$v;
+       } else {
+               $self->{-by_addr}->{lc($v)} = $rv;
+       }
+       if (my $ng = $rv->{newsgroup}) {
+               $self->{-by_newsgroup}->{$ng} = $rv;
        }
-       $rv;
+       $self->{-by_name}->{$name} = $rv;
 }
 
 1;