[PATCH] D63488: [docs] Phabricator, not the lists is the main entry point for new patches

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 18 10:17:30 PDT 2019


lebedev.ri added a comment.

In D63488#1548725 <https://reviews.llvm.org/D63488#1548725>, @hfinkel wrote:

> In D63488#1548514 <https://reviews.llvm.org/D63488#1548514>, @probinson wrote:
>
> > Requiring Phabricator raises the barrier to one-off patches from casual contributors, because using Phabricator requires a registration step.
> >  I don't think we should require it until casual users with drive-by patches can contribute easily.
>
>
> But my impressions is that, in practice, patches sent to the mailing list are not tracked, more difficult to review, and often, don't get looked at. Having a patch sent to the mailing list only to be ignored is a larger barrier to entry than the registration step. We should ask people to follow a procedure that is like to work for them, not one likely to end in frustration.


That's quite precisely my point.
It's friendlier to write the *actual* steps, even if they are more complicated than the
friendly "just submit the patch somewhere, it's their job to track every single entry point"
which is ultimately not true and results in silently ignored patches.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63488/new/

https://reviews.llvm.org/D63488





More information about the llvm-commits mailing list