]> Sergey Matveev's repositories - public-inbox.git/commitdiff
treewide: replace /^I: / prefix with /^# /
authorEric Wong <e@80x24.org>
Thu, 20 Oct 2022 08:43:12 +0000 (08:43 +0000)
committerEric Wong <e@80x24.org>
Mon, 24 Oct 2022 08:18:28 +0000 (08:18 +0000)
This is like more familiar to readers of TAP (Test Anywhere
Protocol) output, as well as shell and Perl scripters which also
use `#' for comments.

AFAIK, nobody is parsing our stderr, and I'm not sure how
standardized the `I:' prefix is (nor `W:' and `E:' are).  It's
already the prevailing style in Lei* code, too, so things have
been moving in that direction for a bit.

examples/grok-pull.post_update_hook.sh
lib/PublicInbox/ExtSearchIdx.pm
lib/PublicInbox/Gcf2.pm
lib/PublicInbox/InboxIdle.pm
lib/PublicInbox/MiscIdx.pm
lib/PublicInbox/OverIdx.pm
lib/PublicInbox/Watch.pm
script/public-inbox-watch
t/extsearch.t
t/imapd.t

index 77489472123192ca1df6522e6a1bbaa8fe6845b5..4d303c0326abc6ebfc3bd2863cf3678a6cad9e0d 100755 (executable)
@@ -111,7 +111,7 @@ case $cfg_dir in
                        "publicinbox.$inbox_name.infourl" "$url"
        done
        curl -sSfv "$remote_inbox_url"/description >"$inbox_dir"/description
-       echo "I: $inbox_name at $inbox_dir ($addresses) $local_url"
+       echo "# $inbox_name at $inbox_dir ($addresses) $local_url"
        ;;
 esac
 
index 7c44a1a406308604c1319b91eaa7130a0be645d2..401b18d00754176a69eb1c2a90d380bc926a84b3 100644 (file)
@@ -406,14 +406,14 @@ EOM
        while (my ($ibx_id, $eidx_key) = $ibx_ck->fetchrow_array) {
                next if $self->{ibx_map}->{$eidx_key};
                $self->{midx}->remove_eidx_key($eidx_key);
-               warn "I: deleting messages for $eidx_key...\n";
+               warn "# deleting messages for $eidx_key...\n";
                $x3_doc->execute($ibx_id);
                my $ibx = { -ibx_id => $ibx_id, -gc_eidx_key => $eidx_key };
                while (my ($docid, $xnum, $oid) = $x3_doc->fetchrow_array) {
                        my $r = _unref_doc($sync, $docid, $ibx, $xnum, $oid);
                        $oid = unpack('H*', $oid);
                        $r = $r ? 'unref' : 'remove';
-                       warn "I: $r #$docid $eidx_key $oid\n";
+                       warn "# $r #$docid $eidx_key $oid\n";
                        if (checkpoint_due($sync)) {
                                $x3_doc = $ibx_ck = undef;
                                reindex_checkpoint($self, $sync);
@@ -433,12 +433,12 @@ SELECT key FROM eidx_meta WHERE key LIKE ? ESCAPE ?
                $lc_i->execute("lc-%:$pat//%", '\\');
                while (my ($key) = $lc_i->fetchrow_array) {
                        next if $key !~ m!\Alc-v[1-9]+:\Q$eidx_key\E//!;
-                       warn "I: removing $key\n";
+                       warn "# removing $key\n";
                        $self->{oidx}->dbh->do(<<'', undef, $key);
 DELETE FROM eidx_meta WHERE key = ?
 
                }
-               warn "I: $eidx_key removed\n";
+               warn "# $eidx_key removed\n";
        }
 }
 
@@ -447,20 +447,20 @@ sub eidx_gc_scan_shards ($$) { # TODO: use for lei/store
        my $nr = $self->{oidx}->dbh->do(<<'');
 DELETE FROM xref3 WHERE docid NOT IN (SELECT num FROM over)
 
-       warn "I: eliminated $nr stale xref3 entries\n" if $nr != 0;
+       warn "# eliminated $nr stale xref3 entries\n" if $nr != 0;
        reindex_checkpoint($self, $sync) if checkpoint_due($sync);
 
        # fixup from old bugs:
        $nr = $self->{oidx}->dbh->do(<<'');
 DELETE FROM over WHERE num > 0 AND num NOT IN (SELECT docid FROM xref3)
 
-       warn "I: eliminated $nr stale over entries\n" if $nr != 0;
+       warn "# eliminated $nr stale over entries\n" if $nr != 0;
        reindex_checkpoint($self, $sync) if checkpoint_due($sync);
 
        $nr = $self->{oidx}->dbh->do(<<'');
 DELETE FROM eidxq WHERE docid NOT IN (SELECT num FROM over)
 
-       warn "I: eliminated $nr stale reindex queue entries\n" if $nr != 0;
+       warn "# eliminated $nr stale reindex queue entries\n" if $nr != 0;
        reindex_checkpoint($self, $sync) if checkpoint_due($sync);
 
        my ($cur) = $self->{oidx}->dbh->selectrow_array(<<EOM);
@@ -490,7 +490,7 @@ SELECT num FROM over WHERE num >= ? ORDER BY num ASC LIMIT 10000
                        reindex_checkpoint($self, $sync);
                }
        }
-       warn "I: eliminated $nr stale Xapian documents\n" if $nr != 0;
+       warn "# eliminated $nr stale Xapian documents\n" if $nr != 0;
 }
 
 sub eidx_gc {
@@ -731,13 +731,11 @@ sub eidxq_lock_acquire ($) {
        my $t = strftime('%Y-%m-%d %k:%M %z', localtime($time));
        local $self->{current_info} = 'eidxq';
        if ($euid == $> && $ident eq host_ident) {
-               if (kill(0, $pid)) {
-                       warn <<EOM; return;
-I: PID:$pid (re)indexing since $t, it will continue our work
+               kill(0, $pid) and warn <<EOM and return;
+# PID:$pid (re)indexing since $t, it will continue our work
 EOM
-               }
                if ($!{ESRCH}) {
-                       warn "I: eidxq_lock is stale ($cur), clobbering\n";
+                       warn "# eidxq_lock is stale ($cur), clobbering\n";
                        return _eidxq_take($self);
                }
                warn "E: kill(0, $pid) failed: $!\n"; # fall-through:
@@ -837,7 +835,7 @@ sub reindex_unseen ($$$$) {
                xnum => $xsmsg->{num},
                # {mids} and {chash} will be filled in at _reindex_unseen
        };
-       warn "I: reindex_unseen ${\$ibx->eidx_key}:$req->{xnum}:$req->{oid}\n";
+       warn "# reindex_unseen ${\$ibx->eidx_key}:$req->{xnum}:$req->{oid}\n";
        $self->git->cat_async($xsmsg->{blob}, \&_reindex_unseen, $req);
 }
 
index 5c35309501d1e2d9436af02519b2342b3b93d801..54b3d6aa17aa6487f6e42ff98f59627338e5bccd 100644 (file)
@@ -128,14 +128,14 @@ sub loop (;$) {
                        # retry once if missing.  We only get unabbreviated OIDs
                        # from SQLite or Xapian DBs, here, so malicious clients
                        # can't trigger excessive retries:
-                       warn "I: $$ $oid missing, retrying in $git_dir\n";
+                       warn "# $$ $oid missing, retrying in $git_dir\n";
 
                        $gcf2 = new();
                        %seen = ($git_dir => add_alt($gcf2, $git_dir));
                        $check_at = clock_gettime(CLOCK_MONOTONIC) + $exp;
 
                        if ($gcf2->cat_oid(1, $oid)) {
-                               warn "I: $$ $oid found after retry\n";
+                               warn "# $$ $oid found after retry\n";
                        } else {
                                warn "W: $$ $oid missing after retry\n";
                                print "$oid missing\n"; # mimic git-cat-file
index ffbbfea7c51c2c554e7ed951fd8e545f0c7ca190..005e263612d409c7a794bcc2eed1e08a47ee793e 100644 (file)
@@ -1,4 +1,4 @@
-# Copyright (C) 2020-2021 all contributors <meta@public-inbox.org>
+# Copyright (C) all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 
 # fields:
@@ -48,11 +48,9 @@ sub in2_arm ($$) { # PublicInbox::Config::each_inbox callback
                $self->{on_unlock}->{$w->name} = $ibx;
        } else {
                warn "E: ".ref($inot)."->watch($lock, IN_MODIFY) failed: $!\n";
-               if ($!{ENOSPC} && $^O eq 'linux') {
-                       warn <<"";
-I: consider increasing /proc/sys/fs/inotify/max_user_watches
+               warn <<"" if $!{ENOSPC} && $^O eq 'linux';
+# consider increasing /proc/sys/fs/inotify/max_user_watches
 
-               }
        }
 
        # TODO: detect deleted packs (and possibly other files)
index 76b33b16462aa0a47196510ad63adf43a096b0ca..19200b92c910c8e22de43acccdbf1bd9e89fa0e6 100644 (file)
@@ -72,7 +72,7 @@ sub remove_eidx_key {
        }
        for my $docid (@docids) {
                $xdb->delete_document($docid);
-               warn "I: remove inbox docid #$docid ($eidx_key)\n";
+               warn "# remove inbox docid #$docid ($eidx_key)\n";
        }
 }
 
index a49ca6db3bb82d62a3e80c4738685cc68955d7b9..6cc86d5d038d3669b0991d270aa74234f3937560 100644 (file)
@@ -1,4 +1,4 @@
-# Copyright (C) 2018-2021 all contributors <meta@public-inbox.org>
+# Copyright (C) all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 
 # for XOVER, OVER in NNTP, and feeds/homepage/threads in PSGI
@@ -509,12 +509,12 @@ EOF
                                next;
                        }
                        $pr->(<<EOM) if $pr;
-I: ghost $r->{num} <$mid> THREADID=$r->{tid} culled
+# ghost $r->{num} <$mid> THREADID=$r->{tid} culled
 EOM
                }
                delete_by_num($self, $r->{num});
        }
-       $pr->("I: rethread culled $total ghosts\n") if $pr && $total;
+       $pr->("# rethread culled $total ghosts\n") if $pr && $total;
 }
 
 # used for cross-inbox search
index 3f6fe21b726f30d3d185032e45ca7e20d358a561..082ecfb96baf5e05021fca165551d5e40cac4a87 100644 (file)
@@ -328,7 +328,7 @@ sub imap_idle_once ($$$$) {
        my ($self, $mic, $intvl, $uri) = @_;
        my $i = $intvl //= (29 * 60);
        my $end = now() + $intvl;
-       warn "I: $uri idling for ${intvl}s\n";
+       warn "# $uri idling for ${intvl}s\n";
        local $0 = "IDLE $0";
        return if $self->{quit};
        unless ($mic->idle) {
@@ -517,7 +517,7 @@ sub poll_fetch_reap {
        if ($?) {
                warn "W: PID=$pid died: \$?=$?\n", map { "$_\n" } @$uris;
        }
-       warn("I: will check $_ in ${intvl}s\n") for @$uris;
+       warn("# will check $_ in ${intvl}s\n") for @$uris;
        add_timer($intvl, \&poll_fetch_fork, $self, $intvl, $uris);
 }
 
index af02d8f358f7b84392744b68df49a4328fc69636..4c50461ff4c02950d14d1aa763e0319bd3ee059f 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/perl -w
-# Copyright (C) 2016-2021 all contributors <meta@public-inbox.org>
+# Copyright (C) all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 my $help = <<EOF;
 usage: public-inbox-watch
@@ -27,7 +27,7 @@ my $reload = sub {
        $watch->quit;
        $watch = PublicInbox::Watch->new(PublicInbox::Config->new);
        if ($watch) {
-               warn("I: reloaded\n");
+               warn "# reloaded\n";
        } else {
                warn("E: reloading failed\n");
                $watch = $prev;
@@ -37,7 +37,7 @@ my $reload = sub {
 if ($watch) {
        my $scan = sub {
                return if !$watch;
-               warn "I: scanning\n";
+               warn "# scanning\n";
                $watch->trigger_scan('full');
        };
        my $quit = sub {
index 6cbe52626830d5886054072a594b06c31725c5d0..03bcad95ee5b162bcddb922bb640d6565a49f46c 100644 (file)
@@ -309,7 +309,7 @@ if ('reindex catches missed messages') {
        is($oidx->eidx_meta($lc_key), $cmt_b, 'lc-v2 stays unchanged');
        my @err = split(/^/, $err);
        is(scalar(@err), 1, 'only one warning') or diag "err=$err";
-       like($err[0], qr/I: reindex_unseen/, 'got reindex_unseen message');
+       like($err[0], qr/# reindex_unseen/, 'got reindex_unseen message');
        my $new = $oidx->get_art($max + 1);
        is($new->{subject}, $eml->header('Subject'), 'new message added');
 
@@ -415,8 +415,8 @@ if ('remove v1test and test gc') {
        my $opt = { 2 => \(my $err = '') };
        ok(run_script([qw(-extindex --gc), "$home/extindex"], undef, $opt),
                'extindex --gc');
-       like($err, qr/^I: remove #1 v1\.example /ms, 'removed v1 message');
-       is(scalar(grep(!/^I:/, split(/^/m, $err))), 0,
+       like($err, qr/^# remove #1 v1\.example /ms, 'removed v1 message');
+       is(scalar(grep(!/^#/, split(/^/m, $err))), 0,
                'no non-informational messages');
        $misc->{xdb}->reopen;
        @it = $misc->mset('')->items;
index 43de867548c651ab89bb98bc51d8211e768cee28..3c74aefd4beeeb806697e55dc503a720aecff5ef 100644 (file)
--- a/t/imapd.t
+++ b/t/imapd.t
@@ -470,7 +470,7 @@ SKIP: {
        PublicInbox::DS::event_loop();
        diag 'inbox unlocked on initial fetch, waiting for IDLE';
 
-       tick until (grep(/I: \S+ idling/, <$err>));
+       tick until (grep(/# \S+ idling/, <$err>));
        open my $fh, '<', 't/iso-2202-jp.eml' or BAIL_OUT $!;
        $old_env->{ORIGINAL_RECIPIENT} = $addr;
        ok(run_script([qw(-mda --no-precheck)], $old_env, { 0 => $fh }),
@@ -487,7 +487,7 @@ SKIP: {
                or BAIL_OUT "git config $?";
        $w->kill('HUP');
        diag 'waiting for -watch reload + initial fetch';
-       tick until (grep(/I: will check/, <$err>));
+       tick until (grep(/# will check/, <$err>));
 
        open $fh, '<', 't/psgi_attach.eml' or BAIL_OUT $!;
        ok(run_script([qw(-mda --no-precheck)], $old_env, { 0 => $fh }),
@@ -516,7 +516,7 @@ SKIP: {
                my @t0 = times;
                $w = start_script(['-watch'], undef, { 2 => $err_wr });
                seek($err, 0, 0);
-               tick until (grep(/I: \S+ idling/, <$err>));
+               tick until (grep(/# \S+ idling/, <$err>));
                diag 'killing imapd, waiting for CPU spins';
                my $delay = 0.11;
                $td->kill(9);