threads / patch / 28953

patchcolor of branches in git status -sb

Subject: [patch] color of branches in git status -sb

## tl;dr

5 messages between Nov 17, 2011 and Nov 17, 2011. Diffs are folded; open one to read it.

replies: 4people: 3as markdown or json

Nicolas Dudebout· Nov 17, 2011, 00:39 UTC · lore

The following patch fixes the fact that two colors of the status output could not be configured in .git/config.

The color of the current branch could not be modified because of the name WT_STATUS_ONBRANCH having been changed to WT_STATUS_LOCAL_BRANCH.

The color of the remote branch was not defined at all.

By the way, when I do 'git status' instead of git 'status -sb' the local and remote branch do not get colored. Is this a desired behavior?

Nicolas

	Modified   Documentation/config.txt

Show changes to 2 files +5 −2

Documentation/config.txt, builtin/commit.c

diff --git a/Documentation/config.txt b/Documentation/config.txt
index 5a841da..7a0ddd6 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
@@ -809 +809,2 @@ color.status.<slot>::
-	`branch` (the current branch), or
+	`branch` (the current branch),
+        `remote` (the remote branch) or
	Modified   builtin/commit.c
diff --git a/builtin/commit.c b/builtin/commit.c
index c46f2d1..4674600 100644
--- a/builtin/commit.c
+++ b/builtin/commit.c
@@ -1118 +1118,3 @@ static int parse_status_slot(const char *var, int offset)
-		return WT_STATUS_ONBRANCH;
+		return WT_STATUS_LOCAL_BRANCH;
+	if (!strcasecmp(var+offset, "remote"))
+		return WT_STATUS_REMOTE_BRANCH;

Junio C Hamano· Nov 17, 2011, 01:13 UTC · re: Nicolas Dudebout · lore

Re: [patch] color of branches in git status -sb

Nicolas Dudebout <nicolas.dudebout@gmail.com> writes:

Show 13 quoted lines

> The following patch fixes the fact that two colors of the status
> output could not be configured in .git/config.
>
> The color of the current branch could not be modified because of the
> name WT_STATUS_ONBRANCH having been changed to WT_STATUS_LOCAL_BRANCH.
>
> The color of the remote branch was not defined at all.
>
> By the way, when I do 'git status' instead of git 'status -sb' the
> local and remote branch do not get colored. Is this a desired
> behavior?
>
> Nicolas

Please follow Documentation/SubmittingPatches.

Also expect that patches to add new feature this deep in the pre-release feature freeze period is not likely to get reviewed by regular list members.

>
> 	Modified   Documentation/config.txt

Don't do this. We can tell what is modified from "diff --git" lines.

Show 8 quoted lines

> diff --git a/Documentation/config.txt b/Documentation/config.txt
> index 5a841da..7a0ddd6 100644
> --- a/Documentation/config.txt
> +++ b/Documentation/config.txt
> @@ -809 +809,2 @@ color.status.<slot>::
> -	`branch` (the current branch), or
> +	`branch` (the current branch),
> +        `remote` (the remote branch) or

Don't do this. Proper patches have context lines for good reasons.

Show 10 quoted lines

> 	Modified   builtin/commit.c
> diff --git a/builtin/commit.c b/builtin/commit.c
> index c46f2d1..4674600 100644
> --- a/builtin/commit.c
> +++ b/builtin/commit.c
> @@ -1118 +1118,3 @@ static int parse_status_slot(const char *var, int offset)
> -		return WT_STATUS_ONBRANCH;
> +		return WT_STATUS_LOCAL_BRANCH;
> +	if (!strcasecmp(var+offset, "remote"))
> +		return WT_STATUS_REMOTE_BRANCH;

Nicolas Dudebout· Nov 17, 2011, 02:25 UTC · re: Junio C Hamano · lore

Re: [patch] color of branches in git status -sb

