From patchwork Tue Jun 28 13:25:52 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Derrick Stolee X-Patchwork-Id: 12898280 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id ECBF4C433EF for ; Tue, 28 Jun 2022 13:28:09 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S245099AbiF1N2H (ORCPT ); Tue, 28 Jun 2022 09:28:07 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56710 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1346450AbiF1N13 (ORCPT ); Tue, 28 Jun 2022 09:27:29 -0400 Received: from mail-wr1-x429.google.com (mail-wr1-x429.google.com [IPv6:2a00:1450:4864:20::429]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B01EC3335E for ; Tue, 28 Jun 2022 06:26:03 -0700 (PDT) Received: by mail-wr1-x429.google.com with SMTP id n1so17667161wrg.12 for ; Tue, 28 Jun 2022 06:26:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=message-id:in-reply-to:references:from:date:subject:fcc :content-transfer-encoding:mime-version:to:cc; bh=5H1/BkGNzKZHdO7ThWj9L1t8uHwFXO4zO/yDHCP54Rw=; b=SilxFcIUqc9l2yJ+nHxNDYypWi06Sr5ji15AQHLz7P7Mu2OP+ojpsTFTTg2wIq1VlB e9LchsyEvP5OFe0+xq8V50HPojD7JiVH6VmR8FqXN0FJbIvcHxjLp2TRmLXRx8HlShdF l4i0N7WUEX1G8JqZx9v3O6+lz4SQ6Uh0VkLzquWgj4rhXJEOw43VAZeoJHEtuknRlPyQ nZDS0HBFVni1StdW843ir1iCrpjpqW75O9RyLn/mlrft/onqpvUiDQsWi1HFqsWxykAN kJXNSYZHba+v6COLfuTZFZ6K0hKO+yEd+Uz94KjJayuvmTLgkvJ4Hz7NMIjgMtxJgZTp Ht0w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:in-reply-to:references:from:date :subject:fcc:content-transfer-encoding:mime-version:to:cc; bh=5H1/BkGNzKZHdO7ThWj9L1t8uHwFXO4zO/yDHCP54Rw=; b=cfo7N9FX4S4kdSuI86becn9NMzik4yyL4fuieWztw88d2ltZKUxLU7YdppY1WBhT3s fxtElnf6/eE2c+QB3Lm/0cgn4F46h6lib8WxVVKiV4nv9ZXQw6K7fTKn5uDZd+3LTGhk tD4zYnGtoYomjpLh0RP1RXY7wyLdqz5M7zPkyHCfe94+hDMBa9DoQsXNkg5fFXJfUYW4 2/RQv0j1yWaice30YTKcfAwU+kaDPMhYkoQMdrV9LUKBIh8Tk1WHsbNbzytKBrc1VTsx KlnGdPOgjxzcIF+7ZzzCtnwlb1YoyKoSy23GrQEcvkJVY5a5MKz55qqXbZq6H/tJEF2M oLBA== X-Gm-Message-State: AJIora9ZE5DgkVWz1Q8Q83jYWchptu3B/9HBHF38Q34XV8UlZz6N89qt p2opyU3cig0FNv4nIexIuKj5z6gWotOhow== X-Google-Smtp-Source: AGRyM1uZYy87pJdagF81T5//YBqT3hdfJ710vV0oJr/eUaeE3kdWCTuPI8amIZg65rs6xxVbTvwHcw== X-Received: by 2002:a5d:518f:0:b0:21b:8a8c:ce4 with SMTP id k15-20020a5d518f000000b0021b8a8c0ce4mr18200882wrv.614.1656422761921; Tue, 28 Jun 2022 06:26:01 -0700 (PDT) Received: from [127.0.0.1] ([13.74.141.28]) by smtp.gmail.com with ESMTPSA id k19-20020a7bc413000000b003a02b135747sm17129003wmi.46.2022.06.28.06.26.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 28 Jun 2022 06:26:01 -0700 (PDT) Message-Id: In-Reply-To: References: Date: Tue, 28 Jun 2022 13:25:52 +0000 Subject: [PATCH v3 1/8] t2407: test branches currently using apply backend Fcc: Sent MIME-Version: 1.0 To: git@vger.kernel.org Cc: gitster@pobox.com, johannes.schindelin@gmx.de, me@ttaylorr.com, Jeff Hostetler , Phillip Wood , Elijah Newren , Derrick Stolee , Derrick Stolee Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org From: Derrick Stolee From: Derrick Stolee The tests in t2407 that verify the branch_checked_out() helper in the case of bisects and rebases were added by 9347303db89 (branch: check for bisects and rebases, 2022-06-08). However, that commit failed to check for rebases that are using the 'apply' backend. Add a test that checks the apply backend. The implementation was already correct here, but it is good to have regression tests before modifying the implementation further. Signed-off-by: Derrick Stolee --- t/t2407-worktree-heads.sh | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/t/t2407-worktree-heads.sh b/t/t2407-worktree-heads.sh index b6be42f74a2..4f59bc21303 100755 --- a/t/t2407-worktree-heads.sh +++ b/t/t2407-worktree-heads.sh @@ -54,7 +54,18 @@ test_expect_success 'refuse to overwrite: worktree in bisect' ' grep "cannot force update the branch '\''fake-2'\'' checked out at.*wt-4" err ' -test_expect_success 'refuse to overwrite: worktree in rebase' ' +test_expect_success 'refuse to overwrite: worktree in rebase (apply)' ' + test_when_finished rm -rf .git/worktrees/wt-*/rebase-apply && + + mkdir -p .git/worktrees/wt-3/rebase-apply && + echo refs/heads/fake-1 >.git/worktrees/wt-3/rebase-apply/head-name && + echo refs/heads/fake-2 >.git/worktrees/wt-3/rebase-apply/onto && + + test_must_fail git branch -f fake-1 HEAD 2>err && + grep "cannot force update the branch '\''fake-1'\'' checked out at.*wt-3" err +' + +test_expect_success 'refuse to overwrite: worktree in rebase (merge)' ' test_when_finished rm -rf .git/worktrees/wt-*/rebase-merge && mkdir -p .git/worktrees/wt-3/rebase-merge && From patchwork Tue Jun 28 13:25:53 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Derrick Stolee X-Patchwork-Id: 12898308 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E3D63C433EF for ; Tue, 28 Jun 2022 13:28:28 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1345728AbiF1N21 (ORCPT ); Tue, 28 Jun 2022 09:28:27 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57096 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1346456AbiF1N13 (ORCPT ); Tue, 28 Jun 2022 09:27:29 -0400 Received: from mail-wr1-x42b.google.com (mail-wr1-x42b.google.com [IPv6:2a00:1450:4864:20::42b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B668F9B for ; Tue, 28 Jun 2022 06:26:04 -0700 (PDT) Received: by mail-wr1-x42b.google.com with SMTP id q9so17698590wrd.8 for ; Tue, 28 Jun 2022 06:26:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=message-id:in-reply-to:references:from:date:subject:fcc :content-transfer-encoding:mime-version:to:cc; bh=+P/IdGGcNRo3SFAFLQ+TOgSB8n0GO2oH0rFfkBNzKtk=; b=oMz/tlWg35H4AXrg3sdt3hrYisVcDVwNd2n0tdLRPCK/E9gVAUZv6oO/jHx53z46Ce 7zwmn4YaYjQImc0qMV1ZAGXSxGGJQEe4MWX04FpRmLhRwIdftEcU/fUCktUaCAuFRRUH T+nHeDaxI+NiWnddUueOlC0YYDMXhTb2gGUUKEQUwVe6wdlCWK4gSfhqS6pOFmoEjp3H rm6WBLsCEi5te5lSyAinNA9o0x8GLB8ilNscW4exoQjWh0Pvsa0MrgLq3wTqm9qKtHOb cofuLb02y+MVWow2acplYMC01kO3dq8kqeluAyZD7Bb8a7o9CV4f+hQXaypqYdJuagUl /qLw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:in-reply-to:references:from:date :subject:fcc:content-transfer-encoding:mime-version:to:cc; bh=+P/IdGGcNRo3SFAFLQ+TOgSB8n0GO2oH0rFfkBNzKtk=; b=d3NS1xP7LmJ7dNRl2aP86u0bS8b5Jx44XVBJLsY9CHRIr536WsWmJw/p48I5pAoTM7 CS8o6v3wJOXuljNKdjVcvj/8+5CbAcAiQ8Xok2I+IVZEESC/ocBue799EeFP9Mu865n8 Xb23Gf67LSYnLm51uzLWp/1Qm3a1hZnEA0sN76Xfk+/R6ft24uYwD7GGeFf8buMeDRtX rfwz3JbIQj3pDBClVjKN8ig+Ef9uLW2UYz00bmgXLQMWmp/vvfvP2YQUhsQMlpSbjSPD tbY7obhC25sAkddIgvI3x6xQ6RWNLo5kYGeApHF7u/lhOwk0hNkH9iUr2HLyJXM1AaqU DkAw== X-Gm-Message-State: AJIora9roBti0mG0f5WdOYu9wuaU98z+jLTLk7bp/y+z6eAbXQISKxqY i8TwWj0QORTWL53ZaHGj2vqLmnBFv3tk2Q== X-Google-Smtp-Source: AGRyM1vhBDDWVFb+eu3meDfvvy+PkKApJUWEXXUcUTrw6hIxIiSKRrHTpETF6RUCuDOaIoSh9hmu5w== X-Received: by 2002:a5d:5047:0:b0:21b:92b2:f34f with SMTP id h7-20020a5d5047000000b0021b92b2f34fmr17118372wrt.677.1656422762967; Tue, 28 Jun 2022 06:26:02 -0700 (PDT) Received: from [127.0.0.1] ([13.74.141.28]) by smtp.gmail.com with ESMTPSA id h8-20020a05600c350800b003a0502c620dsm3867317wmq.44.2022.06.28.06.26.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 28 Jun 2022 06:26:02 -0700 (PDT) Message-Id: <2bc647b6fcd6032a1e97e67739ced31117bfbfce.1656422759.git.gitgitgadget@gmail.com> In-Reply-To: References: Date: Tue, 28 Jun 2022 13:25:53 +0000 Subject: [PATCH v3 2/8] branch: consider refs under 'update-refs' Fcc: Sent MIME-Version: 1.0 To: git@vger.kernel.org Cc: gitster@pobox.com, johannes.schindelin@gmx.de, me@ttaylorr.com, Jeff Hostetler , Phillip Wood , Elijah Newren , Derrick Stolee , Derrick Stolee Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org From: Derrick Stolee From: Derrick Stolee The branch_checked_out() helper helps commands like 'git branch' and 'git fetch' from overwriting refs that are currently checked out in other worktrees. A future update to 'git rebase' will introduce a new '--update-refs' option which will update the local refs that point to commits that are being rebased. To avoid collisions as the rebase completes, we want to make the future data store for these refs to be considered by branch_checked_out(). The data store is a plaintext file inside the 'rebase-merge' directory for that worktree. The file alternates refnames and OIDs. The OIDs will be used to store the to-be-written values as the rebase progresses, but can be ignored at the moment. Create a new sequencer_get_update_refs_state() method that parses this file and populates a struct string_list with the ref-OID pairs. We can then use this list to add to the current_checked_out_branches strmap used by branch_checked_out(). To properly navigate to the rebase directory for a given worktree, extract the static strbuf_worktree_gitdir() method to a public API method. We can test that this works without having Git write this file by artificially creating one in our test script. Signed-off-by: Derrick Stolee --- branch.c | 13 ++++++++++++ sequencer.c | 42 +++++++++++++++++++++++++++++++++++++++ sequencer.h | 9 +++++++++ t/t2407-worktree-heads.sh | 22 ++++++++++++++++---- 4 files changed, 82 insertions(+), 4 deletions(-) diff --git a/branch.c b/branch.c index 526e8237673..f252c4eef6c 100644 --- a/branch.c +++ b/branch.c @@ -365,6 +365,7 @@ static void prepare_checked_out_branches(void) char *old; struct wt_status_state state = { 0 }; struct worktree *wt = worktrees[i++]; + struct string_list update_refs = STRING_LIST_INIT_DUP; if (wt->is_bare) continue; @@ -400,6 +401,18 @@ static void prepare_checked_out_branches(void) strbuf_release(&ref); } wt_status_state_free_buffers(&state); + + if (!sequencer_get_update_refs_state(get_worktree_git_dir(wt), + &update_refs)) { + struct string_list_item *item; + for_each_string_list_item(item, &update_refs) { + old = strmap_put(¤t_checked_out_branches, + item->string, + xstrdup(wt->path)); + free(old); + } + string_list_clear(&update_refs, 1); + } } free_worktrees(worktrees); diff --git a/sequencer.c b/sequencer.c index 8c3ed3532ac..1094e146b99 100644 --- a/sequencer.c +++ b/sequencer.c @@ -5901,3 +5901,45 @@ int sequencer_determine_whence(struct repository *r, enum commit_whence *whence) return 0; } + +int sequencer_get_update_refs_state(const char *wt_dir, + struct string_list *refs) +{ + int result = 0; + struct stat st; + FILE *fp = NULL; + struct strbuf ref = STRBUF_INIT; + struct strbuf hash = STRBUF_INIT; + char *path = xstrfmt("%s/rebase-merge/update-refs", wt_dir); + + if (stat(path, &st)) + goto cleanup; + + fp = fopen(path, "r"); + if (!fp) + goto cleanup; + + while (strbuf_getline(&ref, fp) != EOF) { + struct object_id oid; + struct string_list_item *item; + + if (strbuf_getline(&hash, fp) == EOF || + get_oid_hex(hash.buf, &oid)) { + warning(_("update-refs file at '%s' is invalid"), + path); + result = -1; + goto cleanup; + } + + item = string_list_append(refs, ref.buf); + item->util = oiddup(&oid); + } + +cleanup: + if (fp) + fclose(fp); + free(path); + strbuf_release(&ref); + strbuf_release(&hash); + return result; +} diff --git a/sequencer.h b/sequencer.h index da64473636b..3ae541bb145 100644 --- a/sequencer.h +++ b/sequencer.h @@ -232,4 +232,13 @@ void sequencer_post_commit_cleanup(struct repository *r, int verbose); int sequencer_get_last_command(struct repository* r, enum replay_action *action); int sequencer_determine_whence(struct repository *r, enum commit_whence *whence); + +/** + * Append the set of ref-OID pairs that are currently stored for the 'git + * rebase --update-refs' feature if such a rebase is currently happening. + * + * Localized to a worktree's git dir. + */ +int sequencer_get_update_refs_state(const char *wt_dir, struct string_list *refs); + #endif /* SEQUENCER_H */ diff --git a/t/t2407-worktree-heads.sh b/t/t2407-worktree-heads.sh index 4f59bc21303..f1e9e172a0c 100755 --- a/t/t2407-worktree-heads.sh +++ b/t/t2407-worktree-heads.sh @@ -7,8 +7,11 @@ TEST_PASSES_SANITIZE_LEAK=true test_expect_success 'setup' ' test_commit init && - git branch -f fake-1 && - git branch -f fake-2 && + + for i in 1 2 3 4 + do + git branch -f fake-$i || return 1 + done && for i in 1 2 3 4 do @@ -73,8 +76,19 @@ test_expect_success 'refuse to overwrite: worktree in rebase (merge)' ' echo refs/heads/fake-1 >.git/worktrees/wt-3/rebase-merge/head-name && echo refs/heads/fake-2 >.git/worktrees/wt-3/rebase-merge/onto && - test_must_fail git branch -f fake-1 HEAD 2>err && - grep "cannot force update the branch '\''fake-1'\'' checked out at.*wt-3" err + cat >.git/worktrees/wt-3/rebase-merge/update-refs <<-EOF && + refs/heads/fake-3 + $(git rev-parse HEAD~1) + refs/heads/fake-4 + $(git rev-parse HEAD) + EOF + + for i in 1 3 4 + do + test_must_fail git branch -f fake-$i HEAD 2>err && + grep "cannot force update the branch '\''fake-$i'\'' checked out at.*wt-3" err || + return 1 + done ' test_expect_success !SANITIZE_LEAK 'refuse to fetch over ref: checked out' ' From patchwork Tue Jun 28 13:25:54 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Derrick Stolee X-Patchwork-Id: 12898307 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id C7211C43334 for ; Tue, 28 Jun 2022 13:28:30 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1345885AbiF1N23 (ORCPT ); Tue, 28 Jun 2022 09:28:29 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56836 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1346455AbiF1N13 (ORCPT ); Tue, 28 Jun 2022 09:27:29 -0400 Received: from mail-wm1-x330.google.com (mail-wm1-x330.google.com [IPv6:2a00:1450:4864:20::330]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 97B7C3336C for ; Tue, 28 Jun 2022 06:26:05 -0700 (PDT) Received: by mail-wm1-x330.google.com with SMTP id g39-20020a05600c4ca700b003a03ac7d540so6745848wmp.3 for ; Tue, 28 Jun 2022 06:26:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=message-id:in-reply-to:references:from:date:subject:fcc :content-transfer-encoding:mime-version:to:cc; bh=/DTqwhRJC4KoqsbcyLMTYmiluX61VWs/BXv0a2z1TMA=; b=oOogTZUSsbblhQSEYYDfZ7oLb3AcfR247Yu9DNh7sqcdC6cCemzBHQ26lbV15JYnYW 5FOs8RAydIc3gBjoZmtOmDfuCDaBiZa97PArllksCommIDYVqNKj/y8ee+qiM7bYGwu2 EBKvk4G1YyIYvqvGWVbMUbeN1E0qsg6bDCMbnrk9e5ww6ANc1kJn5WBfHBcXwqnrJnqy rSyoc5CGKu2bEwPyHTXhS478l0VllspVt3bkcHYG4/YYZg1fOIDQYusU9bPbY7T/pZzy 0XXqTxFHYHJji7HTdld7ofujpuZhPkpnnWjFImFcm7yn0SmX7E7+CNUZyX4mNIv77bHD SMMg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:in-reply-to:references:from:date :subject:fcc:content-transfer-encoding:mime-version:to:cc; bh=/DTqwhRJC4KoqsbcyLMTYmiluX61VWs/BXv0a2z1TMA=; b=oG3If93bbQPawqC+8dU810xZ4gGpPC/JChC4niVNgmGhobtSbtNeryBu9h9iZAFPm6 9FSiSU/u02/+LZ7/GnRGNA2J+CgGMK6y/ovvjFIJoixXelV0FAY9BnNrXvyyVXdrHfx5 mZBnIfgANvJM/c80jDpEnx93lQOIKQFjucfwVhiFky7V/wjYtlbQaFkuXIfzcH/41GeM xO4gC9+cmwE9n3adHVRGV99ISiAw6XHTWyd0zxEJP3oIG55YCGfKsbjkev6pGbvYx3Ub EHOnvCee59lcw70j1qO6SR+nDAffwmWpsxjNrnaHWMxyDzReeG0CkoHpyO3lFXAlj7Zo IGwA== X-Gm-Message-State: AJIora+2O/m4PXE8p0MzpewGPwtdLgiNyxqZK2WEf0kOVY0uCanDsbej jANBD53Li1yKe8Bn+Yep6KeqbryKJHqFlA== X-Google-Smtp-Source: AGRyM1uG+OLZLxAv9He2xhrUbqrlOT0qDWUPrdnjRVJeMdzLNtkBopge+iPQqMe67oWPiK5moP5Gew== X-Received: by 2002:a05:600c:3c83:b0:39c:9039:852c with SMTP id bg3-20020a05600c3c8300b0039c9039852cmr21965884wmb.187.1656422763943; Tue, 28 Jun 2022 06:26:03 -0700 (PDT) Received: from [127.0.0.1] ([13.74.141.28]) by smtp.gmail.com with ESMTPSA id ib3-20020a05600ca14300b003a04e6410e0sm4723498wmb.33.2022.06.28.06.26.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 28 Jun 2022 06:26:03 -0700 (PDT) Message-Id: <669f4abd59e7dbb13281e85144d085180934fd15.1656422759.git.gitgitgadget@gmail.com> In-Reply-To: References: Date: Tue, 28 Jun 2022 13:25:54 +0000 Subject: [PATCH v3 3/8] rebase-interactive: update 'merge' description Fcc: Sent MIME-Version: 1.0 To: git@vger.kernel.org Cc: gitster@pobox.com, johannes.schindelin@gmx.de, me@ttaylorr.com, Jeff Hostetler , Phillip Wood , Elijah Newren , Derrick Stolee , Derrick Stolee Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org From: Derrick Stolee From: Derrick Stolee The 'merge' command description for the todo list documentation in an interactive rebase has multiple lines. The lines other than the first one start with dots ('.') while the similar multi-line documentation for 'fixup' does not. The 'merge' command was documented when interactive rebase was first ported to C in 145e05ac44b (rebase -i: rewrite append_todo_help() in C, 2018-08-10). These dots might have been carried over from the previous shell implementation. The 'fixup' command was documented more recently in 9e3cebd97cb (rebase -i: add fixup [-C | -c] command, 2021-01-29). Looking at the output in an editor, my personal opinion is that the dots are unnecessary and noisy. Remove them now before adding more commands with multi-line documentation. Signed-off-by: Derrick Stolee --- rebase-interactive.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/rebase-interactive.c b/rebase-interactive.c index 87649d0c016..22394224faa 100644 --- a/rebase-interactive.c +++ b/rebase-interactive.c @@ -54,9 +54,9 @@ void append_todo_help(int command_count, "l, label