threads / patch / 17706

patchMake 'remote show' distinguish between merged and rebased remote branches

Subject: [PATCH] Make 'remote show' distinguish between merged and rebased remote branches

## tl;dr

9 messages between Feb 10, 2009 and Feb 11, 2009. Diffs are folded; open one to read it.

replies: 8people: 5as markdown or json

Marc Branchaud· Feb 10, 2009, 17:08 UTC · lore
Prints "rebased" instead of "merged" if branch.<name>.rebase is true.
Signed-off-by: Marc Branchaud <marcnarc@xiplink.com>
---
I'd like to be able to tell if my "git pull" is going to merge or rebase...
 builtin-remote.c  |   13 ++++++++++---
 t/t5505-remote.sh |    5 +++++
 2 files changed, 15 insertions(+), 3 deletions(-)
Show changes to 2 files +15 −3

builtin-remote.c, t/t5505-remote.sh

diff --git a/builtin-remote.c b/builtin-remote.c
index abc8dd8..ac3a88e 100644
--- a/builtin-remote.c
+++ b/builtin-remote.c
@@ -145,6 +145,7 @@ static int add(int argc, const char **argv)
 struct branch_info {
 	char *remote;
 	struct string_list merge;
+	int rebase;
 };
 
 static struct string_list branch_list;
@@ -164,7 +165,7 @@ static int config_read_branches(const char *key, const char *value, void *cb)
 		char *name;
 		struct string_list_item *item;
 		struct branch_info *info;
-		enum { REMOTE, MERGE } type;
+		enum { REMOTE, MERGE, REBASE } type;
 
 		key += 7;
 		if (!postfixcmp(key, ".remote")) {
@@ -173,6 +174,9 @@ static int config_read_branches(const char *key, const char *value, void *cb)
 		} else if (!postfixcmp(key, ".merge")) {
 			name = xstrndup(key, strlen(key) - 6);
 			type = MERGE;
+		} else if (!postfixcmp(key, ".rebase")) {
+			name = xstrndup(key, strlen(key) - 7);
+			type = REBASE;
 		} else
 			return 0;
 
@@ -185,7 +189,7 @@ static int config_read_branches(const char *key, const char *value, void *cb)
 			if (info->remote)
 				warning("more than one branch.%s", key);
 			info->remote = xstrdup(value);
-		} else {
+		} else if (type == MERGE) {
 			char *space = strchr(value, ' ');
 			value = abbrev_branch(value);
 			while (space) {
@@ -196,6 +200,8 @@ static int config_read_branches(const char *key, const char *value, void *cb)
 				space = strchr(value, ' ');
 			}
 			string_list_append(xstrdup(value), &info->merge);
+		} else {
+			info->rebase = 1;
 		}
 	}
 	return 0;
@@ -678,9 +684,10 @@ static int show(int argc, const char **argv)
 
 			if (!info->merge.nr || strcmp(*argv, info->remote))
 				continue;
-			printf("  Remote branch%s merged with 'git pull' "
+			printf("  Remote branch%s %s with 'git pull' "
 				"while on branch %s\n   ",
 				info->merge.nr > 1 ? "es" : "",
+				info->rebase ? "rebased" : "merged",
 				branch->string);
 			for (j = 0; j < info->merge.nr; j++)
 				printf(" %s", info->merge.items[j].string);
diff --git a/t/t5505-remote.sh b/t/t5505-remote.sh
index 1f59960..be0316a 100755
--- a/t/t5505-remote.sh
+++ b/t/t5505-remote.sh
@@ -112,6 +112,8 @@ cat > test/expect << EOF
   URL: $(pwd)/one
   Remote branch merged with 'git pull' while on branch master
     master
+  Remote branch rebased with 'git pull' while on branch rebaser
+    side
   New remote branch (next fetch will store in remotes/origin)
     master
   Tracked remote branches
@@ -136,7 +138,10 @@ test_expect_success 'show' '
 		refs/heads/master:refs/heads/upstream &&
 	 git config --add remote.origin.push \
 		+refs/tags/lastbackup &&
+	 git branch --track rebaser origin/side &&
+	 git config --add branch.rebaser.rebase true &&
 	 git remote show origin > output &&
+	 git branch -D rebaser &&
 	 test_cmp expect output)
 '
 
-- 
1.6.1.2.390.gba743
Junio C Hamano· Feb 10, 2009, 21:17 UTC · re: Marc Branchaud · lore

Re: [PATCH] Make 'remote show' distinguish between merged and rebased remote branches

