[LLVMdev] [cfe-dev] !!! 3.2 Release branch patching and the Code Owners

Hal Finkel hfinkel at anl.gov
Sat Nov 17 11:42:20 PST 2012


----- Original Message -----
> From: "Joe Abbey" <joe.abbey at gmail.com>
> To: "Nadav Rotem" <nrotem at apple.com>
> Cc: llvmdev at cs.uiuc.edu
> Sent: Saturday, November 17, 2012 1:25:04 PM
> Subject: Re: [LLVMdev] [cfe-dev] !!! 3.2 Release branch patching and the	Code Owners
> 
> 
> On Nov 17, 2012, at 12:57 PM, Nadav Rotem <nrotem at apple.com> wrote:
> 
> > I think that the code owner process is becoming complicated and I
> > am not sure if it serves Chris's original intent. I don't think
> > that we need to change every file nor do we need an automatic tool
> > to find the owner. I think that a simple text file, or a section
> > in the docs is enough.
> 
> ^^ this

Pawel, please correct me if I'm wrong, but it sounds like your underlying problem is that people are sending you merge requests directly, and you're not sure from whom you need to make sure to get approvals. This being the case, you should stop accepting such requests. Requests should be sent directly to the code owners (on list). Only those code owners should communicate directly with you (either to instruct you to merge in certain patches, or better yet, to merge in approved changes directly). I think this matches the original intent of the system: it partitions the workload among domain experts instead of forcing you to deal explicitly with many of the requests.

 -Hal

> 
> Joe
> 
> 
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
> 

-- 
Hal Finkel
Postdoctoral Appointee
Leadership Computing Facility
Argonne National Laboratory



More information about the llvm-dev mailing list