diff options
author | Eric Wong <e@80x24.org> | 2021-04-19 08:52:13 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2021-04-19 20:15:58 +0000 |
commit | ab42b22acfd9670df0b61f514b4fb89fe7754846 (patch) | |
tree | 003a53391cd10dc7d09a350645cfc4703eb33d43 /lib/PublicInbox/LeiSavedSearch.pm | |
parent | cb0bfb6843650025faed942bfd08338fdaf17b3d (diff) | |
download | public-inbox-ab42b22acfd9670df0b61f514b4fb89fe7754846.tar.gz |
I don't know if it's worth it to sub (or super)class PublicInbox::Config into something more generic for lei, but this change simplifies a good chunk of lei code that reuses the public-inbox config parsing.
Diffstat (limited to 'lib/PublicInbox/LeiSavedSearch.pm')
-rw-r--r-- | lib/PublicInbox/LeiSavedSearch.pm | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/lib/PublicInbox/LeiSavedSearch.pm b/lib/PublicInbox/LeiSavedSearch.pm index 19f4aa5f..d3a32d36 100644 --- a/lib/PublicInbox/LeiSavedSearch.pm +++ b/lib/PublicInbox/LeiSavedSearch.pm @@ -56,9 +56,8 @@ sub list { print $fh "\tpath = ", cquote_val($p), "\n"; } close $fh or die "close $f: $!"; - my $cfg = PublicInbox::Config::git_config_dump($f); + my $cfg = PublicInbox::Config->git_config_dump($f); unlink($f); - bless $cfg, 'PublicInbox::Config'; my $out = $cfg->get_all('lei.q.output') or return (); map {; s!\A(?:maildir|mh|mbox.+|mmdf):!!i; @@ -76,7 +75,6 @@ sub new { output2lssdir($self, $lei, \$dir, \$f) or return $lei->fail("--save was not used with $dst cwd=". $lei->rel2abs('.')); - $self->{-cfg} //= PublicInbox::Config::git_config_dump($f); $self->{'-f'} = $f; } else { # new saved search "lei q --save" $dst = $lei->{ovv}->{dst}; @@ -113,7 +111,6 @@ EOM } close($fh) or return $lei->fail("close $f: $!"); } - bless $self->{-cfg}, 'PublicInbox::Config'; $self->{lock_path} = "$self->{-f}.flock"; $self->{-ovf} = "$dir/over.sqlite3"; $self; @@ -198,7 +195,7 @@ sub output2lssdir { my $dir = lss_dir_for($lei, \$dst); my $f = "$dir/lei.saved-search"; if (-f $f && -r _) { - $self->{-cfg} = PublicInbox::Config::git_config_dump($f); + $self->{-cfg} = PublicInbox::Config->git_config_dump($f); $$dir_ref = $dir; $$fn_ref = $f; return 1; |