# [PATCH 0/1] doc: fixed a typo in git-submodule.txt 5 messages from 2020-01-06 to 2020-01-06. Participants: thomas menzel via GitGitGadget, elonderin via GitGitGadget, Junio C Hamano, Johannes Schindelin. Thread: https://gitlist.dev/t/52573 ## thomas menzel via GitGitGadget, 2020-01-06 14:50 Subject: [PATCH 0/1] doc: fixed a typo in git-submodule.txt Message-ID: URL: https://gitlist.dev/e/pull.688.git.git.1578322213.gitgitgadget%40gmail.com ``` fixed a typo in the doc of submodule elonderin (1): doc: submodule: fix typo for command absorbgitdirs Documentation/git-submodule.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) base-commit: 8679ef24ed64018bb62170c43ce73e0261c0600a Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-688%2Felonderin%2Fpatch-1-v1 Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-688/elonderin/patch-1-v1 Pull-Request: https://github.com/git/git/pull/688 -- gitgitgadget ``` ## elonderin via GitGitGadget, 2020-01-06 14:50 Subject: [PATCH 1/1] doc: submodule: fix typo for command absorbgitdirs Message-ID: URL: https://gitlist.dev/e/b032358ca97df3bd5605ff356196f82a1f16a320.1578322213.git.gitgitgadget%40gmail.com In-Reply-To: ``` From: elonderin <8225602+elonderin@users.noreply.github.com> The sentence wants to talk about the superproject's possesive, not plural form. Signed-off-by: thomas menzel dev@tomsit.de --- Documentation/git-submodule.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/git-submodule.txt b/Documentation/git-submodule.txt index 22425cbc76..5232407f68 100644 --- a/Documentation/git-submodule.txt +++ b/Documentation/git-submodule.txt @@ -248,7 +248,7 @@ registered submodules, and sync any nested submodules within. absorbgitdirs:: If a git directory of a submodule is inside the submodule, - move the git directory of the submodule into its superprojects + move the git directory of the submodule into its superproject's `$GIT_DIR/modules` path and then connect the git directory and its working directory by setting the `core.worktree` and adding a .git file pointing to the git directory embedded in the -- gitgitgadget ``` ## Junio C Hamano, 2020-01-06 17:13 Subject: Re: [PATCH 1/1] doc: submodule: fix typo for command absorbgitdirs Message-ID: URL: https://gitlist.dev/e/xmqqlfqka4oe.fsf%40gitster-ct.c.googlers.com In-Reply-To: ``` "elonderin via GitGitGadget" writes: > From: elonderin <8225602+elonderin@users.noreply.github.com> > > The sentence wants to talk about the superproject's possesive, not plural form. > > Signed-off-by: thomas menzel dev@tomsit.de The author of the patch must sign-off, so the above three lines should look more like so: From: Thomas Menzel The sentence wants to talk about the superproject's possesive, not plural form. From: Thomas Menzel The content of the patch obviously is correct. Thanks. > --- > Documentation/git-submodule.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/git-submodule.txt b/Documentation/git-submodule.txt > index 22425cbc76..5232407f68 100644 > --- a/Documentation/git-submodule.txt > +++ b/Documentation/git-submodule.txt > @@ -248,7 +248,7 @@ registered submodules, and sync any nested submodules within. > > absorbgitdirs:: > If a git directory of a submodule is inside the submodule, > - move the git directory of the submodule into its superprojects > + move the git directory of the submodule into its superproject's > `$GIT_DIR/modules` path and then connect the git directory and > its working directory by setting the `core.worktree` and adding > a .git file pointing to the git directory embedded in the ``` ## Junio C Hamano, 2020-01-06 17:18 Subject: Re: [PATCH 1/1] doc: submodule: fix typo for command absorbgitdirs Message-ID: URL: https://gitlist.dev/e/xmqqblrga4h1.fsf%40gitster-ct.c.googlers.com In-Reply-To: ``` When anybody tries to respond to the quoted message [*1*] that came from GGG in the way recommended for the list, the *.noreply.github.com address gets placed on the Cc:, which is probably not wanted. Can GGG do something about it? As GitHub specific service, it is not wrong for it to know what noreply.github.com means so dropping such an address from its Cc: line shouldn't be too difficult. Thanks. Reference *1* "elonderin via GitGitGadget" writes: > From: elonderin <8225602+elonderin@users.noreply.github.com> > > The sentence wants to talk about the superproject's possesive, not plural form. > > Signed-off-by: thomas menzel dev@tomsit.de > --- > Documentation/git-submodule.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/git-submodule.txt b/Documentation/git-submodule.txt > index 22425cbc76..5232407f68 100644 > --- a/Documentation/git-submodule.txt > +++ b/Documentation/git-submodule.txt > @@ -248,7 +248,7 @@ registered submodules, and sync any nested submodules within. > > absorbgitdirs:: > If a git directory of a submodule is inside the submodule, > - move the git directory of the submodule into its superprojects > + move the git directory of the submodule into its superproject's > `$GIT_DIR/modules` path and then connect the git directory and > its working directory by setting the `core.worktree` and adding > a .git file pointing to the git directory embedded in the ``` ## Johannes Schindelin, 2020-01-06 22:52 Subject: Re: [PATCH 1/1] doc: submodule: fix typo for command absorbgitdirs Message-ID: URL: https://gitlist.dev/e/nycvar.QRO.7.76.6.2001062352010.46%40tvgsbejvaqbjf.bet In-Reply-To: ``` Hi Junio, On Mon, 6 Jan 2020, Junio C Hamano wrote: > When anybody tries to respond to the quoted message [*1*] that came > from GGG in the way recommended for the list, the > *.noreply.github.com address gets placed on the Cc:, which is > probably not wanted. > > Can GGG do something about it? As GitHub specific service, it is > not wrong for it to know what noreply.github.com means so dropping > such an address from its Cc: line shouldn't be too difficult. > > Thanks. > > > Reference *1* > > Right. I added it to the bug tracker: https://github.com/gitgitgadget/gitgitgadget/issues/182 Ciao, Dscho ```