{"thread":{"id":"35054","subject":"[PATCH] push: Enhance unspecified push default warning","startedAt":"2013-10-02T20:48:55Z","lastAt":"2013-10-03T06:48:22Z","messageCount":3,"participants":["Coder Coder5000","Duy Nguyen","Matthieu Moy"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"228525","messageId":"CAKYC+eLGS6ocdE7CTV25E2xMRaHijmQbFBc3tAyx3cNpXfC_sg@mail.gmail.com","threadId":"35054","inReplyTo":null,"subject":"[PATCH] push: Enhance unspecified push default warning","fromName":"Coder Coder5000","fromEmail":"coder5000@gmail.com","sentAt":"2013-10-02T20:48:55Z","receivedAt":"2013-10-02T20:48:55Z","isPatch":true,"sender":{"key":"coder5000@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1053650?v=4"},"body":"When the unset push.default warning message is displayed\nthis may be the first time many users encounter push.default.\nModified the warning message to explain in a compact\nmanner what push.default is and why it is being changed in\nGit 2.0.  Also provided additional information to help users\ndecide if this change will affect their workflow.\n\nSigned-off-by: Greg Jacobson <coder5000@gmail.com>\n---\n builtin/push.c | 9 +++++++++\n po/git.pot     | 8 ++++++++\n 2 files changed, 17 insertions(+)\n\ndiff --git a/builtin/push.c b/builtin/push.c\nindex 7b1b66c..eb7a721 100644\n--- a/builtin/push.c\n+++ b/builtin/push.c\n@@ -174,6 +174,15 @@ N_(\"push.default is unset; its implicit value is\nchanging in\\n\"\n    \"\\n\"\n    \"  git config --global push.default simple\\n\"\n    \"\\n\"\n+   \"When push.default is set to matching git will push all local branches\\n\"\n+   \"to the remote branches with the same (matching) name.  This will no\\n\"\n+   \"longer be the default in Git 2.0 because a branch could be\\n\"\n+   \"unintentionally pushed to a remote.\\n\"\n+   \"\\n\"\n+   \"In Git 2.0 the new push.default of simple will push only the current\\n\"\n+   \"branch to the same remote branch used by git pull.   A push will\\n\"\n+   \"only succeed if the remote and local branches have the same name.\\n\"\n+   \"\\n\"\n    \"See 'git help config' and search for 'push.default' for further\ninformation.\\n\"\n    \"(the 'simple' mode was introduced in Git 1.7.11. Use the similar mode\\n\"\n    \"'current' instead of 'simple' if you sometimes use older versions\nof Git)\");\ndiff --git a/po/git.pot b/po/git.pot\nindex c91e197..6025a54 100644\n--- a/po/git.pot\n+++ b/po/git.pot\n@@ -7240,6 +7240,14 @@ msgid \"\"\n \"\\n\"\n \"  git config --global push.default simple\\n\"\n \"\\n\"\n+\"When push.default is set to matching git will push all local branches\\n\"\n+\"to the remote branches with the same (matching) name.  This will no\\n\"\n+\"longer be the default in Git 2.0 because a branch could be\\n\"\n+\"unintentionally pushed to a remote.\\n\"\n+\"\\n\"\n+\"In Git 2.0 the new push.default of simple will push only the current\\n\"\n+\"branch to the same remote branch used by git pull.   A push will\\n\"\n+\"only succeed if the remote and local branches have the same name.\\n\"\n \"See 'git help config' and search for 'push.default' for further \"\n \"information.\\n\"\n \"(the 'simple' mode was introduced in Git 1.7.11. Use the similar mode\\n\"\n-- \n1.8.4.474.g128a96c.dirty\n"},{"id":"228527","messageId":"CACsJy8Bt_mtXVHjWn+8Ba6qrk=vSh2kGyaowA9B+=2xvhA_o_g@mail.gmail.com","threadId":"35054","inReplyTo":"CAKYC+eLGS6ocdE7CTV25E2xMRaHijmQbFBc3tAyx3cNpXfC_sg@mail.gmail.com","subject":"Re: [PATCH] push: Enhance unspecified push default warning","fromName":"Duy Nguyen","fromEmail":"pclouds@gmail.com","sentAt":"2013-10-03T00:57:41Z","receivedAt":"2013-10-03T00:57:41Z","isPatch":true,"sender":{"key":"pclouds@gmail.com","avatar":"https://avatars.githubusercontent.com/u/720?v=4"},"body":"On Thu, Oct 3, 2013 at 3:48 AM, Coder Coder5000 <coder5000@gmail.com> wrote:\n> When the unset push.default warning message is displayed\n> this may be the first time many users encounter push.default.\n> Modified the warning message to explain in a compact\n> manner what push.default is and why it is being changed in\n> Git 2.0.  Also provided additional information to help users\n> decide if this change will affect their workflow.\n>\n> Signed-off-by: Greg Jacobson <coder5000@gmail.com>\n> ---\n>  builtin/push.c | 9 +++++++++\n>  po/git.pot     | 8 ++++++++\n>  2 files changed, 17 insertions(+)\n\nYou don't have to update git.pot. It'll be regenerated regularly by Jiang Xin.\n-- \nDuy\n"},{"id":"228532","messageId":"vpqr4c2kfyh.fsf@anie.imag.fr","threadId":"35054","inReplyTo":"CAKYC+eLGS6ocdE7CTV25E2xMRaHijmQbFBc3tAyx3cNpXfC_sg@mail.gmail.com","subject":"Re: [PATCH] push: Enhance unspecified push default warning","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@grenoble-inp.fr","sentAt":"2013-10-03T06:48:22Z","receivedAt":"2013-10-03T06:48:22Z","isPatch":true,"sender":{"key":"matthieu.moy@grenoble-inp.fr","avatar":"https://gravatar.com/avatar/72c8a2705971a25dfaff23cece15130d405685845d911aedd5667ace277f3fc5?d=mp&s=160"},"body":"Coder Coder5000 <coder5000@gmail.com> writes:\n^^^^^^^^^^^^^^^\nPlease, use your real identity in your From: field (git send-email may\nhelp).\n\n> --- a/builtin/push.c\n> +++ b/builtin/push.c\n> @@ -174,6 +174,15 @@ N_(\"push.default is unset; its implicit value is\n> changing in\\n\"\n>     \"\\n\"\n>     \"  git config --global push.default simple\\n\"\n>     \"\\n\"\n> +   \"When push.default is set to matching git will push all local branches\\n\"\n> +   \"to the remote branches with the same (matching) name.  This will no\\n\"\n> +   \"longer be the default in Git 2.0 because a branch could be\\n\"\n> +   \"unintentionally pushed to a remote.\\n\"\n> +   \"\\n\"\n> +   \"In Git 2.0 the new push.default of simple will push only the current\\n\"\n> +   \"branch to the same remote branch used by git pull.   A push will\\n\"\n> +   \"only succeed if the remote and local branches have the same name.\\n\"\n> +   \"\\n\"\n\nIt is becoming a monster-warning, but I think this is a good change.\nPeople who still see the warning are those who did not upgrade for a\nwhile (and therefore probably the ones needing the most explanation).\nAnd it's meant to be a one-time warning anyway.\n\nJudging by the question asked on stackoverflow\n( http://stackoverflow.com/questions/13148066/warning-push-default-is-unset-its-implicit-value-is-changing-in-git-2-0 )\nand its popularity, telling the users to read the docs did not work very\nwell.\n\nThanks,\n\n-- \nMatthieu Moy\nhttp://www-verimag.imag.fr/~moy/\n"}]}