◐ Shell
clean mode source ↗

Cut xtrace noise from POSIX-ownership diagnostic steps by EliahKagan · Pull Request #2156 · gitpython-developers/GitPython

The "Show POSIX file ownership" step in each test workflow looped
over a hard-coded path list with one `ls -ld` per iteration. Bash's
xtrace -- active throughout (from `~/.bash_profile` on Cygwin and
from the `-x` flag in GHA's default shell line on Ubuntu / macOS /
Alpine) -- reprints the entire `for` expression's expanded word list
at the start of every iteration. For nine paths that's nine long
traces of the same word list, drowning out the `ls -ld` output we
want to read.

Collapse the loop into a single multi-arg `ls -ld --`: xtrace prints
the expanded command line once, `ls` produces one line per existing
path and a `ls: cannot access '<path>': No such file or directory`
line per missing path. `2>&1` merges those missing-path messages
into the log stream alongside the existing-path output; `|| true`
keeps the step from failing under `set -e` when any path is missing.

The format of missing-path reporting changes from `(missing: <path>)`
to `ls: cannot access '<path>': No such file or directory`. Both
convey the same information; the new form is slightly more verbose
per missing path but eliminates the per-iteration trace reprint that
dominated the original output.

Cosmetic-only; no change to the diagnostic information surfaced.

Flagged on PR gitpython-developers#2154 as a follow-up:
gitpython-developers#2154 (review)

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

@EliahKagan EliahKagan deleted the claude/cygwin-safe-directory-next branch

May 23, 2026 22:04