From owner-cvs-ports@FreeBSD.ORG Wed Dec 14 20:44:26 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 CB620106566B; Wed, 14 Dec 2011 20:44:26 +0000 (UTC) (envelope-from zi@FreeBSD.org) Received: from fast.rit.edu (fast.rit.edu [129.21.182.30]) by mx1.freebsd.org (Postfix) with ESMTP id 8B0F48FC08; Wed, 14 Dec 2011 20:44:26 +0000 (UTC) Received: from fast.rit.edu (localhost.rit.edu [127.0.0.1]) by fast.rit.edu (Postfix) with ESMTP id D03B61D16A; Wed, 14 Dec 2011 15:44:25 -0500 (EST) X-Virus-Scanned: by amavisd-new at fast.rit.edu Received: from fast.rit.edu ([127.0.0.1]) by fast.rit.edu (fast.rit.edu [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 8drt+ubaEFoI; Wed, 14 Dec 2011 15:44:24 -0500 (EST) Received: from syn.rit.edu (syn.rit.edu [129.21.182.15]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by fast.rit.edu (Postfix) with ESMTPS id ABD451D163; Wed, 14 Dec 2011 15:44:24 -0500 (EST) Received: from syn.rit.edu (localhost.rit.edu [127.0.0.1]) by syn.rit.edu (8.14.4/8.14.3) with ESMTP id pBEKiOsf068780; Wed, 14 Dec 2011 15:44:24 -0500 (EST) (envelope-from zi@FreeBSD.org) Received: (from zi@localhost) by syn.rit.edu (8.14.4/8.14.3/Submit) id pBEKiOv7068008; Wed, 14 Dec 2011 15:44:24 -0500 (EST) (envelope-from zi@FreeBSD.org) Date: Wed, 14 Dec 2011 15:44:23 -0500 From: Ryan Steinmetz To: Michael Scheidell Message-ID: <20111214204423.GA5725@fast.rit.edu> References: <6490f693-b019-4260-99d4-4f2dc4d1b8ab@blur> <18a96a03-b593-4431-95bf-70b4a08f6e01@blur> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <18a96a03-b593-4431-95bf-70b4a08f6e01@blur> User-Agent: Mutt/1.5.21 (2010-09-15) Cc: Michael Scheidell , Wesley Shields , "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 20:44:26 -0000 On (12/14/11 17:09), Michael Scheidell wrote: > Hmm. I wonder why I missed something as simple as that. > > Tinderbox didn't complain. > -- > Michael Scheidell, CTO > SECNAP Network Security > > > -----Original message----- > From: Wesley Shields > To: Michael Scheidell > Cc: "ports-committers@FreeBSD.org" , "cvs-ports@FreeBSD.org" , "cvs-all@FreeBSD.org" > Sent: Wed, Dec 14, 2011 16:48:54 GMT+00:00 > Subject: Re: cvs commit: ports/net-mgmt/net-snmp Makefile > > On Wed, Dec 14, 2011 at 01:08:08PM +0000, Michael Scheidell wrote: > > scheidell 2011-12-14 13:08:08 UTC > > > > FreeBSD ports repository > > > > Modified files: > > net-mgmt/net-snmp Makefile > > Log: > > - re-integration which enables the option of building the python bindings [1] > > - with the addition of a options knob (off by default) > > - No portsbump so that its a 'noop' for systems that don't need python bindings > > > > PR: ports/162140 > > Submitted by: Geoffrey Huntley [1] > > Approved by: zi (maintainer timeout, 30 days), gabor (mentor) > > > > Revision Changes Path > > 1.199 +8 -1 ports/net-mgmt/net-snmp/Makefile > > http://cvsweb.FreeBSD.org/ports/net-mgmt/net-snmp/Makefile.diff?r1=1.198&r2=1.199 > > | --- ports/net-mgmt/net-snmp/Makefile 2011/12/07 22:58:31 1.198 > > | +++ ports/net-mgmt/net-snmp/Makefile 2011/12/14 13:08:08 1.199 > > | @@ -2,7 +2,7 @@ > > | # Date created: 26 June 1996 > > | # Whom: gpalmer > > | # > > | -# $FreeBSD: /usr/local/www/cvsroot/FreeBSD/ports/net-mgmt/net-snmp/Makefile,v 1.198 2011/12/07 22:58:31 zi Exp $ > > | +# $FreeBSD: /usr/local/www/cvsroot/FreeBSD/ports/net-mgmt/net-snmp/Makefile,v 1.199 2011/12/14 13:08:08 scheidell Exp $ > > | # > > | > > | PORTNAME= snmp > > | @@ -23,7 +23,8 @@ OPTIONS= IPV6 "Build with IPv6 support" > > | DUMMY "Enable dummy values as placeholders" on \ > > | TKMIB "Install graphical MIB browser" off \ > > | DMALLOC "Enable dmalloc debug memory allocator" off \ > > | - UNPRIVILEGED "Allow unprivileged users to execute net-snmp" off > > | + UNPRIVILEGED "Allow unprivileged users to execute net-snmp" off \ > > | + PYTHON "Install python bindings off > > Missing quote here. > > -- WXS Please revert this change and re-open the PR, my reasoning is as follows: - It doesn't solve the problem in the PR. The submitters patch had problems which caused it not to work properly. - USE_PYDISTUTILS= is inappropriate and doesn't work in this situation. Please review bsd.python.mk for what this actually does and when to use it. - You did not review the tinderbox build log or you would have noticed that the log indicated: "checking for python... no" - You clearly did not test the python bindings prior to committing. - If everything did work, the port would have installed additional files into the python directories which would also have required plist changes. Based on these issues and the issue that wxs@ pointed out, I highly recommend you review your tinderbox settings/practices. It would also be appreciated to ping the maintainer letting them know that you plan to make commits to their port. If you don't hear anything back within a couple days (and it's in a maintainer timeout state), then proceed. I had reviewed this PR and had been investigating a solution. If you had reached out, I would have been able to share this with you. -r -- Ryan Steinmetz PGP: EF36 D45A 5CA9 28B1 A550 18CD A43C D111 7AD7 FAF2