Re: [PATCH v3 3/4] get rid of git submodule summary --for-status

2013-09-03 Thread Jens Lehmann
Am 29.08.2013 21:54, schrieb Jens Lehmann: Am 29.08.2013 15:05, schrieb Matthieu Moy: The --for-status option was an undocumented option used only by wt-status.c, which inserted a header and commented out the output. We can achieve the same result within wt-status.c, without polluting the

Re: [PATCH v3 3/4] get rid of git submodule summary --for-status

2013-09-01 Thread Jens Lehmann
Am 31.08.2013 19:08, schrieb brian m. carlson: On Fri, Aug 30, 2013 at 10:08:53PM +0200, Jens Lehmann wrote: Am 30.08.2013 21:51, schrieb Jens Lehmann: Am 30.08.2013 21:40, schrieb Jens Lehmann: Am 29.08.2013 23:23, schrieb Matthieu Moy: Jens Lehmann jens.lehm...@web.de writes: Am

Re: [PATCH v3 3/4] get rid of git submodule summary --for-status

2013-08-31 Thread brian m. carlson
On Fri, Aug 30, 2013 at 10:08:53PM +0200, Jens Lehmann wrote: Am 30.08.2013 21:51, schrieb Jens Lehmann: Am 30.08.2013 21:40, schrieb Jens Lehmann: Am 29.08.2013 23:23, schrieb Matthieu Moy: Jens Lehmann jens.lehm...@web.de writes: Am 29.08.2013 15:05, schrieb Matthieu Moy: Because of

Re: [PATCH v3 3/4] get rid of git submodule summary --for-status

2013-08-30 Thread Jens Lehmann
Am 30.08.2013 21:51, schrieb Jens Lehmann: Am 30.08.2013 21:40, schrieb Jens Lehmann: Am 29.08.2013 23:23, schrieb Matthieu Moy: Jens Lehmann jens.lehm...@web.de writes: Am 29.08.2013 15:05, schrieb Matthieu Moy: The --for-status option was an undocumented option used only by wt-status.c,

Re: [PATCH v3 3/4] get rid of git submodule summary --for-status

2013-08-30 Thread Jens Lehmann
Am 30.08.2013 21:40, schrieb Jens Lehmann: Am 29.08.2013 23:23, schrieb Matthieu Moy: Jens Lehmann jens.lehm...@web.de writes: Am 29.08.2013 15:05, schrieb Matthieu Moy: The --for-status option was an undocumented option used only by wt-status.c, which inserted a header and commented out the

Re: [PATCH v3 3/4] get rid of git submodule summary --for-status

2013-08-30 Thread Jens Lehmann
Am 29.08.2013 23:23, schrieb Matthieu Moy: Jens Lehmann jens.lehm...@web.de writes: Am 29.08.2013 15:05, schrieb Matthieu Moy: The --for-status option was an undocumented option used only by wt-status.c, which inserted a header and commented out the output. We can achieve the same result

[PATCH v3 3/4] get rid of git submodule summary --for-status

2013-08-29 Thread Matthieu Moy
The --for-status option was an undocumented option used only by wt-status.c, which inserted a header and commented out the output. We can achieve the same result within wt-status.c, without polluting the submodule command-line options. This will make it easier to disable the comments from

Re: [PATCH v3 3/4] get rid of git submodule summary --for-status

2013-08-29 Thread Junio C Hamano
Matthieu Moy matthieu@imag.fr writes: + /* prepend header, only if there's an actual output */ + if (len) { + if (uncommitted) + strbuf_addstr(summary, _(Submodules changed but not updated:)); + else +

Re: [PATCH v3 3/4] get rid of git submodule summary --for-status

2013-08-29 Thread Matthieu Moy
Junio C Hamano gits...@pobox.com writes: Matthieu Moy matthieu@imag.fr writes: +/* prepend header, only if there's an actual output */ +if (len) { +if (uncommitted) +strbuf_addstr(summary, _(Submodules changed but not updated:)); +