All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Johannes Schindelin via GitGitGadget" <gitgitgadget@gmail.com>
To: git@vger.kernel.org
Cc: Johannes Schindelin <Johannes.Schindelin@gmx.de>,
	Johannes Schindelin <johannes.schindelin@gmx.de>
Subject: [PATCH v2 0/9] Prepare for changing the default branch name main (last manual part)
Date: Fri, 23 Oct 2020 13:59:57 +0000	[thread overview]
Message-ID: <pull.758.v2.git.1603461606.gitgitgadget@gmail.com> (raw)
In-Reply-To: <pull.758.git.1603135902.gitgitgadget@gmail.com>

In preparation for changing the default of init.defaultBranch to main, here
are a couple of manual adjustments that cannot be automated (i.e., these
adjustments are not mere search-and-replaces).

This patch series is based on js/default-branch-name-part-3, and is part of
the larger effort tracked at https://github.com/gitgitgadget/git/pull/655.

The next patch series will be largely automated search-and-replaces;
Obviously, the search-and-replace was the easy part, the manual inspection
whether the changes are actually correct was much more involved.

What to expect next, after this patch series? Essentially, there is the
large patch series that adjusts the entire test suite, with two patch series
extracted from that effort: t5411 and t5515 simply require too large
patches, and will therefore be presented as the next patch series after this
one.

Changes since v1:

 * The t4013 patch was moved out of this patch series, to be re-submitted at
   a later date, to avoid clashing with other patches in seen.
 * The commit message of 'fmt-merge-msg: also suppress "into main" by
   default' was adjusted to mention that we do the same for main as we do
   for master.
 * The t5703 patch now uses rain instead of none, to be closer to the
   default branch name.
 * The glob in t6200 was tightened to avoid matching maint.
 * The commit message of 't3200: prepare for main being shorter than master'
   stopped talking about main2 (which itself was a left-over from a previous
   iteration of a previous patch series).

Johannes Schindelin (9):
  fmt-merge-msg: also suppress "into main" by default
  t9801: use `--` in preparation for default branch rename
  tests: start moving to a different default main branch name
  t6200: adjust suppression pattern to also match "main"
  t5703: adjust a test case for the upcoming default branch name
  t3200: prepare for `main` being shorter than `master`
  t9902: prepare a test for the upcoming default branch name
  tests: prepare aligned mentions of the default branch name
  t1400: prepare for `main` being default branch name

 fmt-merge-msg.c                    |  4 +-
 refs.c                             |  5 ++-
 t/lib-submodule-update.sh          |  2 +-
 t/t0001-init.sh                    | 13 ++++--
 t/t1400-update-ref.sh              | 10 ++---
 t/t3200-branch.sh                  | 16 +++----
 t/t3201-branch-contains.sh         |  8 ++--
 t/t3203-branch-output.sh           |  4 +-
 t/t3205-branch-color.sh            |  8 ++--
 t/t5505-remote.sh                  | 30 ++++++-------
 t/t5510-fetch.sh                   |  8 ++--
 t/t5526-fetch-submodules.sh        | 70 +++++++++++++++---------------
 t/t5606-clone-options.sh           |  3 ++
 t/t5703-upload-pack-ref-in-want.sh |  6 +--
 t/t6200-fmt-merge-msg.sh           |  2 +-
 t/t6302-for-each-ref-filter.sh     | 24 +++++-----
 t/t9801-git-p4-branch.sh           | 12 ++---
 t/t9902-completion.sh              |  6 +--
 t/test-lib.sh                      |  7 +++
 19 files changed, 130 insertions(+), 108 deletions(-)


base-commit: 538228ed23a1d5e17e89bb17086d4dda51325bd8
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-758%2Fdscho%2Fprepare-for-main-branch-manual-v2
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-758/dscho/prepare-for-main-branch-manual-v2
Pull-Request: https://github.com/gitgitgadget/git/pull/758

