threads / patch / 34334

patchChange "remote tracking" to "remote-tracking"

Subject: [PATCH] Change "remote tracking" to "remote-tracking"

## tl;dr

6 messages between Jul 3, 2013 and Jul 4, 2013. Diffs are folded; open one to read it.

replies: 5people: 5as markdown or json

Michael Schubert· Jul 3, 2013, 09:12 UTC · lore

Fix a typo ("remote remote-tracking") going back to the big cleanup in 2010 (8b3f3f84 etc). Also, remove some more occurrences of "tracking" and "remote tracking" in favor of "remote-tracking".

Signed-off-by: Michael Schubert <mschub@elegosoft.com>
---
 Documentation/git-p4.txt           | 2 +-
 Documentation/git-submodule.txt    | 2 +-
 Documentation/glossary-content.txt | 4 ++--
 builtin/clone.c                    | 2 +-
 builtin/merge.c                    | 4 ++--
 t/t5505-remote.sh                  | 2 +-
 6 files changed, 8 insertions(+), 8 deletions(-)
Show changes to 6 files +8 −8

Documentation/git-p4.txt, Documentation/git-submodule.txt, Documentation/glossary-content.txt, builtin/clone.c, builtin/merge.c, t/t5505-remote.sh

diff --git a/Documentation/git-p4.txt b/Documentation/git-p4.txt
index c579fbc..609c1d2 100644
--- a/Documentation/git-p4.txt
+++ b/Documentation/git-p4.txt
@@ -180,7 +180,7 @@ subsequent 'sync' operations.
 	Import changes into given branch.  If the branch starts with
 	'refs/', it will be used as is.  Otherwise if it does not start
 	with 'p4/', that prefix is added.  The branch is assumed to
-	name a remote tracking, but this can be modified using
+	name a remote-tracking, but this can be modified using
 	'--import-local', or by giving a full ref name.  The default
 	branch is 'master'.
 +
diff --git a/Documentation/git-submodule.txt b/Documentation/git-submodule.txt
index e576713..bfff090 100644
--- a/Documentation/git-submodule.txt
+++ b/Documentation/git-submodule.txt
@@ -262,7 +262,7 @@ OPTIONS
 --remote::
 	This option is only valid for the update command.  Instead of using
 	the superproject's recorded SHA-1 to update the submodule, use the
-	status of the submodule's remote tracking branch.  The remote used
+	status of the submodule's remote-tracking branch.  The remote used
 	is branch's remote (`branch.<name>.remote`), defaulting to `origin`.
 	The remote branch used defaults to `master`, but the branch name may
 	be overridden by setting the `submodule.<name>.branch` option in
