Jeff King [off-list ref] writes:
quoted
+ if (mailmap < 0)
mailmap = 0;
- }
This should be "mailmap = 1" to match the commit message, no? (Which
also implies we may want a new test).
Gaa, of course.
I'd also be OK with leaving it at "0" for now, making a note of the
upcoming change in the release notes, and flipping it _next_ time. But
IMHO we don't need to be that conservative (and I share your pessimism
that enough people actually read the release notes to merit that kind of
advance notice).
I'd actually think s/need to be/get any benefit from being/; it
would be wasted effort to be extra careful, and any way we cut it,
there will always be people who complain.