Re: [PATCH v3 3/4] git-submodule: New subcommand 'summary' (3) - limit summary size

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Mar 6, 2008 at 10:24 AM, Ping Yin <pkufranky@xxxxxxxxx> wrote:
> On Thu, Mar 6, 2008 at 8:24 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
>  > Ping Yin <pkufranky@xxxxxxxxx> writes:
>  >
>  >  > This patch teaches git-submodule an option '--summary-limit|-n <number>'
>  >  > to limit number of commits in total for the summary. Number 0 will disable
>  >  > summary and minus number means unlimted (the default).
>  >
>  >  I really doubt showing the bottom one specially makes sense.
>  >
>  >  Why not just read from
>  >
>  >         git log '--pretty=%m %h %s' --left-right --topo-order A...B
>  >
>  >  and show N top commits from each?

I doubt '%h' is useful and it may result in overlong message lines.
Is '--topo-order' expensive when only show top N commits?
I prefer to use

    git log --pretty='format:%m %s' --left-right --first-parent A...B

for modificated modules, and

   git log -1 --pretty='format:> %s' $sha1_dst
   git log -1 --pretty='format:< %s' $sha1_src

for added ( or blob -> submodule typechange) and deleted
( or submodule -> blob typechange) modules separately.




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

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux