diff --git a/builtin-log.c b/builtin-log.c index 7f4186f19e..017fcf82ce 100644 --- a/builtin-log.c +++ b/builtin-log.c @@ -108,10 +108,11 @@ static void cmd_log_init(int argc, const char **argv, const char *prefix, } /* - * defeat log.decorate configuration interacting with --pretty + * defeat log.decorate configuration interacting with --pretty=raw * from the command line. */ - if (!decoration_given && rev->pretty_given) + if (!decoration_given && rev->pretty_given + && rev->commit_format == CMIT_FMT_RAW) decoration_style = 0; if (decoration_style) { diff --git a/t/t4202-log.sh b/t/t4202-log.sh index 166de4479c..2230e606ed 100755 --- a/t/t4202-log.sh +++ b/t/t4202-log.sh @@ -390,62 +390,50 @@ test_expect_success 'log --graph with merge' ' test_expect_success 'log.decorate configuration' ' git config --unset-all log.decorate || : - git log >expect.none && - git log --decorate >expect.short && - git log --decorate=full >expect.full && - git log --oneline >expect.oneline && + git log --oneline >expect.none && + git log --oneline --decorate >expect.short && + git log --oneline --decorate=full >expect.full && echo "[log] decorate" >>.git/config && - git log >actual && - test_cmp expect.short actual && git log --oneline >actual && - test_cmp expect.oneline actual && + test_cmp expect.short actual && git config --unset-all log.decorate && git config log.decorate true && - git log >actual && - test_cmp expect.short actual && - git log --decorate=full >actual && - test_cmp expect.full actual && - git log --decorate=no >actual && - test_cmp expect.none actual && git log --oneline >actual && - test_cmp expect.oneline actual && + test_cmp expect.short actual && + git log --oneline --decorate=full >actual && + test_cmp expect.full actual && + git log --oneline --decorate=no >actual && + test_cmp expect.none actual && git config --unset-all log.decorate && git config log.decorate no && - git log >actual && - test_cmp expect.none actual && - git log --decorate >actual && - test_cmp expect.short actual && - git log --decorate=full >actual && - test_cmp expect.full actual && git log --oneline >actual && - test_cmp expect.oneline actual && + test_cmp expect.none actual && + git log --oneline --decorate >actual && + test_cmp expect.short actual && + git log --oneline --decorate=full >actual && + test_cmp expect.full actual && git config --unset-all log.decorate && git config log.decorate short && - git log >actual && - test_cmp expect.short actual && - git log --no-decorate >actual && - test_cmp expect.none actual && - git log --decorate=full >actual && - test_cmp expect.full actual && git log --oneline >actual && - test_cmp expect.oneline actual && + test_cmp expect.short actual && + git log --oneline --no-decorate >actual && + test_cmp expect.none actual && + git log --oneline --decorate=full >actual && + test_cmp expect.full actual && git config --unset-all log.decorate && git config log.decorate full && - git log >actual && - test_cmp expect.full actual && - git log --no-decorate >actual && - test_cmp expect.none actual && - git log --decorate >actual && - test_cmp expect.short actual git log --oneline >actual && - test_cmp expect.oneline actual && + test_cmp expect.full actual && + git log --oneline --no-decorate >actual && + test_cmp expect.none actual && + git log --oneline --decorate >actual && + test_cmp expect.short actual - : ' test_done