git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] t/aggregate-results: fix paste(1) invocation

From
D. Ben Knoble <ben.knoble+github@gmail.com>
Date
Feb 24, 2025, 19:26 UTC
Message-ID
<20250224192724.7625-1-ben.knoble+github@gmail.com>
When running `make test`, when missing prereqs the following is emitted:
    make aggregate-results
    usage: paste [-s] [-d delimiters] file ...
    fixed   1
    success 30066
    failed  0
    broken  218
    total   31274

POSIX says that `paste` requires a file operand; stdin was clearly intended by 49da404070 (test-lib: show missing prereq summary, 2021-11-20). Use it.

Signed-off-by: D. Ben Knoble <ben.knoble+github@gmail.com>
---

The test results are from a real run but on an as-yet-unpublished commit. Not sure if worth keeping that part of the message, so feel free to only keep the "make" and "usage" lines when applying?

 t/aggregate-results.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/t/aggregate-results.sh b/t/aggregate-results.sh
index 6e3bcc4aec..6cb0ff11de 100755
--- a/t/aggregate-results.sh
+++ b/t/aggregate-results.sh
@@ -44,7 +44,7 @@
 		tr -s "," "\n" |
 		grep -v '^$' |
 		sort -u |
-		paste -s -d ' ')
+		paste -s -d ' ' -)
 	if test -n "$unique_missing_prereq"
 	then
 		printf "\nmissing prereq: $unique_missing_prereq\n\n"
-- 
2.48.1
Message 1 of 1 in “t/aggregate-results: fix paste(1) invocation”
  1. t/aggregate-results: fix paste(1) invocationD. Ben Knoble, Feb 24, 2025

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.