Re: [PATCH v2 1/2] rebase doc: document rebase.useBuiltin
From: Junio C Hamano <hidden>
Date: 2018-11-16 09:58:17
Johannes Schindelin [off-list ref] writes:
Meaning: essentially, `rebase.useBuiltin` was defaulting to `false`, and if a user installed Git for Windows with the experimental built-in rebase, it was set to `true` in the system config.
Oh, that changes the picture entirely. If that was what was shipped to Windows users for 2.19.X, then I'd say we should be able to trust the switch well enough. I just somehow thought that everybody in the Windows land has been using the -in-C version.
quoted
Having said that, assuming that the switching back to scripted version works correctly and assuming that we want to expose this to end users, I think the patch text makes sense.Indeed. I would still love to see the built-in rebase to be used by default in v2.20.0, and I am reasonably sure that the escape hatch works (because, as I told you above, it worked in the reverse, making the built-in rebase an opt-in in Git for Windows v2.19.1).
Good. That makes things a lot simpler. Thanks.