<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 9, 2017 at 12:56 AM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><div class="gmail-h5"><div dir="ltr">On Sat, Dec 31, 2016 at 12:49 AM Daniel Berlin <<a href="mailto:dberlin@dberlin.org" target="_blank">dberlin@dberlin.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr" class="gmail-m_-6760421481011919024gmail_msg"><div class="gmail_extra gmail-m_-6760421481011919024gmail_msg"><div class="gmail_quote gmail-m_-6760421481011919024gmail_msg">On Fri, Dec 30, 2016 at 3:36 PM, Mehdi Amini <span dir="ltr" class="gmail-m_-6760421481011919024gmail_msg"><<a href="mailto:mehdi.amini@apple.com" class="gmail-m_-6760421481011919024gmail_msg" target="_blank">mehdi.amini@apple.com</a>></span> wrote:<br class="gmail-m_-6760421481011919024gmail_msg"><blockquote class="gmail_quote gmail-m_-6760421481011919024gmail_msg" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word" class="gmail-m_-6760421481011919024gmail_msg"><div class="gmail-m_-6760421481011919024gmail_msg"><div class="gmail-m_-6760421481011919024m_-7499801369921333899h5 gmail-m_-6760421481011919024gmail_msg"><br class="gmail-m_-6760421481011919024gmail_msg"><div class="gmail-m_-6760421481011919024gmail_msg"><blockquote type="cite" class="gmail-m_-6760421481011919024gmail_msg"><div class="gmail-m_-6760421481011919024gmail_msg">On Dec 30, 2016, at 3:26 PM, Daniel Berlin <<a href="mailto:dberlin@dberlin.org" class="gmail-m_-6760421481011919024gmail_msg" target="_blank">dberlin@dberlin.org</a>> wrote:</div><br class="gmail-m_-6760421481011919024m_-7499801369921333899m_-1612719899131616300Apple-interchange-newline gmail-m_-6760421481011919024gmail_msg"><div class="gmail-m_-6760421481011919024gmail_msg"><div dir="ltr" class="gmail-m_-6760421481011919024gmail_msg"><div class="gmail_extra gmail-m_-6760421481011919024gmail_msg"><div class="gmail_quote gmail-m_-6760421481011919024gmail_msg"><blockquote class="gmail_quote gmail-m_-6760421481011919024gmail_msg" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr" class="gmail-m_-6760421481011919024gmail_msg"><div class="gmail_extra gmail-m_-6760421481011919024gmail_msg"><div class="gmail_quote gmail-m_-6760421481011919024gmail_msg"><div class="gmail-m_-6760421481011919024gmail_msg"><br class="gmail-m_-6760421481011919024gmail_msg"></div><div class="gmail-m_-6760421481011919024gmail_msg">PS: consider sending replies in phabricator, because email replies sometimes doesn't appear there (like this one)</div></div></div></div></blockquote><div class="gmail-m_-6760421481011919024gmail_msg"><br class="gmail-m_-6760421481011919024gmail_msg"></div><div class="gmail-m_-6760421481011919024gmail_msg">I'm strongly against that, in the sense that phabricator needs to integrate with existing workflows.  I do real reviews in phab, but just overall commenting, don't have time or energy to click when i'm reading email :)</div><div class="gmail-m_-6760421481011919024gmail_msg"><br class="gmail-m_-6760421481011919024gmail_msg"></div><div class="gmail-m_-6760421481011919024gmail_msg">If email to phabricator is not currently working, we should fix that.</div><div class="gmail-m_-6760421481011919024gmail_msg">(I believe it is *supposed to be working*)</div></div></div></div></div></blockquote></div><br class="gmail-m_-6760421481011919024gmail_msg"></div></div><div class="gmail-m_-6760421481011919024gmail_msg">Inline answer in emails never worked in Phab. AFAIK.</div></div></blockquote><div class="gmail-m_-6760421481011919024gmail_msg"><br class="gmail-m_-6760421481011919024gmail_msg"></div><div class="gmail-m_-6760421481011919024gmail_msg"><br class="gmail-m_-6760421481011919024gmail_msg"></div></div></div></div><div dir="ltr" class="gmail-m_-6760421481011919024gmail_msg"><div class="gmail_extra gmail-m_-6760421481011919024gmail_msg"><div class="gmail_quote gmail-m_-6760421481011919024gmail_msg"><div class="gmail-m_-6760421481011919024gmail_msg">Sigh.</div></div></div></div><div dir="ltr" class="gmail-m_-6760421481011919024gmail_msg"><div class="gmail_extra gmail-m_-6760421481011919024gmail_msg"><div class="gmail_quote gmail-m_-6760421481011919024gmail_msg"><div class="gmail-m_-6760421481011919024gmail_msg"> </div><blockquote class="gmail_quote gmail-m_-6760421481011919024gmail_msg" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word" class="gmail-m_-6760421481011919024gmail_msg"><div class="gmail-m_-6760421481011919024gmail_msg">It use to be that top answers were included as comment. But even that was not in Phab upstream but in our instance only (IIUC), so it can easily have been lost with any update.</div></div></blockquote></div></div></div><div dir="ltr" class="gmail-m_-6760421481011919024gmail_msg"><div class="gmail_extra gmail-m_-6760421481011919024gmail_msg"><div class="gmail_quote gmail-m_-6760421481011919024gmail_msg"><div class="gmail-m_-6760421481011919024gmail_msg">That seems super annoying.</div><div class="gmail-m_-6760421481011919024gmail_msg"><br class="gmail-m_-6760421481011919024gmail_msg"></div><div class="gmail-m_-6760421481011919024gmail_msg">Even bugzilla can get this right :(</div></div></div></div></blockquote><div><br></div></div></div><div>Generally, only top-posting has worked in replies to update phab. Parsing in-line replies across different email clients is a PITA, unfortunately, so unless we find somebody who'd have fun hacking that up, it's unlikely to happen.</div></div></div></blockquote><div><br></div><div>I have one that will work for a very large set of cases, but not inline replies (ie it handles top and bottom)</div><div>It's python though, i can make it parse whatever and spit out the reply part.<br></div><div><br></div><div>For example, for your email, it comes up with:</div><div>>>> print email_reply_parser.EmailReplyParser.parse_reply(a.get_payload()[0].as_string())</div><div><br>"Content-Type: text/plain; charset=UTF-8</div><div>Generally, only top-posting has worked in replies to update phab. Parsing</div><div>in-line replies across different email clients is a PITA, unfortunately, so</div><div>unless we find somebody who'd have fun hacking that up, it's unlikely to</div><div>happen.</div><div><br></div><div>Currently, incoming email parsing is also completely broken, because the</div><div>https switch invalidated our sendgrid config, and sendgrid now wants us to</div><div>jump through hoops to update the info...</div><div>"</div><div><br></div><div>(note the email is mime, has attached content, blah blah blah)</div><div><br></div><div><br><br></div></div></div></div>