Re: [PATCHv2 0/2] log decorations for HEAD

2015-03-23 Thread Julien Cretel
On Tue, Mar 10, 2015 at 1:53 PM, Michael J Gruber
g...@drmicha.warpmail.net wrote:

 So it didn't take too long to convince me after all :)

 Here comes Junio's version, preceded by a cleanup of the color
 setting and resetting for decorations.

 Junio C Hamano (1):
   log: decorate HEAD with branch name

 Michael J Gruber (1):
   log-tree: properly reset colors

  log-tree.c| 76 
 ++-
  t/t4013/diff.log_--decorate_--all |  2 +-
  t/t4207-log-decoration-colors.sh  | 16 -
  3 files changed, 77 insertions(+), 17 deletions(-)

 --
 2.3.2.346.gb6960c9


I'm so glad you've implemented this. Thanks again.

I hope I'll get familiar enough with the Git codebase to start posting
patches of my own, in the future.

Jubobs
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCHv2 0/2] log decorations for HEAD

2015-03-12 Thread Junio C Hamano
Michael J Gruber g...@drmicha.warpmail.net writes:

 So it didn't take too long to convince me after all :)

 Here comes Junio's version, preceded by a cleanup of the color
 setting and resetting for decorations.

 Junio C Hamano (1):
   log: decorate HEAD with branch name

 Michael J Gruber (1):
   log-tree: properly reset colors

  log-tree.c| 76 
 ++-
  t/t4013/diff.log_--decorate_--all |  2 +-
  t/t4207-log-decoration-colors.sh  | 16 -
  3 files changed, 77 insertions(+), 17 deletions(-)

Hmph, is the do not leak commit color one the same as 5ee87585
(log --decorate: do not leak commit color into the next item,
2015-03-04) aka $gmane/264788 [*1*] or is there something I missed?


[References]

*1* http://thread.gmane.org/gmane.comp.version-control.git/264063/focus=264788




--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCHv2 0/2] log decorations for HEAD

2015-03-11 Thread Michael J Gruber
Junio C Hamano venit, vidit, dixit 10.03.2015 18:06:
 Michael J Gruber g...@drmicha.warpmail.net writes:
 
 So it didn't take too long to convince me after all :)

 Here comes Junio's version, preceded by a cleanup of the color
 setting and resetting for decorations.

 Junio C Hamano (1):
   log: decorate HEAD with branch name

 Michael J Gruber (1):
   log-tree: properly reset colors

  log-tree.c| 76 
 ++-
  t/t4013/diff.log_--decorate_--all |  2 +-
  t/t4207-log-decoration-colors.sh  | 16 -
  3 files changed, 77 insertions(+), 17 deletions(-)
 
 Hmph, is the do not leak commit color one the same as 5ee87585
 (log --decorate: do not leak commit color into the next item,
 2015-03-04) aka $gmane/264788 [*1*] or is there something I missed?
 

Uh, I missed that one. The commit msg confuses me somewhat (color.diff
vs color.decorate), but the patch to log-tree.c is the same, and so I
assume the patch to t4207 is the same, too. 2/2 from here should go on
top of 7/7 from there, accordingly.

 [References]
 
 *1* http://thread.gmane.org/gmane.comp.version-control.git/264063/focus=264788
 
 
 
 
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCHv2 0/2] log decorations for HEAD

2015-03-10 Thread Michael J Gruber
So it didn't take too long to convince me after all :)

Here comes Junio's version, preceded by a cleanup of the color
setting and resetting for decorations.

Junio C Hamano (1):
  log: decorate HEAD with branch name

Michael J Gruber (1):
  log-tree: properly reset colors

 log-tree.c| 76 ++-
 t/t4013/diff.log_--decorate_--all |  2 +-
 t/t4207-log-decoration-colors.sh  | 16 -
 3 files changed, 77 insertions(+), 17 deletions(-)

-- 
2.3.2.346.gb6960c9

--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html