libgit2 [was: Re: t/cindex.t "associate w/o search" test hangs for me]

2023-11-25 Thread Eric Wong
Eric Wong wrote: > Konstantin Ryabitsev wrote: > > I'm quite happy to not require libgit2 -- I've always found it easier to > > just > > use git plumbing commands even if this requires exec'ing an external > > executable. Sorry, I forget, individual inboxes via read-only interfaces still benefi

Re: t/cindex.t "associate w/o search" test hangs for me

2023-11-15 Thread Eric Wong
Konstantin Ryabitsev wrote: > I'm quite happy to not require libgit2 -- I've always found it easier to just > use git plumbing commands even if this requires exec'ing an external > executable. Yeah, I don't have libgit2 installed on most of my systems, either. Hoping git itself eventually makes

Re: t/cindex.t "associate w/o search" test hangs for me

2023-11-15 Thread Konstantin Ryabitsev
On Wed, Nov 15, 2023 at 03:09:28AM +, Eric Wong wrote: > > t/imapd.t 2/? Bailout called. Further testing > > stopped: FETCH socket closed while reading data from server > > FAILED--Further testing stopped: FETCH socket closed while reading data > > from server > > make:

Re: t/cindex.t "associate w/o search" test hangs for me

2023-11-14 Thread Eric Wong
Konstantin Ryabitsev wrote: > t/gcf2_client.t .. 1/? Can't locate object method "fail" via > package "PublicInbox::Gcf2Client" at > /home/mricon/public-inbox-test/blib/lib/PublicInbox/Git.pm line 269. > (in cleanup) Can't locate object method "fail" via package > "PublicInbo

Re: t/cindex.t "associate w/o search" test hangs for me

2023-11-14 Thread Konstantin Ryabitsev
On Wed, Nov 15, 2023 at 01:06:42AM +, Eric Wong wrote: > Konstantin Ryabitsev wrote: > > Looks like the last time I am able to successfully run "make test" is before > > this commit: > > > > b231d91f42d791becf7b6861e723833d71e73237 is the first bad commit > > > > The error I start gettin

Re: t/cindex.t "associate w/o search" test hangs for me

2023-11-14 Thread Eric Wong
Konstantin Ryabitsev wrote: > There are two sources of potential discrepancy: > > - differences in CPAN module versions I have installed > - different git or xapian14 versions Could also be CPU count or CPU speeds since IPC bugs are sensitive to that. That said, I can't get cindex.t to fail wit

Re: t/cindex.t "associate w/o search" test hangs for me

2023-11-14 Thread Eric Wong
Konstantin Ryabitsev wrote: > Looks like the last time I am able to successfully run "make test" is before > this commit: > > b231d91f42d791becf7b6861e723833d71e73237 is the first bad commit > > The error I start getting after this commit is: > > t/extsearch.t 160/? > # F

Re: t/cindex.t "associate w/o search" test hangs for me

2023-11-14 Thread Konstantin Ryabitsev
On Tue, Nov 14, 2023 at 06:51:00PM -0500, Konstantin Ryabitsev wrote: > On Tue, Nov 14, 2023 at 11:46:20PM +, Eric Wong wrote: > > My, that's a lot of pipes... > > > > I should've told you to try this debug patch earlier, but this > > might help... (and our test suite should really be able to

Re: t/cindex.t "associate w/o search" test hangs for me

2023-11-14 Thread Konstantin Ryabitsev
On Tue, Nov 14, 2023 at 11:46:20PM +, Eric Wong wrote: > My, that's a lot of pipes... > > I should've told you to try this debug patch earlier, but this > might help... (and our test suite should really be able to > watch messages like this while capturing) Will try this shorty -- currently

Re: t/cindex.t "associate w/o search" test hangs for me

2023-11-14 Thread Eric Wong
Konstantin Ryabitsev wrote: > On Tue, Nov 14, 2023 at 10:46:57PM +, Eric Wong wrote: > > > I can't do +E because that's not available to me under CentOS7 (I can't > > > wait > > > until we move on, but just when we think the yak is fully shaved, we find > > > more > > > clumps of thick fur w

Re: t/cindex.t "associate w/o search" test hangs for me

2023-11-14 Thread Konstantin Ryabitsev
On Tue, Nov 14, 2023 at 10:46:57PM +, Eric Wong wrote: > > I can't do +E because that's not available to me under CentOS7 (I can't wait > > until we move on, but just when we think the yak is fully shaved, we find > > more > > clumps of thick fur we hadn't considered). Is the output of the reg

Re: t/cindex.t "associate w/o search" test hangs for me

2023-11-14 Thread Eric Wong
Konstantin Ryabitsev wrote: > On Tue, Nov 14, 2023 at 10:16:53PM +, Eric Wong wrote: > > Konstantin Ryabitsev wrote: > > > └─-cindex -u --al,4432 > > > ├─cidx shard[0],4646 > > > └─cidx shard[1],4647 > > > > > > Anything I can do to figure out why this is h

Re: t/cindex.t "associate w/o search" test hangs for me

2023-11-14 Thread Konstantin Ryabitsev
On Tue, Nov 14, 2023 at 10:16:53PM +, Eric Wong wrote: > Konstantin Ryabitsev wrote: > > └─-cindex -u --al,4432 > > ├─cidx shard[0],4646 > > └─cidx shard[1],4647 > > > > Anything I can do to figure out why this is happening? > > You can show me strace and l

Re: t/cindex.t "associate w/o search" test hangs for me

2023-11-14 Thread Eric Wong
Konstantin Ryabitsev wrote: > └─-cindex -u --al,4432 > ├─cidx shard[0],4646 > └─cidx shard[1],4647 > > Anything I can do to figure out why this is happening? You can show me strace and lsof +E of the processes (any other processes (join|sort|awk|perl)?). This

t/cindex.t "associate w/o search" test hangs for me

2023-11-14 Thread Konstantin Ryabitsev
Eric: I'm trying to have tests pass on CentOS7 with the current master and I'm apparently not able to get past the "associate w/o search" test. When I run `prove -bvw t/cindex.t` I get to: ok 76 - xcpdb compact and then it just sits there. If I look at the process table, I can see that the