{"thread":{"id":"18606","subject":"[PATCH 0/4] push.default and branch.<name>.{remote,merge} changes","startedAt":"2009-03-28T23:10:00Z","lastAt":"2009-03-30T12:50:26Z","messageCount":13,"participants":["Santi Béjar","Jay Soffian","Finn Arne Gangstad","Johannes Schindelin","Jeff King"],"isPatch":true,"patchVersion":1,"patchTotal":4},"messages":[{"id":"109738","messageId":"1238281804-30290-1-git-send-email-santi@agolina.net","threadId":"18606","inReplyTo":null,"subject":"[PATCH 0/4] push.default and branch.<name>.{remote,merge} changes","fromName":"Santi Béjar","fromEmail":"santi@agolina.net","sentAt":"2009-03-28T23:10:00Z","receivedAt":"2009-03-28T23:10:00Z","isPatch":true,"sender":{"key":"santi@agolina.net","avatar":null},"body":"Hi *,\n\n  the four patches are independent, but I've send them in a series because\ntheir topic is related. Well, there is a textual dependency between the 3rd\nand the 4th.\n\n  The 1st one can also be applied to 'maint' while the rest is on top of 'next'\nbecause the 'push.default' is only there.\n\nSanti Béjar (4):\n  Documentation: enhance branch.<name>.{remote,merge}\n  Documentation: push.default applies to all remotes\n  Documentation: branch.*.merge can also afect 'git-push'\n  Rename push.default to push.style\n\n Documentation/RelNotes-1.6.3.txt |    2 +-\n Documentation/config.txt         |   21 ++++++++++-----------\n builtin-push.c                   |   16 ++++++++--------\n cache.h                          |   14 +++++++-------\n config.c                         |   10 +++++-----\n environment.c                    |    2 +-\n 6 files changed, 32 insertions(+), 33 deletions(-)\n"},{"id":"109739","messageId":"1238281804-30290-2-git-send-email-santi@agolina.net","threadId":"18606","inReplyTo":"1238281804-30290-1-git-send-email-santi@agolina.net","subject":"[PATCH 1/4] Documentation: enhance branch.<name>.{remote,merge}","fromName":"Santi Béjar","fromEmail":"santi@agolina.net","sentAt":"2009-03-28T23:10:01Z","receivedAt":"2009-03-28T23:10:01Z","isPatch":true,"sender":{"key":"santi@agolina.net","avatar":null},"body":"The documentation for branch.*.merge is very dense, so add a simple\nexplanation on top of it.\n\nAnd branch.*.remote also afects 'git push'. Text taken from\n'push.default'.\n\nSigned-off-by: Santi Béjar <santi@agolina.net>\n---\n Documentation/config.txt |    8 ++++++--\n 1 files changed, 6 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex 089569a..56f0cd7 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -473,10 +473,14 @@ branch.autosetuprebase::\n \tThis option defaults to never.\n \n branch.<name>.remote::\n-\tWhen in branch <name>, it tells 'git-fetch' which remote to fetch.\n-\tIf this option is not given, 'git-fetch' defaults to remote \"origin\".\n+\tWhen in branch <name>, it tells 'git-fetch' and 'git-push' which\n+\tremote to fetch/push, and defaults to `origin` if no remote is\n+\tconfigured. `origin` is also used if you are not on any branch.\n \n branch.<name>.merge::\n+\tIt defines, together with branch.<name>.remote, the tracking branch\n+\tfor the current branch. It tells 'git-fetch'/'git-pull' which\n+\tbranch to merge.\n \tWhen in branch <name>, it tells 'git-fetch' the default\n \trefspec to be marked for merging in FETCH_HEAD. The value is\n \thandled like the remote part of a refspec, and must match a\n-- \n1.6.1.258.g7ff14\n"},{"id":"109740","messageId":"1238281804-30290-3-git-send-email-santi@agolina.net","threadId":"18606","inReplyTo":"1238281804-30290-1-git-send-email-santi@agolina.net","subject":"[PATCH 2/4] Documentation: push.default applies to all remotes","fromName":"Santi Béjar","fromEmail":"santi@agolina.net","sentAt":"2009-03-28T23:10:02Z","receivedAt":"2009-03-28T23:10:02Z","isPatch":true,"sender":{"key":"santi@agolina.net","avatar":null},"body":"push.default is not only for the current remote but setting the default\nbehaviour for all remotes.\n\nSigned-off-by: Santi Béjar <santi@agolina.net>\n---\n Documentation/config.txt |   11 +++--------\n 1 files changed, 3 insertions(+), 8 deletions(-)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex 56f0cd7..07f88f5 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -1219,19 +1219,14 @@ push.default::\n \tDefines the action git push should take if no refspec is given\n \ton the command line, no refspec is configured in the remote, and\n \tno refspec is implied by any of the options given on the command\n-\tline.\n-+\n-The term `current remote` means the remote configured for the current\n-branch, or `origin` if no remote is configured. `origin` is also used\n-if you are not on any branch. Possible values are:\n+\tline. Possible values are:\n +\n * `nothing` do not push anything.\n-* `matching` push all matching branches to the current remote.\n+* `matching` push all matching branches.\n   All branches having the same name in both ends are considered to be\n   matching. This is the current default value.\n * `tracking` push the current branch to the branch it is tracking.\n-* `current` push the current branch to a branch of the same name on the\n-  current remote.\n+* `current` push the current branch to a branch of the same name.\n \n rebase.stat::\n \tWhether to show a diffstat of what changed upstream since the last\n-- \n1.6.1.258.g7ff14\n"},{"id":"109742","messageId":"1238281804-30290-4-git-send-email-santi@agolina.net","threadId":"18606","inReplyTo":"1238281804-30290-1-git-send-email-santi@agolina.net","subject":"[PATCH 3/4] Documentation: branch.*.merge can also afect 'git-push'","fromName":"Santi Béjar","fromEmail":"santi@agolina.net","sentAt":"2009-03-28T23:10:03Z","receivedAt":"2009-03-28T23:10:03Z","isPatch":true,"sender":{"key":"santi@agolina.net","avatar":null},"body":"\nSigned-off-by: Santi Béjar <santi@agolina.net>\n---\n Documentation/config.txt |    2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex 07f88f5..cb392fd 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -480,7 +480,7 @@ branch.<name>.remote::\n branch.<name>.merge::\n \tIt defines, together with branch.<name>.remote, the tracking branch\n \tfor the current branch. It tells 'git-fetch'/'git-pull' which\n-\tbranch to merge.\n+\tbranch to merge and can also afect 'git-push' (see push.default).\n \tWhen in branch <name>, it tells 'git-fetch' the default\n \trefspec to be marked for merging in FETCH_HEAD. The value is\n \thandled like the remote part of a refspec, and must match a\n-- \n1.6.1.258.g7ff14\n"},{"id":"109741","messageId":"1238281804-30290-5-git-send-email-santi@agolina.net","threadId":"18606","inReplyTo":"1238281804-30290-1-git-send-email-santi@agolina.net","subject":"[PATCH 4/4] Rename push.default to push.style","fromName":"Santi Béjar","fromEmail":"santi@agolina.net","sentAt":"2009-03-28T23:10:04Z","receivedAt":"2009-03-28T23:10:04Z","isPatch":true,"sender":{"key":"santi@agolina.net","avatar":null},"body":"push.default was too generic, and also didn't specify if it was about\nremote, refspec, branches, behaviour...\n\nSigned-off-by: Santi Béjar <santi@agolina.net>\n---\n Documentation/RelNotes-1.6.3.txt |    2 +-\n Documentation/config.txt         |    4 ++--\n builtin-push.c                   |   16 ++++++++--------\n cache.h                          |   14 +++++++-------\n config.c                         |   10 +++++-----\n environment.c                    |    2 +-\n 6 files changed, 24 insertions(+), 24 deletions(-)\n\ndiff --git a/Documentation/RelNotes-1.6.3.txt b/Documentation/RelNotes-1.6.3.txt\nindex f0a2e41..f3533e6 100644\n--- a/Documentation/RelNotes-1.6.3.txt\n+++ b/Documentation/RelNotes-1.6.3.txt\n@@ -24,7 +24,7 @@ receive.denyDeleteCurrent in the receiving repository.\n \n When the user does not tell \"git push\" what to push, it has always\n pushed matching refs.  For some people it is unexpected, and a new\n-configuration variable push.default has been introduced to allow\n+configuration variable push.style has been introduced to allow\n changing a different default behaviour.  To advertise the new feature,\n a big warning is issued if this is not configured and a git push without\n arguments is attempted.\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex cb392fd..f2d675a 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -480,7 +480,7 @@ branch.<name>.remote::\n branch.<name>.merge::\n \tIt defines, together with branch.<name>.remote, the tracking branch\n \tfor the current branch. It tells 'git-fetch'/'git-pull' which\n-\tbranch to merge and can also afect 'git-push' (see push.default).\n+\tbranch to merge and can also afect 'git-push' (see push.style).\n \tWhen in branch <name>, it tells 'git-fetch' the default\n \trefspec to be marked for merging in FETCH_HEAD. The value is\n \thandled like the remote part of a refspec, and must match a\n@@ -1215,7 +1215,7 @@ pull.octopus::\n pull.twohead::\n \tThe default merge strategy to use when pulling a single branch.\n \n-push.default::\n+push.style::\n \tDefines the action git push should take if no refspec is given\n \ton the command line, no refspec is configured in the remote, and\n \tno refspec is implied by any of the options given on the command\ndiff --git a/builtin-push.c b/builtin-push.c\nindex 2eabcd3..ba0dd6f 100644\n--- a/builtin-push.c\n+++ b/builtin-push.c\n@@ -72,7 +72,7 @@ static const char *warn_unconfigured_push_msg[] = {\n \t\"not necessarily be what you want to happen.\",\n \t\"\",\n \t\"You can specify what action you want to take in this case, and\",\n-\t\"avoid seeing this message again, by configuring 'push.default' to:\",\n+\t\"avoid seeing this message again, by configuring 'push.style' to:\",\n \t\"  'nothing'  : Do not push anything\",\n \t\"  'matching' : Push all matching branches (default)\",\n \t\"  'tracking' : Push the current branch to whatever it is tracking\",\n@@ -89,26 +89,26 @@ static void warn_unconfigured_push(void)\n static void setup_default_push_refspecs(void)\n {\n \tgit_config(git_default_config, NULL);\n-\tswitch (push_default) {\n-\tcase PUSH_DEFAULT_UNSPECIFIED:\n+\tswitch (push_style) {\n+\tcase PUSH_STYLE_UNSPECIFIED:\n \t\twarn_unconfigured_push();\n \t\t/* fallthrough */\n \n-\tcase PUSH_DEFAULT_MATCHING:\n+\tcase PUSH_STYLE_MATCHING:\n \t\tadd_refspec(\":\");\n \t\tbreak;\n \n-\tcase PUSH_DEFAULT_TRACKING:\n+\tcase PUSH_STYLE_TRACKING:\n \t\tsetup_push_tracking();\n \t\tbreak;\n \n-\tcase PUSH_DEFAULT_CURRENT:\n+\tcase PUSH_STYLE_CURRENT:\n \t\tadd_refspec(\"HEAD\");\n \t\tbreak;\n \n-\tcase PUSH_DEFAULT_NOTHING:\n+\tcase PUSH_STYLE_NOTHING:\n \t\tdie(\"You didn't specify any refspecs to push, and \"\n-\t\t    \"push.default is \\\"nothing\\\".\");\n+\t\t    \"push.style is \\\"nothing\\\".\");\n \t\tbreak;\n \t}\n }\ndiff --git a/cache.h b/cache.h\nindex 641529b..9190046 100644\n--- a/cache.h\n+++ b/cache.h\n@@ -542,17 +542,17 @@ enum rebase_setup_type {\n \tAUTOREBASE_ALWAYS,\n };\n \n-enum push_default_type {\n-\tPUSH_DEFAULT_UNSPECIFIED = -1,\n-\tPUSH_DEFAULT_NOTHING = 0,\n-\tPUSH_DEFAULT_MATCHING,\n-\tPUSH_DEFAULT_TRACKING,\n-\tPUSH_DEFAULT_CURRENT,\n+enum push_style_type {\n+\tPUSH_STYLE_UNSPECIFIED = -1,\n+\tPUSH_STYLE_NOTHING = 0,\n+\tPUSH_STYLE_MATCHING,\n+\tPUSH_STYLE_TRACKING,\n+\tPUSH_STYLE_CURRENT,\n };\n \n extern enum branch_track git_branch_track;\n extern enum rebase_setup_type autorebase;\n-extern enum push_default_type push_default;\n+extern enum push_style_type push_style;\n \n #define GIT_REPO_VERSION 0\n extern int repository_format_version;\ndiff --git a/config.c b/config.c\nindex b76fe4c..86ac830 100644\n--- a/config.c\n+++ b/config.c\n@@ -567,17 +567,17 @@ static int git_default_branch_config(const char *var, const char *value)\n \n static int git_default_push_config(const char *var, const char *value)\n {\n-\tif (!strcmp(var, \"push.default\")) {\n+\tif (!strcmp(var, \"push.style\")) {\n \t\tif (!value)\n \t\t\treturn config_error_nonbool(var);\n \t\telse if (!strcmp(value, \"nothing\"))\n-\t\t\tpush_default = PUSH_DEFAULT_NOTHING;\n+\t\t\tpush_style = PUSH_STYLE_NOTHING;\n \t\telse if (!strcmp(value, \"matching\"))\n-\t\t\tpush_default = PUSH_DEFAULT_MATCHING;\n+\t\t\tpush_style = PUSH_STYLE_MATCHING;\n \t\telse if (!strcmp(value, \"tracking\"))\n-\t\t\tpush_default = PUSH_DEFAULT_TRACKING;\n+\t\t\tpush_style = PUSH_STYLE_TRACKING;\n \t\telse if (!strcmp(value, \"current\"))\n-\t\t\tpush_default = PUSH_DEFAULT_CURRENT;\n+\t\t\tpush_style = PUSH_STYLE_CURRENT;\n \t\telse {\n \t\t\terror(\"Malformed value for %s: %s\", var, value);\n \t\t\treturn error(\"Must be one of nothing, matching, \"\ndiff --git a/environment.c b/environment.c\nindex 4696885..8d2450c 100644\n--- a/environment.c\n+++ b/environment.c\n@@ -42,7 +42,7 @@ enum safe_crlf safe_crlf = SAFE_CRLF_WARN;\n unsigned whitespace_rule_cfg = WS_DEFAULT_RULE;\n enum branch_track git_branch_track = BRANCH_TRACK_REMOTE;\n enum rebase_setup_type autorebase = AUTOREBASE_NEVER;\n-enum push_default_type push_default = PUSH_DEFAULT_UNSPECIFIED;\n+enum push_style_type push_style = PUSH_STYLE_UNSPECIFIED;\n \n /* Parallel index stat data preload? */\n int core_preload_index = 0;\n-- \n1.6.1.258.g7ff14\n"},{"id":"109744","messageId":"76718490903281838s1bc7e908l4702cf9ab020189@mail.gmail.com","threadId":"18606","inReplyTo":"1238281804-30290-2-git-send-email-santi@agolina.net","subject":"Re: [PATCH 1/4] Documentation: enhance branch.<name>.{remote,merge}","fromName":"Jay Soffian","fromEmail":"jaysoffian@gmail.com","sentAt":"2009-03-29T01:38:42Z","receivedAt":"2009-03-29T01:38:42Z","isPatch":true,"sender":{"key":"jaysoffian@gmail.com","avatar":"https://avatars.githubusercontent.com/u/155970?v=4"},"body":"On Sat, Mar 28, 2009 at 7:10 PM, Santi Béjar <santi@agolina.net> wrote:\n>  branch.<name>.merge::\n> +       It defines, together with branch.<name>.remote, the tracking branch\n> +       for the current branch. It tells 'git-fetch'/'git-pull' which\n> +       branch to merge.\n\nI think it would be clearer to say \"the upstream branch for the\ncurrent branch\", since\nit could very well be a local branch, not necessarily a remote tracking branch.\n\nj.\n"},{"id":"109766","messageId":"adf1fd3d0903290354y7ac26542r8d82309a7247eacc@mail.gmail.com","threadId":"18606","inReplyTo":"76718490903281838s1bc7e908l4702cf9ab020189@mail.gmail.com","subject":"Re: [PATCH 1/4] Documentation: enhance branch.<name>.{remote,merge}","fromName":"Santi Béjar","fromEmail":"santi@agolina.net","sentAt":"2009-03-29T10:54:30Z","receivedAt":"2009-03-29T10:54:30Z","isPatch":true,"sender":{"key":"santi@agolina.net","avatar":null},"body":"2009/3/29 Jay Soffian <jaysoffian@gmail.com>:\n> On Sat, Mar 28, 2009 at 7:10 PM, Santi Béjar <santi@agolina.net> wrote:\n>>  branch.<name>.merge::\n>> +       It defines, together with branch.<name>.remote, the tracking branch\n>> +       for the current branch. It tells 'git-fetch'/'git-pull' which\n>> +       branch to merge.\n>\n> I think it would be clearer to say \"the upstream branch for the\n> current branch\", since\n> it could very well be a local branch, not necessarily a remote tracking branch.\n\nIt makes sense, thanks.\n\nSanti\n"},{"id":"109851","messageId":"20090330080115.GA19142@pvv.org","threadId":"18606","inReplyTo":"1238281804-30290-5-git-send-email-santi@agolina.net","subject":"Re: [PATCH 4/4] Rename push.default to push.style","fromName":"Finn Arne Gangstad","fromEmail":"finnag@pvv.org","sentAt":"2009-03-30T08:01:15Z","receivedAt":"2009-03-30T08:01:15Z","isPatch":true,"sender":{"key":"finnag@pvv.org","avatar":"https://gravatar.com/avatar/b421ddd58c3f0f93aa473e17b98bb8d53c221fef741746bc8cb59fae4ec6d95e?d=mp&s=160"},"body":"On Sun, Mar 29, 2009 at 12:10:04AM +0100, Santi Béjar wrote:\n> push.default was too generic, and also didn't specify if it was about\n> remote, refspec, branches, behaviour...\n> [...]\n>  \t\"You can specify what action you want to take in this case, and\",\n> -\t\"avoid seeing this message again, by configuring 'push.default' to:\",\n> +\t\"avoid seeing this message again, by configuring 'push.style' to:\",\n>  \t\"  'nothing'  : Do not push anything\",\n>  \t\"  'matching' : Push all matching branches (default)\",\n>  \t\"  'tracking' : Push the current branch to whatever it is tracking\",\n> [...]\n\nI don't think \"push.style\" is an improvement. \"style\" sounds like it\naffects all pushes. This configuration variable says what push should\ndo when no refspec is given and none are configured, so the word\n\"default\" should be in there at least. Maybe \"defaultref\" would have\nbeen better?\n\nSome people may have started setting push.default already, so changing\nthe name may cause additional grief.\n\n- Finn Arne\n"},{"id":"109854","messageId":"alpine.DEB.1.00.0903301021540.7534@intel-tinevez-2-302","threadId":"18606","inReplyTo":"20090330080115.GA19142@pvv.org","subject":"Re: [PATCH 4/4] Rename push.default to push.style","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2009-03-30T08:23:48Z","receivedAt":"2009-03-30T08:23:48Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Mon, 30 Mar 2009, Finn Arne Gangstad wrote:\n\n> On Sun, Mar 29, 2009 at 12:10:04AM +0100, Santi Béjar wrote:\n> > push.default was too generic, and also didn't specify if it was about\n> > remote, refspec, branches, behaviour...\n> > [...]\n> >  \t\"You can specify what action you want to take in this case, and\",\n> > -\t\"avoid seeing this message again, by configuring 'push.default' to:\",\n> > +\t\"avoid seeing this message again, by configuring 'push.style' to:\",\n> >  \t\"  'nothing'  : Do not push anything\",\n> >  \t\"  'matching' : Push all matching branches (default)\",\n> >  \t\"  'tracking' : Push the current branch to whatever it is tracking\",\n> > [...]\n> \n> I don't think \"push.style\" is an improvement. \"style\" sounds like it \n> affects all pushes. This configuration variable says what push should do \n> when no refspec is given and none are configured, so the word \"default\" \n> should be in there at least. Maybe \"defaultref\" would have been better?\n\nI'd prefer \"push.defaultMode\", but...\n\n> Some people may have started setting push.default already, so changing \n> the name may cause additional grief.\n\n... indeed, it is too late now.  The milk has boiled already.\n\nCiao,\nDscho\n"},{"id":"109859","messageId":"adf1fd3d0903300200v65393b1bif0050392aa44652e@mail.gmail.com","threadId":"18606","inReplyTo":"alpine.DEB.1.00.0903301021540.7534@intel-tinevez-2-302","subject":"Re: [PATCH 4/4] Rename push.default to push.style","fromName":"Santi Béjar","fromEmail":"santi@agolina.net","sentAt":"2009-03-30T09:00:03Z","receivedAt":"2009-03-30T09:00:03Z","isPatch":true,"sender":{"key":"santi@agolina.net","avatar":null},"body":"2009/3/30 Johannes Schindelin <Johannes.Schindelin@gmx.de>:\n> Hi,\n>\n> On Mon, 30 Mar 2009, Finn Arne Gangstad wrote:\n>\n>> On Sun, Mar 29, 2009 at 12:10:04AM +0100, Santi Béjar wrote:\n>> > push.default was too generic, and also didn't specify if it was about\n>> > remote, refspec, branches, behaviour...\n>> > [...]\n>> >     \"You can specify what action you want to take in this case, and\",\n>> > -   \"avoid seeing this message again, by configuring 'push.default' to:\",\n>> > +   \"avoid seeing this message again, by configuring 'push.style' to:\",\n>> >     \"  'nothing'  : Do not push anything\",\n>> >     \"  'matching' : Push all matching branches (default)\",\n>> >     \"  'tracking' : Push the current branch to whatever it is tracking\",\n>> > [...]\n>>\n>> I don't think \"push.style\" is an improvement. \"style\" sounds like it\n>> affects all pushes.\n\nNot for me. I says the style of the push. But maybe push.mode could be better.\n\n>> This configuration variable says what push should do\n>> when no refspec is given and none are configured, so the word \"default\"\n>> should be in there at least. Maybe \"defaultref\" would have been better?\n\nI don't see the point of the word default, a lot of configuration is\nto set the default value. Git has branch.name.remote, not\nbranch.name.defaultremote, or user.email, not user.defaultemail,...\n\n>\n> I'd prefer \"push.defaultMode\", but...\n>\n>> Some people may have started setting push.default already, so changing\n>> the name may cause additional grief.\n>\n> ... indeed, it is too late now.  The milk has boiled already.\n\nI have it configured, but I think we can fix these type of things\nwhile they are in 'next'.\n\nSanti\n"},{"id":"109862","messageId":"adf1fd3d0903300207j479b6e91gc6ed00695a00e157@mail.gmail.com","threadId":"18606","inReplyTo":"adf1fd3d0903300200v65393b1bif0050392aa44652e@mail.gmail.com","subject":"Re: [PATCH 4/4] Rename push.default to push.style","fromName":"Santi Béjar","fromEmail":"santi@agolina.net","sentAt":"2009-03-30T09:07:10Z","receivedAt":"2009-03-30T09:07:10Z","isPatch":true,"sender":{"key":"santi@agolina.net","avatar":null},"body":"2009/3/30 Santi Béjar <santi@agolina.net>:\n> 2009/3/30 Johannes Schindelin <Johannes.Schindelin@gmx.de>:\n>> On Mon, 30 Mar 2009, Finn Arne Gangstad wrote:\n>>> Some people may have started setting push.default already, so changing\n>>> the name may cause additional grief.\n>>\n>> ... indeed, it is too late now.  The milk has boiled already.\n>\n> I have it configured, but I think we can fix these type of things\n> while they are in 'next'.\n>\n\nNow that I've recheck it, it seems to be in 'master', it's less clear,\nbut I still think we can fix these type of things while they are not\nin a relased version.\n\nThe people with that setting is following the development version, and\ncan easily notice/change it.\n\nSanti\n"},{"id":"109882","messageId":"20090330102739.GA5163@sigill.intra.peff.net","threadId":"18606","inReplyTo":"adf1fd3d0903300200v65393b1bif0050392aa44652e@mail.gmail.com","subject":"Re: [PATCH 4/4] Rename push.default to push.style","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2009-03-30T10:29:30Z","receivedAt":"2009-03-30T10:29:30Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Mon, Mar 30, 2009 at 11:00:03AM +0200, Santi Béjar wrote:\n\n> >> This configuration variable says what push should do\n> >> when no refspec is given and none are configured, so the word \"default\"\n> >> should be in there at least. Maybe \"defaultref\" would have been better?\n> \n> I don't see the point of the word default, a lot of configuration is\n> to set the default value. Git has branch.name.remote, not\n> branch.name.defaultremote, or user.email, not user.defaultemail,...\n\nThe usual case is two layers of options: command line and config\noptions. Thus \"git push <remote>\" overrides \"branch.*.remote\".\n\nBut in this case there are actually _three_ layers: command line,\nbranch.*.push, and now push.default. I think a name like \"push.mode\"\ndoesn't make clear the fact that it will never be looked at if you have\n\"branch.*.push\" set up.\n\nI think you have a point that \"default\" is vague, but \"defaultMode\"\nwould be better than simply \"mode\".\n\n-Peff\n"},{"id":"109896","messageId":"adf1fd3d0903300550h6150b8b2xb51da64f35305a78@mail.gmail.com","threadId":"18606","inReplyTo":"20090330102739.GA5163@sigill.intra.peff.net","subject":"Re: [PATCH 4/4] Rename push.default to push.style","fromName":"Santi Béjar","fromEmail":"santi@agolina.net","sentAt":"2009-03-30T12:50:26Z","receivedAt":"2009-03-30T12:50:26Z","isPatch":true,"sender":{"key":"santi@agolina.net","avatar":null},"body":"2009/3/30 Jeff King <peff@peff.net>:\n> On Mon, Mar 30, 2009 at 11:00:03AM +0200, Santi Béjar wrote:\n>\n>> >> This configuration variable says what push should do\n>> >> when no refspec is given and none are configured, so the word \"default\"\n>> >> should be in there at least. Maybe \"defaultref\" would have been better?\n>>\n>> I don't see the point of the word default, a lot of configuration is\n>> to set the default value. Git has branch.name.remote, not\n>> branch.name.defaultremote, or user.email, not user.defaultemail,...\n>\n> The usual case is two layers of options: command line and config\n> options. Thus \"git push <remote>\" overrides \"branch.*.remote\".\n>\n> But in this case there are actually _three_ layers: command line,\n> branch.*.push, and now push.default. I think a name like \"push.mode\"\n> doesn't make clear the fact that it will never be looked at if you have\n> \"branch.*.push\" set up.\n>\n> I think you have a point that \"default\" is vague, but \"defaultMode\"\n> would be better than simply \"mode\".\n\nThanks, it makes sense. Now I find defaultMode better than 'mode'.\n\nSanti\n"}]}