{"thread":{"id":"12744","subject":"bug in git-send-mail (the 'next' branch)","startedAt":"2008-03-18T13:52:58Z","lastAt":"2008-03-19T08:03:18Z","messageCount":3,"participants":["Yakov Lerner","Frank Lichtenheld","Jakub Narebski"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"72354","messageId":"f36b08ee0803180652g68caa4cci21b0c59020c0fd07@mail.gmail.com","threadId":"12744","inReplyTo":null,"subject":"bug in git-send-mail (the 'next' branch)","fromName":"Yakov Lerner","fromEmail":"iler.ml@gmail.com","sentAt":"2008-03-18T13:52:58Z","receivedAt":"2008-03-18T13:52:58Z","isPatch":false,"sender":{"key":"iler.ml@gmail.com","avatar":null},"body":"There is a bug in git-send-mail in the 'next' branch. If first\nparagraph of the file in all indented by 1 space, and 1st line of the\nparagraph is at the first line of the file, then the whole\nparagraph disappears silently. If it's intentional, there should be a warning\nor something. And If it's *not* indented (as if it expects message\nheader lines at this place), then it appears in the message body, so\nthere is some inconsistency here.\n\nYakov\n"},{"id":"72396","messageId":"m3abkwvxfr.fsf@localhost.localdomain","threadId":"12744","inReplyTo":"f36b08ee0803180652g68caa4cci21b0c59020c0fd07@mail.gmail.com","subject":"Re: bug in git-send-mail (the 'next' branch)","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2008-03-18T15:55:37Z","receivedAt":"2008-03-18T15:55:37Z","isPatch":false,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"\"Yakov Lerner\" <iler.ml@gmail.com> writes:\n\n> There is a bug in git-send-mail in the 'next' branch. If first\n> paragraph of the file in all indented by 1 space, and 1st line of the\n> paragraph is at the first line of the file, then the whole\n> paragraph disappears silently. If it's intentional, there should be a warning\n> or something. And If it's *not* indented (as if it expects message\n> header lines at this place), then it appears in the message body, so\n> there is some inconsistency here.\n\nI guess that this bug is in \"header flowing\" code; headers in email\nmessage can span multiple lines by some use of indenting. That is\nwhere one can search for the source of this bug...\n\n-- \nJakub Narebski\nPoland\nShadeHawk on #git\n"},{"id":"72383","messageId":"20080319080318.GX18624@mail-vs.djpig.de","threadId":"12744","inReplyTo":"f36b08ee0803180652g68caa4cci21b0c59020c0fd07@mail.gmail.com","subject":"Re: bug in git-send-mail (the 'next' branch)","fromName":"Frank Lichtenheld","fromEmail":"frank@lichtenheld.de","sentAt":"2008-03-19T08:03:18Z","receivedAt":"2008-03-19T08:03:18Z","isPatch":false,"sender":{"key":"frank@lichtenheld.de","avatar":"https://gravatar.com/avatar/b9f1d4b120e138f157c9e480d0818197c474628923786adb98f30017cdb99c3c?d=mp&s=160"},"body":"On Tue, Mar 18, 2008 at 03:52:58PM +0200, Yakov Lerner wrote:\n> There is a bug in git-send-mail in the 'next' branch. If first\n> paragraph of the file in all indented by 1 space, and 1st line of the\n> paragraph is at the first line of the file, then the whole\n> paragraph disappears silently. If it's intentional, there should be a warning\n> or something. And If it's *not* indented (as if it expects message\n> header lines at this place), then it appears in the message body, so\n> there is some inconsistency here.\n\nIf the first line of the file doesn't match either /^From / or\n/^[-A-Za-z]+:\\s/ send-email assumes the file is using the legacy format\nfrom the original send_lots_of_email.pl script which is apparently\n\"first line of the message is who to CC, and second line is the subject of the\nmessage.\" (comment from source code). The code doesn't use only the\nfirst two lines but eats everything up the first whitespace line.\nWhich explains the behaviour you have seen.\n\nThere are a few things that could be fixed here:\n1) Document the file formats this script expects in the man page. It\n   doesn't talk about this at all atm.\n2) Fix the legacy format support to only use the first two lines and\n   warn/die if there is no whitespace line after that\n3) Warn if the first line looks neither like a email header nor a\n   email address\n4) Perhaps in the future at some point deprecate the legacy support\n   (1.6.0?)\n\nGruesse,\n-- \nFrank Lichtenheld <frank@lichtenheld.de>\nwww: http://www.djpig.de/\n"}]}