]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/Git.pm
mm: min/max: return 0 instead of undef
[public-inbox.git] / lib / PublicInbox / Git.pm
index 2323cecc49f1affe6571820e2653802aa1273a41..917fa4a16e96b76a4a45ebd560035a4bac1b78a4 100644 (file)
@@ -14,10 +14,12 @@ use POSIX ();
 use IO::Handle; # ->autoflush
 use Errno qw(EINTR);
 use File::Glob qw(bsd_glob GLOB_NOSORT);
+use File::Spec ();
 use Time::HiRes qw(stat);
 use PublicInbox::Spawn qw(popen_rd);
 use PublicInbox::Tmpfile;
 use Carp qw(croak);
+use Digest::SHA ();
 our @EXPORT_OK = qw(git_unquote git_quote);
 our $PIPE_BUFSIZ = 65536; # Linux default
 our $in_cleanup;
@@ -204,14 +206,14 @@ sub cat_async_step ($$) {
        } else {
                $self->fail("Unexpected result from async git cat-file: $head");
        }
-       eval { $cb->($bref, $oid, $type, $size, $arg) };
        $self->{cat_rbuf} = $rbuf if $$rbuf ne '';
+       eval { $cb->($bref, $oid, $type, $size, $arg) };
        warn "E: $oid: $@\n" if $@;
 }
 
 sub cat_async_wait ($) {
        my ($self) = @_;
-       my $inflight = delete $self->{inflight} or return;
+       my $inflight = $self->{inflight} or return;
        while (scalar(@$inflight)) {
                cat_async_step($self, $inflight);
        }
@@ -239,7 +241,7 @@ sub check_async_step ($$) {
        my ($self, $inflight_c) = @_;
        die 'BUG: inflight empty or odd' if scalar(@$inflight_c) < 3;
        my ($req, $cb, $arg) = splice(@$inflight_c, 0, 3);
-       my $rbuf = delete($self->{rbuf_c}) // \(my $new = '');
+       my $rbuf = delete($self->{chk_rbuf}) // \(my $new = '');
        chomp(my $line = my_readline($self->{in_c}, $rbuf));
        my ($hex, $type, $size) = split(/ /, $line);
 
@@ -251,14 +253,14 @@ sub check_async_step ($$) {
                my $ret = my_read($self->{in_c}, $rbuf, $type + 1);
                fail($self, defined($ret) ? 'read EOF' : "read: $!") if !$ret;
        }
+       $self->{chk_rbuf} = $rbuf if $$rbuf ne '';
        eval { $cb->($hex, $type, $size, $arg, $self) };
        warn "E: check($req) $@\n" if $@;
-       $self->{rbuf_c} = $rbuf if $$rbuf ne '';
 }
 
 sub check_async_wait ($) {
        my ($self) = @_;
-       my $inflight_c = delete $self->{inflight_c} or return;
+       my $inflight_c = $self->{inflight_c} or return;
        while (scalar(@$inflight_c)) {
                check_async_step($self, $inflight_c);
        }
@@ -275,7 +277,7 @@ sub check_async_begin ($) {
 sub check_async ($$$$) {
        my ($self, $oid, $cb, $arg) = @_;
        my $inflight_c = $self->{inflight_c} // check_async_begin($self);
-       if (scalar(@$inflight_c) >= MAX_INFLIGHT) {
+       while (scalar(@$inflight_c) >= MAX_INFLIGHT) {
                check_async_step($self, $inflight_c);
        }
        print { $self->{out_c} } $oid, "\n" or fail($self, "write error: $!");
@@ -318,13 +320,15 @@ sub _destroy {
 
 sub cat_async_abort ($) {
        my ($self) = @_;
-       if (my $inflight = delete $self->{inflight}) {
+       if (my $inflight = $self->{inflight}) {
                while (@$inflight) {
                        my ($req, $cb, $arg) = splice(@$inflight, 0, 3);
                        $req =~ s/ .*//; # drop git_dir for Gcf2Client
                        eval { $cb->(undef, $req, undef, undef, $arg) };
                        warn "E: $req: $@ (in abort)\n" if $@;
                }
+               delete $self->{cat_rbuf};
+               delete $self->{inflight};
        }
        cleanup($self);
 }
@@ -350,13 +354,25 @@ sub qx {
        <$fh>
 }
 
+# check_async and cat_async may trigger the other, so ensure they're
+# both completely done by using this:
+sub async_wait_all ($) {
+       my ($self) = @_;
+       while (scalar(@{$self->{inflight_c} // []}) ||
+                       scalar(@{$self->{inflight} // []})) {
+               $self->check_async_wait;
+               $self->cat_async_wait;
+       }
+}
+
 # returns true if there are pending "git cat-file" processes
 sub cleanup {
        my ($self) = @_;
        local $in_cleanup = 1;
        delete $self->{async_cat};
-       check_async_wait($self);
-       cat_async_wait($self);
+       async_wait_all($self);
+       delete $self->{inflight};
+       delete $self->{inflight_c};
        _destroy($self, qw(cat_rbuf in out pid));
        _destroy($self, qw(chk_rbuf in_c out_c pid_c err_c));
        !!($self->{pid} || $self->{pid_c});
@@ -416,10 +432,9 @@ sub cat_async_begin {
 sub cat_async ($$$;$) {
        my ($self, $oid, $cb, $arg) = @_;
        my $inflight = $self->{inflight} // cat_async_begin($self);
-       if (scalar(@$inflight) >= MAX_INFLIGHT) {
+       while (scalar(@$inflight) >= MAX_INFLIGHT) {
                cat_async_step($self, $inflight);
        }
-
        print { $self->{out} } $oid, "\n" or fail($self, "write error: $!");
        push(@$inflight, $oid, $cb, $arg);
 }
@@ -462,6 +477,57 @@ sub modified ($) {
        $modified || time;
 }
 
+# for grokmirror, which doesn't read gitweb.description
+# templates/hooks--update.sample and git-multimail in git.git
+# only match "Unnamed repository", not the full contents of
+# templates/this--description in git.git
+sub manifest_entry {
+       my ($self, $epoch, $default_desc) = @_;
+       my ($fh, $pid) = $self->popen('show-ref');
+       my $dig = Digest::SHA->new(1);
+       while (read($fh, my $buf, 65536)) {
+               $dig->add($buf);
+       }
+       close $fh;
+       waitpid($pid, 0);
+       return if $?; # empty, uninitialized git repo
+       my $git_dir = $self->{git_dir};
+       my $ent = {
+               fingerprint => $dig->hexdigest,
+               reference => undef,
+               modified => modified($self),
+       };
+       chomp(my $owner = $self->qx('config', 'gitweb.owner'));
+       utf8::decode($owner);
+       $ent->{owner} = $owner eq '' ? undef : $owner;
+       my $desc = '';
+       if (open($fh, '<', "$git_dir/description")) {
+               local $/ = "\n";
+               chomp($desc = <$fh>);
+               utf8::decode($desc);
+       }
+       $desc = 'Unnamed repository' if $desc eq '';
+       if (defined $epoch && $desc =~ /\AUnnamed repository/) {
+               $desc = "$default_desc [epoch $epoch]";
+       }
+       $ent->{description} = $desc;
+       if (open($fh, '<', "$git_dir/objects/info/alternates")) {
+               # n.b.: GitPython doesn't seem to handle comments or C-quoted
+               # strings like native git does; and we don't for now, either.
+               local $/ = "\n";
+               chomp(my @alt = <$fh>);
+
+               # grokmirror only supports 1 alternate for "reference",
+               if (scalar(@alt) == 1) {
+                       my $objdir = "$git_dir/objects";
+                       my $ref = File::Spec->rel2abs($alt[0], $objdir);
+                       $ref =~ s!/[^/]+/?\z!!; # basename
+                       $ent->{reference} = $ref;
+               }
+       }
+       $ent;
+}
+
 1;
 __END__
 =pod