]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/LeiEditSearch.pm
imap+nntp: share COMPRESS implementation
[public-inbox.git] / lib / PublicInbox / LeiEditSearch.pm
index 30ac65bdc18c7d402abb410f203d27cddc25df15..bcf7c10517c95c7718efb5359ffced25125a57a6 100644 (file)
@@ -7,26 +7,68 @@ use strict;
 use v5.10.1;
 use PublicInbox::LeiSavedSearch;
 use PublicInbox::LeiUp;
+use parent qw(PublicInbox::LeiConfig);
+
+sub cfg_edit_begin {
+       my ($self) = @_;
+       if (ref($self->{lss}->{-cfg}->{'lei.q.output'})) {
+               delete $self->{lss}->{-cfg}->{'lei.q.output'}; # invalid
+               $self->{lei}->pgr_err(<<EOM);
+$self->{lss}->{-f} has multiple values of lei.q.output
+please remove redundant ones
+EOM
+       }
+}
+
+sub cfg_verify {
+       my ($self, $cfg) = @_;
+       my $new_out = $cfg->{'lei.q.output'} // '';
+       return $self->cfg_do_edit(<<EOM) if ref $new_out;
+$self->{-f} has multiple values of lei.q.output
+EOM
+       return $self->cfg_do_edit(<<EOM) if $new_out eq '';
+$self->{-f} needs lei.q.output
+EOM
+       my $lss = $self->{lss};
+       my $old_out = $lss->{-cfg}->{'lei.q.output'} // return;
+       return if $old_out eq $new_out;
+       my $lei = $self->{lei};
+       my $old_path = $old_out;
+       my $new_path = $new_out;
+       s!$PublicInbox::LeiSavedSearch::LOCAL_PFX!! for ($old_path, $new_path);
+       my $dir_old = $lss->can('lss_dir_for')->($lei, \$old_path, 1);
+       my $dir_new = $lss->can('lss_dir_for')->($lei, \$new_path);
+       return if $dir_new eq $dir_old;
+
+       ($old_out =~ m!\Av2:!i || $new_out =~ m!\Av2:!) and
+               return $self->cfg_do_edit(<<EOM);
+conversions from/to v2 inboxes not supported at this time
+EOM
+       return $self->cfg_do_edit(<<EOM) if -e $dir_new;
+lei.q.output changed from `$old_out' to `$new_out'
+However, $dir_new exists
+EOM
+       # start the conversion asynchronously
+       my $old_sq = PublicInbox::Config::squote_maybe($old_out);
+       my $new_sq = PublicInbox::Config::squote_maybe($new_out);
+       $lei->puts("lei.q.output changed from $old_sq to $new_sq");
+       $lei->qerr("# lei convert $old_sq -o $new_sq");
+       my $v = !$lei->{opt}->{quiet};
+       $lei->{opt} = { output => $new_out, verbose => $v };
+       require PublicInbox::LeiConvert;
+       PublicInbox::LeiConvert::lei_convert($lei, $old_out);
+
+       $lei->fail(<<EOM) if -e $dir_old && !rename($dir_old, $dir_new);
+E: rename($dir_old, $dir_new) error: $!
+EOM
+}
 
 sub lei_edit_search {
        my ($lei, $out) = @_;
        my $lss = PublicInbox::LeiSavedSearch->up($lei, $out) or return;
-       my @cmd = (qw(git config --edit -f), $lss->{'-f'});
-       $lei->qerr("# spawning @cmd");
-       $lss->edit_begin($lei);
-       if ($lei->{oneshot}) {
-               require PublicInbox::Spawn;
-               waitpid(PublicInbox::Spawn::spawn(\@cmd), 0);
-               # non-fatal, editor could fail after successful write
-               $lei->child_error($?) if $?;
-               $lss->edit_done($lei);
-       } else { # run in script/lei foreground
-               require PublicInbox::PktOp;
-               my ($op_c, $op_p) = PublicInbox::PktOp->pair;
-               # $op_p will EOF when $EDITOR is done
-               $op_c->{ops} = { '' => [$lss->can('edit_done'), $lss, $lei] };
-               $lei->send_exec_cmd([ @$lei{qw(0 1 2)}, $op_p ], \@cmd, {});
-       }
+       my $f = $lss->{-f};
+       my $self = bless { lei => $lei, lss => $lss, -f => $f }, __PACKAGE__;
+       $self->cfg_do_edit;
 }
 
 *_complete_edit_search = \&PublicInbox::LeiUp::_complete_up;