Pawel Wodnicki
2012-Nov-18 00:40 UTC
[LLVMdev] [cfe-dev] !!! 3.2 Release branch patching and the Code Owners
On 11/17/2012 6:35 PM, Chris Lattner wrote:> > On Nov 17, 2012, at 9:57 AM, 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. > > I agree. What problem are we trying to solve here? Are people approving patches that they shouldn't? > > -Chris >Rather it is the opposite, people are not approving patches they should. Pawel
Justin Holewinski
2012-Nov-18 00:43 UTC
[LLVMdev] [cfe-dev] !!! 3.2 Release branch patching and the Code Owners
On Sat, Nov 17, 2012 at 7:40 PM, Pawel Wodnicki <pawel at 32bitmicro.com>wrote:> On 11/17/2012 6:35 PM, Chris Lattner wrote: > > > > On Nov 17, 2012, at 9:57 AM, 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. > > > > I agree. What problem are we trying to solve here? Are people > approving patches that they shouldn't? > > > > -Chris > > > > Rather it is the opposite, people are not approving patches they should. >Can you provide some examples of the problems you are seeing?> > Pawel > > _______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev >-- Thanks, Justin Holewinski -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20121117/b9a157fb/attachment.html>
Pawel Wodnicki
2012-Nov-18 01:18 UTC
[LLVMdev] [cfe-dev] !!! 3.2 Release branch patching and the Code Owners
> On Sat, Nov 17, 2012 at 7:40 PM, Pawel Wodnicki <pawel at 32bitmicro.com>wrote: > >> On 11/17/2012 6:35 PM, Chris Lattner wrote: >>> >>> On Nov 17, 2012, at 9:57 AM, 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. >>> >>> I agree. What problem are we trying to solve here? Are people >> approving patches that they shouldn't? >>> >>> -Chris >>> >> >> Rather it is the opposite, people are not approving patches they should. >> > > Can you provide some examples of the problems you are seeing?Here is what happens. I get a message "could you please include/add/merge this r16xxxx into 3.2?". And my immediate reaction is sure, no problem this fixes PR/issue/crash so it is important. But are you the code owner and do you approve? So I have to go and start checking because that is the process. In the past few days CODE_OWNERS.TXT on the trunk has been changing while 3.2 has been stable, I work on 3.2 branch so I have sent couple of e-mails to wrong people. Anyway, it was not my intention to cause message storm and this is taking way too much bandwidth on the list. As always, change is causing breakages until we all learn how to do it efficiently. I have now a way to identify the code owners. All I am asking is for the code owners to state clearly that the change they ask for is approved. Couple of examples from real e-mails: approved APPROVED *approved* Pawel P.S. I'll even take "I am the code owner and I approve this message!"
Apparently Analagous Threads
- [LLVMdev] [cfe-dev] !!! 3.2 Release branch patching and the Code Owners
- [LLVMdev] [cfe-dev] !!! 3.2 Release branch patching and the Code Owners
- [LLVMdev] [cfe-dev] !!! 3.2 Release branch patching and the Code Owners
- [LLVMdev] [cfe-dev] !!! 3.2 Release branch patching and the Code Owners
- [LLVMdev] [cfe-dev] !!! 3.2 Release branch patching and the Code Owners