From owner-cvs-ports@FreeBSD.ORG Wed Dec 14 21:00:17 2011 Return-Path: Delivered-To: cvs-ports@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 37C041065670; Wed, 14 Dec 2011 21:00:17 +0000 (UTC) (envelope-from wxs@atarininja.org) Received: from syn.atarininja.org (syn.csh.rit.edu [129.21.49.45]) by mx1.freebsd.org (Postfix) with ESMTP id E1CD18FC0A; Wed, 14 Dec 2011 21:00:16 +0000 (UTC) Received: by syn.atarininja.org (Postfix, from userid 1001) id EC1D65C2E; Wed, 14 Dec 2011 16:00:15 -0500 (EST) Date: Wed, 14 Dec 2011 16:00:15 -0500 From: Wesley Shields To: Michael Scheidell Message-ID: <20111214210015.GA13957@atarininja.org> References: <1ac141b1-6c9f-4023-90f7-270831c9e989@blur> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Cc: Ryan Steinmetz , Michael Scheidell , "cvs-all@FreeBSD.org" , "ports-committers@FreeBSD.org" , "gabor@FreeBSD.org" , "cvs-ports@FreeBSD.org" Subject: Re: cvs commit: ports/net-mgmt/net-snmp Makefile X-BeenThere: cvs-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 14 Dec 2011 21:00:17 -0000 On Wed, Dec 14, 2011 at 08:49:40PM +0000, Michael Scheidell wrote: > Pr logs show pings were made. > Timeout was well past conventional 14 days with no response . > > Sorry but all indications were that noone was working in the port. I don't see anything in the log that indicates zi@ was ever brought in on this discussion before committing. AFAICT the PR was never assigned to him. The PR may have looked stalled but a courtesy "ping" to the current maintainer asking for a quick review would have been the right thing to do, along with recording that in the PR audit trail. Technically speaking zi@ should have grabbed this PR when he became maintainer of this port. As a new committer you may not be familiar with the cultural norms most committers adhere to, but I believe it is the norm to try to contact a current maintainer of a port before committing, especially so if there is no evidence in the audit trail that he/she is working on it. They may be working in private and can help you out before you step into this mess. > As for Python bindings if the default is off it would not have made > those attempts anyway. You can tell your tinderbox to build with custom options. Even if you don't have a tinderbox you should test new options thoroughly. There are plenty of ways to detect the problems this caused. > Back it out and reopen the pr or leave it clised As the one who committed the problem it is, culturally speaking, your responsibility to back it out and work with the requester to find an acceptable solution. -- WXS