{"thread":{"id":"31448","subject":"[PATCH/RFC] l10n: de.po: translate 2 new messages","startedAt":"2012-09-05T17:11:40Z","lastAt":"2012-09-07T21:20:38Z","messageCount":18,"participants":["Ralf Thielow","Thomas Rast","Jiang Xin","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"198392","messageId":"1346865100-10908-1-git-send-email-ralf.thielow@gmail.com","threadId":"31448","inReplyTo":null,"subject":"[PATCH/RFC] l10n: de.po: translate 2 new messages","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2012-09-05T17:11:40Z","receivedAt":"2012-09-05T17:11:40Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"Translate 2 new messages came from git.pot update in\nccfca8d (l10n: Update git.pot (2 new, 4 removed messages)).\n\nSigned-off-by: Ralf Thielow <ralf.thielow@gmail.com>\n---\n\nHi German l10n team,\n\nplease review this small update on German translation.\nThis patch here is based on a preparation commit which\njust contains the \"msgmerge\" change.\nFor an easier review, I'll copy the first message in english\nin this email.\nThe second change shows that there are some newlines, came\nfrom 'msgmerge', on places where we don't want to have them.\nI'll fix that soon.\n\nA hint on this translation:\nI've translated \"See 'git help config' and search for\" as\n\"Führe 'git help config' aus und suche nach...\", not as\n\"Siehe 'git help config' und suche nach...\" because the second\ndoesn't sound good to me. Perhaps someone else has a better\nidea.\n\nThanks,\nRalf\n\n#: builtin/push.c:151\nmsgid \"\"\n\"push.default is unset; its implicit value is changing in\\n\"\n\"Git 2.0 from 'matching' to 'simple'. To squelch this message\\n\"\n\"and maintain the current behavior after the default changes, use:\\n\"\n\"\\n\"\n\"  git config --global push.default matching\\n\"\n\"\\n\"\n\"To squelch this message and adopt the new behavior now, use:\\n\"\n\"\\n\"\n\"  git config --global push.default simple\\n\"\n\"\\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\"'current' instead of 'simple' if you sometimes use older versions of Git)\"\n\n po/de.po | 20 ++++++++++++++++++--\n 1 Datei geändert, 18 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-)\n\ndiff --git a/po/de.po b/po/de.po\nindex f3d232e..df5187e 100644\n--- a/po/de.po\n+++ b/po/de.po\n@@ -4431,6 +4431,23 @@ msgid \"\"\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 of Git)\"\n msgstr \"\"\n+\"'push.default' ist nicht gesetzt; der implizit gesetzte Wert wird in\\n\"\n+\"Git 2.0 von 'matching' nach 'simple' geändert. Um diese Meldung zu unter-\\n\"\n+\"drücken und das aktuelle Verhalten nach Änderung des Standardwertes\\n\"\n+\"beizubehalten, benutze:\"\n+\"\\n\"\n+\"  git config --global push.default matching\\n\"\n+\"\\n\"\n+\"Um diese Meldung zu unterdrücken und das neue Verhalten jetzt zu übernehmen,\\n\"\n+\"benutze:\\n\"\n+\"\\n\"\n+\"  git config --global push.default simple\\n\"\n+\"\\n\"\n+\"Führe 'git help config' aus und suche nach 'push.default' für weitere \"\n+\"Informationen.\\n\"\n+\"(Der Modus 'simple' wurde in Git 1.7.11 eingeführt. Benutze den gleichartigen\"\n+\" Modus 'current' anstatt 'simple', falls du gelegentlich ältere Versionen von\"\n+\" Git benutzt)\"\n \n #: builtin/push.c:199\n msgid \"\"\n@@ -4453,7 +4470,6 @@ msgstr \"\"\n \"für weitere Details.\"\n \n #: builtin/push.c:212\n-#, fuzzy\n msgid \"\"\n \"Updates were rejected because a pushed branch tip is behind its remote\\n\"\n \"counterpart. If you did not intend to push that branch, you may want to\\n\"\n@@ -4465,7 +4481,7 @@ msgstr \"\"\n \"beabsichtigt hast, diesen Zweig zu versenden, kannst du auch den zu \"\n \"versendenden\\n\"\n \"Zweig spezifizieren oder die Konfigurationsvariable 'push.default' zu \"\n-\"'current'\\n\"\n+\"'simple', 'current'\\n\"\n \"oder 'upstream' setzen, um nur den aktuellen Zweig zu versenden.\"\n \n #: builtin/push.c:218\n-- \n1.7.12\n"},{"id":"198393","messageId":"CAN0XMOK7j-cA9jUwF5-1UoZ1HAmDJ6zO9_m2-R9E--igVjrcZg@mail.gmail.com","threadId":"31448","inReplyTo":"1346865100-10908-1-git-send-email-ralf.thielow@gmail.com","subject":"Re: [PATCH/RFC] l10n: de.po: translate 2 new messages","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2012-09-05T17:21:20Z","receivedAt":"2012-09-05T17:21:20Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"On Wed, Sep 5, 2012 at 7:11 PM, Ralf Thielow <ralf.thielow@gmail.com> wrote:\n> +\" Modus 'current' anstatt 'simple', falls du gelegentlich ältere Versionen von\"\n> +\" Git benutzt)\"\n\nOops, i've forgot a full stop before the closing bracket. I'll fix.\n"},{"id":"198460","messageId":"87sjavcd8x.fsf@thomas.inf.ethz.ch","threadId":"31448","inReplyTo":"1346865100-10908-1-git-send-email-ralf.thielow@gmail.com","subject":"Re: [PATCH/RFC] l10n: de.po: translate 2 new messages","fromName":"Thomas Rast","fromEmail":"trast@inf.ethz.ch","sentAt":"2012-09-06T16:44:30Z","receivedAt":"2012-09-06T16:44:30Z","isPatch":true,"sender":{"key":"tr@thomasrast.ch","avatar":"https://avatars.githubusercontent.com/u/153510?v=4"},"body":"Ralf Thielow <ralf.thielow@gmail.com> writes:\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 of Git)\"\n[...]\n> +\"(Der Modus 'simple' wurde in Git 1.7.11 eingeführt. Benutze den gleichartigen\"\n> +\" Modus 'current' anstatt 'simple', falls du gelegentlich ältere Versionen von\"\n> +\" Git benutzt)\"\n\nYou translate \"similar\" as \"gleichartig\", which sounds too much like\n\"the same\" to me.  Perhaps \"ähnlich\"?  Or is there no danger of\nconfusing this in de_DE usage?\n\nAck on the rest.\n\n-- \nThomas Rast\ntrast@{inf,student}.ethz.ch\n"},{"id":"198461","messageId":"CAN0XMO+y4DU5sTWv5dZwXBfER5hS6-U65r73jvQd9TC32YrDUA@mail.gmail.com","threadId":"31448","inReplyTo":"87sjavcd8x.fsf@thomas.inf.ethz.ch","subject":"Re: [PATCH/RFC] l10n: de.po: translate 2 new messages","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2012-09-06T17:15:24Z","receivedAt":"2012-09-06T17:15:24Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"On Thu, Sep 6, 2012 at 6:44 PM, Thomas Rast <trast@inf.ethz.ch> wrote:\n> Ralf Thielow <ralf.thielow@gmail.com> writes:\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 of Git)\"\n> [...]\n>> +\"(Der Modus 'simple' wurde in Git 1.7.11 eingeführt. Benutze den gleichartigen\"\n>> +\" Modus 'current' anstatt 'simple', falls du gelegentlich ältere Versionen von\"\n>> +\" Git benutzt)\"\n>\n> You translate \"similar\" as \"gleichartig\", which sounds too much like\n> \"the same\" to me.  Perhaps \"ähnlich\"?  Or is there no danger of\n> confusing this in de_DE usage?\n>\n> Ack on the rest.\n>\n\nYeah, \"ähnlich\" is probably the better translation in this case. I'll change it.\nThanks\n"},{"id":"198514","messageId":"CANYiYbFBoK7D+9oZ2YGsrHYafOETLRUWUmHzfnAyA7RRhqgEkA@mail.gmail.com","threadId":"31448","inReplyTo":"1346865100-10908-1-git-send-email-ralf.thielow@gmail.com","subject":"Re: [PATCH/RFC] l10n: de.po: translate 2 new messages","fromName":"Jiang Xin","fromEmail":"worldhello.net@gmail.com","sentAt":"2012-09-07T10:31:46Z","receivedAt":"2012-09-07T10:31:46Z","isPatch":true,"sender":{"key":"worldhello.net@gmail.com","avatar":"https://avatars.githubusercontent.com/u/183860?v=4"},"body":"I just notice that the 1st line of the orignal message below has\nonly 56  characters, much shorter than other lines. It is because\nthis is a warning message, and would add a prefix: \"warning: \".\n\n#: builtin/push.c:151\nmsgid \"\"\n\"push.default is unset; its implicit value is changing in\\n\"\n\"Git 2.0 from 'matching' to 'simple'. To squelch this message\\n\"\n\"and maintain the current behavior after the default changes, use:\\n\"\n\"\\n\"\n\"  git config --global push.default matching\\n\"\n\nFor this reason, translations as follows are not well-formed.\nSo I rewind git-po, and your commits may need a bit amend.\n\nSwedish update from Peter:\n\n+\"push.default har inte ställts in; dess underförstådda värde ändras i\\n\"\n+\"Git 2.0 från \\\"matching\\\" till \\\"simple\\\". För att undertrycka det här\\n\"\n+\"meddelandet och behålla nuvarande beteende efter att förvalet ändras,\\n\"\n+\"skriver du:\\n\"\n+\"\\n\"\n+\"  git config --global push.default matching\\n\"\n\nVietnamese update form Trần Ngọc Quân:\n\n+msgstr \"\"\n+\"push.default chưa được đặt, giá trị ngầm định của nó đã được thay đổi trong\\n\"\n+\"Git 2.0 từ 'matching' thành 'simple'. Để chấm dứt lời nhắc nhở này\\n\"\n+\"và duy trì cách xử lý sau những thay đổi mặc định này, hãy chạy lệnh:\\n\"\n+\"\\n\"\n+\"  git config --global push.default matching\\n\"\n\n\n\n2012/9/6 Ralf Thielow <ralf.thielow@gmail.com>:\n> Translate 2 new messages came from git.pot update in\n> ccfca8d (l10n: Update git.pot (2 new, 4 removed messages)).\n>\n> Signed-off-by: Ralf Thielow <ralf.thielow@gmail.com>\n> ---\n>\n> Hi German l10n team,\n>\n> please review this small update on German translation.\n> This patch here is based on a preparation commit which\n> just contains the \"msgmerge\" change.\n> For an easier review, I'll copy the first message in english\n> in this email.\n> The second change shows that there are some newlines, came\n> from 'msgmerge', on places where we don't want to have them.\n> I'll fix that soon.\n>\n> A hint on this translation:\n> I've translated \"See 'git help config' and search for\" as\n> \"Führe 'git help config' aus und suche nach...\", not as\n> \"Siehe 'git help config' und suche nach...\" because the second\n> doesn't sound good to me. Perhaps someone else has a better\n> idea.\n>\n> Thanks,\n> Ralf\n>\n> #: builtin/push.c:151\n> msgid \"\"\n> \"push.default is unset; its implicit value is changing in\\n\"\n> \"Git 2.0 from 'matching' to 'simple'. To squelch this message\\n\"\n> \"and maintain the current behavior after the default changes, use:\\n\"\n> \"\\n\"\n> \"  git config --global push.default matching\\n\"\n> \"\\n\"\n> \"To squelch this message and adopt the new behavior now, use:\\n\"\n> \"\\n\"\n> \"  git config --global push.default simple\\n\"\n> \"\\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> \"'current' instead of 'simple' if you sometimes use older versions of Git)\"\n>\n>  po/de.po | 20 ++++++++++++++++++--\n>  1 Datei geändert, 18 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-)\n>\n> diff --git a/po/de.po b/po/de.po\n> index f3d232e..df5187e 100644\n> --- a/po/de.po\n> +++ b/po/de.po\n> @@ -4431,6 +4431,23 @@ msgid \"\"\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 of Git)\"\n>  msgstr \"\"\n> +\"'push.default' ist nicht gesetzt; der implizit gesetzte Wert wird in\\n\"\n> +\"Git 2.0 von 'matching' nach 'simple' geändert. Um diese Meldung zu unter-\\n\"\n> +\"drücken und das aktuelle Verhalten nach Änderung des Standardwertes\\n\"\n> +\"beizubehalten, benutze:\"\n> +\"\\n\"\n> +\"  git config --global push.default matching\\n\"\n> +\"\\n\"\n> +\"Um diese Meldung zu unterdrücken und das neue Verhalten jetzt zu übernehmen,\\n\"\n> +\"benutze:\\n\"\n> +\"\\n\"\n> +\"  git config --global push.default simple\\n\"\n> +\"\\n\"\n> +\"Führe 'git help config' aus und suche nach 'push.default' für weitere \"\n> +\"Informationen.\\n\"\n> +\"(Der Modus 'simple' wurde in Git 1.7.11 eingeführt. Benutze den gleichartigen\"\n> +\" Modus 'current' anstatt 'simple', falls du gelegentlich ältere Versionen von\"\n> +\" Git benutzt)\"\n>\n>  #: builtin/push.c:199\n>  msgid \"\"\n> @@ -4453,7 +4470,6 @@ msgstr \"\"\n>  \"für weitere Details.\"\n>\n>  #: builtin/push.c:212\n> -#, fuzzy\n>  msgid \"\"\n>  \"Updates were rejected because a pushed branch tip is behind its remote\\n\"\n>  \"counterpart. If you did not intend to push that branch, you may want to\\n\"\n> @@ -4465,7 +4481,7 @@ msgstr \"\"\n>  \"beabsichtigt hast, diesen Zweig zu versenden, kannst du auch den zu \"\n>  \"versendenden\\n\"\n>  \"Zweig spezifizieren oder die Konfigurationsvariable 'push.default' zu \"\n> -\"'current'\\n\"\n> +\"'simple', 'current'\\n\"\n>  \"oder 'upstream' setzen, um nur den aktuellen Zweig zu versenden.\"\n>\n>  #: builtin/push.c:218\n> --\n> 1.7.12\n>\n\n\n\n-- \n蒋鑫\n\n北京群英汇信息技术有限公司\n邮件: worldhello.net@gmail.com\n网址: http://www.ossxp.com/\n博客: http://www.worldhello.net/\n微博: http://weibo.com/gotgit/\n电话: 010-51262007, 18601196889\n"},{"id":"198528","messageId":"CAN0XMOL+2RV3wxp0nCAJzBBVF2vRNsK6H=MFYEqvKTw+nkcuXw@mail.gmail.com","threadId":"31448","inReplyTo":"CANYiYbFBoK7D+9oZ2YGsrHYafOETLRUWUmHzfnAyA7RRhqgEkA@mail.gmail.com","subject":"Re: [PATCH/RFC] l10n: de.po: translate 2 new messages","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2012-09-07T16:36:55Z","receivedAt":"2012-09-07T16:36:55Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"On Fri, Sep 7, 2012 at 12:31 PM, Jiang Xin <worldhello.net@gmail.com> wrote:\n> I just notice that the 1st line of the orignal message below has\n> only 56  characters, much shorter than other lines. It is because\n> this is a warning message, and would add a prefix: \"warning: \".\n>\n> #: builtin/push.c:151\n> msgid \"\"\n> \"push.default is unset; its implicit value is changing in\\n\"\n> \"Git 2.0 from 'matching' to 'simple'. To squelch this message\\n\"\n> \"and maintain the current behavior after the default changes, use:\\n\"\n> \"\\n\"\n> \"  git config --global push.default matching\\n\"\n>\n> For this reason, translations as follows are not well-formed.\n> So I rewind git-po, and your commits may need a bit amend.\n>\n\nI admit that I don't really understand what you mean. I can't\nsee a big difference in line lengths. It's not more than others\nhave.\nDo you mean that the first line just lacks of a \"warning: \"\npart? In this case, the message needs to get fixed.\nImagine that this line starts with \"warning: \", the updated\nGerman translation would looks like this:\n\nmsgstr \"\"\n\"Warnung: 'push.default' ist nicht gesetzt; der implizit gesetzte Wert wird\\n\"\n\"in Git 2.0 von 'matching' nach 'simple' geändert. Um diese Meldung zu\\n\"\n\"unterdrücken und das aktuelle Verhalten nach Änderung des Standardwertes\\n\"\n\"beizubehalten, benutze:\"\n[...]\n\nI'll send you another pull request contains this update.\n"},{"id":"198531","messageId":"CAN0XMOL9cxy8noicF-P-cjLkw76Zf3Uyu4C2RqbcTYrug5+w8Q@mail.gmail.com","threadId":"31448","inReplyTo":"CAN0XMOL+2RV3wxp0nCAJzBBVF2vRNsK6H=MFYEqvKTw+nkcuXw@mail.gmail.com","subject":"Re: [PATCH/RFC] l10n: de.po: translate 2 new messages","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2012-09-07T17:05:12Z","receivedAt":"2012-09-07T17:05:12Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"On Fri, Sep 7, 2012 at 6:36 PM, Ralf Thielow <ralf.thielow@gmail.com> wrote:\n> On Fri, Sep 7, 2012 at 12:31 PM, Jiang Xin <worldhello.net@gmail.com> wrote:\n>> I just notice that the 1st line of the orignal message below has\n>> only 56  characters, much shorter than other lines. It is because\n>> this is a warning message, and would add a prefix: \"warning: \".\n>>\n>> #: builtin/push.c:151\n>> msgid \"\"\n>> \"push.default is unset; its implicit value is changing in\\n\"\n>> \"Git 2.0 from 'matching' to 'simple'. To squelch this message\\n\"\n>> \"and maintain the current behavior after the default changes, use:\\n\"\n>> \"\\n\"\n>> \"  git config --global push.default matching\\n\"\n>>\n>> For this reason, translations as follows are not well-formed.\n>> So I rewind git-po, and your commits may need a bit amend.\n>>\n>\n> I admit that I don't really understand what you mean. I can't\n> see a big difference in line lengths. It's not more than others\n\nI got it. This \"warning: \" is added automatically.\nSorry\n"},{"id":"198533","messageId":"CAN0XMOLUpt+fdT-=wsTg_+m_VGaKpvHuC8H-h7rBozN+iFfVMQ@mail.gmail.com","threadId":"31448","inReplyTo":"CAN0XMOL9cxy8noicF-P-cjLkw76Zf3Uyu4C2RqbcTYrug5+w8Q@mail.gmail.com","subject":"Re: [PATCH/RFC] l10n: de.po: translate 2 new messages","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2012-09-07T17:12:30Z","receivedAt":"2012-09-07T17:12:30Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"On Fri, Sep 7, 2012 at 7:05 PM, Ralf Thielow <ralf.thielow@gmail.com> wrote:\n> On Fri, Sep 7, 2012 at 6:36 PM, Ralf Thielow <ralf.thielow@gmail.com> wrote:\n>> On Fri, Sep 7, 2012 at 12:31 PM, Jiang Xin <worldhello.net@gmail.com> wrote:\n>>> I just notice that the 1st line of the orignal message below has\n>>> only 56  characters, much shorter than other lines. It is because\n>>> this is a warning message, and would add a prefix: \"warning: \".\n>>>\n>>> #: builtin/push.c:151\n>>> msgid \"\"\n>>> \"push.default is unset; its implicit value is changing in\\n\"\n>>> \"Git 2.0 from 'matching' to 'simple'. To squelch this message\\n\"\n>>> \"and maintain the current behavior after the default changes, use:\\n\"\n>>> \"\\n\"\n>>> \"  git config --global push.default matching\\n\"\n>>>\n>>> For this reason, translations as follows are not well-formed.\n>>> So I rewind git-po, and your commits may need a bit amend.\n>>>\n>>\n>> I admit that I don't really understand what you mean. I can't\n>> see a big difference in line lengths. It's not more than others\n>\n> I got it. This \"warning: \" is added automatically.\n> Sorry\n\nThe update would looks like this:\n\nmsgstr \"\"\n\"'push.default' ist nicht gesetzt; der implizit gesetzte Wert\\n\"\n\"wird in Git 2.0 von 'matching' nach 'simple' geändert. Um diese Meldung zu\\n\"\n\"unterdrücken und das aktuelle Verhalten nach Änderung des Standardwertes\\n\"\n...\n"},{"id":"198537","messageId":"7vhar9ybgk.fsf@alter.siamese.dyndns.org","threadId":"31448","inReplyTo":"CAN0XMOL9cxy8noicF-P-cjLkw76Zf3Uyu4C2RqbcTYrug5+w8Q@mail.gmail.com","subject":"Re: [PATCH/RFC] l10n: de.po: translate 2 new messages","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-09-07T17:44:27Z","receivedAt":"2012-09-07T17:44:27Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ralf Thielow <ralf.thielow@gmail.com> writes:\n\n> On Fri, Sep 7, 2012 at 6:36 PM, Ralf Thielow <ralf.thielow@gmail.com> wrote:\n>> On Fri, Sep 7, 2012 at 12:31 PM, Jiang Xin <worldhello.net@gmail.com> wrote:\n>>> I just notice that the 1st line of the orignal message below has\n>>> only 56  characters, much shorter than other lines. It is because\n>>> this is a warning message, and would add a prefix: \"warning: \".\n>>>\n>>> #: builtin/push.c:151\n>>> msgid \"\"\n>>> \"push.default is unset; its implicit value is changing in\\n\"\n>>> \"Git 2.0 from 'matching' to 'simple'. To squelch this message\\n\"\n>>> \"and maintain the current behavior after the default changes, use:\\n\"\n>>> \"\\n\"\n>>> \"  git config --global push.default matching\\n\"\n>>>\n>>> For this reason, translations as follows are not well-formed.\n>>> So I rewind git-po, and your commits may need a bit amend.\n>>>\n>>\n>> I admit that I don't really understand what you mean. I can't\n>> see a big difference in line lengths. It's not more than others\n>\n> I got it. This \"warning: \" is added automatically.\n> Sorry\n\nWe may want to revisit this behaviour, by the way.\n\nIf you have a long string that barely happens to fit on a line with\n\"warning: \" prefix in the original, your translation might want to\nspell it in two lines, i.e.\n\n\twarning(_(\"A quick brown fox jumps over the lazy dog.\"));\n\nwhich gives\n\n\twarning: A quick brown fox jumps over the lazy dog.<LF>\n\nmight want to turn into\n\n\twarning(\"AA qquuiicckk bbrroowwnn ffooxx\\n\"\n        \t\"jjuummppss oovveerr tthhee llaazzyy ddoogg.\");\n\nand currently gives:\n\n\twwaarrnniinngg: AA qquuiicckk bbrroowwnn ffooxx<LF>\n        jjuummppss oovveerr tthhee llaazzyy ddoogg.<LF>\n\nBut we may want to give\n\n\twwaarrnniinngg: AA qquuiicckk bbrroowwnn ffooxx<LF>\n        wwaarrnniinngg: jjuummppss oovveerr tthhee llaazzyy ddoogg.<LF>\n\ninstead.\n\nCheck out the way advice.c::advise() does its \"hints: \" prefix for\nhints, but we probably would want to avoid anything that allocates\nmore memory in the die() codepath.\n"},{"id":"198543","messageId":"1347044705-17268-1-git-send-email-ralf.thielow@gmail.com","threadId":"31448","inReplyTo":"7vhar9ybgk.fsf@alter.siamese.dyndns.org","subject":"[PATCH 0/2] multiline prefixes on messages","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2012-09-07T19:05:03Z","receivedAt":"2012-09-07T19:05:03Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"This is a short implementation of an idea i have, based\non Junio's comment. It might not follows the coding style,\nthe comments can be adverse and so on...\nJust want to know if it's the right direction.\n\nThis series introduces a function that allows to add\na prefix to multilined messages. The function itself\nis an extraction of the code of \"advice.c::advice()\".\nExtract this as a separate function has the advantage\nthat we can mark these prefixes for translation and\ncan produce a nicer output.\n\nJunio said that we should probably avoid memory\nallocation on \"die()\" codepaths, but \"warning\"\nmessages are acceptable, I think. If the series is\nbasically acceptable, the \"usage:\" messages are also a\ncancidate for migration to this.\n\nRalf Thielow (2):\n  advice: extract function to print messages with prefix\n  i18n: mark prefix \"warning:\" for translation\n\n advice.c | 14 ++++++++++----\n advice.h |  1 +\n usage.c  |  2 +-\n 3 Dateien geändert, 12 Zeilen hinzugefügt(+), 5 Zeilen entfernt(-)\n\n-- \n1.7.12.176.g3fc0e4c.dirty\n"},{"id":"198545","messageId":"1347044705-17268-2-git-send-email-ralf.thielow@gmail.com","threadId":"31448","inReplyTo":"1347044705-17268-1-git-send-email-ralf.thielow@gmail.com","subject":"[PATCH 1/2] advice: extract function to print messages with prefix","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2012-09-07T19:05:04Z","receivedAt":"2012-09-07T19:05:04Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"Extract a function that allows to print messages\nwith a prefix.\n\nSigned-off-by: Ralf Thielow <ralf.thielow@gmail.com>\n---\n advice.c | 14 ++++++++++----\n advice.h |  1 +\n 2 Dateien geändert, 11 Zeilen hinzugefügt(+), 4 Zeilen entfernt(-)\n\ndiff --git a/advice.c b/advice.c\nindex edfbd4a..e73d53b 100644\n--- a/advice.c\n+++ b/advice.c\n@@ -25,25 +25,31 @@ static struct {\n \t{ \"detachedhead\", &advice_detached_head },\n };\n \n-void advise(const char *advice, ...)\n+void print_with_prefix(const char *prefix, const char *msg, ...)\n {\n \tstruct strbuf buf = STRBUF_INIT;\n \tva_list params;\n \tconst char *cp, *np;\n \n-\tva_start(params, advice);\n-\tstrbuf_vaddf(&buf, advice, params);\n+\tva_start(params, msg);\n+\tstrbuf_vaddf(&buf, msg, params);\n \tva_end(params);\n \n \tfor (cp = buf.buf; *cp; cp = np) {\n \t\tnp = strchrnul(cp, '\\n');\n-\t\tfprintf(stderr,\t_(\"hint: %.*s\\n\"), (int)(np - cp), cp);\n+\t\tfprintf(stderr,\t \"%s %.*s\\n\", prefix, (int)(np - cp), cp);\n \t\tif (*np)\n \t\t\tnp++;\n \t}\n \tstrbuf_release(&buf);\n }\n \n+void advise(const char *advice, ...)\n+{\n+\tva_list vl;\n+\tprint_with_prefix(_(\"hint:\"), advice, vl);\n+}\n+\n int git_default_advice_config(const char *var, const char *value)\n {\n \tconst char *k = skip_prefix(var, \"advice.\");\ndiff --git a/advice.h b/advice.h\nindex f3cdbbf..328e255 100644\n--- a/advice.h\n+++ b/advice.h\n@@ -14,6 +14,7 @@ extern int advice_implicit_identity;\n extern int advice_detached_head;\n \n int git_default_advice_config(const char *var, const char *value);\n+void print_with_prefix(const char *prefix, const char *msg, ...);\n void advise(const char *advice, ...);\n int error_resolve_conflict(const char *me);\n extern void NORETURN die_resolve_conflict(const char *me);\n-- \n1.7.12.176.g3fc0e4c.dirty\n"},{"id":"198544","messageId":"1347044705-17268-3-git-send-email-ralf.thielow@gmail.com","threadId":"31448","inReplyTo":"1347044705-17268-1-git-send-email-ralf.thielow@gmail.com","subject":"[PATCH 2/2] i18n: mark prefix \"warning:\" for translation","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2012-09-07T19:05:05Z","receivedAt":"2012-09-07T19:05:05Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"Signed-off-by: Ralf Thielow <ralf.thielow@gmail.com>\n---\n usage.c | 2 +-\n 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-)\n\ndiff --git a/usage.c b/usage.c\nindex a2a6678..162a4b2 100644\n--- a/usage.c\n+++ b/usage.c\n@@ -44,7 +44,7 @@ static void error_builtin(const char *err, va_list params)\n \n static void warn_builtin(const char *warn, va_list params)\n {\n-\tvreportf(\"warning: \", warn, params);\n+\tprint_with_prefix(_(\"warning:\"), warn, params);\n }\n \n /* If we are in a dlopen()ed .so write to a global variable would segfault\n-- \n1.7.12.176.g3fc0e4c.dirty\n"},{"id":"198547","messageId":"7v1uidy6ga.fsf@alter.siamese.dyndns.org","threadId":"31448","inReplyTo":"1347044705-17268-2-git-send-email-ralf.thielow@gmail.com","subject":"Re: [PATCH 1/2] advice: extract function to print messages with prefix","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-09-07T19:32:37Z","receivedAt":"2012-09-07T19:32:37Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ralf Thielow <ralf.thielow@gmail.com> writes:\n\n> Extract a function that allows to print messages\n> with a prefix.\n>\n> Signed-off-by: Ralf Thielow <ralf.thielow@gmail.com>\n> ---\n>  advice.c | 14 ++++++++++----\n>  advice.h |  1 +\n>  2 Dateien geändert, 11 Zeilen hinzugefügt(+), 4 Zeilen entfernt(-)\n>\n> diff --git a/advice.c b/advice.c\n> index edfbd4a..e73d53b 100644\n> --- a/advice.c\n> +++ b/advice.c\n> @@ -25,25 +25,31 @@ static struct {\n>  \t{ \"detachedhead\", &advice_detached_head },\n>  };\n>  \n> -void advise(const char *advice, ...)\n> +void print_with_prefix(const char *prefix, const char *msg, ...)\n>  {\n>  \tstruct strbuf buf = STRBUF_INIT;\n>  \tva_list params;\n>  \tconst char *cp, *np;\n>  \n> -\tva_start(params, advice);\n> -\tstrbuf_vaddf(&buf, advice, params);\n> +\tva_start(params, msg);\n> +\tstrbuf_vaddf(&buf, msg, params);\n>  \tva_end(params);\n>  \n>  \tfor (cp = buf.buf; *cp; cp = np) {\n>  \t\tnp = strchrnul(cp, '\\n');\n> -\t\tfprintf(stderr,\t_(\"hint: %.*s\\n\"), (int)(np - cp), cp);\n> +\t\tfprintf(stderr,\t \"%s %.*s\\n\", prefix, (int)(np - cp), cp);\n\nHrm, naively, printf(\"%s: %.*s\\n\", _(\"hint\"), ...) might look more\nnatural, but I vaguely recall that the current code places _()\naround the entire \"hint: %.*s\\n\" on purpose.  IIRC, it was to allow\ntranslations that flow from RTL e.g. \".siht od t'nod :tnih\".\n\nDoesn't this patch break it?\n"},{"id":"198548","messageId":"CAN0XMOK5QnNn8471RF4y7u_X0rfda=-J-KrCtyMsgCGHXNrwVw@mail.gmail.com","threadId":"31448","inReplyTo":"7v1uidy6ga.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 1/2] advice: extract function to print messages with prefix","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2012-09-07T19:56:14Z","receivedAt":"2012-09-07T19:56:14Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"On Fri, Sep 7, 2012 at 9:32 PM, Junio C Hamano <gitster@pobox.com> wrote:\n> Ralf Thielow <ralf.thielow@gmail.com> writes:\n>\n>> Extract a function that allows to print messages\n>> with a prefix.\n>>\n>> Signed-off-by: Ralf Thielow <ralf.thielow@gmail.com>\n>> ---\n>>  advice.c | 14 ++++++++++----\n>>  advice.h |  1 +\n>>  2 Dateien geändert, 11 Zeilen hinzugefügt(+), 4 Zeilen entfernt(-)\n>>\n>> diff --git a/advice.c b/advice.c\n>> index edfbd4a..e73d53b 100644\n>> --- a/advice.c\n>> +++ b/advice.c\n>> @@ -25,25 +25,31 @@ static struct {\n>>       { \"detachedhead\", &advice_detached_head },\n>>  };\n>>\n>> -void advise(const char *advice, ...)\n>> +void print_with_prefix(const char *prefix, const char *msg, ...)\n>>  {\n>>       struct strbuf buf = STRBUF_INIT;\n>>       va_list params;\n>>       const char *cp, *np;\n>>\n>> -     va_start(params, advice);\n>> -     strbuf_vaddf(&buf, advice, params);\n>> +     va_start(params, msg);\n>> +     strbuf_vaddf(&buf, msg, params);\n>>       va_end(params);\n>>\n>>       for (cp = buf.buf; *cp; cp = np) {\n>>               np = strchrnul(cp, '\\n');\n>> -             fprintf(stderr, _(\"hint: %.*s\\n\"), (int)(np - cp), cp);\n>> +             fprintf(stderr,  \"%s %.*s\\n\", prefix, (int)(np - cp), cp);\n>\n> Hrm, naively, printf(\"%s: %.*s\\n\", _(\"hint\"), ...) might look more\n> natural, but I vaguely recall that the current code places _()\n> around the entire \"hint: %.*s\\n\" on purpose.  IIRC, it was to allow\n> translations that flow from RTL e.g. \".siht od t'nod :tnih\".\n>\n> Doesn't this patch break it?\n\nSorry but I don't know what you mean with \"translations that flow\nfrom RTL e.g. \".siht od t'nod :tnih\".\" so I can't check this.\nAs far as I can see the callers only put a simple message in it,\ne.g. advise(_(\"Commit your changes or stash them to proceed.\"));\nSo I don't think that this patch would break anything.\n"},{"id":"198553","messageId":"7vfw6two6o.fsf@alter.siamese.dyndns.org","threadId":"31448","inReplyTo":"CAN0XMOK5QnNn8471RF4y7u_X0rfda=-J-KrCtyMsgCGHXNrwVw@mail.gmail.com","subject":"Re: [PATCH 1/2] advice: extract function to print messages with prefix","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-09-07T20:52:31Z","receivedAt":"2012-09-07T20:52:31Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ralf Thielow <ralf.thielow@gmail.com> writes:\n\n> On Fri, Sep 7, 2012 at 9:32 PM, Junio C Hamano <gitster@pobox.com> wrote:\n>\n>>> -             fprintf(stderr, _(\"hint: %.*s\\n\"), (int)(np - cp), cp);\n>>> +             fprintf(stderr,  \"%s %.*s\\n\", prefix, (int)(np - cp), cp);\n>>\n>> Hrm, naively, printf(\"%s: %.*s\\n\", _(\"hint\"), ...) might look more\n>> natural, but I vaguely recall that the current code places _()\n>> around the entire \"hint: %.*s\\n\" on purpose.  IIRC, it was to allow\n>> translations that flow from RTL e.g. \".siht od t'nod :tnih\".\n>>\n>> Doesn't this patch break it?\n>\n> Sorry but I don't know what you mean with \"translations that flow\n> from RTL e.g. \".siht od t'nod :tnih\".\" so I can't check this.\n> As far as I can see the callers only put a simple message in it,\n> e.g. advise(_(\"Commit your changes or stash them to proceed.\"));\n> So I don't think that this patch would break anything.\n\nYour patch would not allow target languages that want to put the\n_(\"hint\") at the *tail* end of each line of the message.  With the\noriginal, with something like this:\n\n\tmsgid \"hint: %.*s\\n\"\n        msgstr \"%.*s :tnih\\n\"\n\nyou could do that if you wanted to.\n"},{"id":"198554","messageId":"CAN0XMO+jbZeO=2NHisPizHfNwR62ebkC06KVZ8xOhgmRoAT4nw@mail.gmail.com","threadId":"31448","inReplyTo":"7vfw6two6o.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 1/2] advice: extract function to print messages with prefix","fromName":"Ralf Thielow","fromEmail":"ralf.thielow@gmail.com","sentAt":"2012-09-07T21:09:58Z","receivedAt":"2012-09-07T21:09:58Z","isPatch":true,"sender":{"key":"ralf.thielow@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1275832?v=4"},"body":"On Fri, Sep 7, 2012 at 10:52 PM, Junio C Hamano <gitster@pobox.com> wrote:\n> Ralf Thielow <ralf.thielow@gmail.com> writes:\n>\n>> On Fri, Sep 7, 2012 at 9:32 PM, Junio C Hamano <gitster@pobox.com> wrote:\n>>\n>>>> -             fprintf(stderr, _(\"hint: %.*s\\n\"), (int)(np - cp), cp);\n>>>> +             fprintf(stderr,  \"%s %.*s\\n\", prefix, (int)(np - cp), cp);\n>>>\n>>> Hrm, naively, printf(\"%s: %.*s\\n\", _(\"hint\"), ...) might look more\n>>> natural, but I vaguely recall that the current code places _()\n>>> around the entire \"hint: %.*s\\n\" on purpose.  IIRC, it was to allow\n>>> translations that flow from RTL e.g. \".siht od t'nod :tnih\".\n>>>\n>>> Doesn't this patch break it?\n>>\n>> Sorry but I don't know what you mean with \"translations that flow\n>> from RTL e.g. \".siht od t'nod :tnih\".\" so I can't check this.\n>> As far as I can see the callers only put a simple message in it,\n>> e.g. advise(_(\"Commit your changes or stash them to proceed.\"));\n>> So I don't think that this patch would break anything.\n>\n> Your patch would not allow target languages that want to put the\n> _(\"hint\") at the *tail* end of each line of the message.  With the\n> original, with something like this:\n>\n>         msgid \"hint: %.*s\\n\"\n>         msgstr \"%.*s :tnih\\n\"\n>\n> you could do that if you wanted to.\n>\n\nIs there a need actually?\nIt's easy to add a \"_(...)\" around this string, but then we'll have a\nmsgid in \"git.pot\" without ever having a sensible translation. Not?\n"},{"id":"293758","messageId":"7vbohhwn20.fsf@alter.siamese.dyndns.org","threadId":"31448","inReplyTo":"CAN0XMO+jbZeO=2NHisPizHfNwR62ebkC06KVZ8xOhgmRoAT4nw@mail.gmail.com","subject":"Re: [PATCH 1/2] advice: extract function to print messages with prefix","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-09-07T21:16:55Z","receivedAt":"2012-09-07T21:16:55Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ralf Thielow <ralf.thielow@gmail.com> writes:\n\n> On Fri, Sep 7, 2012 at 10:52 PM, Junio C Hamano <gitster@pobox.com> wrote:\n>> Ralf Thielow <ralf.thielow@gmail.com> writes:\n>>\n>>> On Fri, Sep 7, 2012 at 9:32 PM, Junio C Hamano <gitster@pobox.com> wrote:\n>>>\n>>>>> -             fprintf(stderr, _(\"hint: %.*s\\n\"), (int)(np - cp), cp);\n>>>>> +             fprintf(stderr,  \"%s %.*s\\n\", prefix, (int)(np - cp), cp);\n>>>>\n>>>> Hrm, naively, printf(\"%s: %.*s\\n\", _(\"hint\"), ...) might look more\n>>>> natural, but I vaguely recall that the current code places _()\n>>>> around the entire \"hint: %.*s\\n\" on purpose.  IIRC, it was to allow\n>>>> translations that flow from RTL e.g. \".siht od t'nod :tnih\".\n>>>>\n>>>> Doesn't this patch break it?\n>>>\n>>> Sorry but I don't know what you mean with \"translations that flow\n>>> from RTL e.g. \".siht od t'nod :tnih\".\" so I can't check this.\n>>> As far as I can see the callers only put a simple message in it,\n>>> e.g. advise(_(\"Commit your changes or stash them to proceed.\"));\n>>> So I don't think that this patch would break anything.\n>>\n>> Your patch would not allow target languages that want to put the\n>> _(\"hint\") at the *tail* end of each line of the message.  With the\n>> original, with something like this:\n>>\n>>         msgid \"hint: %.*s\\n\"\n>>         msgstr \"%.*s :tnih\\n\"\n>>\n>> you could do that if you wanted to.\n>\n> Is there a need actually?\n> It's easy to add a \"_(...)\" around this string, but then we'll have a\n> msgid in \"git.pot\" without ever having a sensible translation. Not?\n\nAs I said \"vaguely recall\", even though I don't have a first-hand\nexperience in such a language, I know I was talked into doing it\nthis way when we did 23cb5bf (i18n of multi-line advice messages,\n"},{"id":"198555","messageId":"7v7gs5wmvt.fsf@alter.siamese.dyndns.org","threadId":"31448","inReplyTo":"7vbohhwn20.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 1/2] advice: extract function to print messages with prefix","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-09-07T21:20:38Z","receivedAt":"2012-09-07T21:20:38Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> experience in such a language, I know I was talked into doing it\n> this way when we did 23cb5bf (i18n of multi-line advice messages,\n> 2011-12-22).  Could you dig around the list archive to see?\n\nHeh, don't bother.  Instead, start reading from here:\n\n  http://thread.gmane.org/gmane.comp.version-control.git/187592/focus=187601\n\nand learn what happened to the topic by going forward, or why it was\ndone that way by going backwards.\n\nThanks.\n"}]}