From: Johannes Schindelin <Johannes.Schindelin@gmx.de>
To: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
Cc: git@vger.kernel.org, Junio C Hamano <gitster@pobox.com>,
Phillip Wood <phillip.wood123@gmail.com>
Subject: Re: [PATCH] rebase: remove transitory rebase.useBuiltin setting & env
Date: Tue, 23 Mar 2021 21:52:22 +0100 (CET) [thread overview]
Message-ID: <nycvar.QRO.7.76.6.2103232152090.50@tvgsbejvaqbjf.bet> (raw)
In-Reply-To: <patch-1.1-df026b00b67-20210323T152302Z-avarab@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 4346 bytes --]
Hi Ævar,
On Tue, 23 Mar 2021, Ævar Arnfjörð Bjarmason wrote:
> Remove the rebase.useBuiltin setting and the now-obsolete
> GIT_TEST_REBASE_USE_BUILTIN test flag.
>
> This was left in place after my d03ebd411c6 (rebase: remove the
> rebase.useBuiltin setting, 2019-03-18) to help anyone who'd used the
> experimental flag and wanted to know that it was the default, or that
> they should transition their test environment to use the builtin
> rebase unconditionally.
>
> It's been more than long enough for those users to get a headsup about
> this. So remove all the scaffolding that was left inplace after
> d03ebd411c6. I'm also removing the documentation entry, if anyone
> still has this left in their configuration they can do some source
> archaeology to figure out what it used to do, which makes more sense
> than exposing every git user reading the documentation to this legacy
> configuration switch.
>
> Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
ACK!
Dscho
> ---
> Documentation/config/rebase.txt | 7 -------
> builtin/rebase.c | 11 -----------
> t/t3400-rebase.sh | 16 ----------------
> 3 files changed, 34 deletions(-)
>
> diff --git a/Documentation/config/rebase.txt b/Documentation/config/rebase.txt
> index 214f31b451f..8c979cb20f2 100644
> --- a/Documentation/config/rebase.txt
> +++ b/Documentation/config/rebase.txt
> @@ -1,10 +1,3 @@
> -rebase.useBuiltin::
> - Unused configuration variable. Used in Git versions 2.20 and
> - 2.21 as an escape hatch to enable the legacy shellscript
> - implementation of rebase. Now the built-in rewrite of it in C
> - is always used. Setting this will emit a warning, to alert any
> - remaining users that setting this now does nothing.
> -
> rebase.backend::
> Default backend to use for rebasing. Possible choices are
> 'apply' or 'merge'. In the future, if the merge backend gains
> diff --git a/builtin/rebase.c b/builtin/rebase.c
> index de400f9a197..783b526f6e7 100644
> --- a/builtin/rebase.c
> +++ b/builtin/rebase.c
> @@ -100,7 +100,6 @@ struct rebase_options {
> char *strategy, *strategy_opts;
> struct strbuf git_format_patch_opt;
> int reschedule_failed_exec;
> - int use_legacy_rebase;
> int reapply_cherry_picks;
> int fork_point;
> };
> @@ -1102,11 +1101,6 @@ static int rebase_config(const char *var, const char *value, void *data)
> return 0;
> }
>
> - if (!strcmp(var, "rebase.usebuiltin")) {
> - opts->use_legacy_rebase = !git_config_bool(var, value);
> - return 0;
> - }
> -
> if (!strcmp(var, "rebase.backend")) {
> return git_config_string(&opts->default_backend, var, value);
> }
> @@ -1441,11 +1435,6 @@ int cmd_rebase(int argc, const char **argv, const char *prefix)
> gpg_sign = options.gpg_sign_opt ? "" : NULL;
> FREE_AND_NULL(options.gpg_sign_opt);
>
> - if (options.use_legacy_rebase ||
> - !git_env_bool("GIT_TEST_REBASE_USE_BUILTIN", -1))
> - warning(_("the rebase.useBuiltin support has been removed!\n"
> - "See its entry in 'git help config' for details."));
> -
> strbuf_reset(&buf);
> strbuf_addf(&buf, "%s/applying", apply_dir());
> if(file_exists(buf.buf))
> diff --git a/t/t3400-rebase.sh b/t/t3400-rebase.sh
> index 587b408063a..0bb88aa982b 100755
> --- a/t/t3400-rebase.sh
> +++ b/t/t3400-rebase.sh
> @@ -388,22 +388,6 @@ test_expect_success 'rebase--merge.sh and --show-current-patch' '
> )
> '
>
> -test_expect_success 'rebase -c rebase.useBuiltin=false warning' '
> - expected="rebase.useBuiltin support has been removed" &&
> -
> - # Only warn when the legacy rebase is requested...
> - test_must_fail git -c rebase.useBuiltin=false rebase 2>err &&
> - test_i18ngrep "$expected" err &&
> - test_must_fail env GIT_TEST_REBASE_USE_BUILTIN=false git rebase 2>err &&
> - test_i18ngrep "$expected" err &&
> -
> - # ...not when we would have used the built-in anyway
> - test_must_fail git -c rebase.useBuiltin=true rebase 2>err &&
> - test_must_be_empty err &&
> - test_must_fail env GIT_TEST_REBASE_USE_BUILTIN=true git rebase 2>err &&
> - test_must_be_empty err
> -'
> -
> test_expect_success 'switch to branch checked out here' '
> git checkout main &&
> git rebase main main
> --
> 2.31.0.366.g75c23356abd
>
>
prev parent reply other threads:[~2021-03-23 20:53 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-28 10:26 [PATCH] rebase -x: sanity check command Phillip Wood
2019-01-28 18:23 ` Junio C Hamano
2019-01-28 21:56 ` Johannes Schindelin
2019-01-29 11:40 ` Phillip Wood
2019-01-29 15:35 ` Johannes Schindelin
2019-01-28 22:03 ` Johannes Schindelin
2019-01-29 11:34 ` Phillip Wood
2019-01-29 15:32 ` Johannes Schindelin
2019-01-29 18:43 ` [PATCH v2] " Phillip Wood
2019-01-29 21:53 ` Junio C Hamano
2019-01-30 12:25 ` Johannes Schindelin
2019-02-13 13:31 ` Ævar Arnfjörð Bjarmason
2019-02-13 14:22 ` [PATCH] rebase: remove the rebase.useBuiltin setting Ævar Arnfjörð Bjarmason
2019-02-13 16:25 ` Johannes Schindelin
2019-02-13 20:46 ` Junio C Hamano
2019-02-13 21:49 ` [PATCH] rebase: fix regression in rebase.useBuiltin=false test mode Ævar Arnfjörð Bjarmason
2019-02-13 23:21 ` Junio C Hamano
2019-02-14 16:12 ` Phillip Wood
2019-03-14 13:24 ` [PATCH v2] rebase: remove the rebase.useBuiltin setting Ævar Arnfjörð Bjarmason
2019-03-14 14:58 ` Johannes Schindelin
2019-03-14 15:27 ` Ævar Arnfjörð Bjarmason
2019-03-15 13:45 ` [PATCH v3] " Ævar Arnfjörð Bjarmason
2019-03-15 15:44 ` Johannes Schindelin
2019-03-15 16:11 ` Ævar Arnfjörð Bjarmason
2019-03-18 6:06 ` Junio C Hamano
2019-03-18 10:19 ` Phillip Wood
2019-03-18 11:01 ` [PATCH v4] " Ævar Arnfjörð Bjarmason
2019-03-19 10:21 ` Phillip Wood
2021-03-23 15:23 ` [PATCH] rebase: remove transitory rebase.useBuiltin setting & env Ævar Arnfjörð Bjarmason
2021-03-23 20:42 ` Junio C Hamano
2021-03-23 20:52 ` Johannes Schindelin [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=nycvar.QRO.7.76.6.2103232152090.50@tvgsbejvaqbjf.bet \
--to=johannes.schindelin@gmx.de \
--cc=avarab@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=phillip.wood123@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).