Skip to content

Commit

Permalink
pull: configure --rebase via branch.<name>.rebase or pull.rebase
Browse files Browse the repository at this point in the history
Since cd67e4d (Teach 'git pull' about --rebase, 2007-11-28),
fetch+rebase could be set by default by defining the config variable
branch.<name>.rebase. This setting can be overriden on the command line
by --rebase and --no-rebase.

Since 6b37dff (pull: introduce a pull.rebase option to enable --rebase,
2011-11-06), git-pull --rebase can also be configured via the
pull.rebase configuration option.

Re-implement support for these two configuration settings by introducing
config_get_rebase() which is called before parse_options() to set the
default value of opt_rebase.

Helped-by: Stefan Beller <sbeller@google.com>
Helped-by: Duy Nguyen <pclouds@gmail.com>
Signed-off-by: Paul Tan <pyokagan@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Paul Tan authored and Junio C Hamano committed Jun 18, 2015
1 parent 1678b81 commit 81dbd76
Showing 1 changed file with 34 additions and 1 deletion.
35 changes: 34 additions & 1 deletion builtin/pull.c
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ static int opt_verbosity;
static char *opt_progress;

/* Options passed to git-merge or git-rebase */
static enum rebase_type opt_rebase;
static enum rebase_type opt_rebase = -1;
static char *opt_diffstat;
static char *opt_log;
static char *opt_squash;
Expand Down Expand Up @@ -265,6 +265,36 @@ static const char *config_get_ff(void)
die(_("Invalid value for pull.ff: %s"), value);
}

/**
* Returns the default configured value for --rebase. It first looks for the
* value of "branch.$curr_branch.rebase", where $curr_branch is the current
* branch, and if HEAD is detached or the configuration key does not exist,
* looks for the value of "pull.rebase". If both configuration keys do not
* exist, returns REBASE_FALSE.
*/
static enum rebase_type config_get_rebase(void)
{
struct branch *curr_branch = branch_get("HEAD");
const char *value;

if (curr_branch) {
char *key = xstrfmt("branch.%s.rebase", curr_branch->name);

if (!git_config_get_value(key, &value)) {
enum rebase_type ret = parse_config_rebase(key, value, 1);
free(key);
return ret;
}

free(key);
}

if (!git_config_get_value("pull.rebase", &value))
return parse_config_rebase("pull.rebase", value, 1);

return REBASE_FALSE;
}

/**
* Appends merge candidates from FETCH_HEAD that are not marked not-for-merge
* into merge_heads.
Expand Down Expand Up @@ -707,6 +737,9 @@ int cmd_pull(int argc, const char **argv, const char *prefix)
if (!opt_ff)
opt_ff = xstrdup_or_null(config_get_ff());

if (opt_rebase < 0)
opt_rebase = config_get_rebase();

git_config(git_default_config, NULL);

if (read_cache_unmerged())
Expand Down

0 comments on commit 81dbd76

Please sign in to comment.