[PATCH 0/2] builtin/mv: bail out when trying to move child and its parent
- From
Patrick Steinhardt <ps@pks.im>
- Date
- Apr 30, 2025, 12:44 UTC
- Message-ID
- <20250430-pks-mv-parent-child-conflict-v1-0-11a87c55ffb9@pks.im>
Hi,
this series addresses a bug in git-mv(1) where moving both a child and its parent directory may cause an assert to trigger. This discussion came up in the context of [1] as the assert causes problems on our CI systems on Windows.
Thanks!
Patrick
[1]: <pull.1908.git.1745593515875.gitgitgadget@gmail.com>
---
Patrick Steinhardt (2):
builtin/mv: bail out when trying to move child and its parent
builtin/mv: convert assert(3p) into `BUG()`builtin/mv.c | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++--- t/t7001-mv.sh | 24 ++++++++++++++++++---- 2 files changed, 81 insertions(+), 7 deletions(-)
--- base-commit: 6c0bd1fc70efaf053abe4e57c976afdc72d15377 change-id: 20250430-pks-mv-parent-child-conflict-1c15f775e638