The branch main has been updated by jrm:

URL: 
https://cgit.FreeBSD.org/src/commit/?id=6049f1bc210653b126974961b29d6eb71a6c6709

commit 6049f1bc210653b126974961b29d6eb71a6c6709
Author:     Joseph Mingrone <[email protected]>
AuthorDate: 2025-11-27 22:46:43 +0000
Commit:     Joseph Mingrone <[email protected]>
CommitDate: 2025-11-28 00:53:56 +0000

    git-arc: Fix failure to call arc() function
    
    As of b3e53f9fff11, git-arc attempted to call the internal shell
    function, arc(), using env(1).  However, because env(1) does not call
    shell functions, it actually attempted to run the arc utility.  This led
    to errors:
    
        % git arc create -r xxx HEAD
        env: arc: No such file or directory
        git-arc: could not create Phabricator diff
    
    This change removes the unnecessary use of env(1), so the arc() function
    is correctly called.
    
    Reviewed by:    markj
    Fixes:          b3e53f9fff11 ("git-arc: Don't require devel/arcanist")
    Sponsored by:   The FreeBSD Foundation
    Differential Revision:  https://reviews.freebsd.org/D53972
---
 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 eef01b75663e..8507d96fda63 100755
--- a/tools/tools/git/git-arc.sh
+++ b/tools/tools/git/git-arc.sh
@@ -315,7 +315,7 @@ create_one_review()
     printf "\nSubscribers:\n" >> "$msg"
     printf "%s\n" "${subscribers}" >> "$msg"
 
-    yes | env EDITOR=true \
+    yes | EDITOR=true \
         arc diff --message-file "$msg" --never-apply-patches --create \
         --allow-untracked $BROWSE --head "$commit" "${commit}~"
     [ $? -eq 0 ] || err "could not create Phabricator diff"

Reply via email to