Please disregard this as a patch. I do not have the time to understand how they have to be properly formatted. I just pasted the output of my git client. I am just letting you know that there is a problem that can be easily fixed. I can resend a separate email if that makes it easier for your bug tracking.

On Wed, Nov 16, 2011 at 8:13 PM, Junio C Hamano <gitster@pobox.com> wrote:

Show 49 quoted lines

> Nicolas Dudebout <nicolas.dudebout@gmail.com> writes:
>
>> The following patch fixes the fact that two colors of the status
>> output could not be configured in .git/config.
>>
>> The color of the current branch could not be modified because of the
>> name WT_STATUS_ONBRANCH having been changed to WT_STATUS_LOCAL_BRANCH.
>>
>> The color of the remote branch was not defined at all.
>>
>> By the way, when I do 'git status' instead of git 'status -sb' the
>> local and remote branch do not get colored. Is this a desired
>> behavior?
>>
>> Nicolas
>
> Please follow Documentation/SubmittingPatches.
>
> Also expect that patches to add new feature this deep in the pre-release
> feature freeze period is not likely to get reviewed by regular list
> members.
>
>>
>>       Modified   Documentation/config.txt
>
> Don't do this. We can tell what is modified from "diff --git" lines.
>
>> diff --git a/Documentation/config.txt b/Documentation/config.txt
>> index 5a841da..7a0ddd6 100644
>> --- a/Documentation/config.txt
>> +++ b/Documentation/config.txt
>> @@ -809 +809,2 @@ color.status.<slot>::
>> -     `branch` (the current branch), or
>> +     `branch` (the current branch),
>> +        `remote` (the remote branch) or
>
> Don't do this. Proper patches have context lines for good reasons.
>
>>       Modified   builtin/commit.c
>> diff --git a/builtin/commit.c b/builtin/commit.c
>> index c46f2d1..4674600 100644
>> --- a/builtin/commit.c
>> +++ b/builtin/commit.c
>> @@ -1118 +1118,3 @@ static int parse_status_slot(const char *var, int offset)
>> -             return WT_STATUS_ONBRANCH;
>> +             return WT_STATUS_LOCAL_BRANCH;
>> +     if (!strcasecmp(var+offset, "remote"))
>> +             return WT_STATUS_REMOTE_BRANCH;
>

Jonathan Nieder· Nov 17, 2011, 10:55 UTC · re: Nicolas Dudebout · lore

Re: [patch] color of branches in git status -sb

Hi,

Nicolas Dudebout wrote:

> Please disregard this as a patch. I do not have the time to understand
> how they have to be properly formatted. I just pasted the output of my
> git client.

What git client are you using? Maybe it can be fixed to produce patches that others can easily apply.

The 'git diff' and 'git format-patch' commands produce applicable patches when I try, though that doesn't mean much.

Hope that helps, Jonathan

Nicolas Dudebout· Nov 17, 2011, 14:16 UTC · re: Jonathan Nieder · lore

Re: [patch] color of branches in git status -sb

I just tried to redo this patch. I stumbled upon some inconsistency between the coloring of 'git status' and 'git status -sb'. I'll read a little bit more to give a better report with the correct patch.

In a nutshell, the local branch is colored by:
- WT_STATUS_ONBRANCH for git status
- WT_STATUS_LOCAL_BRANCH for git status -sb

Nicolas

On Thu, Nov 17, 2011 at 5:55 AM, Jonathan Nieder <jrnieder@gmail.com> wrote:

Show 17 quoted lines

> Hi,
>
> Nicolas Dudebout wrote:
>
>> Please disregard this as a patch. I do not have the time to understand
>> how they have to be properly formatted. I just pasted the output of my
>> git client.
>
> What git client are you using?  Maybe it can be fixed to produce
> patches that others can easily apply.
>
> The 'git diff' and 'git format-patch' commands produce applicable
> patches when I try, though that doesn't mean much.
>
> Hope that helps,
> Jonathan
>