cvs commit: ports/net-mgmt/net-snmp Makefile
zi at FreeBSD.org
Wed Dec 14 20:44:26 UTC 2011
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 <wxs at FreeBSD.org>
> To: Michael Scheidell <scheidell at FreeBSD.org>
> Cc: "ports-committers at FreeBSD.org" <ports-committers at FreeBSD.org>, "cvs-ports at FreeBSD.org" <cvs-ports at FreeBSD.org>, "cvs-all at FreeBSD.org" <cvs-all at 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 
> > - 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 <ghuntley at ghuntley.com> 
> > 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
- 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
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
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.
PGP: EF36 D45A 5CA9 28B1 A550 18CD A43C D111 7AD7 FAF2
More information about the cvs-all