• Martin Ågren's avatar
    tag: respect `pager.tag` in list-mode only · de121ffe
    Martin Ågren authored
    Using, e.g., `git -c pager.tag tag -a new-tag` results in errors such as
    "Vim: Warning: Output is not to a terminal" and a garbled terminal.
    Someone who makes use of both `git tag -a` and `git tag -l` will
    probably not set `pager.tag`, so that `git tag -a` will actually work,
    at the cost of not paging output of `git tag -l`.
    
    Use the mechanisms introduced in two earlier patches to ignore
    `pager.tag` in git.c and let the `git tag` builtin handle it on its own.
    Only respect `pager.tag` when running in list-mode.
    
    There is a window between where the pager is started before and after
    this patch. This means that early errors can behave slightly different
    before and after this patch. Since operation-parsing has to happen
    inside this window, this can be seen with `git -c pager.tag="echo pager
    is used" tag -l --unknown-option`. This change in paging-behavior should
    be acceptable since it only affects erroneous usages.
    
    Update the documentation and update tests.
    
    If an alias is used to run `git tag -a`, then `pager.tag` will still be
    respected. Document this known breakage. It will be fixed in a later
    commit. Add a similar test for `-l`, which works.
    Noticed-by: Anatoly Borodin's avatarAnatoly Borodin <anatoly.borodin@gmail.com>
    Suggested-by: default avatarJeff King <peff@peff.net>
    Signed-off-by: default avatarMartin Ågren <martin.agren@gmail.com>
    Signed-off-by: default avatarJunio C Hamano <gitster@pobox.com>
    de121ffe
tag.c 15.3 KB