[llvm-dev] whatever happened to this patch?
David Blaikie via llvm-dev
llvm-dev at lists.llvm.org
Wed Apr 15 15:42:26 PDT 2020
Yup, that's the general idea!
On Wed, Apr 15, 2020 at 1:54 PM Andrew Kelley <andrew at ziglang.org> wrote:
> Thanks David, that makes sense. So:
>
> 1. Find someone who has commit access and knowledge about the touched
> areas of code.
>
> 2. Convince them to review & land the patch.
>
> On 4/15/20 4:52 PM, David Blaikie wrote:
> > Generally an author should submit an approved patch. If they do not have
> > commit access they should ask someone who does (usually the approver) to
> > commit it on their behalf.
> >
> > (you generally shouldn't approve patches if you aren't in a position to
> > have committed the patch without review if it was your own patch - so at
> > the very least having commit access, but also being sufficiently
> > familiar with the norms of the part of the project that you're confident
> > committing without review (this doesn't fully cover all cases, some
> > areas do more cross-review than others, etc))
> >
> > On Wed, Apr 15, 2020 at 1:44 PM Andrew Kelley via llvm-dev
> > <llvm-dev at lists.llvm.org <mailto:llvm-dev at lists.llvm.org>> wrote:
> >
> > https://reviews.llvm.org/D62475
> >
> > It looks like it never got merged, what was the missing step to get
> this
> > landed?
> >
> > Regards,
> > Andrew
> >
> > _______________________________________________
> > LLVM Developers mailing list
> > llvm-dev at lists.llvm.org <mailto:llvm-dev at lists.llvm.org>
> > https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
> >
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20200415/63489903/attachment.html>
More information about the llvm-dev
mailing list