[PATCH v3 2/4] format-patch: improve head calculation for cover-letter

2013-04-07 Thread Felipe Contreras
If we do it after the revision traversal we can be sure that this is indeed a commit that will be processed (i.e. not a merge) and it's the top most one (thus removing the NEEDSWORK comment). Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- builtin/log.c | 20 ++--

Re: [PATCH v3 2/4] format-patch: improve head calculation for cover-letter

2013-04-07 Thread Junio C Hamano
Felipe Contreras felipe.contre...@gmail.com writes: If we do it after the revision traversal we can be sure that this is indeed a commit that will be processed (i.e. not a merge) and it's the top most one (thus removing the NEEDSWORK comment). There may not be a single top-most one anyway

Re: [PATCH v3 2/4] format-patch: improve head calculation for cover-letter

2013-04-07 Thread Felipe Contreras
On Sun, Apr 7, 2013 at 1:50 AM, Junio C Hamano gits...@pobox.com wrote: Felipe Contreras felipe.contre...@gmail.com writes: If we do it after the revision traversal we can be sure that this is indeed a commit that will be processed (i.e. not a merge) and it's the top most one (thus removing