Range-diff vs v1:

  1:  cf950c6f6a !  1:  bd2c72a348 fmt-merge-msg: also suppress "into main" by default
     @@ Commit message
          fmt-merge-msg: also suppress "into main" by default
      
          In preparation for changing the default branch name to `main`, let's
     -    skip the suffix "into main" in merge commit messages.
     +    skip the suffix "into main" in merge commit messages, the same way that
     +    "into master" has been skipped by default.
      
          Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
      
  2:  5849eda331 =  2:  e819084eb3 t9801: use `--` in preparation for default branch rename
  3:  d9469202c2 =  3:  9c608c311e tests: start moving to a different default main branch name
  4:  baeb01f706 <  -:  ---------- t4013: prepare for the new default branch name "main"
  5:  7860d8d7df !  4:  9bd0a89a79 t6200: adjust suppression pattern to also match "main"
     @@ t/t6200-fmt-merge-msg.sh: test_expect_success 'merge.suppressDest configuration'
       	grep -e "Merge branch .side. into master$" actual &&
       
      -	git -c merge.suppressDest="ma??er" fmt-merge-msg <.git/FETCH_HEAD >full.3 &&
     -+	git -c merge.suppressDest="ma*" fmt-merge-msg <.git/FETCH_HEAD >full.3 &&
     ++	git -c merge.suppressDest="ma?*[rn]" fmt-merge-msg <.git/FETCH_HEAD >full.3 &&
       	head -n1 full.3 >actual &&
       	grep -e "Merge branch .side." actual &&
       	! grep -e " into master$" actual
  6:  0d2511d18f !  5:  b557ea5ac0 t5703: adjust a test case for the upcoming default branch name
     @@ t/t5703-upload-pack-ref-in-want.sh: test_expect_success 'server is initially beh
       	rm -rf local &&
       	cp -r "$LOCAL_PRISTINE" local &&
      -	echo "s/master/raster/" >"$HTTPD_ROOT_PATH/one-time-perl" &&
     -+	echo "s/main/none/" >"$HTTPD_ROOT_PATH/one-time-perl" &&
     ++	echo "s/main/rain/" >"$HTTPD_ROOT_PATH/one-time-perl" &&
       	test_must_fail git -C local fetch 2>err &&
       
      -	test_i18ngrep "fatal: remote error: unknown ref refs/heads/raster" err
     -+	test_i18ngrep "fatal: remote error: unknown ref refs/heads/none" err
     ++	test_i18ngrep "fatal: remote error: unknown ref refs/heads/rain" err
       '
       
       # DO NOT add non-httpd-specific tests here, because the last part of this
  7:  e1740bb7d7 !  6:  959a770c23 t3200: prepare for `main` being shorter than `master`
     @@ Commit message
      
          In the test case adjusted by this patch, we want to cut just after the
          longest shown ref name. Since `main` is shorter than `master`, we need
     -    to decrease the number of characters. Since `main2` is shown, too, and
     +    to decrease the number of characters. Since `topic` is shown, too, and
          since that is only one character shorter than `master`, we decrement the
     -    length by one.
     +    length by one instead of two.
      
          Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
      
  8:  1641d1950d =  7:  8db960009d t9902: prepare a test for the upcoming default branch name
  9:  5c29a89d88 =  8:  d15e39bb73 tests: prepare aligned mentions of the default branch name
 10:  66365bce55 =  9:  e542010fdd t1400: prepare for `main` being default branch name

