]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/LeiMirror.pm
lei: drop redundant WQ EOF callbacks
[public-inbox.git] / lib / PublicInbox / LeiMirror.pm
index c113c9debf8bfccdc0834e09cd1c56d05f167f3c..6bfa4b6f320ec582e000c4b8189a195ab6cb618b 100644 (file)
@@ -9,9 +9,9 @@ use parent qw(PublicInbox::IPC);
 use IO::Uncompress::Gunzip qw(gunzip $GunzipError);
 use PublicInbox::Spawn qw(popen_rd spawn run_die);
 use File::Temp ();
-use Fcntl qw(SEEK_SET);
+use Fcntl qw(SEEK_SET O_CREAT O_EXCL O_WRONLY);
 
-sub do_finish_mirror { # dwaitpid callback
+sub _wq_done_wait { # dwaitpid callback (via wq_eof)
        my ($arg, $pid) = @_;
        my ($mrr, $lei) = @$arg;
        my $f = "$mrr->{dst}/mirror.done";
@@ -28,12 +28,6 @@ sub do_finish_mirror { # dwaitpid callback
        $lei->dclose;
 }
 
-sub _lei_wq_eof { # EOF callback for main daemon
-       my ($lei) = @_;
-       my $mrr = delete $lei->{wq1} or return $lei->fail;
-       $mrr->wq_wait_old(\&do_finish_mirror, $lei);
-}
-
 # for old installations without manifest.js.gz
 sub try_scrape {
        my ($self) = @_;
@@ -176,12 +170,12 @@ sub index_cloned_inbox {
                PublicInbox::Admin::progress_prepare($opt, $lei->{2});
                PublicInbox::Admin::index_inbox($ibx, undef, $opt);
        }
-       open my $x, '>', "$self->{dst}/mirror.done"; # for do_finish_mirror
+       open my $x, '>', "$self->{dst}/mirror.done"; # for _wq_done_wait
 }
 
 sub run_reap {
        my ($lei, $cmd, $opt) = @_;
-       $lei->qerr("# @$cmd" . ($opt->{-C} ? " (in $opt->{-C})" : ''));
+       $lei->qerr("# @$cmd");
        $opt->{pgid} = 0 if $lei->{sock};
        my $pid = spawn($cmd, undef, $opt);
        my $reap = PublicInbox::OnDestroy->new($lei->can('sigint_reap'), $pid);
@@ -201,6 +195,7 @@ sub clone_v1 {
        my $cerr = run_reap($lei, $cmd, $opt);
        return $lei->child_error($cerr, "@$cmd failed") if $cerr;
        _try_config($self);
+       write_makefile($self->{dst}, 1);
        index_cloned_inbox($self, 1);
 }
 
@@ -234,6 +229,7 @@ failed to extract epoch number from $src
        my $mg = PublicInbox::MultiGit->new($dst, 'all.git', 'git');
        $mg->fill_alternates;
        for my $i ($mg->git_epochs) { $mg->epoch_cfg_set($i) }
+       write_makefile($self->{dst}, 2);
        undef $on_destroy; # unlock
        index_cloned_inbox($self, 2);
 }
@@ -274,11 +270,13 @@ sub try_manifest {
        $uri->path($path . '/manifest.js.gz');
        my $pdir = $lei->rel2abs($self->{dst});
        $pdir =~ s!/[^/]+/?\z!!;
-       my $ft = File::Temp->new(TEMPLATE => 'manifest-XXXX',
-                               UNLINK => 1, DIR => $pdir);
+       my $ft = File::Temp->new(TEMPLATE => 'm-XXXX',
+                               UNLINK => 1, DIR => $pdir, SUFFIX => '.tmp');
        my $fn = $ft->filename;
-       my $cmd = $curl->for_uri($lei, $uri, '-R', '-o', $fn);
-       my $opt = { 0 => $lei->{0}, 1 => $lei->{1}, 2 => $lei->{2} };
+       my ($bn) = ($fn =~ m!/([^/]+)\z!);
+       my $cmd = $curl->for_uri($lei, $uri, '-R', '-o', $bn);
+       my $opt = { -C => $pdir };
+       $opt->{$_} = $lei->{$_} for (0..2);
        my $cerr = run_reap($lei, $cmd, $opt);
        if ($cerr) {
                return try_scrape($self) if ($cerr >> 8) == 22; # 404 missing
@@ -356,4 +354,61 @@ sub ipc_atfork_child {
        $self->SUPER::ipc_atfork_child;
 }
 
+sub write_makefile {
+       my ($dir, $ibx_ver) = @_;
+       my $f = "$dir/Makefile";
+       if (sysopen my $fh, $f, O_CREAT|O_EXCL|O_WRONLY) {
+               print $fh <<EOM or die "print($f) $!";
+# This is a v$ibx_ver public-inbox, see the public-inbox-v$ibx_ver-format(5)
+# manpage for more information on the format.  This Makefile is
+# intended as a familiar wrapper for users unfamiliar with
+# public-inbox-* commands.
+#
+# See the respective manpages for public-inbox-fetch(1),
+# public-inbox-index(1), etc for more information on
+# some of the commands used by this Makefile.
+#
+# This Makefile will not be modified nor read by public-inbox,
+# so you may edit it freely with your own convenience targets
+# and notes.  public-inbox-fetch will recreate it if removed.
+EOM
+               print $fh <<'EOM' or die "print($f): $!";
+# the default target:
+help :
+       @echo Common targets:
+       @echo '    make fetch        - fetch from remote git repostorie(s)'
+       @echo '    make update       - fetch and update index '
+       @echo
+       @echo Rarely needed targets:
+       @echo '    make reindex      - may be needed for new features/bugfixes'
+       @echo '    make compact      - rewrite Xapian storage to save space'
+
+fetch :
+       public-inbox-fetch
+update :
+       @if ! public-inbox-fetch --exit-code; \
+       then \
+               c=$$?; \
+               test $$c -eq 127 && exit 0; \
+               exit $$c; \
+       elif test -f msgmap.sqlite3 || test -f public-inbox/msgmap.sqlite3; \
+       then \
+               public-inbox-index; \
+       else \
+               echo 'public-inbox index not initialized'; \
+               echo 'see public-inbox-index(1) man page'; \
+       fi
+reindex :
+       public-inbox-index --reindex
+compact :
+       public-inbox-compact
+
+.PHONY : help fetch update reindex compact
+EOM
+               close $fh or die "close($f): $!";
+       } else {
+               die "open($f): $!" unless $!{EEXIST};
+       }
+}
+
 1;