log --pretty/--oneline: ignore log.decorate

Many scripts, most notably gitk, rely on output from the log family of
command not to be molested by random user configuration.  This is
especially true when --pretty=raw is given.

Just like we disable notes output unless the command line explicitly
asks for --show-notes, disable the decoration code unless --decorate is
given explicitly from the command line and --pretty or --oneline is
given.

Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/builtin-log.c b/builtin-log.c
index 0afba31c1fc830c021f9a08f9c0442df58cdc6ff..7f4186f19e241ce28ae57f49f3b64da731fab74a 100644 (file)
--- a/builtin-log.c
+++ b/builtin-log.c
@@ -53,6 +53,7 @@ static void cmd_log_init(int argc, const char **argv, const char *prefix,
                      struct rev_info *rev)
 {
        int i;
                      struct rev_info *rev)
 {
        int i;

+       int decoration_given = 0;

 
        rev->abbrev = DEFAULT_ABBREV;
        rev->commit_format = CMIT_FMT_DEFAULT;
 
        rev->abbrev = DEFAULT_ABBREV;
        rev->commit_format = CMIT_FMT_DEFAULT;


@@ -89,11 +90,13 @@ static void cmd_log_init(int argc, const char **argv, const char *prefix,
                const char *arg = argv[i];
                if (!strcmp(arg, "--decorate")) {
                        decoration_style = DECORATE_SHORT_REFS;
                const char *arg = argv[i];
                if (!strcmp(arg, "--decorate")) {
                        decoration_style = DECORATE_SHORT_REFS;

+                       decoration_given = 1;

                } else if (!prefixcmp(arg, "--decorate=")) {
                        const char *v = skip_prefix(arg, "--decorate=");
                        decoration_style = parse_decoration_style(arg, v);
                        if (decoration_style < 0)
                                die("invalid --decorate option: %s", arg);
                } else if (!prefixcmp(arg, "--decorate=")) {
                        const char *v = skip_prefix(arg, "--decorate=");
                        decoration_style = parse_decoration_style(arg, v);
                        if (decoration_style < 0)
                                die("invalid --decorate option: %s", arg);

+                       decoration_given = 1;

                } else if (!strcmp(arg, "--no-decorate")) {
                        decoration_style = 0;
                } else if (!strcmp(arg, "--source")) {
                } else if (!strcmp(arg, "--no-decorate")) {
                        decoration_style = 0;
                } else if (!strcmp(arg, "--source")) {


@@ -103,6 +106,14 @@ static void cmd_log_init(int argc, const char **argv, const char *prefix,
                } else
                        die("unrecognized argument: %s", arg);
        }
                } else
                        die("unrecognized argument: %s", arg);
        }

+
+       /*
+        * defeat log.decorate configuration interacting with --pretty
+        * from the command line.
+        */
+       if (!decoration_given && rev->pretty_given)
+               decoration_style = 0;
+

        if (decoration_style) {
                rev->show_decorations = 1;
                load_ref_decorations(decoration_style);
        if (decoration_style) {
                rev->show_decorations = 1;
                load_ref_decorations(decoration_style);

diff --git a/t/t4202-log.sh b/t/t4202-log.sh
index 2230e606edea0c4f3dcd0b6b99298074d5341e4a..166de4479c1e704990886003f3602aaa5e13e985 100755 (executable)
--- a/t/t4202-log.sh
+++ b/t/t4202-log.sh
@@ -390,50 +390,62 @@ test_expect_success 'log --graph with merge' '
 test_expect_success 'log.decorate configuration' '
        git config --unset-all log.decorate || :
 
 test_expect_success 'log.decorate configuration' '
        git config --unset-all log.decorate || :
 

-       git log --oneline >expect.none &&
-       git log --oneline --decorate >expect.short &&
-       git log --oneline --decorate=full >expect.full &&
+       git log >expect.none &&
+       git log --decorate >expect.short &&
+       git log --decorate=full >expect.full &&
+       git log --oneline >expect.oneline &&

 
        echo "[log] decorate" >>.git/config &&
 
        echo "[log] decorate" >>.git/config &&

-       git log --oneline >actual &&
+       git log >actual &&

        test_cmp expect.short actual &&
        test_cmp expect.short actual &&

+       git log --oneline >actual &&
+       test_cmp expect.oneline actual &&

 
        git config --unset-all log.decorate &&
        git config log.decorate true &&
 
        git config --unset-all log.decorate &&
        git config log.decorate true &&

-       git log --oneline >actual &&
+       git log >actual &&

        test_cmp expect.short actual &&
        test_cmp expect.short actual &&

-       git log --oneline --decorate=full >actual &&
+       git log --decorate=full >actual &&

        test_cmp expect.full actual &&
        test_cmp expect.full actual &&

-       git log --oneline --decorate=no >actual &&
+       git log --decorate=no >actual &&

        test_cmp expect.none actual &&
        test_cmp expect.none actual &&

+       git log --oneline >actual &&
+       test_cmp expect.oneline actual &&

 
        git config --unset-all log.decorate &&
        git config log.decorate no &&
 
        git config --unset-all log.decorate &&
        git config log.decorate no &&

-       git log --oneline >actual &&
+       git log >actual &&

        test_cmp expect.none actual &&
        test_cmp expect.none actual &&

-       git log --oneline --decorate >actual &&
+       git log --decorate >actual &&

        test_cmp expect.short actual &&
        test_cmp expect.short actual &&

-       git log --oneline --decorate=full >actual &&
+       git log --decorate=full >actual &&

        test_cmp expect.full actual &&
        test_cmp expect.full actual &&

+       git log --oneline >actual &&
+       test_cmp expect.oneline actual &&

 
        git config --unset-all log.decorate &&
        git config log.decorate short &&
 
        git config --unset-all log.decorate &&
        git config log.decorate short &&

-       git log --oneline >actual &&
+       git log >actual &&

        test_cmp expect.short actual &&
        test_cmp expect.short actual &&

-       git log --oneline --no-decorate >actual &&
+       git log --no-decorate >actual &&

        test_cmp expect.none actual &&
        test_cmp expect.none actual &&

-       git log --oneline --decorate=full >actual &&
+       git log --decorate=full >actual &&

        test_cmp expect.full actual &&
        test_cmp expect.full actual &&

+       git log --oneline >actual &&
+       test_cmp expect.oneline actual &&

 
        git config --unset-all log.decorate &&
        git config log.decorate full &&
 
        git config --unset-all log.decorate &&
        git config log.decorate full &&

-       git log --oneline >actual &&
+       git log >actual &&

        test_cmp expect.full actual &&
        test_cmp expect.full actual &&

-       git log --oneline --no-decorate >actual &&
+       git log --no-decorate >actual &&

        test_cmp expect.none actual &&
        test_cmp expect.none actual &&

-       git log --oneline --decorate >actual &&
+       git log --decorate >actual &&

        test_cmp expect.short actual
        test_cmp expect.short actual

+       git log --oneline >actual &&
+       test_cmp expect.oneline actual &&

 
 

+       :

 '
 
 test_done
 '
 
 test_done