<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jun 27, 2014 at 6:07 PM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><br>
> On 2014-Jun-27, at 00:36, Manuel Klimek <<a href="mailto:klimek@google.com">klimek@google.com</a>> wrote:<br>
><br>
> On Fri, Jun 27, 2014 at 8:13 AM, Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>> wrote:<br>
> Happened to me twice, it would be really nice if Phab would require confirmation of patches created without CCing one of the two lists, something like:<br>
><br>
> "You have not CCed llvm-commits or cfe-commits, are you creating a private patch?"<br>
><br>
> I filed<br>
> <a href="https://secure.phabricator.com/T5495" target="_blank">https://secure.phabricator.com/T5495</a><br>
><br>
> Cheers,<br>
> /Manuel<br>
><br>
<br>
</div>Thanks for filing this!<br>
<br>
A couple of comments on epriestley's comments; perhaps you can<br>
forward these, if you agree?<br>
<br>
> I don't immediately see a reasonable approach to implement this in a general way. In other cases, I think rules like this are usually expressed as "always use Herald to CC <some list>", but it sounds like there is no way to automatically determine the correct list in this case?<br>

<br>
Users must specify a "repository", and the repository determines<br>
which list needs to be included.  Once the config allows for a<br>
mandatory (or default) list on a per-repository basis, we're golden<br>
(cfe => cfe-commits, llvm => llvm-commits, compiler-rt =><br>
llvm-commits, lldb => lldb-commits, etc.).<br></blockquote><div><br></div><div>Hm, we might be able to set up dummy repositories (because we only have a single large LLVM repository from the code view point of view).</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> We have a buildRevisionWarnings() method, and we can move that into CustomFields and then I can write you a tiny piece of plugin code. This wouldn't prompt or pop a dialog, but would look like this:<br>
<br>
This looks like a good step, but isn't sufficient long-term -- adding<br>
(e.g.) llvm-commits *after the fact* fails to send the patch to the<br>
list.<br></blockquote><div><br></div><div>That's something we want to fix long-term - that is, we'll want to fix the emails to send mails as they look on the first mail when you add people to a review; I've already filed a bug for that.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Also: given that there's a `buildRevisionWarnings()` hook, I wonder:<br>
is there a custom hook for errors, which would reject the revision?<br>
</blockquote></div><br></div><div class="gmail_extra">Oh, we can do anything ourselves, but we'd need somebody to spend some time to hack on it.</div></div>