Marc Branchaud <marcnarc@xiplink.com> writes:
Show 17 quoted lines
> Prints "rebased" instead of "merged" if branch.<name>.rebase is true.
>
> Signed-off-by: Marc Branchaud <marcnarc@xiplink.com>
> ...
> diff --git a/t/t5505-remote.sh b/t/t5505-remote.sh
> index 1f59960..be0316a 100755
> --- a/t/t5505-remote.sh
> +++ b/t/t5505-remote.sh
> @@ -112,6 +112,8 @@ cat > test/expect << EOF
>    URL: $(pwd)/one
>    Remote branch merged with 'git pull' while on branch master
>      master
> +  Remote branch rebased with 'git pull' while on branch rebaser
> +    side
>    New remote branch (next fetch will store in remotes/origin)
>      master
>    Tracked remote branches

I sympathize with what you are trying to do but I do not think this is correct. You will *never* rebase remote on top of your changes; rather you will replay your changes on top of what the updated remote has.

It is more like "if you start git pull while on branch rebaser, it will rebased on top of this branch from the remote".

Marc Branchaud· Feb 10, 2009, 22:05 UTC · re: Junio C Hamano · lore

Re: [PATCH] Make 'remote show' distinguish between merged and rebased remote branches

Junio C Hamano wrote:
Show 8 quoted lines
> 
> I sympathize with what you are trying to do but I do not think this is
> correct.  You will *never* rebase remote on top of your changes; rather
> you will replay your changes on top of what the updated remote has.
> 
> It is more like "if you start git pull while on branch rebaser, it will
> rebased on top of this branch from the remote".
> 
I'm happy to use more accurate phrasing -- I was just going for a minimal change.
How about the following, consistent for both cases:
'git pull' merges branch master with remote branch
  master
'git pull' rebases branch rebaser on top of remote branch
  side
?
I like the above because the keywords & branch names are in consistent locations, making it easier to parse the output.
		Marc
Jay Soffian· Feb 10, 2009, 23:13 UTC · re: Marc Branchaud · lore

Re: [PATCH] Make 'remote show' distinguish between merged and rebased remote branches

On Tue, Feb 10, 2009 at 5:05 PM, Marc Branchaud <marcnarc@xiplink.com> wrote:
Show 12 quoted lines
> I'm happy to use more accurate phrasing -- I was just going for a minimal change.
>
> How about the following, consistent for both cases:
>
> 'git pull' merges branch master with remote branch
>  master
> 'git pull' rebases branch rebaser on top of remote branch
>  side
>
> ?
>
> I like the above because the keywords & branch names are in consistent locations, making it easier to parse the output.

The output of git remote show seems much too verbose for the information it provides. Something like this, I think, provides the same information in much less space:

* remote origin
  URL: git://git.kernel.org/pub/scm/git/git.git
  Remote branches:
    html      Not tracked
    maint     Tracked
    man       Tracked
    master    Tracked
    next      Tracked
    pu        Tracked
    todo      Not tracked
    old-next  Stale (would prune)
  Local branches configured to pull from this remote:
    master           upstream is master (merges)
    wip/remote-HEAD  upstream is next   (rebases)

When run with "-n" the status column would be blank or say "Status not available with -n".

$0.02. :-)
j.
Marc Branchaud· Feb 11, 2009, 16:59 UTC · re: Jay Soffian · lore

Re: [PATCH] Make 'remote show' distinguish between merged and rebased remote branches

Jay Soffian wrote:
> 
> The output of git remote show seems much too verbose for the
> information it provides.
I agree.
I'm happy to hammer out and implement new output for 'remote show', but is there any chance that such a change would be accepted?
Junio, Dscho, (others) -- any opposition to this?
Show 17 quoted lines
> Something like this, I think, provides the
> same information in much less space:
> 
> * remote origin
>   URL: git://git.kernel.org/pub/scm/git/git.git
>   Remote branches:
>     html      Not tracked
>     maint     Tracked
>     man       Tracked
>     master    Tracked
>     next      Tracked
>     pu        Tracked
>     todo      Not tracked
>     old-next  Stale (would prune)
>   Local branches configured to pull from this remote:
>     master           upstream is master (merges)
>     wip/remote-HEAD  upstream is next   (rebases)
How about something a bit tighter, merging the local branch list with the remote branch list:
* remote origin
  URL: git://git.kernel.org/pub/scm/git/git.git
  Remote branches:
    html      Not tracked
    master    Tracked by local branches:
                master (merges)
                mywork (rebases)
    next      Tracked by local branch:
                wip/remote-HEAD (rebases)
    pu        Tracked by local branch:
                pu (merges)
    todo      Not tracked
    old-next  Stale (would prune)
?
		M.
Johannes Sixt· Feb 11, 2009, 17:12 UTC · re: Marc Branchaud · lore

Re: [PATCH] Make 'remote show' distinguish between merged and rebased remote branches

