git: 9e84289f2c7e - main - git-arc: Prefer echo over printf

From: Jose Luis Duran <jlduran_at_FreeBSD.org>
Date: Tue, 05 Nov 2024 02:11:58 UTC
The branch main has been updated by jlduran:

URL: https://cgit.FreeBSD.org/src/commit/?id=9e84289f2c7e6b632047a75b653fcd587287a8b9

commit 9e84289f2c7e6b632047a75b653fcd587287a8b9
Author:     Jose Luis Duran <jlduran@FreeBSD.org>
AuthorDate: 2024-11-05 01:13:22 +0000
Commit:     Jose Luis Duran <jlduran@FreeBSD.org>
CommitDate: 2024-11-05 02:10:16 +0000

    git-arc: Prefer echo over printf
    
    Prefer echo over printf.  Prevent the newline character from being
    translated by using 'blank' instead of 'space' in the re_format.
    
    This change was reviewed in:
    
    https://reviews.freebsd.org/D47289
    
    Committed separately for clarity.
    
    Suggested by:   des
    Approved by:    emaste (mentor)
---
 tools/tools/git/git-arc.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/tools/git/git-arc.sh b/tools/tools/git/git-arc.sh
index 64d1ee2bd63f..050ec4ec1f2c 100644
--- a/tools/tools/git/git-arc.sh
+++ b/tools/tools/git/git-arc.sh
@@ -341,7 +341,7 @@ diff2reviewers()
         jq '.response.data[0].attachments.reviewers.reviewers[] | select(.status == "accepted").reviewerPHID')
     if [ -n "$userids" ]; then
         echo '{
-        "constraints": {"phids": ['"$(printf "%s" "$userids" | tr '[:space:]' ',')"']}
+        "constraints": {"phids": ['"$(echo "$userids" | tr '[:blank:]' ',')"']}
         }' |
         arc_call_conduit -- user.search |
         jq -r '.response.data[].fields.username'