Re: [PATCH 2/2] send-email: add series-cc-cmd option

2012-11-12 Thread Joe Perches
On Tue, 2012-11-13 at 03:21 +0530, Ramkumar Ramachandra wrote: Felipe Contreras wrote: cc-cmd is only per-file, and many times receipients get lost without seing the full patch series. s/seing/seeing [...] Looks good otherwise. s/receipients/recipients/ too Practically this is ok

Re: [PATCH 2/2] send-email: add series-cc-cmd option

2012-11-12 Thread Felipe Contreras
On Mon, Nov 12, 2012 at 11:52 PM, Joe Perches j...@perches.com wrote: On Tue, 2012-11-13 at 03:21 +0530, Ramkumar Ramachandra wrote: Felipe Contreras wrote: cc-cmd is only per-file, and many times receipients get lost without seing the full patch series. s/seing/seeing [...] Looks

Re: [PATCH 2/2] send-email: add series-cc-cmd option

2012-11-12 Thread Joe Perches
On Tue, 2012-11-13 at 00:03 +0100, Felipe Contreras wrote: On Mon, Nov 12, 2012 at 11:52 PM, Joe Perches j...@perches.com wrote: On Tue, 2012-11-13 at 03:21 +0530, Ramkumar Ramachandra wrote: Felipe Contreras wrote: cc-cmd is only per-file, and many times receipients get lost without

Re: [PATCH 2/2] send-email: add series-cc-cmd option

2012-11-12 Thread Felipe Contreras
On Tue, Nov 13, 2012 at 12:13 AM, Joe Perches j...@perches.com wrote: On Tue, 2012-11-13 at 00:03 +0100, Felipe Contreras wrote: On Mon, Nov 12, 2012 at 11:52 PM, Joe Perches j...@perches.com wrote: On Tue, 2012-11-13 at 03:21 +0530, Ramkumar Ramachandra wrote: Felipe Contreras wrote:

Re: [PATCH 2/2] send-email: add series-cc-cmd option

2012-11-12 Thread Joe Perches
On Tue, 2012-11-13 at 00:37 +0100, Felipe Contreras wrote: On Tue, Nov 13, 2012 at 12:13 AM, Joe Perches j...@perches.com wrote: On Tue, 2012-11-13 at 00:03 +0100, Felipe Contreras wrote: [] For --to-cmd and --cc-cmd? So basically you check the dirname of the argument passed? yes.

Re: [PATCH 2/2] send-email: add series-cc-cmd option

2012-11-12 Thread Junio C Hamano
Joe Perches j...@perches.com writes: On Tue, 2012-11-13 at 03:21 +0530, Ramkumar Ramachandra wrote: Felipe Contreras wrote: cc-cmd is only per-file, and many times receipients get lost without seing the full patch series. s/seing/seeing [...] Looks good otherwise.

Re: [PATCH 2/2] send-email: add series-cc-cmd option

2012-11-12 Thread Felipe Contreras
On Tue, Nov 13, 2012 at 1:37 AM, Junio C Hamano gits...@pobox.com wrote: Joe Perches j...@perches.com writes: On Tue, 2012-11-13 at 03:21 +0530, Ramkumar Ramachandra wrote: Felipe Contreras wrote: cc-cmd is only per-file, and many times receipients get lost without seing the full patch

[PATCH 2/2] send-email: add series-cc-cmd option

2012-11-11 Thread Felipe Contreras
cc-cmd is only per-file, and many times receipients get lost without seing the full patch series. So, add an option for series-cc-cmd, which receives as an argument rev-list options, just like format-patch. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com ---