Marc Branchaud schrieb:
Show 15 quoted lines
> Jay Soffian wrote:
>> * remote origin
>>   URL: git://git.kernel.org/pub/scm/git/git.git
>>   Remote branches:
>>     html      Not tracked
>>     maint     Tracked
>>     man       Tracked
>>     master    Tracked
>>     next      Tracked
>>     pu        Tracked
>>     todo      Not tracked
>>     old-next  Stale (would prune)
>>   Local branches configured to pull from this remote:
>>     master           upstream is master (merges)
>>     wip/remote-HEAD  upstream is next   (rebases)

I find this form more useful than the one below because of the clear separation into remote an local branchs.

Show 15 quoted lines
> How about something a bit tighter, merging the local branch list with the remote branch list:
> 
> * remote origin
>   URL: git://git.kernel.org/pub/scm/git/git.git
>   Remote branches:
>     html      Not tracked
>     master    Tracked by local branches:
>                 master (merges)
>                 mywork (rebases)
>     next      Tracked by local branch:
>                 wip/remote-HEAD (rebases)
>     pu        Tracked by local branch:
>                 pu (merges)
>     todo      Not tracked
>     old-next  Stale (would prune)
-- Hannes
Junio C Hamano· Feb 11, 2009, 17:35 UTC · re: Johannes Sixt · lore

Re: [PATCH] Make 'remote show' distinguish between merged and rebased remote branches

Johannes Sixt <j.sixt@viscovery.net> writes:
Show 19 quoted lines
> Marc Branchaud schrieb:
>> Jay Soffian wrote:
>>> * remote origin
>>>   URL: git://git.kernel.org/pub/scm/git/git.git
>>>   Remote branches:
>>>     html      Not tracked
>>>     maint     Tracked
>>>     man       Tracked
>>>     master    Tracked
>>>     next      Tracked
>>>     pu        Tracked
>>>     todo      Not tracked
>>>     old-next  Stale (would prune)
>>>   Local branches configured to pull from this remote:
>>>     master           upstream is master (merges)
>>>     wip/remote-HEAD  upstream is next   (rebases)
>
> I find this form more useful than the one below because of the clear
> separation into remote an local branchs.
Good point.
Marc Branchaud· Feb 11, 2009, 21:35 UTC · re: Jay Soffian · lore

Re: [PATCH] Make 'remote show' distinguish between merged and rebased remote branches

Jay Soffian wrote:
Show 19 quoted lines
> 
> The output of git remote show seems much too verbose for the
> information it provides. Something like this, I think, provides the
> same information in much less space:
> 
> * remote origin
>   URL: git://git.kernel.org/pub/scm/git/git.git
>   Remote branches:
>     html      Not tracked
>     maint     Tracked
>     man       Tracked
>     master    Tracked
>     next      Tracked
>     pu        Tracked
>     todo      Not tracked
>     old-next  Stale (would prune)
>   Local branches configured to pull from this remote:
>     master           upstream is master (merges)
>     wip/remote-HEAD  upstream is next   (rebases)
The current test case for 'remote show' (without my rebase patch) is:
* remote origin
  URL: $(pwd)/one
  Remote branch merged with 'git pull' while on branch master
    master
  New remote branch (next fetch will store in remotes/origin)
    master
  Tracked remote branches
    side
    master
  Local branches pushed with 'git push'
    master:upstream
    +refs/tags/lastbackup
Should that last bit about 'git push' change as well (I admit I don't really understand what that part is saying)?  Or should we just be consistent?  Maybe something like:
* remote origin
  URL: $(pwd)/one
  Remote branches:
    master  Tracked
    side    Not tracked
    flip    Tracked
  New remote branch (next fetch will store in remotes/origin)
    master
  Local branches tracking this remote:
    master   upstream is master (merges)
    rebaser  upstream is flip   (rebases)
  Local branches pushed with 'git push'
    master:upstream
    +refs/tags/lastbackup
		M.
Johannes Schindelin· Feb 10, 2009, 21:22 UTC · re: Marc Branchaud · lore

Re: [PATCH] Make 'remote show' distinguish between merged and rebased remote branches

Hi,
On Tue, 10 Feb 2009, Marc Branchaud wrote:
Show 10 quoted lines
> diff --git a/t/t5505-remote.sh b/t/t5505-remote.sh
> index 1f59960..be0316a 100755
> --- a/t/t5505-remote.sh
> +++ b/t/t5505-remote.sh
> @@ -112,6 +112,8 @@ cat > test/expect << EOF
>    URL: $(pwd)/one
>    Remote branch merged with 'git pull' while on branch master
>      master
> +  Remote branch rebased with 'git pull' while on branch rebaser
> +    side

I'm not a native speaker... but "rebased with" sounds wrong, but so does "Remote branch rebased onto with 'git pull' while on branch rebaser"...

Otherwise, the patch looks good to me.

Thanks, Dscho

← back to recent threads