<div dir="ltr">Here's an example of a use case that would be nice to fix:<div><br></div><div><a href="http://reviews.llvm.org/D4425">http://reviews.llvm.org/D4425</a><br></div><div><br></div><div>It's possible this has already been pointed out earlier in the thread.  The situation was, I forgot to include lldb-commits on the original patch, and then added it subsequently.  I could not find any way to get it to send out a new email containing the full patch + summary, so as a result I had to manually copy/paste the summary + patch text into an email response, and manually attach the patch as a file.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Jul 6, 2014 at 8:28 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="">On Sun, Jul 6, 2014 at 4:42 AM, Nick Lewycky <span dir="ltr"><<a href="mailto:nicholas@mxc.ca" target="_blank">nicholas@mxc.ca</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Chandler Carruth wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>
<br>
On Thu, Jul 3, 2014 at 11:00 PM, Nick Lewycky <<a href="mailto:nicholas@mxc.ca" target="_blank">nicholas@mxc.ca</a><br></div><div>
<mailto:<a href="mailto:nicholas@mxc.ca" target="_blank">nicholas@mxc.ca</a>>> wrote:<br>
<br>
    I don't like the lack attached patch files on the mailing list to do<br>
    a normal review.<br>
<br>
<br>
Wait what? The emails I get from phab *have* an attached patch file.<br>
That was a hard requirement when we first set up Phabricator.<br>
</div></blockquote>
<br>
Aaron nailed it. The initial emails come with attached patches. The problem is when people comment with the changes they made to the code, but there's no updated patch attached to that email. Aaron found examples so I'll defer to those. I can also keep an eye out for the next time it happens if you want.</blockquote>

<div><br></div></div><div>You should usually see 2 messages directly after each other - one with the patch, and one with the comment updates.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<span><font color="#888888"><br>
<br>
Nick<br>
</font></span></blockquote></div><br></div></div>
<br>_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
<br></blockquote></div><br></div>