{"thread":{"id":"62995","subject":"[PATCH] t/aggregate-results: fix paste(1) invocation","startedAt":"2025-02-24T19:27:37Z","lastAt":"2025-02-24T19:27:37Z","messageCount":1,"participants":["D. Ben Knoble"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"512928","messageId":"20250224192724.7625-1-ben.knoble+github@gmail.com","threadId":"62995","inReplyTo":null,"subject":"[PATCH] t/aggregate-results: fix paste(1) invocation","fromName":"D. Ben Knoble","fromEmail":"ben.knoble+github@gmail.com","sentAt":"2025-02-24T19:26:02Z","receivedAt":"2025-02-24T19:27:37Z","isPatch":true,"sender":{"key":"ben.knoble@gmail.com","avatar":"https://avatars.githubusercontent.com/u/22802209?v=4"},"body":"When running `make test`, when missing prereqs the following is emitted:\n\n    make aggregate-results\n    usage: paste [-s] [-d delimiters] file ...\n    fixed   1\n    success 30066\n    failed  0\n    broken  218\n    total   31274\n\nPOSIX says that `paste` requires a file operand; stdin was clearly\nintended by 49da404070 (test-lib: show missing prereq summary,\n2021-11-20). Use it.\n\nSigned-off-by: D. Ben Knoble <ben.knoble+github@gmail.com>\n---\n\nThe test results are from a real run but on an as-yet-unpublished commit. Not\nsure if worth keeping that part of the message, so feel free to only keep the\n\"make\" and \"usage\" lines when applying?\n\n t/aggregate-results.sh | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/t/aggregate-results.sh b/t/aggregate-results.sh\nindex 6e3bcc4aec..6cb0ff11de 100755\n--- a/t/aggregate-results.sh\n+++ b/t/aggregate-results.sh\n@@ -44,7 +44,7 @@\n \t\ttr -s \",\" \"\\n\" |\n \t\tgrep -v '^$' |\n \t\tsort -u |\n-\t\tpaste -s -d ' ')\n+\t\tpaste -s -d ' ' -)\n \tif test -n \"$unique_missing_prereq\"\n \tthen\n \t\tprintf \"\\nmissing prereq: $unique_missing_prereq\\n\\n\"\n-- \n2.48.1\n\n"}]}