From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27783 invoked by alias); 18 Nov 2005 21:32:39 -0000 Received: (qmail 27776 invoked by uid 22791); 18 Nov 2005 21:32:39 -0000 X-Spam-Check-By: sourceware.org Received: from nevyn.them.org (HELO nevyn.them.org) (66.93.172.17) by sourceware.org (qpsmtpd/0.31) with ESMTP; Fri, 18 Nov 2005 21:32:38 +0000 Received: from drow by nevyn.them.org with local (Exim 4.54) id 1EdDqT-0004ut-Nu; Fri, 18 Nov 2005 16:32:33 -0500 Date: Fri, 18 Nov 2005 21:32:00 -0000 From: Daniel Jacobowitz To: Eli Zaretskii Cc: Ian Lance Taylor , gdb@sourceware.org Subject: Re: Maintainer policy for GDB Message-ID: <20051118213233.GA18853@nevyn.them.org> Mail-Followup-To: Eli Zaretskii , Ian Lance Taylor , gdb@sourceware.org References: <20051117044801.GA4705@nevyn.them.org> <8f2776cb0511162240q6f550008udda9803b5253fd88@mail.gmail.com> <20051118030711.GB31581@nevyn.them.org> <20051118152618.GB9100@nevyn.them.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.8i Mailing-List: contact gdb-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sourceware.org X-SW-Source: 2005-11/txt/msg00404.txt.bz2 On Fri, Nov 18, 2005 at 11:09:44PM +0200, Eli Zaretskii wrote: > > If you are "responsible" for patch review in a specific area, it means > > that you have promised that you will review all patches in that area > > in a reasonably timely fashion. > > > > If you are "authorized" to commit patches in a specific area, it means > > that you can commit patches without anybody else's approval. > > Thanks, I think I understand now. What fooled me was that Daniel said > "authority to review patches", not "authority to commit patches". And > since almost anyone can comment on a patch posted to gdb-patches, it > wasn't clear what kind of authority we were talking about. Ahh! Thanks, that's very helpful feedback. I've been using "review" sloppily when I really mean "approve and apply, or reject". > > At least in the U.S., anybody is "authorized" to make a citizen's > > arrest if they see a crime being committted. But only the police are > > "responsible" for doing so. If a civilian sees a crime being > > committed and does nothing, nothing happens to the civilian. If a > > policeman sees a crime being committed and does nothing, he gets fired > > from his job. > > But, unless I'm mistaken, Daniel didn't suggest to ``fire'' the > responsible person(s) in our case, did he? So the analogy is not > really full; in particular, what kind of responsibility is that if you > aren't going to be fired for failing to do that for which you are > responsible? You are mistaken. It's not a zero-tolerance policy, of course, but if you are listed as responsible and not doing your job, the global maintainers or SC should address it: Some maintainers are listed as responsible for patch review in particular areas. If a maintainer is not currently able or willing to review patches, please contact the global maintainers or the steering committee, who will resolve the situation and find a replacement or assistant if necessary. This is one of the big differences that having clearly listed responsibility gives us. We don't have to take away their commit access just to resolve this particular problem. -- Daniel Jacobowitz CodeSourcery, LLC