Hi Junio,
On Tue, Sep 6, 2022 at 12:41 AM Junio C Hamano <gitster@xxxxxxxxx> wrote:
> * en/remerge-diff-fixes (2022-09-02) 3 commits
> - diff: fix filtering of merge commits under --remerge-diff
> - diff: fix filtering of additional headers under --remerge-diff
> - diff: have submodule_format logic avoid additional diff headers
>
> Fix a few "git log --remerge-diff" bugs.
>
> Will probably need to rebase to make it mergeable to 'maint'.
> source: <pull.1342.v3.git.1662090810.gitgitgadget@xxxxxxxxx>
It rebases to 'maint' without conflict and the result correctly
applies there...but rebasing to maint causes it to drop a line of code
("TEST_PASSES_SANITIZE_LEAK=true") which is called out in the commit
message, making the commit message look funny. Also, while that line
removal turns out to not matter to maint (because the leak checker was
already off), it is critical for main. So, if you merge the rebased
result back to main, the leak check will fail -- unless the merge back
is an "evil merge" that restores the removal of that line.
So, which of the following would you prefer?
* A separate series for maint and main?
* Just ignore maint?
* Me to rebase on maint to tweak the commit message, and then you
carefully reinstate the line removal as part of the merge back to
main?
