Re: builtin-status submodule summary

From: Ping Yin <pkufranky@gmail.com>
Date: Mar 14, 2008, 14:30 UTC
Message-ID: <46dff0320803140730jeef5808r7da67a103ce7b171@mail.gmail.com>
In-Reply-To: <1205416085-23431-1-git-send-email-pkufranky@gmail.com>

On Thu, Mar 13, 2008 at 9:48 PM, Ping Yin <pkufranky@gmail.com> wrote:

Show 17 quoted lines

> This patch series builds on top of next. It teaches builtin-status/commit
>  summary for submodules.
>
>  It contains
>       git-submodule summary: --for-status option
>       builtin-status: submodule summary support
>       builtin-status: configurable submodule summary size
>       buitin-status: Add tests for submodule summary
>
>  with diffstat
>
>   git-submodule.sh             |   17 +++++-
>   t/t7401-submodule-summary.sh |   13 ++++
>   t/t7502-status.sh            |  136 ++++++++++++++++++++++++++++++++++++++++++
>   wt-status.c                  |   38 ++++++++++++
>   4 files changed, 203 insertions(+), 1 deletions(-)
>

IMO, git submodule summary is not so useful for me if it's not integrated into git-status. In fact i never use "git submodule summary" directly. git-status with submodule summary support is very useful to help user figure out what is going on in a more global level when cooking the commit message.

So i think this series should go along with the submodule summary series.

-- 
Ping Yin