-- 
gitgitgadget

  parent reply	other threads:[~2020-10-23 14:00 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-19 19:31 [PATCH 00/10] Prepare for changing the default branch name main (last manual part) Johannes Schindelin via GitGitGadget
2020-10-19 19:31 ` [PATCH 01/10] fmt-merge-msg: also suppress "into main" by default Johannes Schindelin via GitGitGadget
2020-10-21 22:12   ` Junio C Hamano
2020-10-22 12:28     ` Johannes Schindelin
2020-10-19 19:31 ` [PATCH 02/10] t9801: use `--` in preparation for default branch rename Johannes Schindelin via GitGitGadget
2020-10-21 22:19   ` Junio C Hamano
2020-10-19 19:31 ` [PATCH 03/10] tests: start moving to a different default main branch name Johannes Schindelin via GitGitGadget
2020-10-21 22:22   ` Junio C Hamano
2020-10-19 19:31 ` [PATCH 04/10] t4013: prepare for the new default branch name "main" Johannes Schindelin via GitGitGadget
2020-10-21 22:23   ` Junio C Hamano
2020-10-22 12:27     ` Johannes Schindelin
2020-10-19 19:31 ` [PATCH 05/10] t6200: adjust suppression pattern to also match "main" Johannes Schindelin via GitGitGadget
2020-10-21 22:27   ` Junio C Hamano
2020-10-22 13:01     ` Johannes Schindelin
2020-10-19 19:31 ` [PATCH 06/10] t5703: adjust a test case for the upcoming default branch name Johannes Schindelin via GitGitGadget
2020-10-21 22:34   ` Junio C Hamano
2020-10-19 19:31 ` [PATCH 07/10] t3200: prepare for `main` being shorter than `master` Johannes Schindelin via GitGitGadget
2020-10-21 22:39   ` Junio C Hamano
2020-10-22 13:08     ` Johannes Schindelin
2020-10-19 19:31 ` [PATCH 08/10] t9902: prepare a test for the upcoming default branch name Johannes Schindelin via GitGitGadget
2020-10-19 19:31 ` [PATCH 09/10] tests: prepare aligned mentions of the " Johannes Schindelin via GitGitGadget
2020-10-19 19:31 ` [PATCH 10/10] t1400: prepare for `main` being " Johannes Schindelin via GitGitGadget
2020-10-21 21:59 ` [PATCH 00/10] Prepare for changing the default branch name main (last manual part) Junio C Hamano
2020-10-22 13:11   ` Johannes Schindelin
2020-10-23 13:59 ` Johannes Schindelin via GitGitGadget [this message]
2020-10-23 13:59   ` [PATCH v2 1/9] fmt-merge-msg: also suppress "into main" by default Johannes Schindelin via GitGitGadget
2020-10-23 13:59   ` [PATCH v2 2/9] t9801: use `--` in preparation for default branch rename Johannes Schindelin via GitGitGadget
2020-10-23 14:00   ` [PATCH v2 3/9] tests: start moving to a different default main branch name Johannes Schindelin via GitGitGadget
2020-10-23 14:00   ` [PATCH v2 4/9] t6200: adjust suppression pattern to also match "main" Johannes Schindelin via GitGitGadget
2020-10-23 14:00   ` [PATCH v2 5/9] t5703: adjust a test case for the upcoming default branch name Johannes Schindelin via GitGitGadget
2020-10-23 14:00   ` [PATCH v2 6/9] t3200: prepare for `main` being shorter than `master` Johannes Schindelin via GitGitGadget
2020-10-23 14:00   ` [PATCH v2 7/9] t9902: prepare a test for the upcoming default branch name Johannes Schindelin via GitGitGadget
2020-10-23 14:00   ` [PATCH v2 8/9] tests: prepare aligned mentions of the " Johannes Schindelin via GitGitGadget
2020-10-23 14:00   ` [PATCH v2 9/9] t1400: prepare for `main` being " Johannes Schindelin via GitGitGadget
2020-10-23 15:59   ` [PATCH v2 0/9] Prepare for changing the default branch name main (last manual part) Junio C Hamano

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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=pull.758.v2.git.1603461606.gitgitgadget@gmail.com \
    --to=gitgitgadget@gmail.com \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=git@vger.kernel.org \
    /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.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.