git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2] diff: add --compact-summary option to complement --stat

From
Duy Nguyen <pclouds@gmail.com>
Date
Jan 19, 2018, 00:06 UTC
Message-ID
<CACsJy8CXPP_75TEHvcWStm1ZneMDP+ioA=7iYcL25oEUiVHKxw@mail.gmail.com>
In-Reply-To
<87bmhqj1y6.fsf@evledraar.gmail.com>

On Fri, Jan 19, 2018 at 4:23 AM, Ævar Arnfjörð Bjarmason <avarab@gmail.com> wrote:

Show 51 quoted lines
> Wait, isn't there a bug here in the existing --summary code, its
> documentation says it'll show information "such as creations, renames
> and mode changes".
>
> But even though your --compact-summary shows that the file is being
> added and its mode changed:
>
>     $ diff -ru <(./git-show --stat 0433d533f1) <(./git-show --stat --compact-summary 0433d533f1)
>     --- /dev/fd/63  2018-01-18 21:11:51.186570555 +0000
>     +++ /dev/fd/62  2018-01-18 21:11:51.186570555 +0000
>     @@ -14,8 +14,8 @@
>            t: add tests for pull --verify-signatures
>            merge: add config option for verifySignatures
>
>     - Documentation/merge-config.txt     |  4 ++
>     - builtin/merge.c                    |  2 +
>     - t/t5573-pull-verify-signatures.sh  | 81 ++++++++++++++++++++++++++++++++++++++
>     - t/t7612-merge-verify-signatures.sh | 45 +++++++++++++++++++++
>     +     Documentation/merge-config.txt     |  4 ++
>     +     builtin/merge.c                    |  2 +
>     + A+x t/t5573-pull-verify-signatures.sh  | 81 ++++++++++++++++++++++++++++++++++
>     +     t/t7612-merge-verify-signatures.sh | 45 +++++++++++++++++++
>       4 files changed, 132 insertions(+)
>
> There is no difference between --stat with and without --summary on the
> same commit, shouldn't it show "create mode [...]" ?
>
> E.g. 95450bbbaa will do the trick for both:
>
>     $ diff -ru <(./git-show --stat 95450bbbaa) <(./git-show --stat --summary 95450bbbaa)
>     --- /dev/fd/63  2018-01-18 21:14:20.770050599 +0000
>     +++ /dev/fd/62  2018-01-18 21:14:20.770050599 +0000
>     @@ -14,3 +14,4 @@
>       git-svn.perl                    |  1 +
>       t/t9169-git-svn-dcommit-crlf.sh | 27 +++++++++++++++++++++++++++
>       2 files changed, 28 insertions(+)
>     + create mode 100755 t/t9169-git-svn-dcommit-crlf.sh
>
>     $ diff -ru <(./git-show --stat --summary 95450bbbaa) <(./git-show --stat --compact-summary 95450bbbaa)
>     --- /dev/fd/63  2018-01-18 21:14:30.646016210 +0000
>     +++ /dev/fd/62  2018-01-18 21:14:30.646016210 +0000
>     @@ -11,7 +11,6 @@
>          Reported-by: Brian Bennett <Brian.Bennett@Transamerica.com>
>          Signed-off-by: Eric Wong <e@80x24.org>
>
>     - git-svn.perl                    |  1 +
>     - t/t9169-git-svn-dcommit-crlf.sh | 27 +++++++++++++++++++++++++++
>     +     git-svn.perl                    |  1 +
>     + A+x t/t9169-git-svn-dcommit-crlf.sh | 27 +++++++++++++++++++++++++++
>       2 files changed, 28 insertions(+)
>     - create mode 100755 t/t9169-git-svn-dcommit-crlf.sh

Interesting. 0433d533f1 is a merge commit, perhaps that has something to do with this. Adding --first-parent does show "create mode" line. I'll check this later.

-- 
Duy
Previous: Ævar Arnfjörð BjarmasonNext: Jeff King
Message 9 of 14 in “diff: add --compact-summary option to complement --stat”
  1. diff: add --compact-summary option to complement --statNguyễn Thái Ngọc Duy, Jan 13, 2018
  2. Philip OakleyJan 13, 2018
  3. Simon RuderichJan 14, 2018
  4. Duy NguyenJan 14, 2018
  5. diff: add --compact-summary option to complement --statNguyễn Thái Ngọc Duy, Jan 18, 2018
  6. Eric SunshineJan 18, 2018
  7. Duy NguyenJan 19, 2018
  8. Ævar Arnfjörð BjarmasonJan 18, 2018
  9. Duy NguyenJan 19, 2018
  10. Jeff KingJan 18, 2018
  11. Duy NguyenJan 19, 2018
  12. Jeff KingJan 19, 2018
  13. Junio C HamanoJan 19, 2018
  14. Jeff KingJan 19, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.