diff --git a/Documentation/glossary-content.txt b/Documentation/glossary-content.txt
index db2a74d..dba5062 100644
--- a/Documentation/glossary-content.txt
+++ b/Documentation/glossary-content.txt
@@ -113,7 +113,7 @@ Note that commands that operate on the history of the current branch
 while the HEAD is detached. They update the HEAD to point at the tip
 of the updated history without affecting any branch.  Commands that
 update or inquire information _about_ the current branch (e.g. `git
-branch --set-upstream-to` that sets what remote tracking branch the
+branch --set-upstream-to` that sets what remote-tracking branch the
 current branch integrates with) obviously do not work, as there is no
 (real) current branch to ask about in this state.
 
@@ -267,7 +267,7 @@ This commit is referred to as a "merge commit", or sometimes just a
 	The default upstream <<def_repository,repository>>. Most projects have
 	at least one upstream project which they track. By default
 	'origin' is used for that purpose. New upstream updates
-	will be fetched into remote <<def_remote_tracking_branch,remote-tracking branches>> named
+	will be fetched into <<def_remote_tracking_branch,remote-tracking branches>> named
 	origin/name-of-upstream-branch, which you can see using
 	`git branch -r`.
 
diff --git a/builtin/clone.c b/builtin/clone.c
index 14b1323..17f57cd 100644
--- a/builtin/clone.c
+++ b/builtin/clone.c
@@ -701,7 +701,7 @@ static void write_refspec_config(const char* src_ref_prefix,
 			/*
 			 * otherwise, the next "git fetch" will
 			 * simply fetch from HEAD without updating
-			 * any remote tracking branch, which is what
+			 * any remote-tracking branch, which is what
 			 * we want.
 			 */
 		} else {
diff --git a/builtin/merge.c b/builtin/merge.c
index 2ebe732..bad4536 100644
--- a/builtin/merge.c
+++ b/builtin/merge.c
@@ -948,7 +948,7 @@ static int evaluate_result(void)
 }
 
 /*
- * Pretend as if the user told us to merge with the tracking
+ * Pretend as if the user told us to merge with the remote-tracking
  * branch we have for the upstream of the current branch
  */
 static int setup_with_upstream(const char ***argv)
@@ -967,7 +967,7 @@ static int setup_with_upstream(const char ***argv)
 	args = xcalloc(branch->merge_nr + 1, sizeof(char *));
 	for (i = 0; i < branch->merge_nr; i++) {
 		if (!branch->merge[i]->dst)
-			die(_("No remote tracking branch for %s from %s"),
+			die(_("No remote-tracking branch for %s from %s"),
 			    branch->merge[i]->src, branch->remote_name);
 		args[i] = branch->merge[i]->dst;
 	}
diff --git a/t/t5505-remote.sh b/t/t5505-remote.sh
index ee5d65d..8f6e392 100755
--- a/t/t5505-remote.sh
+++ b/t/t5505-remote.sh
@@ -74,7 +74,7 @@ test_expect_success 'add another remote' '
 	)
 '
 
-test_expect_success C_LOCALE_OUTPUT 'check remote tracking' '
+test_expect_success C_LOCALE_OUTPUT 'check remote-tracking' '
 	(
 		cd test &&
 		check_remote_track origin master side &&
-- 
1.8.3.2.860.gf7f8a72
Johan Herland· Jul 3, 2013, 10:02 UTC · re: Michael Schubert · lore

Re: [PATCH] Change "remote tracking" to "remote-tracking"

On Wed, Jul 3, 2013 at 11:12 AM, Michael Schubert <mschub@elegosoft.com> wrote:
Show 5 quoted lines
> Fix a typo ("remote remote-tracking") going back to the big cleanup
> in 2010 (8b3f3f84 etc). Also, remove some more occurrences of
> "tracking" and "remote tracking" in favor of "remote-tracking".
>
> Signed-off-by: Michael Schubert <mschub@elegosoft.com>
Reviewed-by: Johan Herland <johan@herland.net>
-- 
Johan Herland, <johan@herland.net>
www.herland.net
Jonathan Nieder· Jul 3, 2013, 18:38 UTC · re: Michael Schubert · lore

Re: [PATCH] Change "remote tracking" to "remote-tracking"

Michael Schubert wrote:
Show 10 quoted lines
> --- a/Documentation/git-p4.txt
> +++ b/Documentation/git-p4.txt
> @@ -180,7 +180,7 @@ subsequent 'sync' operations.
>  	Import changes into given branch.  If the branch starts with
>  	'refs/', it will be used as is.  Otherwise if it does not start
>  	with 'p4/', that prefix is added.  The branch is assumed to
> -	name a remote tracking, but this can be modified using
> +	name a remote-tracking, but this can be modified using
>  	'--import-local', or by giving a full ref name.  The default
>  	branch is 'master'.

This is confusing both before and after the patch. What is "a remote tracking"?

Perhaps:
	--branch <ref>::
		Import changes into <ref> instead of refs/remotes/p4/master.
		If <ref> starts with refs/, it is used as is.  Otherwise, if
		it does not start with p4/, that prefix is added.
	+
	By default a <ref> not starting with refs/ is treated as the
	name of a remote-tracking branch (under refs/remotes/).  This
	behavior can be modified using the --import-local option.
	+
	The default <ref> is "master".
The rest of the patch looks good.

Thanks, Jonathan

Junio C Hamano· Jul 3, 2013, 20:33 UTC · re: Jonathan Nieder · lore

Re: [PATCH] Change "remote tracking" to "remote-tracking"

Jonathan Nieder <jrnieder@gmail.com> writes:
Show 30 quoted lines
> Michael Schubert wrote:
>
>> --- a/Documentation/git-p4.txt
>> +++ b/Documentation/git-p4.txt
>> @@ -180,7 +180,7 @@ subsequent 'sync' operations.
>>  	Import changes into given branch.  If the branch starts with
>>  	'refs/', it will be used as is.  Otherwise if it does not start
>>  	with 'p4/', that prefix is added.  The branch is assumed to
>> -	name a remote tracking, but this can be modified using
>> +	name a remote-tracking, but this can be modified using
>>  	'--import-local', or by giving a full ref name.  The default
>>  	branch is 'master'.
>
> This is confusing both before and after the patch.  What is "a remote
> tracking"?
>
> Perhaps:
>
> 	--branch <ref>::
> 		Import changes into <ref> instead of refs/remotes/p4/master.
> 		If <ref> starts with refs/, it is used as is.  Otherwise, if
> 		it does not start with p4/, that prefix is added.
> 	+
> 	By default a <ref> not starting with refs/ is treated as the
> 	name of a remote-tracking branch (under refs/remotes/).  This
> 	behavior can be modified using the --import-local option.
> 	+
> 	The default <ref> is "master".
>
> The rest of the patch looks good.

Myy reading did hiccup at the same "remote-tracking" used as if it were a noun, and your rewritten version reads much better.

Thanks.
Pete Wyckoff· Jul 3, 2013, 22:43 UTC · re: Junio C Hamano · lore

Re: [PATCH] Change "remote tracking" to "remote-tracking"

gitster@pobox.com wrote on Wed, 03 Jul 2013 13:33 -0700:
Show 35 quoted lines
> Jonathan Nieder <jrnieder@gmail.com> writes:
> 
> > Michael Schubert wrote:
> >
> >> --- a/Documentation/git-p4.txt
> >> +++ b/Documentation/git-p4.txt
> >> @@ -180,7 +180,7 @@ subsequent 'sync' operations.
> >>  	Import changes into given branch.  If the branch starts with
> >>  	'refs/', it will be used as is.  Otherwise if it does not start
> >>  	with 'p4/', that prefix is added.  The branch is assumed to
> >> -	name a remote tracking, but this can be modified using
> >> +	name a remote-tracking, but this can be modified using
> >>  	'--import-local', or by giving a full ref name.  The default
> >>  	branch is 'master'.
> >
> > This is confusing both before and after the patch.  What is "a remote
> > tracking"?
> >
> > Perhaps:
> >
> > 	--branch <ref>::
> > 		Import changes into <ref> instead of refs/remotes/p4/master.
> > 		If <ref> starts with refs/, it is used as is.  Otherwise, if
> > 		it does not start with p4/, that prefix is added.
> > 	+
> > 	By default a <ref> not starting with refs/ is treated as the
> > 	name of a remote-tracking branch (under refs/remotes/).  This
> > 	behavior can be modified using the --import-local option.
> > 	+
> > 	The default <ref> is "master".
> >
> > The rest of the patch looks good.
> 
> Myy reading did hiccup at the same "remote-tracking" used as if it
> were a noun, and your rewritten version reads much better.

Yes, very clear and complete rewrite; thanks. The final paragraph is perhaps duplicative of the first sentence, but adds clarity, so I'm happy as it stands.

Acked-by: Pete Wyckoff <pw@padd.com>
		-- Pete
Michael Schubert· Jul 4, 2013, 09:05 UTC · re: Jonathan Nieder · lore

Re: [PATCH] Change "remote tracking" to "remote-tracking"

On Wed, Jul 03, 2013 at 11:38:51AM -0700, Jonathan Nieder wrote:
Show 28 quoted lines
> Michael Schubert wrote:
> 
> > --- a/Documentation/git-p4.txt
> > +++ b/Documentation/git-p4.txt
> > @@ -180,7 +180,7 @@ subsequent 'sync' operations.
> >  	Import changes into given branch.  If the branch starts with
> >  	'refs/', it will be used as is.  Otherwise if it does not start
> >  	with 'p4/', that prefix is added.  The branch is assumed to
> > -	name a remote tracking, but this can be modified using
> > +	name a remote-tracking, but this can be modified using
> >  	'--import-local', or by giving a full ref name.  The default
> >  	branch is 'master'.
> 
> This is confusing both before and after the patch.  What is "a remote
> tracking"?
> 
> Perhaps:
> 
> 	--branch <ref>::
> 		Import changes into <ref> instead of refs/remotes/p4/master.
> 		If <ref> starts with refs/, it is used as is.  Otherwise, if
> 		it does not start with p4/, that prefix is added.
> 	+
> 	By default a <ref> not starting with refs/ is treated as the
> 	name of a remote-tracking branch (under refs/remotes/).  This
> 	behavior can be modified using the --import-local option.
> 	+
> 	The default <ref> is "master".
Thanks.

← back to recent threads