From owner-freebsd-python@FreeBSD.ORG Fri Feb 24 16:50:06 2012 Return-Path: Delivered-To: freebsd-python@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 2DE14106564A for ; Fri, 24 Feb 2012 16:50:06 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 0D2548FC0C for ; Fri, 24 Feb 2012 16:50:06 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.5/8.14.5) with ESMTP id q1OGo5iH098440 for ; Fri, 24 Feb 2012 16:50:05 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.5/8.14.5/Submit) id q1OGo5VA098438; Fri, 24 Feb 2012 16:50:05 GMT (envelope-from gnats) Date: Fri, 24 Feb 2012 16:50:05 GMT Message-Id: <201202241650.q1OGo5VA098438@freefall.freebsd.org> To: freebsd-python@FreeBSD.org From: Michael Scheidell Cc: Subject: Re: ports/118301: [patch] devel/py-setuptools easy-install.pth contents lost on upgrade clobbering ports using it X-BeenThere: freebsd-python@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Michael Scheidell List-Id: FreeBSD-specific Python issues List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 24 Feb 2012 16:50:06 -0000 The following reply was made to PR ports/118301; it has been noted by GNATS. From: Michael Scheidell To: bug-followup@FreeBSD.org, gabor@freebsd.org, lwhsu@FreeBSD.org, sunpoet@FreeBSD.org, python@freebsd.org Cc: nivit@FreeBSD.org, vivanov@ivanov-nest.com, lars.eggert@nokia.com Subject: Re: ports/118301: [patch] devel/py-setuptools easy-install.pth contents lost on upgrade clobbering ports using it Date: Fri, 24 Feb 2012 11:43:23 -0500 (EST) --ELM1330101803-86909-0_ Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="US-ASCII" As promissed, attached is a patch to back out portrevision 3, or, most specifically, leave it as an option knob, default off. It packages just fine, with and without: http://people.freebsd.org/~scheidell/py27-setuptools-0.6c11_4.WITHOUT.log (the default, just like it did before portrevision 3) and, with: (with option knob set) by looking at Makefile, and pkg-plist changes, I think I correctly wrapped !defines() around all the changed things. (note: this sat for 3 years. python@ got 'open pr' emails, once a week, so, speak up now, or forever hold your peace. or nra members, your piece) I can tell you this packages, both ways, leaves no left overs, but I can't tell you if it solves the original problem with the knob, or the problem created with revision 3. But I can tell you, it looks like this attached patch reverses revision 3. Please, if you have any stake in this, test it, or, in when this gets committed, don't complain. -- Michael Scheidell, CTO SECNAP Network Security Corporation http://people.freebsd.org/~scheidell ______________________________________________________________________ This email has been scanned and certified safe by SpammerTrap(r). For Information please see http://www.spammertrap.com/ ______________________________________________________________________ --ELM1330101803-86909-0_ Content-Transfer-Encoding: 7bit Content-Type: text/x-patch Content-Disposition: attachment; filename="pr_118301_devel_py-setuptools.patch" Index: Makefile =================================================================== RCS file: /home/pcvs/ports/devel/py-setuptools/Makefile,v retrieving revision 1.24 diff -u -r1.24 Makefile --- Makefile 8 Jan 2012 01:51:58 -0000 1.24 +++ Makefile 24 Feb 2012 16:35:57 -0000 @@ -7,7 +7,7 @@ PORTNAME= setuptools PORTVERSION= 0.6c11 -PORTREVISION= 3 +PORTREVISION= 4 CATEGORIES= devel python MASTER_SITES= CHEESESHOP PKGNAMEPREFIX= ${PYTHON_PKGNAMEPREFIX} @@ -21,6 +21,8 @@ PYDISTUTILS_NOEGGINFO= yes # XXX convert easy_install support to bsd.python.mk's PLIST_SUB= VERSION=${PORTVERSION} PYVER=${PYTHON_VERSION:S/python//} +OPTIONS= EASYINSTALL "Use With Python easy-install" off + .include .if ${PYTHON_SUFFIX} < 25 @@ -31,13 +33,18 @@ PLIST_SUB+= PYTHON_OLD="@comment " .endif +.if defined(WITH_EASYINSTALL) SUB_FILES= pkg-install SUB_LIST= EASYINSTALL_PTH=${EASYINSTALL_PTH} +PLIST_SUB+= EASYINSTALL="" EASYINSTALL_PTH= ${PYTHON_SITELIBDIR}/easy-install.pth pre-install: PKG_PREFIX=${PREFIX} ${SH} ${PKGINSTALL} ${PKGNAME} PRE-INSTALL +.else +PLIST_SUB+= EASYINSTALL="@comment " +.endif post-install: ${INSTALL_DATA} ${WRKSRC}/site.py ${PYTHON_SITELIBDIR} @@ -45,8 +52,10 @@ ${PYTHON_CMD} ${opt} -m compileall -l -x \ '^[^s][^i]?[^t]?[^e]?[^.][^p][^y]$$' ${PYTHON_SITELIBDIR} .endfor +.if defined(WITH_EASYINSTALL) ${CP} ${EASYINSTALL_PTH} ${EASYINSTALL_PTH}.dist PKG_PREFIX=${PREFIX} ${SH} ${PKGINSTALL} ${PKGNAME} POST-INSTALL +.endif .if !defined(NOPORTDOCS) @${MKDIR} ${DOCSDIR} Index: pkg-plist =================================================================== RCS file: /home/pcvs/ports/devel/py-setuptools/pkg-plist,v retrieving revision 1.13 diff -u -r1.13 pkg-plist --- pkg-plist 8 Jan 2012 01:51:58 -0000 1.13 +++ pkg-plist 24 Feb 2012 16:35:57 -0000 @@ -3,8 +3,9 @@ bin/easy_install-%%PYVER%% %%PYTHON_SITELIBDIR%%/setuptools.pth @unexec if cmp -s %B/easy-install.pth %B/easy-install.pth.dist; then rm %B/easy-install.pth; fi -%%PYTHON_SITELIBDIR%%/easy-install.pth.dist -@exec if [ ! -f %B/easy-install.pth ]; then cp %B/%f %B/easy-install.pth; fi +@unexec if [ ! -f %B/easy-install.pth.dist ]; then rm %B/easy-install.pth; fi +%%EASYINSTALL%%%%PYTHON_SITELIBDIR%%/easy-install.pth.dist +%%EASYINSTALL%%@exec if [ ! -f %B/easy-install.pth ]; then cp %B/%f %B/easy-install.pth; fi %%PYTHON_SITELIBDIR%%/site.py %%PYTHON_SITELIBDIR%%/site.pyc %%PYTHON_SITELIBDIR%%/site.pyo --ELM1330101803-86909-0_--