{"thread":{"id":"27103","subject":"[PATCH] doc: Use the term \"remark\" in \"cherry-pick -x\" to disambiguate from \"git notes\"","startedAt":"2011-04-15T07:43:55Z","lastAt":"2011-04-17T07:20:50Z","messageCount":11,"participants":["Sebastian Schuberth","Junio C Hamano","Jay Soffian"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"165884","messageId":"4DA7F73B.9050007@gmail.com","threadId":"27103","inReplyTo":null,"subject":"[PATCH] doc: Use the term \"remark\" in \"cherry-pick -x\" to disambiguate from \"git notes\"","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2011-04-15T07:43:55Z","receivedAt":"2011-04-15T07:43:55Z","isPatch":true,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"The documentation for \"cherry-pick -x\" can be misread in the way that a\n\"git notes\" object is attached to the new commit, which is not the case.\nInstead, just some text is appended to the original commit message. Make this\nmore clear by simply using the term \"remark\" instead of \"note\".\n\nSigned-off-by: Sebastian Schuberth \n---\n Documentation/git-cherry-pick.txt | 4 ++--\n 1 files changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/git-cherry-pick.txt b/Documentation/git-cherry-pick.txt\nindex 01db830..5afc2b0 100644\n--- a/Documentation/git-cherry-pick.txt\n+++ b/Documentation/git-cherry-pick.txt\n@@ -52,8 +52,8 @@ OPTIONS\n \n -x::\n \tWhen recording the commit, append to the original commit\n-\tmessage a note that indicates which commit this change\n-\twas cherry-picked from. Append the note only for cherry\n+\tmessage a remark that indicates which commit this change\n+\twas cherry-picked from. Append the remark only for cherry\n \tpicks without conflicts. Do not use this option if\n \tyou are cherry-picking from your private branch because\n \tthe information is useless to the recipient. If on the\n-- \n1.7.3.2.msysgit.11\n"},{"id":"165909","messageId":"7v7have8wo.fsf@alter.siamese.dyndns.org","threadId":"27103","inReplyTo":"4DA7F73B.9050007@gmail.com","subject":"Re: [PATCH] doc: Use the term \"remark\" in \"cherry-pick -x\" to disambiguate from \"git notes\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2011-04-15T17:12:07Z","receivedAt":"2011-04-15T17:12:07Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sebastian Schuberth writes:\n\n> The documentation for \"cherry-pick -x\" can be misread in the way that a\n> \"git notes\" object is attached to the new commit, which is not the case.\n> Instead, just some text is appended to the original commit message. Make this\n> more clear by simply using the term \"remark\" instead of \"note\".\n\nA sensible concern, I guess, even though \"_append_ to the original _commit\nmessage_\" should signal that this sentence cannot be referring to a note\nobject separate from the commit.\n\n> diff --git a/Documentation/git-cherry-pick.txt b/Documentation/git-cherry-pick.txt\n> index 01db830..5afc2b0 100644\n> --- a/Documentation/git-cherry-pick.txt\n> +++ b/Documentation/git-cherry-pick.txt\n> @@ -52,8 +52,8 @@ OPTIONS\n> \n> -x::\n> \tWhen recording the commit, append to the original commit\n> +\tmessage a remark that indicates which commit this change\n> +\twas cherry-picked from. Append the remark only for cherry\n> \tpicks without conflicts. ...\n\nI wonder if it is better to be even more explicit and say:\n\n\t... append a line that says \"(cherry picked from commit ...)\" to\n the original commit message in order to indicate which commit this\n change was cherry-picked from. This is done only for ...\n\nso that we don't have to worry about having to change this in 10 years\nwhen we notice that description can easily be misread that it is done\nusing the remark feature introduced a few years back?\n"},{"id":"165915","messageId":"4DA8862F.5070102@gmail.com","threadId":"27103","inReplyTo":"7v7have8wo.fsf@alter.siamese.dyndns.org","subject":"[PATCHv2] doc: Clarify that \"cherry-pick -x\" does not use \"git notes\"","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2011-04-15T17:53:51Z","receivedAt":"2011-04-15T17:53:51Z","isPatch":false,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"The documentation for \"cherry-pick -x\" could be misread in the way that a\n\"git notes\" object is attached to the new commit, which is not the case.\n\nSigned-off-by: Sebastian Schuberth \n---\n Documentation/git-cherry-pick.txt | 7 ++++---\n 1 files changed, 4 insertions(+), 3 deletions(-)\n\ndiff --git a/Documentation/git-cherry-pick.txt b/Documentation/git-cherry-pick.txt\nindex 01db830..9d8fe0d 100644\n--- a/Documentation/git-cherry-pick.txt\n+++ b/Documentation/git-cherry-pick.txt\n@@ -51,9 +51,10 @@ OPTIONS\n \tmessage prior to committing.\n \n -x::\n-\tWhen recording the commit, append to the original commit\n-\tmessage a note that indicates which commit this change\n-\twas cherry-picked from. Append the note only for cherry\n+\tWhen recording the commit, append a line that says\n+\t\"(cherry picked from commit ...)\" to the original commit\n+\tmessage in order to indicate which commit this change was\n+\tcherry-picked from. This is done only for cherry\n \tpicks without conflicts. Do not use this option if\n \tyou are cherry-picking from your private branch because\n \tthe information is useless to the recipient. If on the\n-- \n1.7.3.2.msysgit.11\n"},{"id":"165934","messageId":"BANLkTi=HecmdGbEX5inqUi4FZ5Ty+L1Ybg@mail.gmail.com","threadId":"27103","inReplyTo":"4DA8862F.5070102@gmail.com","subject":"Re: [PATCHv2] doc: Clarify that \"cherry-pick -x\" does not use \"git notes\"","fromName":"Jay Soffian","fromEmail":"jaysoffian@gmail.com","sentAt":"2011-04-15T22:22:07Z","receivedAt":"2011-04-15T22:22:07Z","isPatch":false,"sender":{"key":"jaysoffian@gmail.com","avatar":"https://avatars.githubusercontent.com/u/155970?v=4"},"body":"On Fri, Apr 15, 2011 at 1:53 PM, Sebastian Schuberth\n wrote:\n> The documentation for \"cherry-pick -x\" could be misread in the way that a\n> \"git notes\" object is attached to the new commit, which is not the case.\n>\n> Signed-off-by: Sebastian Schuberth \n> ---\n>  Documentation/git-cherry-pick.txt |    7 ++++---\n>  1 files changed, 4 insertions(+), 3 deletions(-)\n>\n> diff --git a/Documentation/git-cherry-pick.txt b/Documentation/git-cherry-pick.txt\n> index 01db830..9d8fe0d 100644\n> --- a/Documentation/git-cherry-pick.txt\n> +++ b/Documentation/git-cherry-pick.txt\n> @@ -51,9 +51,10 @@ OPTIONS\n>        message prior to committing.\n>\n>  -x::\n> -       When recording the commit, append to the original commit\n> -       message a note that indicates which commit this change\n> -       was cherry-picked from.  Append the note only for cherry\n> +       When recording the commit, append a line that says\n> +       \"(cherry picked from commit ...)\" to the original commit\n> +       message in order to indicate which commit this change was\n> +       cherry-picked from.  This is done only for cherry\n>        picks without conflicts.  Do not use this option if\n>        you are cherry-picking from your private branch because\n>        the information is useless to the recipient.  If on the\n\nAs long as you're nearby, \"This is done only for cherry picks without\nconflicts.\" can be dropped. It's no longer true.\n\nj.\n"},{"id":"165956","messageId":"BANLkTi=jr0hkmBUGj7tHURSj5XiJkyyQqg@mail.gmail.com","threadId":"27103","inReplyTo":"BANLkTi=HecmdGbEX5inqUi4FZ5Ty+L1Ybg@mail.gmail.com","subject":"Re: [PATCHv2] doc: Clarify that \"cherry-pick -x\" does not use \"git notes\"","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2011-04-16T07:19:57Z","receivedAt":"2011-04-16T07:19:57Z","isPatch":false,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"On Sat, Apr 16, 2011 at 00:22, Jay Soffian wrote:\n\n>>  -x::\n>> -       When recording the commit, append to the original commit\n>> -       message a note that indicates which commit this change\n>> -       was cherry-picked from.  Append the note only for cherry\n>> +       When recording the commit, append a line that says\n>> +       \"(cherry picked from commit ...)\" to the original commit\n>> +       message in order to indicate which commit this change was\n>> +       cherry-picked from.  This is done only for cherry\n>>        picks without conflicts.  Do not use this option if\n>>        you are cherry-picking from your private branch because\n>>        the information is useless to the recipient.  If on the\n>\n> As long as you're nearby, \"This is done only for cherry picks without\n> conflicts.\" can be dropped. It's no longer true.\n\nHow can this be not true anymore? If you get a conflict while picking,\nyou're asked to resolve the conflict and then commit via \"git commit\n-c \". Doing so will *not* give you the mentioned \"(cherry\npicked from commit ...)\" line.\n\n-- \nSebastian Schuberth\n"},{"id":"165967","messageId":"7vk4eub1g9.fsf@alter.siamese.dyndns.org","threadId":"27103","inReplyTo":"BANLkTi=jr0hkmBUGj7tHURSj5XiJkyyQqg@mail.gmail.com","subject":"Re: [PATCHv2] doc: Clarify that \"cherry-pick -x\" does not use \"git notes\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2011-04-16T16:33:42Z","receivedAt":"2011-04-16T16:33:42Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sebastian Schuberth writes:\n\n> How can this be not true anymore? If you get a conflict while picking,\n> you're asked to resolve the conflict and then commit via \"git commit\n> -c \". Doing so will *not* give you the mentioned \"(cherry\n> picked from commit ...)\" line.\n\nI suspect Jay had the recent CHERRY_PICK_HEAD series that lets you omit\nsaying \"-c \" in mind. We should not leave the cherry-picked from\nin such a case, I think, but the new code may be buggy.\n"},{"id":"165971","messageId":"BANLkTimX6QQB8uASjNw4azG4Gdchc-S=rQ@mail.gmail.com","threadId":"27103","inReplyTo":"7vk4eub1g9.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCHv2] doc: Clarify that \"cherry-pick -x\" does not use \"git notes\"","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2011-04-16T16:55:41Z","receivedAt":"2011-04-16T16:55:41Z","isPatch":false,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"On Sat, Apr 16, 2011 at 18:33, Junio C Hamano wrote:\n\n>> How can this be not true anymore? If you get a conflict while picking,\n>> you're asked to resolve the conflict and then commit via \"git commit\n>> -c \". Doing so will *not* give you the mentioned \"(cherry\n>> picked from commit ...)\" line.\n>\n> I suspect Jay had the recent CHERRY_PICK_HEAD series that lets you omit\n> saying \"-c \" in mind.  We should not leave the cherry-picked from\n> in such a case, I think, but the new code may be buggy.\n\nThanks for explaining. I keep my patchv2 as-is in this case.\n\n-- \nSebastian Schuberth\n"},{"id":"165973","messageId":"BANLkTim2=Jg4QcgKwO=J6343zWAoCkyXZQ@mail.gmail.com","threadId":"27103","inReplyTo":"7vk4eub1g9.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCHv2] doc: Clarify that \"cherry-pick -x\" does not use \"git notes\"","fromName":"Jay Soffian","fromEmail":"jaysoffian@gmail.com","sentAt":"2011-04-16T17:11:04Z","receivedAt":"2011-04-16T17:11:04Z","isPatch":false,"sender":{"key":"jaysoffian@gmail.com","avatar":"https://avatars.githubusercontent.com/u/155970?v=4"},"body":"On Sat, Apr 16, 2011 at 12:33 PM, Junio C Hamano wrote:\n> I suspect Jay had the recent CHERRY_PICK_HEAD series that lets you omit\n> saying \"-c \" in mind.\n\nCorrect.\n\n>  We should not leave the cherry-picked from\n> in such a case, I think, but the new code may be buggy.\n\nDisagree. Here was my thought process (it was a conscious decision):\nafter conflict, when the user runs \"git commit\", the message is\nbrought up in the editor, leaving it up to the user to decide whether\nit is appropriate to remove the \"cherry-picked from\" message. I\npersonally think it is correct to keept it, along with the list of\n\"Conflicts:\". I understand some users think it's not appropriate to\nkeep the \"cherry-picked from\" message, but git should not force the\ndecision.\n\nj.\n"},{"id":"165977","messageId":"7vaafqay9f.fsf@alter.siamese.dyndns.org","threadId":"27103","inReplyTo":"BANLkTim2=Jg4QcgKwO=J6343zWAoCkyXZQ@mail.gmail.com","subject":"Re: [PATCHv2] doc: Clarify that \"cherry-pick -x\" does not use \"git notes\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2011-04-16T17:42:36Z","receivedAt":"2011-04-16T17:42:36Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jay Soffian writes:\n\n> Disagree. Here was my thought process (it was a conscious decision):\n> after conflict, when the user runs \"git commit\", the message is\n> brought up in the editor, leaving it up to the user to decide whether\n> it is appropriate to remove the \"cherry-picked from\" message. I\n> personally think it is correct to keept it, along with the list of\n> \"Conflicts:\".\n\nI agree that the user can decide to remove that line in the editor when\nthe necessary conflict resolution was so extensive that the most part of\nthe original change became irrelevant.\n\nIt requires extra work that was unnecessary to the end user before the\nseries, and the series is a regression in that sense. But removing a\nsingle extra line is much easier to do than adding the line from scratch,\nso this particular potential regression is not really a big deal.\n"},{"id":"165998","messageId":"7v39lhbnzj.fsf@alter.siamese.dyndns.org","threadId":"27103","inReplyTo":"7vaafqay9f.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCHv2] doc: Clarify that \"cherry-pick -x\" does not use \"git notes\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2011-04-17T02:39:12Z","receivedAt":"2011-04-17T02:39:12Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano writes:\n\n> I agree that the user can decide to remove that line in the editor when\n> the necessary conflict resolution was so extensive that the most part of\n> the original change became irrelevant.\n> ...\n\nSo to conclude,...\n\n>> + cherry-picked from. This is done only for cherry\n>> picks without conflicts. Do not use this option if\n>> you are cherry-picking from your private branch because\n>> the information is useless to the recipient. If on the\n\n... we could just remove the \"This is done only for cherry picks without\nconflicts.\" We could explain that the user may want to remove the line\ndepending on the nature of the conflict resolution, but I guess it can\nbe left without saying.\n"},{"id":"166006","messageId":"BANLkTink3uzx3xz5_pG6FmFctD_2W6HiCg@mail.gmail.com","threadId":"27103","inReplyTo":"7v39lhbnzj.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCHv2] doc: Clarify that \"cherry-pick -x\" does not use \"git notes\"","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2011-04-17T07:20:50Z","receivedAt":"2011-04-17T07:20:50Z","isPatch":false,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"On Sun, Apr 17, 2011 at 04:39, Junio C Hamano wrote:\n\n> So to conclude,...\n>\n>>> +       cherry-picked from.  This is done only for cherry\n>>>        picks without conflicts.  Do not use this option if\n>>>        you are cherry-picking from your private branch because\n>>>        the information is useless to the recipient.  If on the\n>\n> ... we could just remove the \"This is done only for cherry picks without\n> conflicts.\"  We could explain that the user may want to remove the line\n> depending on the nature of the conflict resolution, but I guess it can\n> be left without saying.\n\nStill, I believe it would be more consistent to do this change as part\nof the CHERRY_PICK_HEAD series itself, then.\n\n-- \nSebastian Schuberth\n"}]}