From: Eric Wong <e@80x24.org> To: meta@public-inbox.org Subject: [PATCH 6/5] t/v2mirror: check dependencies for legacy test Date: Sat, 25 Sep 2021 03:21:01 +0000 [thread overview] Message-ID: <YU6VnTZaEt6NNi2D@dcvr> (raw) In-Reply-To: <20210924105645.8627-1-e@80x24.org> We still need Email::MIME to test against old revisions. We'll also depend on the revision just prior to the manifest.js.gz introduction to avoid loading Danga::Socket, since it was getting loaded even with `plackup'. Finally, we'll disable Inline::C usage with old Spawn.pm since our old code included alloca.h, which is not portable to FreeBSD. --- t/v2mirror.t | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/t/v2mirror.t b/t/v2mirror.t index a625646d..63d17ebf 100644 --- a/t/v2mirror.t +++ b/t/v2mirror.t @@ -315,23 +315,26 @@ if ('test read-only epoch dirs') { } my $err = ''; -my $v110 = xqx([qw(git rev-parse v1.1.0-pre1)], undef, { 2 => \$err }); +my $oldrev = '0b3e19584c90d958a723ac2d3dec3f84f5513688~1'; +# 3e0e596105198cfa (wwwlisting: allow hiding entries from manifest, 2019-06-09) +$oldrev = xqx([qw(git rev-parse), $oldrev], undef, { 2 => \$err }); SKIP: { skip("no detected public-inbox GIT_DIR ($err)", 1) if $?; + require_mods('Email::MIME', 1); # for legacy revision # using plackup to test old PublicInbox::WWW since -httpd from # back then relied on some packages we no longer depend on my $plackup = which('plackup') or skip('no plackup in path', 1); require PublicInbox::Lock; - chomp $v110; + chomp $oldrev; my ($base) = ($0 =~ m!\b([^/]+)\.[^\.]+\z!); - my $wt = "t/data-gen/$base.pre-manifest"; + my $wt = "t/data-gen/$base.pre-manifest-$oldrev"; my $lk = bless { lock_path => __FILE__ }, 'PublicInbox::Lock'; $lk->lock_acquire; my $psgi = "$wt/app.psgi"; if (!-f $psgi) { # checkout a pre-manifest.js.gz version my $t = File::Temp->new(TEMPLATE => 'g-XXXX', TMPDIR => 1); my $env = { GIT_INDEX_FILE => $t->filename }; - xsys([qw(git read-tree), $v110], $env) and xbail 'read-tree'; + xsys([qw(git read-tree), $oldrev], $env) and xbail 'read-tree'; xsys([qw(git checkout-index -a), "--prefix=$wt/"], $env) and xbail 'checkout-index'; my $f = "$wt/app.psgi.tmp.$$"; @@ -353,7 +356,8 @@ EOM $td->join('TERM'); open $rdr->{2}, '>>', "$tmpdir/plackup.err.log" or xbail "open: $!"; open $rdr->{1}, '>>&', $rdr->{2} or xbail "open: $!"; - $td = start_script($cmd, { PERL5LIB => 'lib' }, $rdr); + my $env = { PERL5LIB => 'lib', PERL_INLINE_DIRECTORY => undef }; + $td = start_script($cmd, $env, $rdr); # wait for plackup socket()+bind()+listen() my %opt = ( Proto => 'tcp', Type => Socket::SOCK_STREAM(), PeerAddr => "$host:$port" ); @@ -363,7 +367,7 @@ EOM } my $dst = "$tmpdir/scrape"; @cmd = (qw(-clone -q), "http://$host:$port/v2", $dst); - run_script(\@cmd, undef, { 2 => \(my $err = '') }); + run_script(\@cmd, undef, { 2 => \($err = '') }); is($?, 0, 'scraping clone on old PublicInbox::WWW') or diag $err; my @g_all = glob("$dst/git/*.git");
prev parent reply other threads:[~2021-09-25 3:21 UTC|newest] Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-09-24 10:56 [PATCH 0/5] clone|fetch: flesh out partial mirror support Eric Wong 2021-09-24 10:56 ` [PATCH 1/5] clone|--mirror: support --epoch=RANGE for partial clones Eric Wong 2021-09-24 10:56 ` [PATCH 2/5] fetch: fix skipping with multi-epoch inboxes Eric Wong 2021-09-24 10:56 ` [PATCH 3/5] clone|--mirror: fix and test against pre-manifest WWW Eric Wong 2021-09-24 10:56 ` [PATCH 4/5] clone|fetch|--mirror: cull manifest in partial mirrors Eric Wong 2021-09-24 10:56 ` [PATCH 5/5] fetch: support v2 w/o manifest on old WWW Eric Wong 2021-09-25 3:21 ` Eric Wong [this message]
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style List information: https://public-inbox.org/README * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=YU6VnTZaEt6NNi2D@dcvr \ --to=e@80x24.org \ --cc=meta@public-inbox.org \ --subject='Re: [PATCH 6/5] t/v2mirror: check dependencies for legacy test' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Code repositories for project(s) associated with this inbox: https://80x24.org/public-inbox.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).