mbox series

[v3,0/3] document deprecation of log.mailmap=false default

Message ID 20190715124106.12642-1-ariadne@dereferenced.org (mailing list archive)
Headers show
Series document deprecation of log.mailmap=false default | expand

Message

Ariadne Conill July 15, 2019, 12:41 p.m. UTC
Based on the discussion of the previous patches, we concluded that
changing the default will require a transitional period.

As such, I have introduced a deprecation warning that is printed when
the log builtin commands are used.

Thanks to Junio and everyone else for providing feedback on how to
proceed.

New in version 3:
- Split interactive session detection in auto_decoration_style() into
  it's own function.
- Defang the tests in a more efficient way.

New in version 2:
- The warning is disabled when `--format` is used.
- The warning is disabled when not called from a controlling terminal.
- Tests which fake a controlling terminal have been defanged.

Ariadne Conill (3):
  log: add warning for unspecified log.mailmap setting
  documentation: mention --no-use-mailmap and log.mailmap false setting
  tests: defang pager tests by explicitly disabling the log.mailmap
    warning

 Documentation/config/log.txt |  3 ++-
 Documentation/git-log.txt    |  2 +-
 builtin/log.c                | 26 ++++++++++++++++++++++++--
 t/t7006-pager.sh             |  2 ++
 4 files changed, 29 insertions(+), 4 deletions(-)