From: Koosha Khajehmoogahi <koosha@posteo•de>
To: git <git@vger•kernel.org>
Subject: Re: [PATCH] [GSoC] Add configuration options for some commonly used command-line options
Date: Sun, 15 Mar 2015 20:35:16 +0100 [thread overview]
Message-ID: <5505DEF4.3060208@posteo.de> (raw)
In-Reply-To: <5505DD9D.8010102@posteo.de>
On 03/15/2015 08:29 PM, Koosha Khajehmoogahi wrote:
> This patch adds a 'showmerges' config. option for git-log.
> This option determines whether the log should contain merge
> commits or not. In essence, if this option is set to true,
Sorry, this should be 'false'.
> git-log will be run as 'git-log --no-merges'.
>
> Signed-off-by: Koosha Khajehmoogahi <koosha@posteo•de>
> ---
> Documentation/config.txt | 3 +++
> builtin/log.c | 8 ++++++++
> 2 files changed, 11 insertions(+)
>
> diff --git a/Documentation/config.txt b/Documentation/config.txt
> index 1530255..7775b8c 100644
> --- a/Documentation/config.txt
> +++ b/Documentation/config.txt
> @@ -1735,6 +1735,9 @@ log.showroot::
> Tools like linkgit:git-log[1] or linkgit:git-whatchanged[1], which
> normally hide the root commit will now show it. True by default.
>
> +log.showmerges::
> + If true, merges will be shown in the log list. True by default.
> +
> log.mailmap::
> If true, makes linkgit:git-log[1], linkgit:git-show[1], and
> linkgit:git-whatchanged[1] assume `--use-mailmap`.
> diff --git a/builtin/log.c b/builtin/log.c
> index dd8f3fc..bb36f61 100644
> --- a/builtin/log.c
> +++ b/builtin/log.c
> @@ -31,6 +31,7 @@ static const char *default_date_mode = NULL;
>
> static int default_abbrev_commit;
> static int default_show_root = 1;
> +static int default_max_parents = -1;
> static int decoration_style;
> static int decoration_given;
> static int use_mailmap_config;
> @@ -108,6 +109,7 @@ static void cmd_log_init_defaults(struct rev_info *rev)
> rev->diffopt.stat_graph_width = -1; /* respect statGraphWidth config */
> rev->abbrev_commit = default_abbrev_commit;
> rev->show_root_diff = default_show_root;
> + rev->max_parents = default_max_parents;
> rev->subject_prefix = fmt_patch_subject_prefix;
> DIFF_OPT_SET(&rev->diffopt, ALLOW_TEXTCONV);
>
> @@ -390,6 +392,12 @@ static int git_log_config(const char *var, const char *value, void *cb)
> default_show_root = git_config_bool(var, value);
> return 0;
> }
> +
> + if (!strcmp(var, "log.showmerges")) {
> + default_max_parents = git_config_bool(var, value) ? -1 : 1;
> + return 0;
> + }
> +
> if (skip_prefix(var, "color.decorate.", &slot_name))
> return parse_decorate_color_config(var, slot_name, value);
> if (!strcmp(var, "log.mailmap")) {
>
--
Koosha
next prev parent reply other threads:[~2015-03-15 19:36 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-15 19:29 [PATCH] [GSoC] Add configuration options for some commonly used command-line options Koosha Khajehmoogahi
2015-03-15 19:35 ` Koosha Khajehmoogahi [this message]
2015-03-15 20:24 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5505DEF4.3060208@posteo.de \
--to=koosha@posteo$(echo .)de \
--cc=git@vger$(echo .)kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox