Thread (1 message) 1 message, 1 author, 2018-09-27

Re: Triggering "BUG: wt-status.c:476: multiple renames on the same target? how?"

From: Elijah Newren <hidden>
Date: 2018-09-27 04:20:35

On Wed, Sep 26, 2018, 2:27 PM Ævar Arnfjörð Bjarmason [off-list ref] wrote:
On Wed, Sep 26 2018, Andrea Stacchiotti wrote:
quoted
I'm very sorry, I indeed forgot the `diff.renames=copies`.

The following script can reproduce the bug even with a blank config:
Thanks for the bug report and the simple testcase.
quoted
---------------------

# Make a test repo
git init testrepo
cd testrepo
git config user.name A
git config user.email B
git config diff.renames copies

# Add a file called orig
echo 'a' > orig
git add orig
git commit -m'orig'

# Copy orig in new and modify orig
cp orig new
echo 'b' > orig

# add -N and then commit trigger the bug
git add -N new
git commit

# Cleanup
cd ..
rm -rf testrepo
Thanks. Bisecting shows that the bug is in dc6b1d92ca ("wt-status: use
settings from git_diff_ui_config", 2018-05-04) first released with
2.18.0.
The bisect is slightly misleading; the bug was introduced in 2.17.0
for renames, and when copy detection became a thing in 2.18.0 it also
incidentally would trigger with copies.  I'll post a patch soon.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help