user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
Search results ordered by [date|relevance]  view[summary|nested|Atom feed]
thread overview below | download mbox.gz: |
* [PATCH] test_common: fix excessive wait for GNU tail inotify
@ 2023-11-20  7:39  7% Eric Wong
  0 siblings, 0 replies; 1+ results
From: Eric Wong @ 2023-11-20  7:39 UTC (permalink / raw)
  To: meta

We want to use the filenames tail will watch, not the number of
args passed to the `tail_f' subroutine.

Fixes: 9231d2e7b93f (tests: map CLOFORK->FD_CLOEXEC temporarily for `tail -f')
---
 lib/PublicInbox/TestCommon.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/PublicInbox/TestCommon.pm b/lib/PublicInbox/TestCommon.pm
index 8bfa30f2..0d17e6e7 100644
--- a/lib/PublicInbox/TestCommon.pm
+++ b/lib/PublicInbox/TestCommon.pm
@@ -512,7 +512,7 @@ sub tail_f (@) {
 	while (my ($io, $fl) = splice(@cfmap, 0, 2)) {
 		fcntl($io, F_SETFD, $fl);
 	}
-	wait_for_tail($pid, scalar @_);
+	wait_for_tail($pid, scalar @f);
 	require PublicInbox::AutoReap;
 	PublicInbox::AutoReap->new($pid, \&wait_for_tail);
 }

^ permalink raw reply related	[relevance 7%]

Results 1-1 of 1 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2023-11-20  7:39  7% [PATCH] test_common: fix excessive wait for GNU tail inotify Eric Wong

Code repositories for project(s) associated with this public 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).